Linux Sound subsystem development
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: John Veness <john-linux@pelago.org.uk>
Cc: Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>,
	linux-sound@vger.kernel.org
Subject: Re: [PATCH] ALSA: hda/conexant: Add quirk for HP ProBook 450 G4 mute LED
Date: Mon, 17 Feb 2025 10:56:44 +0100	[thread overview]
Message-ID: <87ldu499nn.wl-tiwai@suse.de> (raw)
In-Reply-To: <5eb4305d-48b2-43a4-a30e-2bbe8d409146@pelago.org.uk>

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
> 

  reply	other threads:[~2025-02-17  9:56 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
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 [this message]
2025-02-17 12:01   ` John Veness

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=87ldu499nn.wl-tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=john-linux@pelago.org.uk \
    --cc=linux-sound@vger.kernel.org \
    --cc=perex@perex.cz \
    --cc=tiwai@suse.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