linux-kernel - Re: [PATCH] ALSA: hda/realtek: fix mute/micmute LEDs for a HP ZBook

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <ZZOQamQsHUR7eEPd@geday>
Date: Tue, 2 Jan 2024 01:26:18 -0300
From: Geraldo Nascimento <geraldogabriel@...il.com>
To: Andy Chi <andy.chi@...onical.com>
Cc: Jaroslav Kysela <perex@...ex.cz>, Takashi Iwai <tiwai@...e.com>,
	Stefan Binding <sbinding@...nsource.cirrus.com>,
	Kailang Yang <kailang@...ltek.com>,
	"Luke D. Jones" <luke@...nes.dev>,
	Shenghao Ding <shenghao-ding@...com>,
	Matthew Anderson <ruinairas1992@...il.com>,
	Luka Guzenko <l.guzenko@....de>, Yuchi Yang <yangyuchi66@...il.com>,
	Vitaly Rodionov <vitalyr@...nsource.cirrus.com>,
	linux-sound@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH] ALSA: hda/realtek: fix mute/micmute LEDs for a HP ZBook

On Tue, Jan 02, 2024 at 10:49:15AM +0800, Andy Chi wrote:
> There is a HP ZBook which using ALC236 codec and need the
> ALC236_FIXUP_HP_MUTE_LED_MICMUTE_VREF quirk to make mute LED
> and micmute LED work.

Hi Andy,

I know Takashi in the name of smooth operation usually takes this kind
of simple patch, quickly, since it's a simple bugfix, unless there are
any objections, so I must raise one.

Which HP ZBook? Which model?

Is this a still unreleased model Canonical has had access under a NDA
for example? Or perhaps there is some other reason why you cannot name
the model yet?

Either way, there is a need to publicly know the reason for the missing
model information.

> 
> Signed-off-by: Andy Chi <andy.chi@...onical.com>
> ---
>  sound/pci/hda/patch_realtek.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
> index 7e9d8c1cedfc..1c8d5b58eb64 100644
> --- a/sound/pci/hda/patch_realtek.c
> +++ b/sound/pci/hda/patch_realtek.c
> @@ -9939,6 +9939,7 @@ static const struct snd_pci_quirk alc269_fixup_tbl[] = {
>  	SND_PCI_QUIRK(0x103c, 0x8c70, "HP EliteBook 835 G11", ALC287_FIXUP_CS35L41_I2C_2_HP_GPIO_LED),
>  	SND_PCI_QUIRK(0x103c, 0x8c71, "HP EliteBook 845 G11", ALC287_FIXUP_CS35L41_I2C_2_HP_GPIO_LED),
>  	SND_PCI_QUIRK(0x103c, 0x8c72, "HP EliteBook 865 G11", ALC287_FIXUP_CS35L41_I2C_2_HP_GPIO_LED),
> +	SND_PCI_QUIRK(0x103c, 0x8c96, "HP", ALC236_FIXUP_HP_MUTE_LED_MICMUTE_VREF),

Same questioning as above. "HP" is IMHO unacceptable without a clear
reason for the missing model information.

Thanks,
Geraldo Nascimento

>  	SND_PCI_QUIRK(0x103c, 0x8ca4, "HP ZBook Fury", ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED),
>  	SND_PCI_QUIRK(0x103c, 0x8ca7, "HP ZBook Fury", ALC245_FIXUP_CS35L41_SPI_2_HP_GPIO_LED),
>  	SND_PCI_QUIRK(0x103c, 0x8cf5, "HP ZBook Studio 16", ALC245_FIXUP_CS35L41_SPI_4_HP_GPIO_LED),
> -- 
> 2.34.1
> 
>