Re: Conexant 5051 lenovo-x200 fixes

From: Takashi Iwai <tiwai@suse.de>
To: Ulo Mets <umets@osa.pri.ee>
Cc: alsa-devel@alsa-project.org
Subject: Re: Conexant 5051 lenovo-x200 fixes
Date: Fri, 19 Aug 2011 07:43:05 +0200	[thread overview]
Message-ID: <s5h62lulzx2.wl%tiwai@suse.de> (raw)
In-Reply-To: <4E4DA7BA.5020704@osa.pri.ee>

At Fri, 19 Aug 2011 03:00:58 +0300,
Ulo Mets wrote:
> 
> On 08/18/2011 01:08 PM, Takashi Iwai wrote:
> > At Thu, 18 Aug 2011 03:45:33 +0300,
> > Ulo Mets wrote:
> >> Hi,
> >> I am not a kernel developer, but occasionally dig into it if something gets in my way.
> >> On Lenovo X200 Tablet none of the mixers showed Mic Boost adjustments, but HDA-Analyzer found them
> >> and adjusted properly. So I inserted the corresponding controls, and it worked.
> >> Then, during testing I found that the labels of External and Docking Mic are swapped,
> >> so I exchanged these too. Since I could not test it on other hardware, I put it all on lenovo-x200 branch.
> >> The patch applies to 3.1-rc2, I hope it is sufficiently recent.
> >> The Mic Switches don't have any effect on my unit (X200T without modem)
> > Could you try model=auto instead of patching?
> > The recent auto-parser should work better.
> Model=auto seems to give a very generic configuration, which indeed does 
> include working
> Mic boost for Internal Mic. (Ext?) Mic Boost and gain are also present, 
> but jack sensing does not work,
> so these cannot be tested.

The automatic mic-switch by jack-sensing works without model=auto,
right?  Then we need to fix it.  Could you give alsa-info.sh output?

> Also, the Dock Microphone is not detected, so I would conclude that for 
> X200T "auto" does not work.

OK, it means that BIOS doesn't set up the pins for docking-station.

> It would be surprising if Auto was better than dedicated model. (X200 
> and X200T accept
> same part numbers for audio card).

The auto-parser is nowadays much better than the model specific
quirks.  I'm going to reduce model quirks as much as possible now.
So, basically, I won't add a model fix any more unless it's
unavoidable.  In your case, we'll need to fix pin configurations (like
in patch_sigmatel.c or patch_realtek.c) and fix the auto-parser.

> The kernel version that I used for this latest testing is 3.0-ARCH
> 
> It would be nice to get the mute switches to work as well.

Which mute switch do you mean exactly?

thanks,

Takashi