All of lore.kernel.org
 help / color / mirror / Atom feed
* Re: [PATCH v2] ALSA: hda: cs35l41: Support HP OmniBook 7 Laptop 16-bh0xxx
@ 2026-10-15 22:15 Jon Harrison
  2026-10-16 14:16 ` Takashi Iwai
  0 siblings, 1 reply; 10+ messages in thread
From: Jon Harrison @ 2026-10-15 22:15 UTC (permalink / raw)
  To: skbotnl; +Cc: david.rhodes, linux-kernel, linux-sound, patches, perex, rf,
	tiwai

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

^ permalink raw reply	[flat|nested] 10+ messages in thread
* [PATCH v2] ALSA: hda: cs35l41: Support HP OmniBook 7 Laptop 16-bh0xxx
@ 2026-07-09 21:43 Sven Jans
  2026-07-15 14:34 ` Takashi Iwai
  2026-07-23 10:42 ` Sven
  0 siblings, 2 replies; 10+ messages in thread
From: Sven Jans @ 2026-07-09 21:43 UTC (permalink / raw)
  Cc: Sven Jans, David Rhodes, Richard Fitzgerald, Jaroslav Kysela,
	Takashi Iwai, linux-sound, patches, linux-kernel

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


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

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

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-10-15 22:15 [PATCH v2] ALSA: hda: cs35l41: Support HP OmniBook 7 Laptop 16-bh0xxx Jon Harrison
2026-10-16 14:16 ` Takashi Iwai
2026-10-16 14:42   ` Sven
2026-10-16 14:46     ` Sven Jans
2026-10-16 14:53       ` Sven Jans
  -- strict thread matches above, loose matches on Subject: below --
2026-07-09 21:43 Sven Jans
2026-07-15 14:34 ` Takashi Iwai
2026-07-15 14:54   ` Sven Jans
2026-07-15 16:13   ` Stefan Binding (Opensource)
2026-07-23 10:42 ` Sven

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.