From: Takashi Iwai <tiwai@suse.de>
To: "Jian-Hong Pan" <jian-hong@endlessm.com>
Cc: <alsa-devel@alsa-project.org>,
"Daniel Drake" <drake@endlessm.com>, <linux@endlessm.com>,
"Kailang Yang" <kailang@realtek.com>,
<linux-kernel@vger.kernel.org>
Subject: Re: [PATCH] ALSA: hda/realtek: Enable headset MIC of Acer TravelMate B114-21 with ALC233
Date: Mon, 01 Apr 2019 19:08:49 +0200 [thread overview]
Message-ID: <s5hmul9lkke.wl-tiwai@suse.de> (raw)
In-Reply-To: <20190401032503.2821-1-jian-hong@endlessm.com>
On Mon, 01 Apr 2019 05:25:05 +0200,
Jian-Hong Pan wrote:
>
> The Acer TravelMate B114-21 laptop cannot detect and record sound from
> headset MIC. This patch adds the ALC233_FIXUP_ACER_HEADSET_MIC HDA verb
> quirk chained with ALC233_FIXUP_ASUS_MIC_NO_PRESENCE pin quirk to fix
> this issue.
>
> Signed-off-by: Jian-Hong Pan <jian-hong@endlessm.com>
> Signed-off-by: Daniel Drake <drake@endlessm.com>
> ---
> sound/pci/hda/patch_realtek.c | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
> diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
> index a3fb3d4c5730..bdb2227be4eb 100644
> --- a/sound/pci/hda/patch_realtek.c
> +++ b/sound/pci/hda/patch_realtek.c
> @@ -5690,6 +5690,7 @@ enum {
> ALC286_FIXUP_ACER_AIO_HEADSET_MIC,
> ALC256_FIXUP_ASUS_MIC_NO_PRESENCE,
> ALC299_FIXUP_PREDATOR_SPK,
> + ALC233_FIXUP_ACER_HEADSET_MIC,
> };
>
> static const struct hda_fixup alc269_fixups[] = {
> @@ -6713,6 +6714,15 @@ static const struct hda_fixup alc269_fixups[] = {
> { 0x21, 0x90170150 }, /* use as headset mic, without its own jack detect */
> { }
> }
> + [ALC233_FIXUP_ACER_HEADSET_MIC] = {
> + .type = HDA_FIXUP_VERBS,
> + .v.verbs = (const struct hda_verb[]) {
> + { 0x20, AC_VERB_SET_COEF_INDEX, 0x45 },
> + { 0x20, AC_VERB_SET_PROC_COEF, 0x5089 },
> + { }
For the additional COEF application, I'd like to hear ack from
Kailang. It's still a black magic from Realtek, so need some
confirmation.
thanks,
Takashi
> + },
> + .chained = true,
> + .chain_id = ALC233_FIXUP_ASUS_MIC_NO_PRESENCE
> },
> };
>
> @@ -6737,6 +6747,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = {
> SND_PCI_QUIRK(0x1025, 0x1290, "Acer Veriton Z4860G", ALC286_FIXUP_ACER_AIO_HEADSET_MIC),
> SND_PCI_QUIRK(0x1025, 0x1291, "Acer Veriton Z4660G", ALC286_FIXUP_ACER_AIO_HEADSET_MIC),
> SND_PCI_QUIRK(0x1025, 0x1308, "Acer Aspire Z24-890", ALC286_FIXUP_ACER_AIO_HEADSET_MIC),
> + SND_PCI_QUIRK(0x1025, 0x132a, "Acer TravelMate B114-21", ALC233_FIXUP_ACER_HEADSET_MIC),
> SND_PCI_QUIRK(0x1025, 0x1330, "Acer TravelMate X514-51T", ALC255_FIXUP_ACER_HEADSET_MIC),
> SND_PCI_QUIRK(0x1028, 0x0470, "Dell M101z", ALC269_FIXUP_DELL_M101Z),
> SND_PCI_QUIRK(0x1028, 0x054b, "Dell XPS one 2710", ALC275_FIXUP_DELL_XPS),
> --
> 2.20.1
>
>
next prev parent reply other threads:[~2019-04-01 18:09 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-04-01 3:25 Jian-Hong Pan
2019-04-01 17:08 ` Takashi Iwai [this message]
2019-04-02 6:38 ` Kailang
2019-04-02 6:43 ` Takashi Iwai
2019-04-02 6:45 ` Kailang
2019-04-02 7:18 ` 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=s5hmul9lkke.wl-tiwai@suse.de \
--to=tiwai@suse.de \
--cc=alsa-devel@alsa-project.org \
--cc=drake@endlessm.com \
--cc=jian-hong@endlessm.com \
--cc=kailang@realtek.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@endlessm.com \
/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
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®