All of lore.kernel.org
 help / color / mirror / Atom feed
From: Takashi Iwai <tiwai@suse.de>
To: Uzair Mughal <contact@uzair.is-a.dev>
Cc: perex@perex.cz, alsa-devel@alsa-project.org,
	linux-sound@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] ALSA: hda/realtek: Add headset jack quirk for Thinkpad X390
Date: Mon, 09 Mar 2026 10:02:01 +0100	[thread overview]
Message-ID: <87ms0htlyu.wl-tiwai@suse.de> (raw)
In-Reply-To: <20260307012906.20093-1-contact@uzair.is-a.dev>

On Sat, 07 Mar 2026 02:29:06 +0100,
Uzair Mughal wrote:
> 
> The Lenovo ThinkPad X390 (ALC257 codec, subsystem ID 0x17aa2288)
> does not report headset button press events. Headphone insertion is
> detected (SW_HEADPHONE_INSERT), but pressing the inline microphone
> button on a headset produces no input events.
> 
> Add a SND_PCI_QUIRK entry that maps this subsystem ID to
> ALC285_FIXUP_THINKPAD_NO_BASS_SPK_HEADSET_JACK, which enables
> headset jack button detection through alc_fixup_headset_jack()
> and ThinkPad ACPI integration. This is the same fixup used by
> similar ThinkPad models (P1 Gen 3, X1 Extreme Gen 3).
> 
> Signed-off-by: Uzair Mughal <contact@uzair.is-a.dev>
> ---
>  sound/hda/codecs/realtek/alc269.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/sound/hda/codecs/realtek/alc269.c b/sound/hda/codecs/realtek/alc269.c
> index XXXXXXX..XXXXXXX 100644
> --- a/sound/hda/codecs/realtek/alc269.c
> +++ b/sound/hda/codecs/realtek/alc269.c
> @@ -6998,6 +6998,7 @@
>  	SND_PCI_QUIRK(0x17aa, 0x224d, "Thinkpad", ALC298_FIXUP_TPT470_DOCK),
>  	SND_PCI_QUIRK(0x17aa, 0x225d, "Thinkpad T480", ALC269_FIXUP_LIMIT_INT_MIC_BOOST),
> +	SND_PCI_QUIRK(0x17aa, 0x2288, "Thinkpad X390", ALC285_FIXUP_THINKPAD_NO_BASS_SPK_HEADSET_JACK),
>  	SND_PCI_QUIRK(0x17aa, 0x2292, "Thinkpad X1 Carbon 7th", ALC285_FIXUP_THINKPAD_HEADSET_JACK),
>  	SND_PCI_QUIRK(0x17aa, 0x22be, "Thinkpad X1 Carbon 8th", ALC285_FIXUP_THINKPAD_HEADSET_JACK),
> -- 
> 2.43.0

It looks like that the patch is malformed, failing one context line,
and neither git-am nor patch can't apply it cleanly.
Since the code change is trivial, I fixed it locally now.  But at the
next time, please make sure that the patch is really applicable.


thanks,

Takashi

      reply	other threads:[~2026-03-09  9:02 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-03-07  1:29 [PATCH] ALSA: hda/realtek: Add headset jack quirk for Thinkpad X390 Uzair Mughal
2026-03-09  9:02 ` 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=87ms0htlyu.wl-tiwai@suse.de \
    --to=tiwai@suse.de \
    --cc=alsa-devel@alsa-project.org \
    --cc=contact@uzair.is-a.dev \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-sound@vger.kernel.org \
    --cc=perex@perex.cz \
    /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.