At Mon, 14 Sep 2009 16:41:10 +0300,Ozan Çağlayan wrote:> > Takashi Iwai wrote On 14-09-2009 16:23:> > I looked through alsa-info output now, and found the culprit.> > NID 0x0f is set as a line-out, and currently the driver doesn't> > don't do auto-muting for line-out unless a headphone is present.> > Changing it to a headphone should fix. Or you can give hp_detect=1> > hint via sysfs and reconfigure.> >> > > > Yes! hp_detect=1 worked altogether with passing hp-dv5 model. I'm ready> to test any patch you suggest.
Good to hear. The final patch is already merged to sound git tree,and a new snapshot tarball is being created and updated now. FWIW,the essential change is below.
thanks,
Takashi
--->From 6e34c033210bb86b3f5f45fd6497f74150bdf620 Mon Sep 17 00:00:00 2001From: Takashi Iwai <tiwai@xxxxxxx>Date: Mon, 14 Sep 2009 15:42:18 +0200Subject: [PATCH] ALSA: hda - Add support for HP dv6
Add the quirk entry for HP dv6. Also add a workaround for the headphonedetection by setting hp_detect=1 beforehand. Without this, the driverwon't do auto-muting because BIOS doesn't give any HP pin but only aline-out pin.
Signed-off-by: Takashi Iwai <tiwai@xxxxxxx>--- sound/pci/hda/patch_sigmatel.c | 7 +++++++ 1 files changed, 7 insertions(+), 0 deletions(-)
diff --git a/sound/pci/hda/patch_sigmatel.c b/sound/pci/hda/patch_sigmatel.cindex fe2ba76..cc0e26c 100644--- a/sound/pci/hda/patch_sigmatel.c+++ b/sound/pci/hda/patch_sigmatel.c@@ -1690,6 +1690,8 @@ static struct snd_pci_quirk stac92hd71bxx_cfg_tbl[] = { "HP mini 1000", STAC_HP_M4), SND_PCI_QUIRK(PCI_VENDOR_ID_HP, 0x361b, "HP HDX", STAC_HP_HDX), /* HDX16 */+ SND_PCI_QUIRK_MASK(PCI_VENDOR_ID_HP, 0xfff0, 0x3620,+ "HP dv6", STAC_HP_DV5), SND_PCI_QUIRK_MASK(PCI_VENDOR_ID_HP, 0xfff0, 0x7010, "HP", STAC_HP_DV5), SND_PCI_QUIRK(PCI_VENDOR_ID_DELL, 0x0233,@@ -5383,6 +5385,11 @@ again: case STAC_HP_DV5: snd_hda_codec_set_pincfg(codec, 0x0d, 0x90170010); stac92xx_auto_set_pinctl(codec, 0x0d, AC_PINCTL_OUT_EN);+ /* HP dv6 gives the headphone pin as a line-out. Thus we+ * need to set hp_detect flag here to force to enable HP+ * detection.+ */+ spec->hp_detect = 1; break; case STAC_HP_HDX: spec->num_dmics = 1;-- 1.6.4.2
_______________________________________________Alsa-devel mailing listAlsa-devel@xxxxxxxxxxxxxxxxxxxx://mailman.alsa-project.org/mailman/listinfo/alsa-devel
