From mboxrd@z Thu Jan 1 00:00:00 1970 From: Takashi Iwai Subject: Re: [PATCH] ALSA: hda - restore the MIC FIXUP for some Dell machines Date: Fri, 26 Jun 2015 07:06:42 +0200 Message-ID: References: <1435293317-386-1-git-send-email-hui.wang@canonical.com> Mime-Version: 1.0 (generated by SEMI 1.14.6 - "Maruoka") Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: Received: from mx2.suse.de (cantor2.suse.de [195.135.220.15]) by alsa0.perex.cz (Postfix) with ESMTP id D96162607E8 for ; Fri, 26 Jun 2015 07:06:42 +0200 (CEST) In-Reply-To: <1435293317-386-1-git-send-email-hui.wang@canonical.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org To: Hui Wang Cc: alsa-devel@alsa-project.org List-Id: alsa-devel@alsa-project.org At Fri, 26 Jun 2015 12:35:17 +0800, Hui Wang wrote: > > Those FIXUPs were applied to the machines through pin quirks, but > recently the PCI_QUIRK makes them can't apply to the machines. > > Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=99851 > Signed-off-by: Hui Wang Thanks, applied. Takashi > --- > sound/pci/hda/patch_realtek.c | 18 +++++++++++++++++- > 1 file changed, 17 insertions(+), 1 deletion(-) > > diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c > index 8e02cdf..78dfec9d 100644 > --- a/sound/pci/hda/patch_realtek.c > +++ b/sound/pci/hda/patch_realtek.c > @@ -4522,6 +4522,8 @@ enum { > ALC288_FIXUP_DELL_HEADSET_MODE, > ALC288_FIXUP_DELL1_MIC_NO_PRESENCE, > ALC288_FIXUP_DELL_XPS_13_GPIO6, > + ALC288_FIXUP_DELL_XPS_13, > + ALC288_FIXUP_DISABLE_AAMIX, > ALC292_FIXUP_DELL_E7X, > ALC292_FIXUP_DISABLE_AAMIX, > ALC298_FIXUP_DELL1_MIC_NO_PRESENCE, > @@ -5053,9 +5055,23 @@ static const struct hda_fixup alc269_fixups[] = { > .chained = true, > .chain_id = ALC288_FIXUP_DELL1_MIC_NO_PRESENCE > }, > + [ALC288_FIXUP_DISABLE_AAMIX] = { > + .type = HDA_FIXUP_FUNC, > + .v.func = alc_fixup_disable_aamix, > + .chained = true, > + .chain_id = ALC288_FIXUP_DELL_XPS_13_GPIO6 > + }, > + [ALC288_FIXUP_DELL_XPS_13] = { > + .type = HDA_FIXUP_FUNC, > + .v.func = alc_fixup_dell_xps13, > + .chained = true, > + .chain_id = ALC288_FIXUP_DISABLE_AAMIX > + }, > [ALC292_FIXUP_DISABLE_AAMIX] = { > .type = HDA_FIXUP_FUNC, > .v.func = alc_fixup_disable_aamix, > + .chained = true, > + .chain_id = ALC269_FIXUP_DELL2_MIC_NO_PRESENCE > }, > [ALC292_FIXUP_DELL_E7X] = { > .type = HDA_FIXUP_FUNC, > @@ -5096,7 +5112,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = { > SND_PCI_QUIRK(0x1028, 0x0638, "Dell Inspiron 5439", ALC290_FIXUP_MONO_SPEAKERS_HSJACK), > SND_PCI_QUIRK(0x1028, 0x064a, "Dell", ALC293_FIXUP_DELL1_MIC_NO_PRESENCE), > SND_PCI_QUIRK(0x1028, 0x064b, "Dell", ALC293_FIXUP_DELL1_MIC_NO_PRESENCE), > - SND_PCI_QUIRK(0x1028, 0x0665, "Dell XPS 13", ALC292_FIXUP_DELL_E7X), > + SND_PCI_QUIRK(0x1028, 0x0665, "Dell XPS 13", ALC288_FIXUP_DELL_XPS_13), > SND_PCI_QUIRK(0x1028, 0x06c7, "Dell", ALC255_FIXUP_DELL1_MIC_NO_PRESENCE), > SND_PCI_QUIRK(0x1028, 0x06d9, "Dell", ALC293_FIXUP_DELL1_MIC_NO_PRESENCE), > SND_PCI_QUIRK(0x1028, 0x06da, "Dell", ALC293_FIXUP_DELL1_MIC_NO_PRESENCE), > -- > 1.9.1 >