* [PATCH] ALSA: hda/realtek - Fix 2 front mics of codec 0x623 @ 2019-10-22 15:38 Aaron Ma 2019-10-22 16:08 ` Takashi Iwai ` (2 more replies) 0 siblings, 3 replies; 11+ messages in thread From: Aaron Ma @ 2019-10-22 15:38 UTC (permalink / raw) To: perex, tiwai, kailang, hui.wang, alsa-devel, aaron.ma, linux-kernel These 2 ThinkCentres installed a new realtek codec ID 0x623, it has 2 front mics with the same location on pin 0x18 and 0x19. Apply fixup ALC283_FIXUP_HEADSET_MIC to change 1 front mic location to right, then pulseaudio can handle them. One "Front Mic" and one "Mic" will be shown, and audio output works fine. Signed-off-by: Aaron Ma <aaron.ma@canonical.com> --- sound/pci/hda/patch_realtek.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index b000b36ac3c6..c34d8b435f58 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -7186,6 +7186,8 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = { SND_PCI_QUIRK(0x17aa, 0x312f, "ThinkCentre Station", ALC294_FIXUP_LENOVO_MIC_LOCATION), SND_PCI_QUIRK(0x17aa, 0x313c, "ThinkCentre Station", ALC294_FIXUP_LENOVO_MIC_LOCATION), SND_PCI_QUIRK(0x17aa, 0x3151, "ThinkCentre Station", ALC283_FIXUP_HEADSET_MIC), + SND_PCI_QUIRK(0x17aa, 0x3178, "ThinkCentre Station", ALC283_FIXUP_HEADSET_MIC), + SND_PCI_QUIRK(0x17aa, 0x3176, "ThinkCentre Station", ALC283_FIXUP_HEADSET_MIC), SND_PCI_QUIRK(0x17aa, 0x3902, "Lenovo E50-80", ALC269_FIXUP_DMIC_THINKPAD_ACPI), SND_PCI_QUIRK(0x17aa, 0x3977, "IdeaPad S210", ALC283_FIXUP_INT_MIC), SND_PCI_QUIRK(0x17aa, 0x3978, "Lenovo B50-70", ALC269_FIXUP_DMIC_THINKPAD_ACPI), @@ -9187,6 +9189,7 @@ static const struct hda_device_id snd_hda_id_realtek[] = { HDA_CODEC_ENTRY(0x10ec0298, "ALC298", patch_alc269), HDA_CODEC_ENTRY(0x10ec0299, "ALC299", patch_alc269), HDA_CODEC_ENTRY(0x10ec0300, "ALC300", patch_alc269), + HDA_CODEC_ENTRY(0x10ec0623, "ALC623", patch_alc269), HDA_CODEC_REV_ENTRY(0x10ec0861, 0x100340, "ALC660", patch_alc861), HDA_CODEC_ENTRY(0x10ec0660, "ALC660-VD", patch_alc861vd), HDA_CODEC_ENTRY(0x10ec0861, "ALC861", patch_alc861), -- 2.17.1 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH] ALSA: hda/realtek - Fix 2 front mics of codec 0x623 2019-10-22 15:38 [PATCH] ALSA: hda/realtek - Fix 2 front mics of codec 0x623 Aaron Ma @ 2019-10-22 16:08 ` Takashi Iwai 2019-10-23 8:44 ` Kailang 2019-10-24 11:18 ` [PATCH v2] " Aaron Ma 2019-10-24 11:44 ` [PATCH v3] " Aaron Ma 2 siblings, 1 reply; 11+ messages in thread From: Takashi Iwai @ 2019-10-22 16:08 UTC (permalink / raw) To: Aaron Ma; +Cc: perex, kailang, hui.wang, alsa-devel, linux-kernel On Tue, 22 Oct 2019 17:38:55 +0200, Aaron Ma wrote: > > These 2 ThinkCentres installed a new realtek codec ID 0x623, > it has 2 front mics with the same location on pin 0x18 and 0x19. > > Apply fixup ALC283_FIXUP_HEADSET_MIC to change 1 front mic > location to right, then pulseaudio can handle them. > One "Front Mic" and one "Mic" will be shown, and audio output works > fine. > > Signed-off-by: Aaron Ma <aaron.ma@canonical.com> I'd like to have Kailang's review about the new codec before applying. Kailang, could you take a look? thanks, Takashi > --- > sound/pci/hda/patch_realtek.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c > index b000b36ac3c6..c34d8b435f58 100644 > --- a/sound/pci/hda/patch_realtek.c > +++ b/sound/pci/hda/patch_realtek.c > @@ -7186,6 +7186,8 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = { > SND_PCI_QUIRK(0x17aa, 0x312f, "ThinkCentre Station", ALC294_FIXUP_LENOVO_MIC_LOCATION), > SND_PCI_QUIRK(0x17aa, 0x313c, "ThinkCentre Station", ALC294_FIXUP_LENOVO_MIC_LOCATION), > SND_PCI_QUIRK(0x17aa, 0x3151, "ThinkCentre Station", ALC283_FIXUP_HEADSET_MIC), > + SND_PCI_QUIRK(0x17aa, 0x3178, "ThinkCentre Station", ALC283_FIXUP_HEADSET_MIC), > + SND_PCI_QUIRK(0x17aa, 0x3176, "ThinkCentre Station", ALC283_FIXUP_HEADSET_MIC), > SND_PCI_QUIRK(0x17aa, 0x3902, "Lenovo E50-80", ALC269_FIXUP_DMIC_THINKPAD_ACPI), > SND_PCI_QUIRK(0x17aa, 0x3977, "IdeaPad S210", ALC283_FIXUP_INT_MIC), > SND_PCI_QUIRK(0x17aa, 0x3978, "Lenovo B50-70", ALC269_FIXUP_DMIC_THINKPAD_ACPI), > @@ -9187,6 +9189,7 @@ static const struct hda_device_id snd_hda_id_realtek[] = { > HDA_CODEC_ENTRY(0x10ec0298, "ALC298", patch_alc269), > HDA_CODEC_ENTRY(0x10ec0299, "ALC299", patch_alc269), > HDA_CODEC_ENTRY(0x10ec0300, "ALC300", patch_alc269), > + HDA_CODEC_ENTRY(0x10ec0623, "ALC623", patch_alc269), > HDA_CODEC_REV_ENTRY(0x10ec0861, 0x100340, "ALC660", patch_alc861), > HDA_CODEC_ENTRY(0x10ec0660, "ALC660-VD", patch_alc861vd), > HDA_CODEC_ENTRY(0x10ec0861, "ALC861", patch_alc861), > -- > 2.17.1 > ^ permalink raw reply [flat|nested] 11+ messages in thread
* RE: [PATCH] ALSA: hda/realtek - Fix 2 front mics of codec 0x623 2019-10-22 16:08 ` Takashi Iwai @ 2019-10-23 8:44 ` Kailang 2019-10-23 10:06 ` Aaron Ma 0 siblings, 1 reply; 11+ messages in thread From: Kailang @ 2019-10-23 8:44 UTC (permalink / raw) To: Takashi Iwai, Aaron Ma; +Cc: perex, hui.wang, alsa-devel, linux-kernel > -----Original Message----- > From: Takashi Iwai <tiwai@suse.de> > Sent: Wednesday, October 23, 2019 12:08 AM > To: Aaron Ma <aaron.ma@canonical.com> > Cc: perex@perex.cz; Kailang <kailang@realtek.com>; > hui.wang@canonical.com; alsa-devel@alsa-project.org; > linux-kernel@vger.kernel.org > Subject: Re: [PATCH] ALSA: hda/realtek - Fix 2 front mics of codec 0x623 > > On Tue, 22 Oct 2019 17:38:55 +0200, > Aaron Ma wrote: > > > > These 2 ThinkCentres installed a new realtek codec ID 0x623, it has 2 > > front mics with the same location on pin 0x18 and 0x19. > > > > Apply fixup ALC283_FIXUP_HEADSET_MIC to change 1 front mic location to > > right, then pulseaudio can handle them. > > One "Front Mic" and one "Mic" will be shown, and audio output works > > fine. > > > > Signed-off-by: Aaron Ma <aaron.ma@canonical.com> > > I'd like to have Kailang's review about the new codec before applying. > > Kailang, could you take a look? OK. I will post you the patch for ALC623 codec tomorrow. Thanks. > > > thanks, > > Takashi > > > --- > > sound/pci/hda/patch_realtek.c | 3 +++ > > 1 file changed, 3 insertions(+) > > > > diff --git a/sound/pci/hda/patch_realtek.c > > b/sound/pci/hda/patch_realtek.c index b000b36ac3c6..c34d8b435f58 > > 100644 > > --- a/sound/pci/hda/patch_realtek.c > > +++ b/sound/pci/hda/patch_realtek.c > > @@ -7186,6 +7186,8 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] > = { > > SND_PCI_QUIRK(0x17aa, 0x312f, "ThinkCentre Station", > ALC294_FIXUP_LENOVO_MIC_LOCATION), > > SND_PCI_QUIRK(0x17aa, 0x313c, "ThinkCentre Station", > ALC294_FIXUP_LENOVO_MIC_LOCATION), > > SND_PCI_QUIRK(0x17aa, 0x3151, "ThinkCentre Station", > > ALC283_FIXUP_HEADSET_MIC), > > + SND_PCI_QUIRK(0x17aa, 0x3178, "ThinkCentre Station", > ALC283_FIXUP_HEADSET_MIC), > > + SND_PCI_QUIRK(0x17aa, 0x3176, "ThinkCentre Station", > > +ALC283_FIXUP_HEADSET_MIC), > > SND_PCI_QUIRK(0x17aa, 0x3902, "Lenovo E50-80", > ALC269_FIXUP_DMIC_THINKPAD_ACPI), > > SND_PCI_QUIRK(0x17aa, 0x3977, "IdeaPad S210", > ALC283_FIXUP_INT_MIC), > > SND_PCI_QUIRK(0x17aa, 0x3978, "Lenovo B50-70", > > ALC269_FIXUP_DMIC_THINKPAD_ACPI), @@ -9187,6 +9189,7 @@ static > const struct hda_device_id snd_hda_id_realtek[] = { > > HDA_CODEC_ENTRY(0x10ec0298, "ALC298", patch_alc269), > > HDA_CODEC_ENTRY(0x10ec0299, "ALC299", patch_alc269), > > HDA_CODEC_ENTRY(0x10ec0300, "ALC300", patch_alc269), > > + HDA_CODEC_ENTRY(0x10ec0623, "ALC623", patch_alc269), > > HDA_CODEC_REV_ENTRY(0x10ec0861, 0x100340, "ALC660", > patch_alc861), > > HDA_CODEC_ENTRY(0x10ec0660, "ALC660-VD", patch_alc861vd), > > HDA_CODEC_ENTRY(0x10ec0861, "ALC861", patch_alc861), > > -- > > 2.17.1 > > > > ------Please consider the environment before printing this e-mail. ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] ALSA: hda/realtek - Fix 2 front mics of codec 0x623 2019-10-23 8:44 ` Kailang @ 2019-10-23 10:06 ` Aaron Ma 2019-10-24 7:20 ` Kailang 0 siblings, 1 reply; 11+ messages in thread From: Aaron Ma @ 2019-10-23 10:06 UTC (permalink / raw) To: Kailang, Takashi Iwai; +Cc: perex, hui.wang, alsa-devel, linux-kernel On 10/23/19 4:44 PM, Kailang wrote: > > >> -----Original Message----- >> From: Takashi Iwai <tiwai@suse.de> >> Sent: Wednesday, October 23, 2019 12:08 AM >> To: Aaron Ma <aaron.ma@canonical.com> >> Cc: perex@perex.cz; Kailang <kailang@realtek.com>; >> hui.wang@canonical.com; alsa-devel@alsa-project.org; >> linux-kernel@vger.kernel.org >> Subject: Re: [PATCH] ALSA: hda/realtek - Fix 2 front mics of codec 0x623 >> >> On Tue, 22 Oct 2019 17:38:55 +0200, >> Aaron Ma wrote: >>> >>> These 2 ThinkCentres installed a new realtek codec ID 0x623, it has 2 >>> front mics with the same location on pin 0x18 and 0x19. >>> >>> Apply fixup ALC283_FIXUP_HEADSET_MIC to change 1 front mic location to >>> right, then pulseaudio can handle them. >>> One "Front Mic" and one "Mic" will be shown, and audio output works >>> fine. >>> >>> Signed-off-by: Aaron Ma <aaron.ma@canonical.com> >> >> I'd like to have Kailang's review about the new codec before applying. >> >> Kailang, could you take a look? > OK. > I will post you the patch for ALC623 codec tomorrow. > Thanks. Cc me too. Thank you. Aaron > >> >> >> thanks, >> >> Takashi >> >>> --- >>> sound/pci/hda/patch_realtek.c | 3 +++ >>> 1 file changed, 3 insertions(+) >>> >>> diff --git a/sound/pci/hda/patch_realtek.c >>> b/sound/pci/hda/patch_realtek.c index b000b36ac3c6..c34d8b435f58 >>> 100644 >>> --- a/sound/pci/hda/patch_realtek.c >>> +++ b/sound/pci/hda/patch_realtek.c >>> @@ -7186,6 +7186,8 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] >> = { >>> SND_PCI_QUIRK(0x17aa, 0x312f, "ThinkCentre Station", >> ALC294_FIXUP_LENOVO_MIC_LOCATION), >>> SND_PCI_QUIRK(0x17aa, 0x313c, "ThinkCentre Station", >> ALC294_FIXUP_LENOVO_MIC_LOCATION), >>> SND_PCI_QUIRK(0x17aa, 0x3151, "ThinkCentre Station", >>> ALC283_FIXUP_HEADSET_MIC), >>> + SND_PCI_QUIRK(0x17aa, 0x3178, "ThinkCentre Station", >> ALC283_FIXUP_HEADSET_MIC), >>> + SND_PCI_QUIRK(0x17aa, 0x3176, "ThinkCentre Station", >>> +ALC283_FIXUP_HEADSET_MIC), >>> SND_PCI_QUIRK(0x17aa, 0x3902, "Lenovo E50-80", >> ALC269_FIXUP_DMIC_THINKPAD_ACPI), >>> SND_PCI_QUIRK(0x17aa, 0x3977, "IdeaPad S210", >> ALC283_FIXUP_INT_MIC), >>> SND_PCI_QUIRK(0x17aa, 0x3978, "Lenovo B50-70", >>> ALC269_FIXUP_DMIC_THINKPAD_ACPI), @@ -9187,6 +9189,7 @@ static >> const struct hda_device_id snd_hda_id_realtek[] = { >>> HDA_CODEC_ENTRY(0x10ec0298, "ALC298", patch_alc269), >>> HDA_CODEC_ENTRY(0x10ec0299, "ALC299", patch_alc269), >>> HDA_CODEC_ENTRY(0x10ec0300, "ALC300", patch_alc269), >>> + HDA_CODEC_ENTRY(0x10ec0623, "ALC623", patch_alc269), >>> HDA_CODEC_REV_ENTRY(0x10ec0861, 0x100340, "ALC660", >> patch_alc861), >>> HDA_CODEC_ENTRY(0x10ec0660, "ALC660-VD", patch_alc861vd), >>> HDA_CODEC_ENTRY(0x10ec0861, "ALC861", patch_alc861), >>> -- >>> 2.17.1 >>> >> >> ------Please consider the environment before printing this e-mail. ^ permalink raw reply [flat|nested] 11+ messages in thread
* RE: [PATCH] ALSA: hda/realtek - Fix 2 front mics of codec 0x623 2019-10-23 10:06 ` Aaron Ma @ 2019-10-24 7:20 ` Kailang 2019-10-24 10:35 ` Takashi Iwai 0 siblings, 1 reply; 11+ messages in thread From: Kailang @ 2019-10-24 7:20 UTC (permalink / raw) To: Aaron Ma, Takashi Iwai; +Cc: perex, hui.wang, alsa-devel, linux-kernel [-- Attachment #1: Type: text/plain, Size: 3557 bytes --] Hi Takashi, Attach patch was support ALC623. BR, Kailang > -----Original Message----- > From: Aaron Ma <aaron.ma@canonical.com> > Sent: Wednesday, October 23, 2019 6:06 PM > To: Kailang <kailang@realtek.com>; Takashi Iwai <tiwai@suse.de> > Cc: perex@perex.cz; hui.wang@canonical.com; alsa-devel@alsa-project.org; > linux-kernel@vger.kernel.org > Subject: Re: [PATCH] ALSA: hda/realtek - Fix 2 front mics of codec 0x623 > > On 10/23/19 4:44 PM, Kailang wrote: > > > > > >> -----Original Message----- > >> From: Takashi Iwai <tiwai@suse.de> > >> Sent: Wednesday, October 23, 2019 12:08 AM > >> To: Aaron Ma <aaron.ma@canonical.com> > >> Cc: perex@perex.cz; Kailang <kailang@realtek.com>; > >> hui.wang@canonical.com; alsa-devel@alsa-project.org; > >> linux-kernel@vger.kernel.org > >> Subject: Re: [PATCH] ALSA: hda/realtek - Fix 2 front mics of codec > >> 0x623 > >> > >> On Tue, 22 Oct 2019 17:38:55 +0200, > >> Aaron Ma wrote: > >>> > >>> These 2 ThinkCentres installed a new realtek codec ID 0x623, it has > >>> 2 front mics with the same location on pin 0x18 and 0x19. > >>> > >>> Apply fixup ALC283_FIXUP_HEADSET_MIC to change 1 front mic location > >>> to right, then pulseaudio can handle them. > >>> One "Front Mic" and one "Mic" will be shown, and audio output works > >>> fine. > >>> > >>> Signed-off-by: Aaron Ma <aaron.ma@canonical.com> > >> > >> I'd like to have Kailang's review about the new codec before applying. > >> > >> Kailang, could you take a look? > > OK. > > I will post you the patch for ALC623 codec tomorrow. > > Thanks. > > Cc me too. > > Thank you. > Aaron > > > > >> > >> > >> thanks, > >> > >> Takashi > >> > >>> --- > >>> sound/pci/hda/patch_realtek.c | 3 +++ > >>> 1 file changed, 3 insertions(+) > >>> > >>> diff --git a/sound/pci/hda/patch_realtek.c > >>> b/sound/pci/hda/patch_realtek.c index b000b36ac3c6..c34d8b435f58 > >>> 100644 > >>> --- a/sound/pci/hda/patch_realtek.c > >>> +++ b/sound/pci/hda/patch_realtek.c > >>> @@ -7186,6 +7186,8 @@ static const struct snd_pci_quirk > >>> alc269_fixup_tbl[] > >> = { > >>> SND_PCI_QUIRK(0x17aa, 0x312f, "ThinkCentre Station", > >> ALC294_FIXUP_LENOVO_MIC_LOCATION), > >>> SND_PCI_QUIRK(0x17aa, 0x313c, "ThinkCentre Station", > >> ALC294_FIXUP_LENOVO_MIC_LOCATION), > >>> SND_PCI_QUIRK(0x17aa, 0x3151, "ThinkCentre Station", > >>> ALC283_FIXUP_HEADSET_MIC), > >>> + SND_PCI_QUIRK(0x17aa, 0x3178, "ThinkCentre Station", > >> ALC283_FIXUP_HEADSET_MIC), > >>> + SND_PCI_QUIRK(0x17aa, 0x3176, "ThinkCentre Station", > >>> +ALC283_FIXUP_HEADSET_MIC), > >>> SND_PCI_QUIRK(0x17aa, 0x3902, "Lenovo E50-80", > >> ALC269_FIXUP_DMIC_THINKPAD_ACPI), > >>> SND_PCI_QUIRK(0x17aa, 0x3977, "IdeaPad S210", > >> ALC283_FIXUP_INT_MIC), > >>> SND_PCI_QUIRK(0x17aa, 0x3978, "Lenovo B50-70", > >>> ALC269_FIXUP_DMIC_THINKPAD_ACPI), @@ -9187,6 +9189,7 @@ static > >> const struct hda_device_id snd_hda_id_realtek[] = { > >>> HDA_CODEC_ENTRY(0x10ec0298, "ALC298", patch_alc269), > >>> HDA_CODEC_ENTRY(0x10ec0299, "ALC299", patch_alc269), > >>> HDA_CODEC_ENTRY(0x10ec0300, "ALC300", patch_alc269), > >>> + HDA_CODEC_ENTRY(0x10ec0623, "ALC623", patch_alc269), > >>> HDA_CODEC_REV_ENTRY(0x10ec0861, 0x100340, "ALC660", > >> patch_alc861), > >>> HDA_CODEC_ENTRY(0x10ec0660, "ALC660-VD", patch_alc861vd), > >>> HDA_CODEC_ENTRY(0x10ec0861, "ALC861", patch_alc861), > >>> -- > >>> 2.17.1 > >>> > >> > >> ------Please consider the environment before printing this e-mail. [-- Attachment #2: 0000-add-support-alc623.patch --] [-- Type: application/octet-stream, Size: 1990 bytes --] From 3103f6683093e47368d87671f3726f1a11f9ef28 Mon Sep 17 00:00:00 2001 From: Kailang Yang <kailang@realtek.com> Date: Thu, 24 Oct 2019 15:13:32 +0800 Subject: [PATCH] ALSA: hda/realtek - Add support for ALC623 Support new codec ALC623. Signed-off-by: Kailang Yang <kailang@realtek.com> diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index 085a2f95e076..a0c237cc13d4 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -409,6 +409,9 @@ static void alc_fill_eapd_coef(struct hda_codec *codec) case 0x10ec0672: alc_update_coef_idx(codec, 0xd, 0, 1<<14); /* EAPD Ctrl */ break; + case 0x10ec0623: + alc_update_coef_idx(codec, 0x19, 1<<13, 0); + break; case 0x10ec0668: alc_update_coef_idx(codec, 0x7, 3<<13, 0); break; @@ -2920,6 +2923,7 @@ enum { ALC269_TYPE_ALC225, ALC269_TYPE_ALC294, ALC269_TYPE_ALC300, + ALC269_TYPE_ALC623, ALC269_TYPE_ALC700, }; @@ -2955,6 +2959,7 @@ static int alc269_parse_auto_config(struct hda_codec *codec) case ALC269_TYPE_ALC225: case ALC269_TYPE_ALC294: case ALC269_TYPE_ALC300: + case ALC269_TYPE_ALC623: case ALC269_TYPE_ALC700: ssids = alc269_ssids; break; @@ -8017,6 +8022,9 @@ static int patch_alc269(struct hda_codec *codec) spec->codec_variant = ALC269_TYPE_ALC300; spec->gen.mixer_nid = 0; /* no loopback on ALC300 */ break; + case 0x10ec0623: + spec->codec_variant = ALC269_TYPE_ALC623; + break; case 0x10ec0700: case 0x10ec0701: case 0x10ec0703: @@ -9218,6 +9226,7 @@ static const struct hda_device_id snd_hda_id_realtek[] = { HDA_CODEC_ENTRY(0x10ec0298, "ALC298", patch_alc269), HDA_CODEC_ENTRY(0x10ec0299, "ALC299", patch_alc269), HDA_CODEC_ENTRY(0x10ec0300, "ALC300", patch_alc269), + HDA_CODEC_ENTRY(0x10ec0623, "ALC623", patch_alc269), HDA_CODEC_REV_ENTRY(0x10ec0861, 0x100340, "ALC660", patch_alc861), HDA_CODEC_ENTRY(0x10ec0660, "ALC660-VD", patch_alc861vd), HDA_CODEC_ENTRY(0x10ec0861, "ALC861", patch_alc861), ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH] ALSA: hda/realtek - Fix 2 front mics of codec 0x623 2019-10-24 7:20 ` Kailang @ 2019-10-24 10:35 ` Takashi Iwai 2019-10-24 10:54 ` Aaron Ma 0 siblings, 1 reply; 11+ messages in thread From: Takashi Iwai @ 2019-10-24 10:35 UTC (permalink / raw) To: Kailang; +Cc: Aaron Ma, perex, hui.wang, alsa-devel, linux-kernel On Thu, 24 Oct 2019 09:20:19 +0200, Kailang wrote: > > Hi Takashi, > > Attach patch was support ALC623. Thanks, applied now. Aaron, could you rebase your patch and resubmit? Takashi > > BR, > Kailang > > > -----Original Message----- > > From: Aaron Ma <aaron.ma@canonical.com> > > Sent: Wednesday, October 23, 2019 6:06 PM > > To: Kailang <kailang@realtek.com>; Takashi Iwai <tiwai@suse.de> > > Cc: perex@perex.cz; hui.wang@canonical.com; alsa-devel@alsa-project.org; > > linux-kernel@vger.kernel.org > > Subject: Re: [PATCH] ALSA: hda/realtek - Fix 2 front mics of codec 0x623 > > > > On 10/23/19 4:44 PM, Kailang wrote: > > > > > > > > >> -----Original Message----- > > >> From: Takashi Iwai <tiwai@suse.de> > > >> Sent: Wednesday, October 23, 2019 12:08 AM > > >> To: Aaron Ma <aaron.ma@canonical.com> > > >> Cc: perex@perex.cz; Kailang <kailang@realtek.com>; > > >> hui.wang@canonical.com; alsa-devel@alsa-project.org; > > >> linux-kernel@vger.kernel.org > > >> Subject: Re: [PATCH] ALSA: hda/realtek - Fix 2 front mics of codec > > >> 0x623 > > >> > > >> On Tue, 22 Oct 2019 17:38:55 +0200, > > >> Aaron Ma wrote: > > >>> > > >>> These 2 ThinkCentres installed a new realtek codec ID 0x623, it has > > >>> 2 front mics with the same location on pin 0x18 and 0x19. > > >>> > > >>> Apply fixup ALC283_FIXUP_HEADSET_MIC to change 1 front mic location > > >>> to right, then pulseaudio can handle them. > > >>> One "Front Mic" and one "Mic" will be shown, and audio output works > > >>> fine. > > >>> > > >>> Signed-off-by: Aaron Ma <aaron.ma@canonical.com> > > >> > > >> I'd like to have Kailang's review about the new codec before applying. > > >> > > >> Kailang, could you take a look? > > > OK. > > > I will post you the patch for ALC623 codec tomorrow. > > > Thanks. > > > > Cc me too. > > > > Thank you. > > Aaron > > > > > > > >> > > >> > > >> thanks, > > >> > > >> Takashi > > >> > > >>> --- > > >>> sound/pci/hda/patch_realtek.c | 3 +++ > > >>> 1 file changed, 3 insertions(+) > > >>> > > >>> diff --git a/sound/pci/hda/patch_realtek.c > > >>> b/sound/pci/hda/patch_realtek.c index b000b36ac3c6..c34d8b435f58 > > >>> 100644 > > >>> --- a/sound/pci/hda/patch_realtek.c > > >>> +++ b/sound/pci/hda/patch_realtek.c > > >>> @@ -7186,6 +7186,8 @@ static const struct snd_pci_quirk > > >>> alc269_fixup_tbl[] > > >> = { > > >>> SND_PCI_QUIRK(0x17aa, 0x312f, "ThinkCentre Station", > > >> ALC294_FIXUP_LENOVO_MIC_LOCATION), > > >>> SND_PCI_QUIRK(0x17aa, 0x313c, "ThinkCentre Station", > > >> ALC294_FIXUP_LENOVO_MIC_LOCATION), > > >>> SND_PCI_QUIRK(0x17aa, 0x3151, "ThinkCentre Station", > > >>> ALC283_FIXUP_HEADSET_MIC), > > >>> + SND_PCI_QUIRK(0x17aa, 0x3178, "ThinkCentre Station", > > >> ALC283_FIXUP_HEADSET_MIC), > > >>> + SND_PCI_QUIRK(0x17aa, 0x3176, "ThinkCentre Station", > > >>> +ALC283_FIXUP_HEADSET_MIC), > > >>> SND_PCI_QUIRK(0x17aa, 0x3902, "Lenovo E50-80", > > >> ALC269_FIXUP_DMIC_THINKPAD_ACPI), > > >>> SND_PCI_QUIRK(0x17aa, 0x3977, "IdeaPad S210", > > >> ALC283_FIXUP_INT_MIC), > > >>> SND_PCI_QUIRK(0x17aa, 0x3978, "Lenovo B50-70", > > >>> ALC269_FIXUP_DMIC_THINKPAD_ACPI), @@ -9187,6 +9189,7 @@ static > > >> const struct hda_device_id snd_hda_id_realtek[] = { > > >>> HDA_CODEC_ENTRY(0x10ec0298, "ALC298", patch_alc269), > > >>> HDA_CODEC_ENTRY(0x10ec0299, "ALC299", patch_alc269), > > >>> HDA_CODEC_ENTRY(0x10ec0300, "ALC300", patch_alc269), > > >>> + HDA_CODEC_ENTRY(0x10ec0623, "ALC623", patch_alc269), > > >>> HDA_CODEC_REV_ENTRY(0x10ec0861, 0x100340, "ALC660", > > >> patch_alc861), > > >>> HDA_CODEC_ENTRY(0x10ec0660, "ALC660-VD", patch_alc861vd), > > >>> HDA_CODEC_ENTRY(0x10ec0861, "ALC861", patch_alc861), > > >>> -- > > >>> 2.17.1 > > >>> > > >> > > >> ------Please consider the environment before printing this e-mail. > [2 0000-add-support-alc623.patch <application/octet-stream (base64)>] > ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PATCH] ALSA: hda/realtek - Fix 2 front mics of codec 0x623 2019-10-24 10:35 ` Takashi Iwai @ 2019-10-24 10:54 ` Aaron Ma 0 siblings, 0 replies; 11+ messages in thread From: Aaron Ma @ 2019-10-24 10:54 UTC (permalink / raw) To: Takashi Iwai, Kailang; +Cc: perex, hui.wang, alsa-devel, linux-kernel On 10/24/19 6:35 PM, Takashi Iwai wrote: > On Thu, 24 Oct 2019 09:20:19 +0200, > Kailang wrote: >> >> Hi Takashi, >> >> Attach patch was support ALC623. > > Thanks, applied now. > > Aaron, could you rebase your patch and resubmit? > > Yes I will. One line of same changes. Thanks, Aaron > Takashi > >> >> BR, >> Kailang >> >>> -----Original Message----- >>> From: Aaron Ma <aaron.ma@canonical.com> >>> Sent: Wednesday, October 23, 2019 6:06 PM >>> To: Kailang <kailang@realtek.com>; Takashi Iwai <tiwai@suse.de> >>> Cc: perex@perex.cz; hui.wang@canonical.com; alsa-devel@alsa-project.org; >>> linux-kernel@vger.kernel.org >>> Subject: Re: [PATCH] ALSA: hda/realtek - Fix 2 front mics of codec 0x623 >>> >>> On 10/23/19 4:44 PM, Kailang wrote: >>>> >>>> >>>>> -----Original Message----- >>>>> From: Takashi Iwai <tiwai@suse.de> >>>>> Sent: Wednesday, October 23, 2019 12:08 AM >>>>> To: Aaron Ma <aaron.ma@canonical.com> >>>>> Cc: perex@perex.cz; Kailang <kailang@realtek.com>; >>>>> hui.wang@canonical.com; alsa-devel@alsa-project.org; >>>>> linux-kernel@vger.kernel.org >>>>> Subject: Re: [PATCH] ALSA: hda/realtek - Fix 2 front mics of codec >>>>> 0x623 >>>>> >>>>> On Tue, 22 Oct 2019 17:38:55 +0200, >>>>> Aaron Ma wrote: >>>>>> >>>>>> These 2 ThinkCentres installed a new realtek codec ID 0x623, it has >>>>>> 2 front mics with the same location on pin 0x18 and 0x19. >>>>>> >>>>>> Apply fixup ALC283_FIXUP_HEADSET_MIC to change 1 front mic location >>>>>> to right, then pulseaudio can handle them. >>>>>> One "Front Mic" and one "Mic" will be shown, and audio output works >>>>>> fine. >>>>>> >>>>>> Signed-off-by: Aaron Ma <aaron.ma@canonical.com> >>>>> >>>>> I'd like to have Kailang's review about the new codec before applying. >>>>> >>>>> Kailang, could you take a look? >>>> OK. >>>> I will post you the patch for ALC623 codec tomorrow. >>>> Thanks. >>> >>> Cc me too. >>> >>> Thank you. >>> Aaron >>> >>>> >>>>> >>>>> >>>>> thanks, >>>>> >>>>> Takashi >>>>> >>>>>> --- >>>>>> sound/pci/hda/patch_realtek.c | 3 +++ >>>>>> 1 file changed, 3 insertions(+) >>>>>> >>>>>> diff --git a/sound/pci/hda/patch_realtek.c >>>>>> b/sound/pci/hda/patch_realtek.c index b000b36ac3c6..c34d8b435f58 >>>>>> 100644 >>>>>> --- a/sound/pci/hda/patch_realtek.c >>>>>> +++ b/sound/pci/hda/patch_realtek.c >>>>>> @@ -7186,6 +7186,8 @@ static const struct snd_pci_quirk >>>>>> alc269_fixup_tbl[] >>>>> = { >>>>>> SND_PCI_QUIRK(0x17aa, 0x312f, "ThinkCentre Station", >>>>> ALC294_FIXUP_LENOVO_MIC_LOCATION), >>>>>> SND_PCI_QUIRK(0x17aa, 0x313c, "ThinkCentre Station", >>>>> ALC294_FIXUP_LENOVO_MIC_LOCATION), >>>>>> SND_PCI_QUIRK(0x17aa, 0x3151, "ThinkCentre Station", >>>>>> ALC283_FIXUP_HEADSET_MIC), >>>>>> + SND_PCI_QUIRK(0x17aa, 0x3178, "ThinkCentre Station", >>>>> ALC283_FIXUP_HEADSET_MIC), >>>>>> + SND_PCI_QUIRK(0x17aa, 0x3176, "ThinkCentre Station", >>>>>> +ALC283_FIXUP_HEADSET_MIC), >>>>>> SND_PCI_QUIRK(0x17aa, 0x3902, "Lenovo E50-80", >>>>> ALC269_FIXUP_DMIC_THINKPAD_ACPI), >>>>>> SND_PCI_QUIRK(0x17aa, 0x3977, "IdeaPad S210", >>>>> ALC283_FIXUP_INT_MIC), >>>>>> SND_PCI_QUIRK(0x17aa, 0x3978, "Lenovo B50-70", >>>>>> ALC269_FIXUP_DMIC_THINKPAD_ACPI), @@ -9187,6 +9189,7 @@ static >>>>> const struct hda_device_id snd_hda_id_realtek[] = { >>>>>> HDA_CODEC_ENTRY(0x10ec0298, "ALC298", patch_alc269), >>>>>> HDA_CODEC_ENTRY(0x10ec0299, "ALC299", patch_alc269), >>>>>> HDA_CODEC_ENTRY(0x10ec0300, "ALC300", patch_alc269), >>>>>> + HDA_CODEC_ENTRY(0x10ec0623, "ALC623", patch_alc269), >>>>>> HDA_CODEC_REV_ENTRY(0x10ec0861, 0x100340, "ALC660", >>>>> patch_alc861), >>>>>> HDA_CODEC_ENTRY(0x10ec0660, "ALC660-VD", patch_alc861vd), >>>>>> HDA_CODEC_ENTRY(0x10ec0861, "ALC861", patch_alc861), >>>>>> -- >>>>>> 2.17.1 >>>>>> >>>>> >>>>> ------Please consider the environment before printing this e-mail. >> [2 0000-add-support-alc623.patch <application/octet-stream (base64)>] >> ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH v2] ALSA: hda/realtek - Fix 2 front mics of codec 0x623 2019-10-22 15:38 [PATCH] ALSA: hda/realtek - Fix 2 front mics of codec 0x623 Aaron Ma 2019-10-22 16:08 ` Takashi Iwai @ 2019-10-24 11:18 ` Aaron Ma 2019-10-24 11:30 ` Hui Wang 2019-10-24 11:44 ` [PATCH v3] " Aaron Ma 2 siblings, 1 reply; 11+ messages in thread From: Aaron Ma @ 2019-10-24 11:18 UTC (permalink / raw) To: perex, tiwai, kailang, hui.wang, alsa-devel, aaron.ma, linux-kernel These 2 ThinkCentres installed a new realtek codec ID 0x623, it has 2 front mics with the same location on pin 0x18 and 0x19. Apply fixup ALC283_FIXUP_HEADSET_MIC to change 1 front mic location to right, then pulseaudio can handle them. One "Front Mic" and one "Mic" will be shown, and audio output works fine. Signed-off-by: Aaron Ma <aaron.ma@canonical.com> --- sound/pci/hda/patch_realtek.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index a0c237cc13d4..173a7867bb45 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -7221,6 +7221,8 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = { SND_PCI_QUIRK(0x17aa, 0x312f, "ThinkCentre Station", ALC294_FIXUP_LENOVO_MIC_LOCATION), SND_PCI_QUIRK(0x17aa, 0x313c, "ThinkCentre Station", ALC294_FIXUP_LENOVO_MIC_LOCATION), SND_PCI_QUIRK(0x17aa, 0x3151, "ThinkCentre Station", ALC283_FIXUP_HEADSET_MIC), + SND_PCI_QUIRK(0x17aa, 0x3178, "ThinkCentre Station", ALC283_FIXUP_HEADSET_MIC), + SND_PCI_QUIRK(0x17aa, 0x3176, "ThinkCentre Station", ALC283_FIXUP_HEADSET_MIC), SND_PCI_QUIRK(0x17aa, 0x3902, "Lenovo E50-80", ALC269_FIXUP_DMIC_THINKPAD_ACPI), SND_PCI_QUIRK(0x17aa, 0x3977, "IdeaPad S210", ALC283_FIXUP_INT_MIC), SND_PCI_QUIRK(0x17aa, 0x3978, "Lenovo B50-70", ALC269_FIXUP_DMIC_THINKPAD_ACPI), -- 2.17.1 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH v2] ALSA: hda/realtek - Fix 2 front mics of codec 0x623 2019-10-24 11:18 ` [PATCH v2] " Aaron Ma @ 2019-10-24 11:30 ` Hui Wang 0 siblings, 0 replies; 11+ messages in thread From: Hui Wang @ 2019-10-24 11:30 UTC (permalink / raw) To: Aaron Ma, perex, tiwai, kailang, alsa-devel, linux-kernel On 2019/10/24 下午7:18, Aaron Ma wrote: > These 2 ThinkCentres installed a new realtek codec ID 0x623, > it has 2 front mics with the same location on pin 0x18 and 0x19. > > Apply fixup ALC283_FIXUP_HEADSET_MIC to change 1 front mic > location to right, then pulseaudio can handle them. > One "Front Mic" and one "Mic" will be shown, and audio output works > fine. > > Signed-off-by: Aaron Ma <aaron.ma@canonical.com> > --- > sound/pci/hda/patch_realtek.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c > index a0c237cc13d4..173a7867bb45 100644 > --- a/sound/pci/hda/patch_realtek.c > +++ b/sound/pci/hda/patch_realtek.c > @@ -7221,6 +7221,8 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = { > SND_PCI_QUIRK(0x17aa, 0x312f, "ThinkCentre Station", ALC294_FIXUP_LENOVO_MIC_LOCATION), > SND_PCI_QUIRK(0x17aa, 0x313c, "ThinkCentre Station", ALC294_FIXUP_LENOVO_MIC_LOCATION), > SND_PCI_QUIRK(0x17aa, 0x3151, "ThinkCentre Station", ALC283_FIXUP_HEADSET_MIC), > + SND_PCI_QUIRK(0x17aa, 0x3178, "ThinkCentre Station", ALC283_FIXUP_HEADSET_MIC), > + SND_PCI_QUIRK(0x17aa, 0x3176, "ThinkCentre Station", ALC283_FIXUP_HEADSET_MIC), Please sort them in numerical order, 0x3176 is ahead of 0x3178. thanks > SND_PCI_QUIRK(0x17aa, 0x3902, "Lenovo E50-80", ALC269_FIXUP_DMIC_THINKPAD_ACPI), > SND_PCI_QUIRK(0x17aa, 0x3977, "IdeaPad S210", ALC283_FIXUP_INT_MIC), > SND_PCI_QUIRK(0x17aa, 0x3978, "Lenovo B50-70", ALC269_FIXUP_DMIC_THINKPAD_ACPI), ^ permalink raw reply [flat|nested] 11+ messages in thread
* [PATCH v3] ALSA: hda/realtek - Fix 2 front mics of codec 0x623 2019-10-22 15:38 [PATCH] ALSA: hda/realtek - Fix 2 front mics of codec 0x623 Aaron Ma 2019-10-22 16:08 ` Takashi Iwai 2019-10-24 11:18 ` [PATCH v2] " Aaron Ma @ 2019-10-24 11:44 ` Aaron Ma 2019-10-24 12:23 ` Takashi Iwai 2 siblings, 1 reply; 11+ messages in thread From: Aaron Ma @ 2019-10-24 11:44 UTC (permalink / raw) To: perex, tiwai, kailang, hui.wang, alsa-devel, aaron.ma, linux-kernel These 2 ThinkCentres installed a new realtek codec ID 0x623, it has 2 front mics with the same location on pin 0x18 and 0x19. Apply fixup ALC283_FIXUP_HEADSET_MIC to change 1 front mic location to right, then pulseaudio can handle them. One "Front Mic" and one "Mic" will be shown, and audio output works fine. Signed-off-by: Aaron Ma <aaron.ma@canonical.com> --- sound/pci/hda/patch_realtek.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c index a0c237cc13d4..80f66ba85f87 100644 --- a/sound/pci/hda/patch_realtek.c +++ b/sound/pci/hda/patch_realtek.c @@ -7221,6 +7221,8 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = { SND_PCI_QUIRK(0x17aa, 0x312f, "ThinkCentre Station", ALC294_FIXUP_LENOVO_MIC_LOCATION), SND_PCI_QUIRK(0x17aa, 0x313c, "ThinkCentre Station", ALC294_FIXUP_LENOVO_MIC_LOCATION), SND_PCI_QUIRK(0x17aa, 0x3151, "ThinkCentre Station", ALC283_FIXUP_HEADSET_MIC), + SND_PCI_QUIRK(0x17aa, 0x3176, "ThinkCentre Station", ALC283_FIXUP_HEADSET_MIC), + SND_PCI_QUIRK(0x17aa, 0x3178, "ThinkCentre Station", ALC283_FIXUP_HEADSET_MIC), SND_PCI_QUIRK(0x17aa, 0x3902, "Lenovo E50-80", ALC269_FIXUP_DMIC_THINKPAD_ACPI), SND_PCI_QUIRK(0x17aa, 0x3977, "IdeaPad S210", ALC283_FIXUP_INT_MIC), SND_PCI_QUIRK(0x17aa, 0x3978, "Lenovo B50-70", ALC269_FIXUP_DMIC_THINKPAD_ACPI), -- 2.17.1 ^ permalink raw reply related [flat|nested] 11+ messages in thread
* Re: [PATCH v3] ALSA: hda/realtek - Fix 2 front mics of codec 0x623 2019-10-24 11:44 ` [PATCH v3] " Aaron Ma @ 2019-10-24 12:23 ` Takashi Iwai 0 siblings, 0 replies; 11+ messages in thread From: Takashi Iwai @ 2019-10-24 12:23 UTC (permalink / raw) To: Aaron Ma; +Cc: perex, tiwai, kailang, hui.wang, alsa-devel, linux-kernel On Thu, 24 Oct 2019 13:44:39 +0200, Aaron Ma wrote: > > These 2 ThinkCentres installed a new realtek codec ID 0x623, > it has 2 front mics with the same location on pin 0x18 and 0x19. > > Apply fixup ALC283_FIXUP_HEADSET_MIC to change 1 front mic > location to right, then pulseaudio can handle them. > One "Front Mic" and one "Mic" will be shown, and audio output works > fine. > > Signed-off-by: Aaron Ma <aaron.ma@canonical.com> Applied, thanks. Takashi ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2019-10-24 12:23 UTC | newest] Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-10-22 15:38 [PATCH] ALSA: hda/realtek - Fix 2 front mics of codec 0x623 Aaron Ma 2019-10-22 16:08 ` Takashi Iwai 2019-10-23 8:44 ` Kailang 2019-10-23 10:06 ` Aaron Ma 2019-10-24 7:20 ` Kailang 2019-10-24 10:35 ` Takashi Iwai 2019-10-24 10:54 ` Aaron Ma 2019-10-24 11:18 ` [PATCH v2] " Aaron Ma 2019-10-24 11:30 ` Hui Wang 2019-10-24 11:44 ` [PATCH v3] " Aaron Ma 2019-10-24 12:23 ` Takashi Iwai
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).