Skip to content

Conversation

@bardliao
Copy link
Collaborator

@bardliao bardliao commented Aug 8, 2025

The DMIC and HDMI BE IDs depends on how many SDW BEs are created. Add some missing function topologies.

Copilot AI review requested due to automatic review settings August 8, 2025 01:40
@bardliao bardliao requested review from jsarha and ranj063 as code owners August 8, 2025 01:40
Copy link

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

This PR adds additional topology configurations to support different Backend Element (BE) IDs for DMIC and HDMI interfaces when SDW BEs are present, addressing the dependency between SDW BE count and DMIC/HDMI BE ID assignment.

  • Adds HDMI topology variant with ID 6 for SDW configurations
  • Adds DMIC topology variants with IDs 3 and 4 for both 2-channel and 4-channel configurations across ACE1 and ACE2 platforms
  • Extends support for multiple platform variants (ARL, MTL, LNL) with consistent ID patterns

Reviewed Changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
tplg-targets-sdca-generic.cmake Adds HDMI PCM topology with ID 6 for SDW configurations
tplg-targets-ace2.cmake Adds DMIC topologies with IDs 3-4 for LNL platform in 2ch and 4ch variants
tplg-targets-ace1.cmake Adds DMIC topologies with IDs 3-4 for both ARL and MTL platforms in 2ch and 4ch variants

@simontrimmer
Copy link
Contributor

In testing one of the scenarios I booted was codec-only and that attempted to load an sof-lnl-dmic-2ch-id2.tplg It's a bit of a weird one as far as products are concerned but I guess it's theoretically possible?

@bardliao bardliao force-pushed the various-function-topologies branch from 9e7ad24 to 9a2fa17 Compare August 11, 2025 05:25
@bardliao
Copy link
Collaborator Author

In testing one of the scenarios I booted was codec-only and that attempted to load an sof-lnl-dmic-2ch-id2.tplg It's a bit of a weird one as far as products are concerned but I guess it's theoretically possible?

Yes, it is possible. Updated.


"cavs-sdw\;sof-arl-dmic-4ch-id2\;PLATFORM=mtl,SDW_JACK=false,NUM_HDMIS=0,NUM_DMICS=4,\
PDM1_MIC_A_ENABLE=1,PDM1_MIC_B_ENABLE=1,DMIC0_ID=2,DMIC1_ID=3,PREPROCESS_PLUGINS=nhlt,\
NHLT_BIN=nhlt-sof-arl-dmic-4ch-id3.bin"
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this correct? The tplg file is named sof-arl-dmic-4ch-id2, but the NHLT .bin is nhlt-sof-arl-dmic-4ch-id3.bin ?

Copy link
Contributor

Choose a reason for hiding this comment

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

Good catch!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Thanks @jsarha, fixed.

The DMIC BE ID could be 3 or 4 depends on how many SDW BEs are created.

Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
The HDMI BE ID depends on how many SDW BEs are created. Add missing
BE ID 4 and 6.

Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
@bardliao bardliao force-pushed the various-function-topologies branch from 9a2fa17 to 6eb37a4 Compare August 14, 2025 12:26
@lgirdwood
Copy link
Member

@simontrimmer @jsarha pls review/approve if good for you now. Thanks!

@kv2019i kv2019i merged commit d677107 into thesofproject:main Aug 18, 2025
39 of 45 checks passed
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.

5 participants