-
Notifications
You must be signed in to change notification settings - Fork 349
Topology2: Add function topologies with different BE IDs #10157
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Topology2: Add function topologies with different BE IDs #10157
Conversation
There was a problem hiding this 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 |
|
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? |
9e7ad24 to
9a2fa17
Compare
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" |
There was a problem hiding this comment.
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 ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch!
There was a problem hiding this comment.
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>
9a2fa17 to
6eb37a4
Compare
|
@simontrimmer @jsarha pls review/approve if good for you now. Thanks! |
The DMIC and HDMI BE IDs depends on how many SDW BEs are created. Add some missing function topologies.