All of lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Preston Lam <plamlam99@gmail.com>
Cc: Takashi Iwai <tiwai@suse.com>, Jaroslav Kysela <perex@perex.cz>,
	linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] ALSA: hda/realtek: Add mute LED quirk for HP Laptop 15-fd2xxx
Date: Tue, 29 Sep 2026 07:31:52 +0200	[thread overview]
Message-ID: <87wls439o7.wl-tiwai@suse.de> (raw)
In-Reply-To: <20260928231621.1672704-1-plamlam99@gmail.com>

On Tue, 29 Sep 2026 01:16:21 +0200,
Preston Lam wrote:
> 
> The HP Laptop 15-fd2xxx (PCI SSID 103c:8e43, Realtek ALC236) has a
> speaker mute LED on the F5 key and a microphone mute LED on the F8 key.
> Neither lights up, because the model has no quirk entry and only the
> generic HP vendor fallback (ALC269_FIXUP_HP_MUTE_LED) is applied.
> 
> Probing the codec with hda-verb showed how the LEDs are wired:
> 
>  - mute LED: COEF index 0x07, bit 0, on NID 0x20 (set = LED on)
>  - mic mute LED: codec GPIO0, active low (driven low = LED on)
> 
> Driving GPIO1 and GPIO2 did not light the mute LED. This wiring is
> exactly what the existing ALC236_FIXUP_HP_MUTE_LED_MICMUTE_GPIO fixup
> implements (COEF 0x07 bit 0 for the mute LED, GPIO0 with inverted
> polarity for the micmute LED), so use it for this model. The sibling
> HP Laptop 15-fd0xxx (103c:8dd7) already uses the same fixup.
> 
> Tested on this machine running Ubuntu 7.0.0-34-generic (SOF driver, the
> quirk built into snd-hda-codec-alc269 from Ubuntu's 7.0.0-34.34 source
> tree): the hda::mute and hda::micmute LEDs are created, follow the
> Master mute state and the Dmic0 capture switch respectively, and the
> F5 and F8 keys toggle them correctly. Muting the microphone also
> silences the capture stream. The LEDs also kept the correct state across
> an s2idle suspend/resume cycle (speaker muted in one run, microphone in
> another) and still responded to mute changes afterwards; long suspends
> were not tested. The desktop session was SwayFX (wlroots) with PipeWire,
> which only changes the mute state through the normal mixer controls; no
> other desktop was tried. Not tested on a mainline kernel tree directly;
> the change is only the quirk table entry.
> 
> AI assistance: this patch, the hardware analysis behind it (probing the
> codec with hda-verb and choosing the existing fixup) and this commit
> message were written by Claude Code (Anthropic, model claude-sonnet-5-5).
> The submitter ran the privileged commands and reboots and reported the
> LED behavior; Claude Code ran the tests and read the kernel logs.
> 
> Assisted-by: Claude:claude-sonnet-5-5
> Signed-off-by: Preston Lam <plamlam99@gmail.com>

Applied now.  Thanks.


Takashi

      reply	other threads:[~2026-09-29  5:32 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-28 23:16 [PATCH] ALSA: hda/realtek: Add mute LED quirk for HP Laptop 15-fd2xxx Preston Lam
2026-09-29  5:31 ` Takashi Iwai [this message]

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=87wls439o7.wl-tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-sound@vger.kernel.org \
    --cc=perex@perex.cz \
    --cc=plamlam99@gmail.com \
    --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 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.