[PATCH] ALSA: hda/realtek: Add quirk for Legion S7 15IMH

Eric Naim posted 1 patch 2 months ago
sound/hda/codecs/realtek/alc269.c | 1 +
1 file changed, 1 insertion(+)
[PATCH] ALSA: hda/realtek: Add quirk for Legion S7 15IMH
Posted by Eric Naim 2 months ago
Fix speaker output on the Lenovo Legion S7 15IMH05.

Cc: stable@vger.kernel.org
Signed-off-by: Eric Naim <dnaim@cachyos.org>
---
 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 45f9d6487388..ae74e1b69eb3 100644
--- a/sound/hda/codecs/realtek/alc269.c
+++ b/sound/hda/codecs/realtek/alc269.c
@@ -7605,6 +7605,7 @@ static const struct hda_quirk alc269_fixup_tbl[] = {
 	SND_PCI_QUIRK(0x17aa, 0x3801, "Lenovo Yoga9 14IAP7", ALC287_FIXUP_YOGA9_14IAP7_BASS_SPK_PIN),
 	HDA_CODEC_QUIRK(0x17aa, 0x3802, "DuetITL 2021", ALC287_FIXUP_YOGA7_14ITL_SPEAKERS),
 	SND_PCI_QUIRK(0x17aa, 0x3802, "Lenovo Yoga Pro 9 14IRP8", ALC287_FIXUP_TAS2781_I2C),
+	SND_PCI_QUIRK(0x17aa, 0x3811, "Legion S7 15IMH05", ALC287_FIXUP_LEGION_15IMHG05_SPEAKERS),
 	SND_PCI_QUIRK(0x17aa, 0x3813, "Legion 7i 15IMHG05", ALC287_FIXUP_LEGION_15IMHG05_SPEAKERS),
 	SND_PCI_QUIRK(0x17aa, 0x3818, "Lenovo C940 / Yoga Duet 7", ALC298_FIXUP_LENOVO_C940_DUET7),
 	SND_PCI_QUIRK(0x17aa, 0x3819, "Lenovo 13s Gen2 ITL", ALC287_FIXUP_13S_GEN2_SPEAKERS),
-- 
2.53.0
Re: [PATCH] ALSA: hda/realtek: Add quirk for Legion S7 15IMH
Posted by Takashi Iwai 2 months ago
On Mon, 13 Apr 2026 17:48:17 +0200,
Eric Naim wrote:
> 
> Fix speaker output on the Lenovo Legion S7 15IMH05.
> 
> Cc: stable@vger.kernel.org
> Signed-off-by: Eric Naim <dnaim@cachyos.org>

Thanks, applied now.


Takashi
Re: [PATCH] ALSA: hda/realtek: Add quirk for Legion S7 15IMH
Posted by Eric Naim 2 months ago
On 4/13/26 11:56 PM, Takashi Iwai wrote:
> On Mon, 13 Apr 2026 17:48:17 +0200,
> Eric Naim wrote:
>>
>> Fix speaker output on the Lenovo Legion S7 15IMH05.
>>
>> Cc: stable@vger.kernel.org
>> Signed-off-by: Eric Naim <dnaim@cachyos.org>
> 
> Thanks, applied now.
> 
> 
> Takashi

Sorry Takashi, can you remove this from your tree? I seem to have gotten the
PID wrong for this device. I'll follow up with a v2 or fixup once I've
confirmed I got the correct PID. Let me know which of the two resolutions you
prefer.

-- 
Regards,
  Eric
Re: [PATCH] ALSA: hda/realtek: Add quirk for Legion S7 15IMH
Posted by Takashi Iwai 2 months ago
On Tue, 14 Apr 2026 05:28:00 +0200,
Eric Naim wrote:
> 
> On 4/13/26 11:56 PM, Takashi Iwai wrote:
> > On Mon, 13 Apr 2026 17:48:17 +0200,
> > Eric Naim wrote:
> >>
> >> Fix speaker output on the Lenovo Legion S7 15IMH05.
> >>
> >> Cc: stable@vger.kernel.org
> >> Signed-off-by: Eric Naim <dnaim@cachyos.org>
> > 
> > Thanks, applied now.
> > 
> > 
> > Takashi
> 
> Sorry Takashi, can you remove this from your tree? I seem to have gotten the
> PID wrong for this device. I'll follow up with a v2 or fixup once I've
> confirmed I got the correct PID. Let me know which of the two resolutions you
> prefer.

As the tree was published, could you rather a correction patch on the
top?  Put Fixes tag for pointing to the corrected commit.


thanks,

Takashi
Re: [PATCH] ALSA: hda/realtek: Add quirk for Legion S7 15IMH
Posted by Eric Naim 1 month, 3 weeks ago
On 4/14/26 3:06 PM, Takashi Iwai wrote:
> On Tue, 14 Apr 2026 05:28:00 +0200,
> Eric Naim wrote:
>>
>> On 4/13/26 11:56 PM, Takashi Iwai wrote:
>>> On Mon, 13 Apr 2026 17:48:17 +0200,
>>> Eric Naim wrote:
>>>>
>>>> Fix speaker output on the Lenovo Legion S7 15IMH05.
>>>>
>>>> Cc: stable@vger.kernel.org
>>>> Signed-off-by: Eric Naim <dnaim@cachyos.org>
>>>
>>> Thanks, applied now.
>>>
>>>
>>> Takashi
>>
>> Sorry Takashi, can you remove this from your tree? I seem to have gotten the
>> PID wrong for this device. I'll follow up with a v2 or fixup once I've
>> confirmed I got the correct PID. Let me know which of the two resolutions you
>> prefer.
> 
> As the tree was published, could you rather a correction patch on the
> top?  Put Fixes tag for pointing to the corrected commit.
> 
> 
> thanks,
> 
> Takashi

Alright, I finally got a reply from the user that requested this from me.
After looking at [1] and reading through [2], it looks like the HDA verbs for
ALC287_FIXUP_LEGION_15IMHG05_SPEAKERS are just copied from
ALC287_FIXUP_YOGA7_14ITL_SPEAKERS, and is different from the original patch
that was tested in [2] and the provided verbs from [1].

Cameron, you submitted the original patch for this. Can you confirm?

Attached is my proposal fix for the ALC287_FIXUP_LEGION_15IMHG05_SPEAKERS
quirk. I'll get the reporter to test this and report back.

[1]
https://github.com/thiagotei/linux-realtek-alc287/blob/2a3a57c14adf3313d4c14df1a0a8a8bd3f544369/lenovo-legion/verbs-legion.txt

[2] https://bugzilla.kernel.org/show_bug.cgi?id=208555

-- 
Regards,
  Eric