Alsa-Devel Archive on lore.kernel.org
 help / color / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Martin Kepplinger <martink@posteo.de>
Cc: perex@perex.cz, david.henningsson@canonical.com,
	hui.wang@canonical.com, alsa-devel@alsa-project.org,
	linux-kernel@vger.kernel.org
Subject: Re: [BUG]3.14-rc2 [PATCH] Revert "ALSA: hda/realtek - Avoid invalid COEFs for ALC271X"
Date: Wed, 12 Feb 2014 17:45:14 +0100
Message-ID: <s5hwqh046w5.wl%tiwai@suse.de> (raw)
In-Reply-To: <s5h4n445mm2.wl%tiwai@suse.de>

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:
-- 
1.8.5.2

  reply index

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-12 16:09 Martin Kepplinger
2014-02-12 16:20 ` Takashi Iwai
2014-02-12 16:45   ` Takashi Iwai [this message]
2014-02-12 18:31     ` Martin Kepplinger
2014-02-12 18:58       ` Takashi Iwai

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=s5hwqh046w5.wl%tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=alsa-devel@alsa-project.org \
    --cc=david.henningsson@canonical.com \
    --cc=hui.wang@canonical.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=martink@posteo.de \
    --cc=perex@perex.cz \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

Alsa-Devel Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/alsa-devel/0 alsa-devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 alsa-devel alsa-devel/ https://lore.kernel.org/alsa-devel \
		alsa-devel@alsa-project.org
	public-inbox-index alsa-devel

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.alsa-project.alsa-devel


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git