Skip to content

soundwire: dmi-quirks: add a global ghost list#5826

Open
bardliao wants to merge 1 commit into
thesofproject:topic/sof-devfrom
bardliao:ghost-adr
Open

soundwire: dmi-quirks: add a global ghost list#5826
bardliao wants to merge 1 commit into
thesofproject:topic/sof-devfrom
bardliao:ghost-adr

Conversation

@bardliao

Copy link
Copy Markdown
Collaborator

Not like other ghost devices, the 0x000000D010010500 ADR doesn't belong to any codec. We should disable it in all devices.

Not like other ghost devices, the 0x000000D010010500 ADR doesn't belong
to any codec. We should disable it in all devices.

Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a global SoundWire _ADR “ghost” remap so that a known-bogus ACPI address (0x000000D010010500) is disabled (remapped to 0) on all systems, independent of DMI-specific quirk tables.

Changes:

  • Introduces a global_ghost_adr remap table containing the ghost _ADR → 0 mapping.
  • Extends sdw_dmi_override_adr() to apply the global ghost remap in addition to existing DMI-based remaps.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 188 to +193
for (map = dmi_id->driver_data; map->adr; map++) {
if (map->adr == addr) {
dev_dbg(bus->dev, "remapped _ADR 0x%llx as 0x%llx\n",
addr, map->remapped_adr);
addr = map->remapped_adr;
break;
goto out;

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is on purpose. If we already remapped an addr, we should return that addr. Also, addr should not be 0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants