On Thu, Apr 26, 2012 at 11:11:33PM +0200, Christoph Fritz wrote:
On Thu, 2012-04-26 at 18:20 +0100, Mark Brown wrote:
On Thu, Apr 26, 2012 at 03:49:02PM +0200, Christoph Fritz wrote:
Is "SOC_ENUM_SINGLE(0x5c, 8, 2, wm9712_diff_sel)," correct? Because of the ">> 1" shift in ac97_write() ?
What makes you think it might not be correct?
ac97_write() doesn't get called when I try to change "Item: Differential Source".
Oh, right - you're not actually seeing any attempt to write. There won't be anything on the bus, then. Still, why are you identifying the shift there? The write will be suppresed if the register contents don't change which looks like what you're seeing here - though I can't see anything obviously wrong with the enum from looking at the code, it's saying that the enumeration exists in bit 8 of register 0x5c...