mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH] ALSA: hda: cs35l41: Add SSID quirk for ASUS ROG Strix G16 G614PR
@ 2026-06-10 15:57 Marco Scardovi
  2026-06-10 16:25 ` Stefan Binding (Opensource)
  0 siblings, 1 reply; 2+ messages in thread
From: Marco Scardovi @ 2026-06-10 15:57 UTC (permalink / raw)
  To: david.rhodes, rf, perex, tiwai; +Cc: linux-sound, patches, linux-kernel

The ASUS ROG Strix G16 G614PR laptop (SSID: 1043:1074) uses I2C to connect
two CS35L41 smart amplifiers, but its BIOS ACPI tables lack the necessary
_DSD properties to describe this hardware setup. This leads to the driver
failing to enable the amplifiers during boot, resulting in a -110
connection timeout:

    cs35l41-hda i2c-CSC3551:00-cs35l41-hda.0: Enable(1) failed: -110

Add the SSID quirk entry in cs35l41_config_table and
cs35l41_prop_model_table to provide the configuration override. The
setup uses internal boost with a 1.0 uH inductor, a peak current limit of
4.5 A, and a 24 uF capacitor. The GPIO indices used are 0 for the reset
GPIO and 1 for the speaker ID GPIO.

Assisted-by: Antigravity:gemini-3.5-flash
Signed-off-by: Marco Scardovi <scardracs@disroot.org>
---
 sound/hda/codecs/side-codecs/cs35l41_hda_property.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/sound/hda/codecs/side-codecs/cs35l41_hda_property.c b/sound/hda/codecs/side-codecs/cs35l41_hda_property.c
index 732ae534db36..4a3b14468dd8 100644
--- a/sound/hda/codecs/side-codecs/cs35l41_hda_property.c
+++ b/sound/hda/codecs/side-codecs/cs35l41_hda_property.c
@@ -85,6 +85,7 @@ static const struct cs35l41_config cs35l41_config_table[] = {
 	{ "103C8C51", 2, INTERNAL, { CS35L41_LEFT, CS35L41_RIGHT, 0, 0 }, 0, 1, -1, 1000, 4100, 24 },
 	{ "103C8CDD", 2, INTERNAL, { CS35L41_LEFT, CS35L41_RIGHT, 0, 0 }, 0, 1, -1, 1000, 4100, 24 },
 	{ "103C8CDE", 2, INTERNAL, { CS35L41_LEFT, CS35L41_RIGHT, 0, 0 }, 0, 1, -1, 1000, 3900, 24 },
+	{ "10431074", 2, INTERNAL, { CS35L41_LEFT, CS35L41_RIGHT, 0, 0 }, 0, 1, -1, 1000, 4500, 24 },
 	{ "104312AF", 2, INTERNAL, { CS35L41_LEFT, CS35L41_RIGHT, 0, 0 }, 1, 2, 0, 1000, 4500, 24 },
 	{ "10431433", 2, INTERNAL, { CS35L41_LEFT, CS35L41_RIGHT, 0, 0 }, 0, 1, -1, 1000, 4500, 24 },
 	{ "10431463", 2, INTERNAL, { CS35L41_LEFT, CS35L41_RIGHT, 0, 0 }, 0, 1, -1, 1000, 4500, 24 },
@@ -510,6 +511,7 @@ static const struct cs35l41_prop_model cs35l41_prop_model_table[] = {
 	{ "CSC3551", "103C8C6A", hp_i2c_int_2amp_dual_spkid },
 	{ "CSC3551", "103C8CDD", generic_dsd_config },
 	{ "CSC3551", "103C8CDE", generic_dsd_config },
+	{ "CSC3551", "10431074", generic_dsd_config },
 	{ "CSC3551", "104312AF", generic_dsd_config },
 	{ "CSC3551", "10431433", generic_dsd_config },
 	{ "CSC3551", "10431463", generic_dsd_config },
-- 
2.54.0


^ permalink raw reply	[flat|nested] 2+ messages in thread

* RE: [PATCH] ALSA: hda: cs35l41: Add SSID quirk for ASUS ROG Strix G16 G614PR
  2026-06-10 15:57 [PATCH] ALSA: hda: cs35l41: Add SSID quirk for ASUS ROG Strix G16 G614PR Marco Scardovi
@ 2026-06-10 16:25 ` Stefan Binding (Opensource)
  0 siblings, 0 replies; 2+ messages in thread
From: Stefan Binding (Opensource) @ 2026-06-10 16:25 UTC (permalink / raw)
  To: 'Marco Scardovi', david.rhodes, rf, perex, tiwai
  Cc: linux-sound, patches, linux-kernel

Hi,

My understanding is that this laptop should have correct _DSD.
Can you please create a bugzilla ticket at the below address and CC Cirrus
Patches (patches@opensource.cirrus.com) to it, so we can investigate.

https://bugzilla.kernel.org/

Please attach an acpi dump and dmesg of the failure to the ticket.

Creating a public Bugzilla allows others with the same laptop to find
relevant information pertaining to this issue.

Thanks,
Stefan

> -----Original Message-----
> From: Marco Scardovi <scardracs@disroot.org>
> Sent: Wednesday, June 10, 2026 4:57 PM
> To: david.rhodes@cirrus.com; rf@opensource.cirrus.com; perex@perex.cz;
> tiwai@suse.com
> Cc: linux-sound@vger.kernel.org; patches@opensource.cirrus.com; linux-
> kernel@vger.kernel.org
> Subject: [PATCH] ALSA: hda: cs35l41: Add SSID quirk for ASUS ROG Strix G16
> G614PR
> 
> The ASUS ROG Strix G16 G614PR laptop (SSID: 1043:1074) uses I2C to connect
> two CS35L41 smart amplifiers, but its BIOS ACPI tables lack the necessary
_DSD
> properties to describe this hardware setup. This leads to the driver
failing to
> enable the amplifiers during boot, resulting in a -110 connection timeout:
> 
>     cs35l41-hda i2c-CSC3551:00-cs35l41-hda.0: Enable(1) failed: -110
> 
> Add the SSID quirk entry in cs35l41_config_table and
cs35l41_prop_model_table
> to provide the configuration override. The setup uses internal boost with
a 1.0 uH
> inductor, a peak current limit of
> 4.5 A, and a 24 uF capacitor. The GPIO indices used are 0 for the reset
GPIO and 1
> for the speaker ID GPIO.
> 
> Assisted-by: Antigravity:gemini-3.5-flash
> Signed-off-by: Marco Scardovi <scardracs@disroot.org>
> ---
>  sound/hda/codecs/side-codecs/cs35l41_hda_property.c | 2 ++
>  1 file changed, 2 insertions(+)
> 
> diff --git a/sound/hda/codecs/side-codecs/cs35l41_hda_property.c
> b/sound/hda/codecs/side-codecs/cs35l41_hda_property.c
> index 732ae534db36..4a3b14468dd8 100644
> --- a/sound/hda/codecs/side-codecs/cs35l41_hda_property.c
> +++ b/sound/hda/codecs/side-codecs/cs35l41_hda_property.c
> @@ -85,6 +85,7 @@ static const struct cs35l41_config
cs35l41_config_table[] = {
>  	{ "103C8C51", 2, INTERNAL, { CS35L41_LEFT, CS35L41_RIGHT, 0, 0 }, 0,
1,
> -1, 1000, 4100, 24 },
>  	{ "103C8CDD", 2, INTERNAL, { CS35L41_LEFT, CS35L41_RIGHT, 0, 0 }, 0,
1,
> -1, 1000, 4100, 24 },
>  	{ "103C8CDE", 2, INTERNAL, { CS35L41_LEFT, CS35L41_RIGHT, 0, 0 }, 0,
1,
> -1, 1000, 3900, 24 },
> +	{ "10431074", 2, INTERNAL, { CS35L41_LEFT, CS35L41_RIGHT, 0, 0 }, 0,
> +1, -1, 1000, 4500, 24 },
>  	{ "104312AF", 2, INTERNAL, { CS35L41_LEFT, CS35L41_RIGHT, 0, 0 }, 1,
2,
> 0, 1000, 4500, 24 },
>  	{ "10431433", 2, INTERNAL, { CS35L41_LEFT, CS35L41_RIGHT, 0, 0 }, 0,
1,
> -1, 1000, 4500, 24 },
>  	{ "10431463", 2, INTERNAL, { CS35L41_LEFT, CS35L41_RIGHT, 0, 0 }, 0,
1,
> -1, 1000, 4500, 24 }, @@ -510,6 +511,7 @@ static const struct
> cs35l41_prop_model cs35l41_prop_model_table[] = {
>  	{ "CSC3551", "103C8C6A", hp_i2c_int_2amp_dual_spkid },
>  	{ "CSC3551", "103C8CDD", generic_dsd_config },
>  	{ "CSC3551", "103C8CDE", generic_dsd_config },
> +	{ "CSC3551", "10431074", generic_dsd_config },
>  	{ "CSC3551", "104312AF", generic_dsd_config },
>  	{ "CSC3551", "10431433", generic_dsd_config },
>  	{ "CSC3551", "10431463", generic_dsd_config },
> --
> 2.54.0
> 



^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2026-06-10 16:25 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2026-06-10 15:57 [PATCH] ALSA: hda: cs35l41: Add SSID quirk for ASUS ROG Strix G16 G614PR Marco Scardovi
2026-06-10 16:25 ` Stefan Binding (Opensource)

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

Powered by JetHome