ASoC: Intel: soc-acpi-intel-ptl-match: Remove unnecessary cs42l43 match#5773
ASoC: Intel: soc-acpi-intel-ptl-match: Remove unnecessary cs42l43 match#5773charleskeepax wants to merge 3 commits into
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates Intel PTL SoundWire machine matching to prefer function topology selection over a per-platform match entry, and adds a SoundWire DMI quirk to ignore a ghost RT722 ACPI device on specific PTL systems to avoid incorrect enumeration.
Changes:
- Remove the PTL SoundWire machine match entry for the cs42l43 + 6x cs35l56 configuration to avoid preempting function-topology based selection.
- Add a new DMI-based _ADR remap quirk to drop a ghost RT722 device (by remapping its _ADR to 0) on select ASUS/Lenovo PTL platforms.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| sound/soc/intel/common/soc-acpi-intel-ptl-match.c | Removes an explicit cs42l43+cs35l56x6 match entry so PTL systems rely on function topologies instead. |
| drivers/soundwire/dmi-quirks.c | Adds a DMI quirk table entry to ignore a ghost Realtek RT722 ACPI device on specific PTL systems. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
86ab49f to
c04dbd2
Compare
For PTL onwards Cirrus are intending to rely on function topologies, rather than using a match table for each system type. Remove this unnecessary match table entry. Having the match entries can mean that systems match when they should use function topologies instead, resulting in incorrect audio configurations. Although, admittedly this is not too likely with this 6x amp configuration as those are quite rare, but best to follow best practice. Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
For PTL onwards Cirrus are intending to rely on function topologies, rather than using a match table for each system type. Chrome systems tend to have custom magic in the topology and thus need to load a specific file. This causes problems as these system can have the same layout as generic laptops causing the match to apply to other laptops. Add a DMI quirk that forces these matches to only apply to specific devices. Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
Many systems ship with a Realtek audio codec in the ACPI that doesn't physically exist in the system. This confuses the newer function topology system that creates the soundcard, as it builds the card based on the ACPI information. Whilst we are working with the laptop vendors to try and stop this happening there are quite a few systems where this has shipped. Add a quirk to disable this "ghost" device. Currently this patch should cover: - Asus UX5406AA - Lenovo Yoga Pro 9i (83SF) - Lenovo Yoga Slim 7 Ultra (83QK) Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
|
Tested on ASUS Zenbook S14 UX5406AA (Panther Lake, Core Ultra 9 386H), BIOS v303, against torvalds linux v7.1-rc2 with this PR's current HEAD ( Boot evidence ( What's confirmed:
One observation worth flagging on the path the kernel takes: after the rt722 is removed, the For my @Pegorim/omarchy-pkgs#90, I'm dropping the prior backport of #5754 and shipping just #5728 + this PR as the linux-ptl audio set, since the result on hardware is identical and the patch surface is much smaller. cc @itisyb for the Lenovo 83QK/83SF coverage now in the same shared |
|
@Pegorim Thanks very much for the testing there is one slight oddity reported on the Yoga Pro 9 that I want to attempt to track down, then I shall get the series upstreamed. |
For PTL onwards we are intending to rely on function topologies, rather than using a match table for each system type. Remove this unnecessary match table entry. Having the match entries can mean that systems match when they should use function topologies instead, resulting in incorrect audio configurations. Although, admittedly this is not too likely with this 6x amp configuration as those are quite rare, but best to follow best practice.
Note there is a single remaining match which does currently need a match as it contains custom magic in the topology. More work will be required to figure out how we guard off situations like that from interfering with more generic systems.