* [PATCH] ALSA: hda/conexant: Add quirk for HP ProBook 450 G4 mute LED
@ 2025-02-16 23:21 John Veness
2025-02-17 9:56 ` Takashi Iwai
0 siblings, 1 reply; 4+ messages in thread
From: John Veness @ 2025-02-16 23:21 UTC (permalink / raw)
To: Jaroslav Kysela, Takashi Iwai, linux-sound
Allows the LED on the dedicated mute button on the HP ProBook 450 G4 laptop
to change colour correctly.
Signed-off-by: John Veness <john-linux@pelago.org.uk>
---
I have only tested this with 6.13, but I expect it could be applied to older
stable kernels too. The HP ProBook 450 G4 laptop was released in 2017.
sound/pci/hda/patch_conexant.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/sound/pci/hda/patch_conexant.c b/sound/pci/hda/patch_conexant.c
index 4985e72b9..34874039a 100644
--- a/sound/pci/hda/patch_conexant.c
+++ b/sound/pci/hda/patch_conexant.c
@@ -1090,6 +1090,7 @@ static const struct hda_quirk cxt5066_fixups[] = {
SND_PCI_QUIRK(0x103c, 0x814f, "HP ZBook 15u G3", CXT_FIXUP_MUTE_LED_GPIO),
SND_PCI_QUIRK(0x103c, 0x8174, "HP Spectre x360", CXT_FIXUP_HP_SPECTRE),
SND_PCI_QUIRK(0x103c, 0x822e, "HP ProBook 440 G4", CXT_FIXUP_MUTE_LED_GPIO),
+ SND_PCI_QUIRK(0x103c, 0x8231, "HP ProBook 450 G4", CXT_FIXUP_MUTE_LED_GPIO),
SND_PCI_QUIRK(0x103c, 0x828c, "HP EliteBook 840 G4", CXT_FIXUP_HP_DOCK),
SND_PCI_QUIRK(0x103c, 0x8299, "HP 800 G3 SFF", CXT_FIXUP_HP_MIC_NO_PRESENCE),
SND_PCI_QUIRK(0x103c, 0x829a, "HP 800 G3 DM", CXT_FIXUP_HP_MIC_NO_PRESENCE),
--
2.48.1
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] ALSA: hda/conexant: Add quirk for HP ProBook 450 G4 mute LED
2025-02-16 23:21 [PATCH] ALSA: hda/conexant: Add quirk for HP ProBook 450 G4 mute LED John Veness
@ 2025-02-17 9:56 ` Takashi Iwai
2025-02-17 12:01 ` John Veness
0 siblings, 1 reply; 4+ messages in thread
From: Takashi Iwai @ 2025-02-17 9:56 UTC (permalink / raw)
To: John Veness; +Cc: Jaroslav Kysela, Takashi Iwai, linux-sound
On Mon, 17 Feb 2025 00:21:57 +0100,
John Veness wrote:
>
> Allows the LED on the dedicated mute button on the HP ProBook 450 G4 laptop
> to change colour correctly.
>
> Signed-off-by: John Veness <john-linux@pelago.org.uk>
> ---
> I have only tested this with 6.13, but I expect it could be applied to older
> stable kernels too. The HP ProBook 450 G4 laptop was released in 2017.
It's fine to put Cc to stable in the patch, then.
But, your patch seems inapplicable, likely because your mailer
converted tabs to spaces. Could you fix the mailer setup and
resubmit?
thanks,
Takashi
>
> sound/pci/hda/patch_conexant.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/sound/pci/hda/patch_conexant.c b/sound/pci/hda/patch_conexant.c
> index 4985e72b9..34874039a 100644
> --- a/sound/pci/hda/patch_conexant.c
> +++ b/sound/pci/hda/patch_conexant.c
> @@ -1090,6 +1090,7 @@ static const struct hda_quirk cxt5066_fixups[] = {
> SND_PCI_QUIRK(0x103c, 0x814f, "HP ZBook 15u G3", CXT_FIXUP_MUTE_LED_GPIO),
> SND_PCI_QUIRK(0x103c, 0x8174, "HP Spectre x360", CXT_FIXUP_HP_SPECTRE),
> SND_PCI_QUIRK(0x103c, 0x822e, "HP ProBook 440 G4", CXT_FIXUP_MUTE_LED_GPIO),
> + SND_PCI_QUIRK(0x103c, 0x8231, "HP ProBook 450 G4", CXT_FIXUP_MUTE_LED_GPIO),
> SND_PCI_QUIRK(0x103c, 0x828c, "HP EliteBook 840 G4", CXT_FIXUP_HP_DOCK),
> SND_PCI_QUIRK(0x103c, 0x8299, "HP 800 G3 SFF", CXT_FIXUP_HP_MIC_NO_PRESENCE),
> SND_PCI_QUIRK(0x103c, 0x829a, "HP 800 G3 DM", CXT_FIXUP_HP_MIC_NO_PRESENCE),
> --
> 2.48.1
>
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] ALSA: hda/conexant: Add quirk for HP ProBook 450 G4 mute LED
2025-02-17 9:56 ` Takashi Iwai
@ 2025-02-17 12:01 ` John Veness
0 siblings, 0 replies; 4+ messages in thread
From: John Veness @ 2025-02-17 12:01 UTC (permalink / raw)
To: Takashi Iwai; +Cc: Jaroslav Kysela, Takashi Iwai, linux-sound
On 17/02/2025 09:56, Takashi Iwai wrote:
> On Mon, 17 Feb 2025 00:21:57 +0100,
> John Veness wrote:
>>
>> Allows the LED on the dedicated mute button on the HP ProBook 450 G4 laptop
>> to change colour correctly.
>>
>> Signed-off-by: John Veness <john-linux@pelago.org.uk>
>> ---
>> I have only tested this with 6.13, but I expect it could be applied to older
>> stable kernels too. The HP ProBook 450 G4 laptop was released in 2017.
>
> It's fine to put Cc to stable in the patch, then.
>
> But, your patch seems inapplicable, likely because your mailer
> converted tabs to spaces. Could you fix the mailer setup and
> resubmit?
Ah darn it! I think this was a copy-and-paste issue. I'll resubmit with
tabs, and CC stable too. Thanks for the feedback.
John
> thanks,
>
> Takashi
>
>>
>> sound/pci/hda/patch_conexant.c | 1 +
>> 1 file changed, 1 insertion(+)
>>
>> diff --git a/sound/pci/hda/patch_conexant.c b/sound/pci/hda/patch_conexant.c
>> index 4985e72b9..34874039a 100644
>> --- a/sound/pci/hda/patch_conexant.c
>> +++ b/sound/pci/hda/patch_conexant.c
>> @@ -1090,6 +1090,7 @@ static const struct hda_quirk cxt5066_fixups[] = {
>> SND_PCI_QUIRK(0x103c, 0x814f, "HP ZBook 15u G3", CXT_FIXUP_MUTE_LED_GPIO),
>> SND_PCI_QUIRK(0x103c, 0x8174, "HP Spectre x360", CXT_FIXUP_HP_SPECTRE),
>> SND_PCI_QUIRK(0x103c, 0x822e, "HP ProBook 440 G4", CXT_FIXUP_MUTE_LED_GPIO),
>> + SND_PCI_QUIRK(0x103c, 0x8231, "HP ProBook 450 G4", CXT_FIXUP_MUTE_LED_GPIO),
>> SND_PCI_QUIRK(0x103c, 0x828c, "HP EliteBook 840 G4", CXT_FIXUP_HP_DOCK),
>> SND_PCI_QUIRK(0x103c, 0x8299, "HP 800 G3 SFF", CXT_FIXUP_HP_MIC_NO_PRESENCE),
>> SND_PCI_QUIRK(0x103c, 0x829a, "HP 800 G3 DM", CXT_FIXUP_HP_MIC_NO_PRESENCE),
>> --
>> 2.48.1
>>
^ permalink raw reply [flat|nested] 4+ messages in thread
* [PATCH] ALSA: hda/conexant: Add quirk for HP ProBook 450 G4 mute LED
@ 2025-02-10 15:50 John Veness
0 siblings, 0 replies; 4+ messages in thread
From: John Veness @ 2025-02-10 15:50 UTC (permalink / raw)
To: alsa-devel
Allows the LED on the dedicated mute button on the HP ProBook 450 G4 laptop
to change colour correctly.
Signed-off-by: John Veness <john-linux@pelago.org.uk>
---
sound/pci/hda/patch_conexant.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/sound/pci/hda/patch_conexant.c b/sound/pci/hda/patch_conexant.c
index 538c37a78..84ab357b8 100644
--- a/sound/pci/hda/patch_conexant.c
+++ b/sound/pci/hda/patch_conexant.c
@@ -1080,6 +1080,7 @@ static const struct hda_quirk cxt5066_fixups[] = {
SND_PCI_QUIRK(0x103c, 0x814f, "HP ZBook 15u G3", CXT_FIXUP_MUTE_LED_GPIO),
SND_PCI_QUIRK(0x103c, 0x8174, "HP Spectre x360", CXT_FIXUP_HP_SPECTRE),
SND_PCI_QUIRK(0x103c, 0x822e, "HP ProBook 440 G4", CXT_FIXUP_MUTE_LED_GPIO),
+ SND_PCI_QUIRK(0x103c, 0x8231, "HP ProBook 450 G4", CXT_FIXUP_MUTE_LED_GPIO),
SND_PCI_QUIRK(0x103c, 0x828c, "HP EliteBook 840 G4", CXT_FIXUP_HP_DOCK),
SND_PCI_QUIRK(0x103c, 0x8299, "HP 800 G3 SFF", CXT_FIXUP_HP_MIC_NO_PRESENCE),
SND_PCI_QUIRK(0x103c, 0x829a, "HP 800 G3 DM", CXT_FIXUP_HP_MIC_NO_PRESENCE),
--
2.48.1
^ permalink raw reply related [flat|nested] 4+ messages in thread
end of thread, other threads:[~2025-02-17 12:24 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-02-16 23:21 [PATCH] ALSA: hda/conexant: Add quirk for HP ProBook 450 G4 mute LED John Veness
2025-02-17 9:56 ` Takashi Iwai
2025-02-17 12:01 ` John Veness
-- strict thread matches above, loose matches on Subject: below --
2025-02-10 15:50 John Veness
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.