Re: [bisected] snd_hda_intel, 2.6.37: automuting on headphone connect broken; headphone connecter no longer shown in userspace

[Takashi Iwai]

At Mon, 10 Jan 2011 14:54:25 +0100,
Takashi Iwai wrote:
> 
> At Wed, 5 Jan 2011 13:07:41 +0530,
> Nirbheek Chauhan wrote:
> > 
> > Hello,
> > 
> > Upon upgrading to 2.6.37,
> > auto-muting-laptop-speakers-on-headphone-connect stopped working, and
> > the headphone connector completely disappeared from alsamixer as well
> > as the pulseaudio volume control panel. The headphone jack was still
> > transmitting sound, though.
> > 
> > I bisected the problem, with 'good' being v2.6.36, and found the
> > commit below as the first bad commit. Reverting it on v2.6.37 fixes
> > the problem. I also tried the latest alsa-kernel master git tree,
> > which had the same problem.
> > 
> > alsa-info when 'good':
> > 
> > http://www.alsa-project.org/db/?f=a0b5c95f764a0b01746be46f989685996e1f9210
> > 
> > alsa-info when 'bad':
> > 
> > http://www.alsa-project.org/db/?f=30f30ca5461f03b98ab6c7c611cf24ea4526f537
> > 
> > Note: both were taken without X running, before pulseaudio came up.
> > 
> > ================
> > 03642c9a444079aa13f0864383a8f9ca04bfd198 is the first bad commit
> > commit 03642c9a444079aa13f0864383a8f9ca04bfd198
> > Author: Takashi Iwai <tiwai@xxxxxxx>
> > Date:   Wed Sep 8 15:28:19 2010 +0200
> > 
> >     ALSA: hda - Clear left-over hp_pins in snd_hda_parse_pin_def_config()
> > 
> >     In snd_hda_parse_def_config(), some unused values may remain in hp_pins[]
> >     array during the headphone-reassignment workaround.  This patch clears
> >     the unused array members.
> > 
> >     Signed-off-by: Takashi Iwai <tiwai@xxxxxxx>
> > 
> > :040000 040000 34724336a75f74263423f28d64450d65e5ed948d
> > 8d136fce8661ad2df35ac3c4a9ca1c712c9833ed M	sound
> > 
> > ================
> 
> It seems that the driver worked casually in the earlier versions :)
> Does the patch below fix your problem?
> 
> Note that the "Headphone" control will still not appear even with this
> patch, but these two outputs are controlled by "Front" and "Surround"
> controls.  The lack of "Headphone" is no regression but simply it
> shouldn't have been there.
> 
> I'll work on 2.6.38 tree for better handling of multi headphone
> cases.

The patch is below.  Give it a try.


Takashi

===