[PATCH v2] ALSA: hda: cs35l41: Support HP OmniBook 7 Laptop 16-bh0xxx

[PATCH v2] ALSA: hda: cs35l41: Support HP OmniBook 7 Laptop 16-bh0xxx

[Sven Jans]

The laptop has a malformed _DSD which causes the cirrus,dev-index
property to fail to parse, so it needs to be supported using the
configuration table.

Signed-off-by: Sven Jans <skbotnl@disroot.org>
---
Resubmission, discussion of the previous submission stalled.
The values are equal to the values reported in the _DSD.
You can find my ACPI dump to verify here:
https://bugzilla.kernel.org/attachment.cgi?id=310050
Link to v1:
https://lore.kernel.org/linux-sound/20260506181807.22547-1-skbotnl@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 416d7bf3e289..e14a87336478 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 },
+	{ "103C8E60", 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 },
@@ -511,6 +512,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", "103C8E60", generic_dsd_config },
 	{ "CSC3551", "104312AF", generic_dsd_config },
 	{ "CSC3551", "10431433", generic_dsd_config },
 	{ "CSC3551", "10431463", generic_dsd_config },
-- 
2.55.0

Re: [PATCH v2] ALSA: hda: cs35l41: Support HP OmniBook 7 Laptop 16-bh0xxx

[Takashi Iwai]

On Thu, 09 Jul 2026 23:43:53 +0200,
Sven Jans wrote:
> 
> The laptop has a malformed _DSD which causes the cirrus,dev-index
> property to fail to parse, so it needs to be supported using the
> configuration table.
> 
> Signed-off-by: Sven Jans <skbotnl@disroot.org>
> ---
> Resubmission, discussion of the previous submission stalled.
> The values are equal to the values reported in the _DSD.
> You can find my ACPI dump to verify here:
> https://bugzilla.kernel.org/attachment.cgi?id=310050
> Link to v1:
> https://lore.kernel.org/linux-sound/20260506181807.22547-1-skbotnl@disroot.org/

Stefan, is this also one of models with odd firmware?


thanks,

Takashi

> 
>  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 416d7bf3e289..e14a87336478 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 },
> +	{ "103C8E60", 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 },
> @@ -511,6 +512,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", "103C8E60", generic_dsd_config },
>  	{ "CSC3551", "104312AF", generic_dsd_config },
>  	{ "CSC3551", "10431433", generic_dsd_config },
>  	{ "CSC3551", "10431463", generic_dsd_config },
> -- 
> 2.55.0
> 

Re: [PATCH v2] ALSA: hda: cs35l41: Support HP OmniBook 7 Laptop 16-bh0xxx

[Sven Jans]

For what it's worth Stefan has confirmed that there is an issue with 
this laptop here: https://bugzilla.kernel.org/show_bug.cgi?id=221185#c11
The mentioned bug report 221479 is mine 
(https://bugzilla.kernel.org/show_bug.cgi?id=221479).

On 15/07/2026 16:34, Takashi Iwai wrote:
> On Thu, 09 Jul 2026 23:43:53 +0200,
> Sven Jans wrote:
>>
>> The laptop has a malformed _DSD which causes the cirrus,dev-index
>> property to fail to parse, so it needs to be supported using the
>> configuration table.
>>
>> Signed-off-by: Sven Jans <skbotnl@disroot.org>
>> ---
>> Resubmission, discussion of the previous submission stalled.
>> The values are equal to the values reported in the _DSD.
>> You can find my ACPI dump to verify here:
>> https://bugzilla.kernel.org/attachment.cgi?id=310050
>> Link to v1:
>> https://lore.kernel.org/linux-sound/20260506181807.22547-1-skbotnl@disroot.org/
> 
> Stefan, is this also one of models with odd firmware?
> 
> 
> thanks,
> 
> Takashi
> 
>>
>>   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 416d7bf3e289..e14a87336478 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 },
>> +	{ "103C8E60", 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 },
>> @@ -511,6 +512,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", "103C8E60", generic_dsd_config },
>>   	{ "CSC3551", "104312AF", generic_dsd_config },
>>   	{ "CSC3551", "10431433", generic_dsd_config },
>>   	{ "CSC3551", "10431463", generic_dsd_config },
>> -- 
>> 2.55.0
>>

RE: [PATCH v2] ALSA: hda: cs35l41: Support HP OmniBook 7 Laptop 16-bh0xxx

[Stefan Binding (Opensource)]

Hi Takashi,

Yes, this is one of the laptops with malformed ACPI.

Thanks,
Stefan

> -----Original Message-----
> From: Takashi Iwai <tiwai@suse.de>
> Sent: Wednesday, July 15, 2026 3:35 PM
> To: Stefan Binding <sbinding@opensource.cirrus.com>
> Cc: Sven Jans <skbotnl@disroot.org>; David Rhodes
> <david.rhodes@cirrus.com>; Richard Fitzgerald <rf@opensource.cirrus.com>;
> Jaroslav Kysela <perex@perex.cz>; Takashi Iwai <tiwai@suse.com>; linux-
> sound@vger.kernel.org; patches@opensource.cirrus.com; linux-
> kernel@vger.kernel.org
> Subject: Re: [PATCH v2] ALSA: hda: cs35l41: Support HP OmniBook 7 Laptop
> 16-bh0xxx
> 
> On Thu, 09 Jul 2026 23:43:53 +0200,
> Sven Jans wrote:
> >
> > The laptop has a malformed _DSD which causes the cirrus,dev-index
> > property to fail to parse, so it needs to be supported using the
> > configuration table.
> >
> > Signed-off-by: Sven Jans <skbotnl@disroot.org>
> > ---
> > Resubmission, discussion of the previous submission stalled.
> > The values are equal to the values reported in the _DSD.
> > You can find my ACPI dump to verify here:
> > https://bugzilla.kernel.org/attachment.cgi?id=310050
> > Link to v1:
> > https://lore.kernel.org/linux-sound/20260506181807.22547-1-
> skbotnl@dis
> > root.org/
> 
> Stefan, is this also one of models with odd firmware?
> 
> 
> thanks,
> 
> Takashi
> 
> >
> >  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 416d7bf3e289..e14a87336478 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 },
> > +	{ "103C8E60", 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 }, @@ -511,6 +512,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", "103C8E60", generic_dsd_config },
> >  	{ "CSC3551", "104312AF", generic_dsd_config },
> >  	{ "CSC3551", "10431433", generic_dsd_config },
> >  	{ "CSC3551", "10431463", generic_dsd_config },
> > --
> > 2.55.0
> >

Re: [PATCH v2] ALSA: hda: cs35l41: Support HP OmniBook 7 Laptop 16-bh0xxx

[Sven]

Hello,
Is there anything I can do to help this patch move forward?

Re: [PATCH v2] ALSA: hda: cs35l41: Support HP OmniBook 7 Laptop 16-bh0xxx

[Jon Harrison]

On Mon, 24 Aug 2026, Stefan Binding wrote:
> We are currently awaiting confirmation of the correct settings before we do
> so.

I have another HP OmniBook 7 Laptop 16-bh0xxx, SKU D2KR6UA#ABA,
BIOS F.12, subsystem 103c:8e60, running Linux 7.2.5. Both CS35L41
amplifiers still fail to probe:

  cs35l41-hda i2c-CSC3551:00-cs35l41-hda.0: Failed property
cirrus,dev-index: -22
  cs35l41-hda i2c-CSC3551:00-cs35l41-hda.0: error -EINVAL: Platform
not supported

The speakers only play through the Realtek fallback and are very quiet.
Has that settings confirmation arrived, and is there an expected time
for the quirk?

Thanks,
Jon Harrison

Re: [PATCH v2] ALSA: hda: cs35l41: Support HP OmniBook 7 Laptop 16-bh0xxx

[Takashi Iwai]

On Fri, 16 Oct 2026 00:15:24 +0200,
Jon Harrison wrote:
> 
> On Mon, 24 Aug 2026, Stefan Binding wrote:
> > We are currently awaiting confirmation of the correct settings before we do
> > so.
> 
> I have another HP OmniBook 7 Laptop 16-bh0xxx, SKU D2KR6UA#ABA,
> BIOS F.12, subsystem 103c:8e60, running Linux 7.2.5. Both CS35L41
> amplifiers still fail to probe:
> 
>   cs35l41-hda i2c-CSC3551:00-cs35l41-hda.0: Failed property
> cirrus,dev-index: -22
>   cs35l41-hda i2c-CSC3551:00-cs35l41-hda.0: error -EINVAL: Platform
> not supported
> 
> The speakers only play through the Realtek fallback and are very quiet.
> Has that settings confirmation arrived, and is there an expected time
> for the quirk?

OK, we've waited enough long, so I take the patch to for-next branch
now (for 7.4-rc1).

If this is really breaking something, let me know.  We'll fix up again
before 7.4 release.


thanks,

Takashi

Re: [PATCH v2] ALSA: hda: cs35l41: Support HP OmniBook 7 Laptop 16-bh0xxx

[Sven]

On 16/10/2026 16:16, Takashi Iwai wrote:
> On Fri, 16 Oct 2026 00:15:24 +0200,
> Jon Harrison wrote:
>>
>> On Mon, 24 Aug 2026, Stefan Binding wrote:
>>> We are currently awaiting confirmation of the correct settings before we do
>>> so.
>>
>> I have another HP OmniBook 7 Laptop 16-bh0xxx, SKU D2KR6UA#ABA,
>> BIOS F.12, subsystem 103c:8e60, running Linux 7.2.5. Both CS35L41
>> amplifiers still fail to probe:
>>
>>    cs35l41-hda i2c-CSC3551:00-cs35l41-hda.0: Failed property
>> cirrus,dev-index: -22
>>    cs35l41-hda i2c-CSC3551:00-cs35l41-hda.0: error -EINVAL: Platform
>> not supported
>>
>> The speakers only play through the Realtek fallback and are very quiet.
>> Has that settings confirmation arrived, and is there an expected time
>> for the quirk?
> 
> OK, we've waited enough long, so I take the patch to for-next branch
> now (for 7.4-rc1).
> 
> If this is really breaking something, let me know.  We'll fix up again
> before 7.4 release.
> 
> 
> thanks,
> 
> Takashi

Hello,
I saw the commit in tiwai/sound. I noticed that 3900 was used for the 
boost_peak_milliamp value. I just wanted to note that as far as I can 
remember that is the value that the Windows driver uses, while the 
malformed but readable _DSD specifies 4500 (at least mine does). I don't 
know which one is correct, but 3900 is probably on the safer side I guess.

Re: [PATCH v2] ALSA: hda: cs35l41: Support HP OmniBook 7 Laptop 16-bh0xxx

[Sven Jans]

On 16/10/2026 16:42, Sven wrote:
> On 16/10/2026 16:16, Takashi Iwai wrote:
>> On Fri, 16 Oct 2026 00:15:24 +0200,
>> Jon Harrison wrote:
>>>
>>> On Mon, 24 Aug 2026, Stefan Binding wrote:
>>>> We are currently awaiting confirmation of the correct settings 
>>>> before we do
>>>> so.
>>>
>>> I have another HP OmniBook 7 Laptop 16-bh0xxx, SKU D2KR6UA#ABA,
>>> BIOS F.12, subsystem 103c:8e60, running Linux 7.2.5. Both CS35L41
>>> amplifiers still fail to probe:
>>>
>>>    cs35l41-hda i2c-CSC3551:00-cs35l41-hda.0: Failed property
>>> cirrus,dev-index: -22
>>>    cs35l41-hda i2c-CSC3551:00-cs35l41-hda.0: error -EINVAL: Platform
>>> not supported
>>>
>>> The speakers only play through the Realtek fallback and are very quiet.
>>> Has that settings confirmation arrived, and is there an expected time
>>> for the quirk?
>>
>> OK, we've waited enough long, so I take the patch to for-next branch
>> now (for 7.4-rc1).
>>
>> If this is really breaking something, let me know.  We'll fix up again
>> before 7.4 release.
>>
>>
>> thanks,
>>
>> Takashi
> 
> Hello,
> I saw the commit in tiwai/sound. I noticed that 3900 was used for the 
> boost_peak_milliamp value. I just wanted to note that as far as I can 
> remember that is the value that the Windows driver uses, while the 
> malformed but readable _DSD specifies 4500 (at least mine does). I don't 
> know which one is correct, but 3900 is probably on the safer side I guess.

Small correction, 3850 mA to be precise.

Re: [PATCH v2] ALSA: hda: cs35l41: Support HP OmniBook 7 Laptop 16-bh0xxx

[Sven Jans]

On 16/10/2026 16:46, Sven Jans wrote:
> On 16/10/2026 16:42, Sven wrote:
>> On 16/10/2026 16:16, Takashi Iwai wrote:
>>> On Fri, 16 Oct 2026 00:15:24 +0200,
>>> Jon Harrison wrote:
>>>>
>>>> On Mon, 24 Aug 2026, Stefan Binding wrote:
>>>>> We are currently awaiting confirmation of the correct settings 
>>>>> before we do
>>>>> so.
>>>>
>>>> I have another HP OmniBook 7 Laptop 16-bh0xxx, SKU D2KR6UA#ABA,
>>>> BIOS F.12, subsystem 103c:8e60, running Linux 7.2.5. Both CS35L41
>>>> amplifiers still fail to probe:
>>>>
>>>>    cs35l41-hda i2c-CSC3551:00-cs35l41-hda.0: Failed property
>>>> cirrus,dev-index: -22
>>>>    cs35l41-hda i2c-CSC3551:00-cs35l41-hda.0: error -EINVAL: Platform
>>>> not supported
>>>>
>>>> The speakers only play through the Realtek fallback and are very quiet.
>>>> Has that settings confirmation arrived, and is there an expected time
>>>> for the quirk?
>>>
>>> OK, we've waited enough long, so I take the patch to for-next branch
>>> now (for 7.4-rc1).
>>>
>>> If this is really breaking something, let me know.  We'll fix up again
>>> before 7.4 release.
>>>
>>>
>>> thanks,
>>>
>>> Takashi
>>
>> Hello,
>> I saw the commit in tiwai/sound. I noticed that 3900 was used for the 
>> boost_peak_milliamp value. I just wanted to note that as far as I can 
>> remember that is the value that the Windows driver uses, while the 
>> malformed but readable _DSD specifies 4500 (at least mine does). I 
>> don't know which one is correct, but 3900 is probably on the safer 
>> side I guess.
> 
> Small correction, 3850 mA to be precise.

Apologies for the multiple emails, but after re-reading I thought I 
should specify that in my last email (about the 3850 mA) I was talking 
about the Windows driver. Just in case.