[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <DS0PR84MB341799A29B99290A1FB10F58BB4D2@DS0PR84MB3417.NAMPRD84.PROD.OUTLOOK.COM>
Date: Thu, 15 Feb 2024 15:25:05 +0000
From: "Zhang, Eniac" <eniac-xw.zhang@...com>
To: Takashi Iwai <tiwai@...e.de>, "Gagniuc, Alexandru"
<alexandru.gagniuc@...com>
CC: "linux-sound@...r.kernel.org" <linux-sound@...r.kernel.org>,
"perex@...ex.cz" <perex@...ex.cz>, "tiwai@...e.com" <tiwai@...e.com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, Eniac Zhang
<eniacz@...il.com>, Alexandru Gagniuc <mr.nuke.me@...il.com>,
"stable@...r.kernel.org" <stable@...r.kernel.org>
Subject: RE: [PATCH] ALSA: hda/realtek: fix mute/micmute LED For HP mt645
Hi Takashi,
I can sign off this merge request. Alex is ThinPro's new kernel maintainer He is trying to push all those old HP patches upstream to make our life (and other HP machine user's life) easier.
Let me know if there's anything else I can do.
Regards/Eniac
-----Original Message-----
From: Takashi Iwai <tiwai@...e.de>
Sent: Thursday, February 15, 2024 5:50 AM
To: Gagniuc, Alexandru <alexandru.gagniuc@...com>
Cc: linux-sound@...r.kernel.org; perex@...ex.cz; tiwai@...e.com; linux-kernel@...r.kernel.org; Zhang, Eniac <eniac-xw.zhang@...com>; Eniac Zhang <eniacz@...il.com>; Alexandru Gagniuc <mr.nuke.me@...il.com>; stable@...r.kernelorg
Subject: Re: [PATCH] ALSA: hda/realtek: fix mute/micmute LED For HP mt645
CAUTION: External Email
On Wed, 14 Feb 2024 19:45:07 +0100,
Alexandru Gagniuc wrote:
>
> From: Eniac Zhang <eniacz@...il.com>
>
> The HP mt645 G7 Thin Client uses an ALC236 codec and needs the
> ALC236_FIXUP_HP_MUTE_LED_MICMUTE_VREF quirk to make the mute and
> micmute LEDs work.
>
> There are two variants of the USB-C PD chip on this device. Each uses
> a different BIOS and board ID, hence the two entries.
>
> Signed-off-by: Alexandru Gagniuc <mr.nuke.me@...il.com>
> Signed-off-by: Alexandru Gagniuc <alexandru.gagniuc@...com>
Any reason to have two your sign-offs?
Also, can we get a sign-off from the original author?
thanks,
Takashi
> Cc: <stable@...r.kernel.org>
> ---
> 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 6994c4c5073c..c837470ef5b8
> 100644
> --- a/sound/pci/hda/patch_realtek.c
> +++ b/sound/pci/hda/patch_realtek.c
> @@ -9928,6 +9928,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = {
> SND_PCI_QUIRK(0x103c, 0x8abb, "HP ZBook Firefly 14 G9", ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED),
> SND_PCI_QUIRK(0x103c, 0x8ad1, "HP EliteBook 840 14 inch G9 Notebook PC", ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED),
> SND_PCI_QUIRK(0x103c, 0x8ad2, "HP EliteBook 860 16 inch G9
> Notebook PC", ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED),
> + SND_PCI_QUIRK(0x103c, 0x8b0f, "HP Elite mt645 G7 Mobile Thin
> + Client U81", ALC236_FIXUP_HP_MUTE_LED_MICMUTE_VREF),
> SND_PCI_QUIRK(0x103c, 0x8b2f, "HP 255 15.6 inch G10 Notebook PC", ALC236_FIXUP_HP_MUTE_LED_COEFBIT2),
> SND_PCI_QUIRK(0x103c, 0x8b42, "HP", ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED),
> SND_PCI_QUIRK(0x103c, 0x8b43, "HP",
> ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED),
> @@ -9935,6 +9936,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = {
> SND_PCI_QUIRK(0x103c, 0x8b45, "HP", ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED),
> SND_PCI_QUIRK(0x103c, 0x8b46, "HP", ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED),
> SND_PCI_QUIRK(0x103c, 0x8b47, "HP",
> ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED),
> + SND_PCI_QUIRK(0x103c, 0x8b59, "HP Elite mt645 G7 Mobile Thin
> + Client U89", ALC236_FIXUP_HP_MUTE_LED_MICMUTE_VREF),
> SND_PCI_QUIRK(0x103c, 0x8b5d, "HP", ALC236_FIXUP_HP_MUTE_LED_MICMUTE_VREF),
> SND_PCI_QUIRK(0x103c, 0x8b5e, "HP", ALC236_FIXUP_HP_MUTE_LED_MICMUTE_VREF),
> SND_PCI_QUIRK(0x103c, 0x8b63, "HP Elite Dragonfly 13.5 inch G4",
> ALC245_FIXUP_CS35L41_SPI_4_HP_GPIO_LED),
> --
> 2.42.0
>
Powered by blists - more mailing lists