Re: Headphone output doesn't work with 3.0.0-rc* anymore

[Takashi Iwai <tiwai@suse.de>]

At Thu, 09 Jun 2011 08:09:18 +0200,
Takashi Iwai wrote:
> 
> At Wed, 8 Jun 2011 19:48:55 +0200,
> Michal Hocko wrote:
> > 
> > On Wed 08-06-11 18:03:24, Takashi Iwai wrote:
> > > At Tue, 7 Jun 2011 20:05:58 +0200,
> > > Michal Hocko wrote:
> > > > 
> > > > Hi,
> > > > before 3.0.0-rc1, when I plugged anything into headphone output of my
> > > > laptop, speakers were disabled and the sound has been routed to the
> > > > headphone output. This doesn't seem to work anymore starting with
> > > > 3.0.0-rc1. Speakers stay on even if I plug something to the headphone
> > > > output.
> > > > Is this a bug or a new behavior that userspace has to cope
> > > > with?
> > > > 
> > > > I am sorry for the lack of information but I have no idea what to
> > > > provide.
> > > 
> > > Please give alsa-info.sh output at best.  Run it with --no-upload
> > > option, and attach the output file.  This will contain most of
> > > necessary information.
> > 
> > See attached. I have added also .39 output where things work as expected
> > (just in case it could be useful).
> 
> Thanks, that's helpful.
> As a quirk solution, could you try to pass model=auto option to
> snd-hda-intel module?  You may need to adjust the mixer after changing
> this, but it should provide the full (even better) functionality.
> It'll support also the auto-mic jack switching.
> 
> If auto-parser works with this hardware, we don't need to debug the
> old code but just switch to the auto-parser statically in the driver.

OK, found out the culprit.  The patch below should fix the problem.

But, still the test with model=auto would be appreciated.  Basically
I'd like to kill all model-specific quirks as much as possible.  If
the auto-parser works, we'll switch to it (but maybe for 3.1).


thanks,

Takashi

---
diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
index 8187320..bef7a77 100644
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -11925,7 +11925,7 @@ static const struct hda_verb alc262_nec_verbs[] = {
  *  0x1b = port replicator headphone out
  */
 
-#define ALC_HP_EVENT	0x37
+#define ALC_HP_EVENT	ALC880_HP_EVENT
 
 static const struct hda_verb alc262_fujitsu_unsol_verbs[] = {
 	{0x14, AC_VERB_SET_UNSOLICITED_ENABLE, AC_USRSP_EN | ALC_HP_EVENT},