ASoC: kirkwood-i2s: fix DCO lock detection

Russell King · broonie · commit 2424d458108e · 2012-11-21T10:38:38.000+09:00
This is part of a patch found in Rabeeh Khoury's git tree for the
cubox, which is further attributed to Sebastian Hesselbrath.

Rather than masking the KIRKWOOD_DCO_SPCR_STATUS register contents
against the registers virtual address, let's actually use the bit
definition for the locked status, as required in the documentation.

Signed-off-by: Russell King &lt;rmk+kernel@arm.linux.org.uk&gt;
Signed-off-by: Mark Brown &lt;broonie@opensource.wolfsonmicro.com&gt;
diff --git a/sound/soc/kirkwood/kirkwood-i2s.c b/sound/soc/kirkwood/kirkwood-i2s.c
@@ -95,7 +95,7 @@ static inline void kirkwood_set_dco(void __iomem *io, unsigned long rate)
 	do {
 		cpu_relax();
 		value = readl(io + KIRKWOOD_DCO_SPCR_STATUS);
-		value &= KIRKWOOD_DCO_SPCR_STATUS;
+		value &= KIRKWOOD_DCO_SPCR_STATUS_DCO_LOCK;
 	} while (value == 0);
 }
 

Original file line number	Diff line number	Diff line change
`@@ -95,7 +95,7 @@ static inline void kirkwood_set_dco(void __iomem *io, unsigned long rate)`
`95`	`95`	`do {`
`96`	`96`	`cpu_relax();`
`97`	`97`	`value = readl(io + KIRKWOOD_DCO_SPCR_STATUS);`
`98`		`- value &= KIRKWOOD_DCO_SPCR_STATUS;`
	`98`	`+ value &= KIRKWOOD_DCO_SPCR_STATUS_DCO_LOCK;`
`99`	`99`	`} while (value == 0);`
`100`	`100`	`}`
`101`	`101`