Message ID | s5hwqh046w5.wl%tiwai@suse.de (mailing list archive) |
---|---|
State | Deferred |
Delegated to: | Takashi Iwai |
Headers | show |
Am 2014-02-12 17:45, schrieb Takashi Iwai: > At Wed, 12 Feb 2014 17:20:21 +0100, > Takashi Iwai wrote: >> >> At Wed, 12 Feb 2014 17:09:23 +0100, >> Martin Kepplinger wrote: >>> >>> This reverts commit d3c56568f43807135f2c2a09582a69f809f0d8b7. >>> >>> The reverted commit breaks audio through headphone line out on >>> the Acer TravelMate B113 (Type1Sku0) Notebook, my main work >>> machine. I don't know much about it but this fixes my problem. >>> Bisected and tested. >>> >>> Tested-by: Martin Kepplinger <martink@posteo.de> >>> Signed-off-by: Martin Kepplinger <martink@posteo.de> >> >> Too bad, we need COEF for some machine but it breaks for some. >> Since reverting breaks obviously another machine, we need a different >> approach, e.g. checking the machine ID. Please give alsa-info.sh >> output of your machine. > > Thinking it again, I'll take your patch as is, and put an additional > fix for AO725 as below. Could you try it to see whether it brings any > regressions? It's to be applied after your revert patch. > > > thanks, > > Takashi > > -- 8< -- > From: Takashi Iwai <tiwai@suse.de> > Subject: [PATCH] ALSA: hda - Better fix for invalid COEF setup on Acer AO725 > > Instead of disabling the COEF setup for all ALC271X codec (like commit > d3c56568), do it only if needed. Currently, Acer AO725 is known to > show the problem, so clear the bad init_hook in the fixup. > > The explicit call of alc269_fill_coef() in patch_alc269() is also > removed, since the function will be called anyway at init callback. > > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=52181 > Cc: <stable@vger.kernel.org> > Signed-off-by: Takashi Iwai <tiwai@suse.de> > --- > sound/pci/hda/patch_realtek.c | 18 ++++++++++++++++-- > 1 file changed, 16 insertions(+), 2 deletions(-) > > diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c > index a9a83b85517a..eaff10598f67 100644 > --- a/sound/pci/hda/patch_realtek.c > +++ b/sound/pci/hda/patch_realtek.c > @@ -3817,6 +3817,14 @@ static void alc290_fixup_mono_speakers(struct hda_codec *codec, > } > } > > +static void alc_fixup_clear_init_hook(struct hda_codec *codec, > + const struct hda_fixup *fix, int action) > +{ > + struct alc_spec *spec = codec->spec; > + if (action == HDA_FIXUP_ACT_PROBE) > + spec->init_hook = NULL; > +} > + > /* for hda_fixup_thinkpad_acpi() */ > #include "thinkpad_helper.c" > > @@ -3858,6 +3866,7 @@ enum { > ALC271_FIXUP_HP_GATE_MIC_JACK, > ALC271_FIXUP_HP_GATE_MIC_JACK_E1_572, > ALC269_FIXUP_ACER_AC700, > + ALC271_FIXUP_ACER_AO725, > ALC269_FIXUP_LIMIT_INT_MIC_BOOST, > ALC269VB_FIXUP_ASUS_ZENBOOK, > ALC269VB_FIXUP_ASUS_ZENBOOK_UX31A, > @@ -4250,13 +4259,19 @@ static const struct hda_fixup alc269_fixups[] = { > .type = HDA_FIXUP_FUNC, > .v.func = alc_fixup_headset_mode_alc255, > }, > + [ALC271_FIXUP_ACER_AO725] = { > + .type = HDA_FIXUP_FUNC, > + .v.func = alc_fixup_clear_init_hook, > + .chained = true, > + .chain_id = ALC271_FIXUP_HP_GATE_MIC_JACK, > + }, > }; > > static const struct snd_pci_quirk alc269_fixup_tbl[] = { > SND_PCI_QUIRK(0x1025, 0x029b, "Acer 1810TZ", ALC269_FIXUP_INV_DMIC), > SND_PCI_QUIRK(0x1025, 0x0349, "Acer AOD260", ALC269_FIXUP_INV_DMIC), > SND_PCI_QUIRK(0x1025, 0x047c, "Acer AC700", ALC269_FIXUP_ACER_AC700), > - SND_PCI_QUIRK(0x1025, 0x0740, "Acer AO725", ALC271_FIXUP_HP_GATE_MIC_JACK), > + SND_PCI_QUIRK(0x1025, 0x0740, "Acer AO725", ALC271_FIXUP_ACER_AO725), > SND_PCI_QUIRK(0x1025, 0x0742, "Acer AO756", ALC271_FIXUP_HP_GATE_MIC_JACK), > SND_PCI_QUIRK_VENDOR(0x1025, "Acer Aspire", ALC271_FIXUP_DMIC), > SND_PCI_QUIRK(0x1025, 0x0775, "Acer Aspire E1-572", ALC271_FIXUP_HP_GATE_MIC_JACK_E1_572), > @@ -4523,7 +4538,6 @@ static int patch_alc269(struct hda_codec *codec) > if (err < 0) > goto error; > spec->init_hook = alc269_fill_coef; > - alc269_fill_coef(codec); > break; > > case 0x10ec0280: > So my alsa-info.sh is http://www.alsa-project.org/db/?f=3e9c5d39ff057106d6ae307f9b86c1e562056c1b and I'm running your patch on top of mine without any problems. thanks, martin
At Wed, 12 Feb 2014 19:31:36 +0100, Martin Kepplinger wrote: > > Am 2014-02-12 17:45, schrieb Takashi Iwai: > > At Wed, 12 Feb 2014 17:20:21 +0100, > > Takashi Iwai wrote: > >> > >> At Wed, 12 Feb 2014 17:09:23 +0100, > >> Martin Kepplinger wrote: > >>> > >>> This reverts commit d3c56568f43807135f2c2a09582a69f809f0d8b7. > >>> > >>> The reverted commit breaks audio through headphone line out on > >>> the Acer TravelMate B113 (Type1Sku0) Notebook, my main work > >>> machine. I don't know much about it but this fixes my problem. > >>> Bisected and tested. > >>> > >>> Tested-by: Martin Kepplinger <martink@posteo.de> > >>> Signed-off-by: Martin Kepplinger <martink@posteo.de> > >> > >> Too bad, we need COEF for some machine but it breaks for some. > >> Since reverting breaks obviously another machine, we need a different > >> approach, e.g. checking the machine ID. Please give alsa-info.sh > >> output of your machine. > > > > Thinking it again, I'll take your patch as is, and put an additional > > fix for AO725 as below. Could you try it to see whether it brings any > > regressions? It's to be applied after your revert patch. > > > > > > thanks, > > > > Takashi > > > > -- 8< -- > > From: Takashi Iwai <tiwai@suse.de> > > Subject: [PATCH] ALSA: hda - Better fix for invalid COEF setup on Acer AO725 > > > > Instead of disabling the COEF setup for all ALC271X codec (like commit > > d3c56568), do it only if needed. Currently, Acer AO725 is known to > > show the problem, so clear the bad init_hook in the fixup. > > > > The explicit call of alc269_fill_coef() in patch_alc269() is also > > removed, since the function will be called anyway at init callback. > > > > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=52181 > > Cc: <stable@vger.kernel.org> > > Signed-off-by: Takashi Iwai <tiwai@suse.de> > > --- > > sound/pci/hda/patch_realtek.c | 18 ++++++++++++++++-- > > 1 file changed, 16 insertions(+), 2 deletions(-) > > > > diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c > > index a9a83b85517a..eaff10598f67 100644 > > --- a/sound/pci/hda/patch_realtek.c > > +++ b/sound/pci/hda/patch_realtek.c > > @@ -3817,6 +3817,14 @@ static void alc290_fixup_mono_speakers(struct hda_codec *codec, > > } > > } > > > > +static void alc_fixup_clear_init_hook(struct hda_codec *codec, > > + const struct hda_fixup *fix, int action) > > +{ > > + struct alc_spec *spec = codec->spec; > > + if (action == HDA_FIXUP_ACT_PROBE) > > + spec->init_hook = NULL; > > +} > > + > > /* for hda_fixup_thinkpad_acpi() */ > > #include "thinkpad_helper.c" > > > > @@ -3858,6 +3866,7 @@ enum { > > ALC271_FIXUP_HP_GATE_MIC_JACK, > > ALC271_FIXUP_HP_GATE_MIC_JACK_E1_572, > > ALC269_FIXUP_ACER_AC700, > > + ALC271_FIXUP_ACER_AO725, > > ALC269_FIXUP_LIMIT_INT_MIC_BOOST, > > ALC269VB_FIXUP_ASUS_ZENBOOK, > > ALC269VB_FIXUP_ASUS_ZENBOOK_UX31A, > > @@ -4250,13 +4259,19 @@ static const struct hda_fixup alc269_fixups[] = { > > .type = HDA_FIXUP_FUNC, > > .v.func = alc_fixup_headset_mode_alc255, > > }, > > + [ALC271_FIXUP_ACER_AO725] = { > > + .type = HDA_FIXUP_FUNC, > > + .v.func = alc_fixup_clear_init_hook, > > + .chained = true, > > + .chain_id = ALC271_FIXUP_HP_GATE_MIC_JACK, > > + }, > > }; > > > > static const struct snd_pci_quirk alc269_fixup_tbl[] = { > > SND_PCI_QUIRK(0x1025, 0x029b, "Acer 1810TZ", ALC269_FIXUP_INV_DMIC), > > SND_PCI_QUIRK(0x1025, 0x0349, "Acer AOD260", ALC269_FIXUP_INV_DMIC), > > SND_PCI_QUIRK(0x1025, 0x047c, "Acer AC700", ALC269_FIXUP_ACER_AC700), > > - SND_PCI_QUIRK(0x1025, 0x0740, "Acer AO725", ALC271_FIXUP_HP_GATE_MIC_JACK), > > + SND_PCI_QUIRK(0x1025, 0x0740, "Acer AO725", ALC271_FIXUP_ACER_AO725), > > SND_PCI_QUIRK(0x1025, 0x0742, "Acer AO756", ALC271_FIXUP_HP_GATE_MIC_JACK), > > SND_PCI_QUIRK_VENDOR(0x1025, "Acer Aspire", ALC271_FIXUP_DMIC), > > SND_PCI_QUIRK(0x1025, 0x0775, "Acer Aspire E1-572", ALC271_FIXUP_HP_GATE_MIC_JACK_E1_572), > > @@ -4523,7 +4538,6 @@ static int patch_alc269(struct hda_codec *codec) > > if (err < 0) > > goto error; > > spec->init_hook = alc269_fill_coef; > > - alc269_fill_coef(codec); > > break; > > > > case 0x10ec0280: > > > > So my alsa-info.sh is > http://www.alsa-project.org/db/?f=3e9c5d39ff057106d6ae307f9b86c1e562056c1b > and I'm running your patch on top of mine without any problems. OK. I queued the patches now. Thanks! Takashi
diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index a9a83b85517a..eaff10598f67 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -3817,6 +3817,14 @@ static void alc290_fixup_mono_speakers(struct hda_codec *codec, } } +static void alc_fixup_clear_init_hook(struct hda_codec *codec, + const struct hda_fixup *fix, int action) +{ + struct alc_spec *spec = codec->spec; + if (action == HDA_FIXUP_ACT_PROBE) + spec->init_hook = NULL; +} + /* for hda_fixup_thinkpad_acpi() */ #include "thinkpad_helper.c" @@ -3858,6 +3866,7 @@ enum { ALC271_FIXUP_HP_GATE_MIC_JACK, ALC271_FIXUP_HP_GATE_MIC_JACK_E1_572, ALC269_FIXUP_ACER_AC700, + ALC271_FIXUP_ACER_AO725, ALC269_FIXUP_LIMIT_INT_MIC_BOOST, ALC269VB_FIXUP_ASUS_ZENBOOK, ALC269VB_FIXUP_ASUS_ZENBOOK_UX31A, @@ -4250,13 +4259,19 @@ static const struct hda_fixup alc269_fixups[] = { .type = HDA_FIXUP_FUNC, .v.func = alc_fixup_headset_mode_alc255, }, + [ALC271_FIXUP_ACER_AO725] = { + .type = HDA_FIXUP_FUNC, + .v.func = alc_fixup_clear_init_hook, + .chained = true, + .chain_id = ALC271_FIXUP_HP_GATE_MIC_JACK, + }, }; static const struct snd_pci_quirk alc269_fixup_tbl[] = { SND_PCI_QUIRK(0x1025, 0x029b, "Acer 1810TZ", ALC269_FIXUP_INV_DMIC), SND_PCI_QUIRK(0x1025, 0x0349, "Acer AOD260", ALC269_FIXUP_INV_DMIC), SND_PCI_QUIRK(0x1025, 0x047c, "Acer AC700", ALC269_FIXUP_ACER_AC700), - SND_PCI_QUIRK(0x1025, 0x0740, "Acer AO725", ALC271_FIXUP_HP_GATE_MIC_JACK), + SND_PCI_QUIRK(0x1025, 0x0740, "Acer AO725", ALC271_FIXUP_ACER_AO725), SND_PCI_QUIRK(0x1025, 0x0742, "Acer AO756", ALC271_FIXUP_HP_GATE_MIC_JACK), SND_PCI_QUIRK_VENDOR(0x1025, "Acer Aspire", ALC271_FIXUP_DMIC), SND_PCI_QUIRK(0x1025, 0x0775, "Acer Aspire E1-572", ALC271_FIXUP_HP_GATE_MIC_JACK_E1_572), @@ -4523,7 +4538,6 @@ static int patch_alc269(struct hda_codec *codec) if (err < 0) goto error; spec->init_hook = alc269_fill_coef; - alc269_fill_coef(codec); break; case 0x10ec0280: