From: Sasha Levin <sashal@kernel.org>
To: patches@lists.linux.dev, stable@vger.kernel.org
Cc: Stefan Binding <sbinding@opensource.cirrus.com>,
Takashi Iwai <tiwai@suse.de>, Sasha Levin <sashal@kernel.org>,
perex@perex.cz, tiwai@suse.com, kailang@realtek.com,
gregkh@linuxfoundation.org, edson.drosdeck@gmail.com,
oder_chiou@realtek.com, desikumar81@gmail.com,
alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org
Subject: [PATCH AUTOSEL 6.15 14/30] ALSA: hda/realtek: Add support for Acer Helios Laptops using CS35L41 HDA
Date: Fri, 30 May 2025 08:38:36 -0400 [thread overview]
Message-ID: <20250530123852.2574030-14-sashal@kernel.org> (raw)
In-Reply-To: <20250530123852.2574030-1-sashal@kernel.org>
From: Stefan Binding <sbinding@opensource.cirrus.com>
[ Upstream commit d64cbb5ed9227566c068ac9300a85912234d10aa ]
Laptops use 2 CS35L41 Amps with HDA, using External boost with I2C.
Similar to previous Acer laptops, these laptops also need the
ALC255_FIXUP_PREDATOR_SUBWOOFER quirk to function properly.
Signed-off-by: Stefan Binding <sbinding@opensource.cirrus.com>
Link: https://patch.msgid.link/20250515162848.405055-2-sbinding@opensource.cirrus.com
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Sasha Levin <sashal@kernel.org>
---
**YES** This commit should be backported to stable kernel trees. ##
Analysis ### What the commit does: This commit adds audio support for
Acer Helios laptops using CS35L41 HDA amplifiers. The specific changes
are: 1. **Adds new enum**: `ALC287_FIXUP_PREDATOR_SPK_CS35L41_I2C_2` 2.
**Adds new fixup entry**: Chains `cs35l41_fixup_i2c_two` with the
existing `ALC255_FIXUP_PREDATOR_SUBWOOFER` quirk 3. **Adds 3 PCI quirk
entries** for specific Acer models: - `0x1025, 0x1826, "Acer Helios
ZPC"` - `0x1025, 0x182c, "Acer Helios ZPD"` - `0x1025, 0x1844, "Acer
Helios ZPS"` ### Comparison with approved similar commits: This commit
follows **exactly the same pattern** as the four similar commits marked
"YES": - **Similar Commits #1-4**: All added PCI quirk entries for ASUS
laptops using CS35L41 HDA, with identical structure - **Same approach**:
Extending the quirk table to map new hardware IDs to existing, proven
fixup mechanisms - **Same scope**: Hardware-specific audio support using
well-established CS35L41 amplifier functionality ### Hardware support vs
bug fixing: This is **new hardware support** that enables audio
functionality on specific Acer Helios laptop models. The CS35L41
amplifier support and ALC255_FIXUP_PREDATOR_SUBWOOFER quirk already
exist - this simply extends support to new hardware variants. ### Risk
assessment: **Very low risk** because: 1. **Isolated impact**: Only
affects the three specific Acer Helios models listed (PCI IDs 0x1826,
0x182c, 0x1844) 2. **No architectural changes**: Pure quirk table
additions with no modifications to core audio logic 3. **Proven fixup
chain**: Both `cs35l41_fixup_i2c_two` and
`ALC255_FIXUP_PREDATOR_SUBWOOFER` are existing, stable fixups 4. **No
regression risk**: Doesn't modify behavior for any existing hardware ###
Stable kernel criteria alignment: ✅ **Fixes important bugs**: Enables
audio functionality that would otherwise be broken on these laptops ✅
**No new features**: Uses existing CS35L41 and Predator subwoofer
infrastructure ✅ **No architectural changes**: Only extends PCI quirk
tables ✅ **Minimal regression risk**: Changes are completely isolated to
specific hardware IDs ✅ **Subsystem-confined**: Limited to ALSA HDA
realtek driver quirks This commit perfectly matches the established
pattern of stable-worthy ALSA HDA hardware support commits and should be
backported to ensure audio functionality works properly for users with
these Acer Helios laptops.
sound/pci/hda/patch_realtek.c | 10 ++++++++++
1 file changed, 10 insertions(+)
diff --git a/sound/pci/hda/patch_realtek.c b/sound/pci/hda/patch_realtek.c
index 20ab1fb2195ff..cd0d7ba7320ef 100644
--- a/sound/pci/hda/patch_realtek.c
+++ b/sound/pci/hda/patch_realtek.c
@@ -8029,6 +8029,7 @@ enum {
ALC283_FIXUP_DELL_HP_RESUME,
ALC294_FIXUP_ASUS_CS35L41_SPI_2,
ALC274_FIXUP_HP_AIO_BIND_DACS,
+ ALC287_FIXUP_PREDATOR_SPK_CS35L41_I2C_2,
};
/* A special fixup for Lenovo C940 and Yoga Duet 7;
@@ -9301,6 +9302,12 @@ static const struct hda_fixup alc269_fixups[] = {
{ }
}
},
+ [ALC287_FIXUP_PREDATOR_SPK_CS35L41_I2C_2] = {
+ .type = HDA_FIXUP_FUNC,
+ .v.func = cs35l41_fixup_i2c_two,
+ .chained = true,
+ .chain_id = ALC255_FIXUP_PREDATOR_SUBWOOFER
+ },
[ALC256_FIXUP_MEDION_HEADSET_NO_PRESENCE] = {
.type = HDA_FIXUP_PINS,
.v.pins = (const struct hda_pintbl[]) {
@@ -10456,6 +10463,9 @@ static const struct hda_quirk alc269_fixup_tbl[] = {
SND_PCI_QUIRK(0x1025, 0x1534, "Acer Predator PH315-54", ALC255_FIXUP_ACER_MIC_NO_PRESENCE),
SND_PCI_QUIRK(0x1025, 0x159c, "Acer Nitro 5 AN515-58", ALC2XX_FIXUP_HEADSET_MIC),
SND_PCI_QUIRK(0x1025, 0x169a, "Acer Swift SFG16", ALC256_FIXUP_ACER_SFG16_MICMUTE_LED),
+ SND_PCI_QUIRK(0x1025, 0x1826, "Acer Helios ZPC", ALC287_FIXUP_PREDATOR_SPK_CS35L41_I2C_2),
+ SND_PCI_QUIRK(0x1025, 0x182c, "Acer Helios ZPD", ALC287_FIXUP_PREDATOR_SPK_CS35L41_I2C_2),
+ SND_PCI_QUIRK(0x1025, 0x1844, "Acer Helios ZPS", ALC287_FIXUP_PREDATOR_SPK_CS35L41_I2C_2),
SND_PCI_QUIRK(0x1028, 0x0470, "Dell M101z", ALC269_FIXUP_DELL_M101Z),
SND_PCI_QUIRK(0x1028, 0x053c, "Dell Latitude E5430", ALC292_FIXUP_DELL_E7X),
SND_PCI_QUIRK(0x1028, 0x054b, "Dell XPS one 2710", ALC275_FIXUP_DELL_XPS),
--
2.39.5
next prev parent reply other threads:[~2025-05-30 12:39 UTC|newest]
Thread overview: 30+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-05-30 12:38 [PATCH AUTOSEL 6.15 01/30] ACPICA: fix acpi operand cache leak in dswstate.c Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 02/30] ASoC: amd: yc: Add quirk for Lenovo Yoga Pro 7 14ASP9 Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 03/30] clocksource: Fix the CPUs' choice in the watchdog per CPU verification Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 04/30] power: supply: gpio-charger: Fix wakeup source leaks on device unbind Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 05/30] tools/nolibc: use intmax definitions from compiler Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 06/30] power: supply: collie: Fix wakeup source leaks on device unbind Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 07/30] mmc: Add quirk to disable DDR50 tuning Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 08/30] ACPICA: Avoid sequence overread in call to strncmp() Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 09/30] mmc: sdhci-esdhc-imx: Save tuning value when card stays powered in suspend Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 10/30] EDAC/igen6: Skip absent memory controllers Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 11/30] ASoC: tas2770: Power cycle amp on ISENSE/VSENSE change Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 12/30] ASoC: intel/sdw_utils: Assign initial value in asoc_sdw_rt_amp_spk_rtd_init() Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 13/30] ACPI: bus: Bail out if acpi_kobj registration fails Sasha Levin
2025-05-30 12:38 ` Sasha Levin [this message]
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 15/30] ACPI: Add missing prototype for non CONFIG_SUSPEND/CONFIG_X86 case Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 16/30] ACPICA: fix acpi parse and parseext cache leaks Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 17/30] ACPICA: Apply pack(1) to union aml_resource Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 18/30] ALSA: hda: cs35l41: Fix swapped l/r audio channels for Acer Helios laptops Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 19/30] power: supply: bq27xxx: Retrieve again when busy Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 20/30] ASoC: simple-card-utils: fixup dlc->xxx handling for error case Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 21/30] pmdomain: core: Reset genpd->states to avoid freeing invalid data Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 22/30] ACPICA: utilities: Fix overflow check in vsnprintf() Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 23/30] platform-msi: Add msi_remove_device_irq_domain() in platform_device_msi_free_irqs_all() Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 24/30] ASoC: tegra210_ahub: Add check to of_device_get_match_data() Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 25/30] Make 'cc-option' work correctly for the -Wno-xyzzy pattern Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 26/30] gpiolib: of: Add polarity quirk for s5m8767 Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 27/30] PM: runtime: fix denying of auto suspend in pm_suspend_timer_fn() Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 28/30] tools/nolibc: use pselect6_time64 if available Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 29/30] power: supply: max17040: adjust thermal channel scaling Sasha Levin
2025-05-30 12:38 ` [PATCH AUTOSEL 6.15 30/30] ACPI: battery: negate current when discharging Sasha Levin
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=20250530123852.2574030-14-sashal@kernel.org \
--to=sashal@kernel.org \
--cc=alsa-devel@alsa-project.org \
--cc=desikumar81@gmail.com \
--cc=edson.drosdeck@gmail.com \
--cc=gregkh@linuxfoundation.org \
--cc=kailang@realtek.com \
--cc=linux-kernel@vger.kernel.org \
--cc=oder_chiou@realtek.com \
--cc=patches@lists.linux.dev \
--cc=perex@perex.cz \
--cc=sbinding@opensource.cirrus.com \
--cc=stable@vger.kernel.org \
--cc=tiwai@suse.com \
--cc=tiwai@suse.de \
/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