Re: ASoC: wm9712: Microphone doesn't work, "Capture Volume" inverted

[Christoph Fritz <chf.fritz@googlemail.com>]

On Thu, 2012-04-26 at 22:37 +0100, Mark Brown wrote:
> 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 shift should be fine here.

>   The write will be suppresed if the register contents don't
> change which looks like what you're seeing here - 

Can you imagine why the registers don't change?

> 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...

Yes I rechecked this too.


Thanks,
 -- Christoph