Skip to content

ASoC: sdw_utils: Add quirk to ignore sdca RT712/RT721 CODEC_MIC#5739

Open
macchian wants to merge 1 commit intothesofproject:topic/sof-devfrom
macchian:dev-pch-dmic
Open

ASoC: sdw_utils: Add quirk to ignore sdca RT712/RT721 CODEC_MIC#5739
macchian wants to merge 1 commit intothesofproject:topic/sof-devfrom
macchian:dev-pch-dmic

Conversation

@macchian
Copy link
Copy Markdown

When the sof driver selects the ACPI machine tables, some SKUs use the host PCH_DMIC;add quirk to ignore the CODEC_MIC if found in disco table.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds a SoundWire machine-driver quirk mapping so specific SDCA Realtek codecs (RT712/RT721) can skip exposing their CODEC_MIC endpoint on SKUs that instead route microphone capture through the host PCH DMIC when ACPI machine tables are used.

Changes:

  • Mark RT712-SDCA MIC endpoint (aif3) with SOC_SDW_CODEC_MIC quirk and set it as an exclude-quirtk.
  • Mark RT721-SDCA MIC endpoint (aif3) with SOC_SDW_CODEC_MIC quirk and set it as an exclude-quirk.

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

@macchian
Copy link
Copy Markdown
Author

@lgirdwood , could you help review it? Thank you.

Comment thread sound/soc/sdw_utils/soc_sdw_utils.c
@macchian macchian requested a review from ujfalusi April 23, 2026 07:44
@macchian macchian requested a review from bardliao May 4, 2026 03:20
Some devices do not use CODEC_MIC but use the host PCH_DMIC
instead. Add a quirk to skip the CODEC_MIC DAI when it is not present
in disco table, ensuring the correct capture device is used.

If CODEC_MIC is present, it continues to be used as default.

Fixes: f54d87d ("ASoC: rt712: avoid skipping the blind write")
Signed-off-by: Mac Chiang <mac.chiang@intel.com>
@bardliao
Copy link
Copy Markdown
Collaborator

bardliao commented May 4, 2026

The Fixes: commit looks weird. Shouldn't it be 9489db9 ("ASoC: sdw_utils: add SmartMic DAI for RT712 VB")? Or split into 2 commits and the other Fixes commit should be 846a8d3 ("ASoC: Intel: soc-acpi-intel-ptl-match: Add rt721 support")

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.

4 participants