Skip to content

Audio: add CTC to PTL cs42l43 path.#10825

Open
eddy1021 wants to merge 3 commits into
thesofproject:mainfrom
eddy1021:ctc_ptl_cs42l43
Open

Audio: add CTC to PTL cs42l43 path.#10825
eddy1021 wants to merge 3 commits into
thesofproject:mainfrom
eddy1021:ctc_ptl_cs42l43

Conversation

@eddy1021
Copy link
Copy Markdown

Include CTC component in PTL with cs42l43.

Eddy Hsu added 2 commits May 28, 2026 18:07
Include CTC component in PTL with cs42l43.

Signed-off-by: Eddy Hsu <eddyhsu@google.com>
Include CTC component in PTL with rt721.

Signed-off-by: Eddy Hsu <eddyhsu@google.com>
@eddy1021 eddy1021 requested a review from johnylin76 June 1, 2026 15:28
@eddy1021 eddy1021 marked this pull request as ready for review June 1, 2026 17:25
@eddy1021 eddy1021 requested review from jsarha and ranj063 as code owners June 1, 2026 17:25
@lgirdwood
Copy link
Copy Markdown
Member

@bardliao pls check

Comment thread tools/topology/topology2/platform/intel/sdw-amp-dax-ctc.conf Outdated
DEEPBUFFER_FW_DMA_MS=10,DEEP_BUF_SPK=true,BT_PCM_ID=20,BT_ID=8,BT_PCM_NAME=Bluetooth,ADD_BT=true,\
SDW_SPK_ENHANCED_PLAYBACK=false"

"cavs-sdw\;sof-ptl-cs42l43-agg-l3-cs35l56-l2-ctc\;PLATFORM=ptl,SDW_DMIC=1,NUM_SDW_AMP_CTC_LINKS=2,\
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It's not easy for me to catch the change until I realize the deviation is brought by either "NUM_SDW_AMP_LINKS" or "NUM_SDW_AMP_CTC_LINKS" in use.

That also makes sense to me, but I remember NUM_SDW_AMP_LINKS has been referenced by some codes to get the number of speaker. Could you confirm that doesn't matter for the cases using NUM_SDW_AMP_CTC_LINKS?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

NUM_SDW_AMP_LINKS means how many SoundWire links do the amps connect to. For example, NUM_SDW_AMP_LINKS should be 2 if 4 amps connect to 2 SDW links. And NUM_SDW_AMP_LINKS should be 1 if 4 amps connect to 1 SDW link.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I have updated the CTC conf files to use NUM_SDW_AMP_CTC_LINKS. This variable correctly scales the DAIs and channels (e.g., 2 links = 4 speakers/channels). NUM_SDW_AMP_LINKS is only used in topology files.

Fix NUM_SDW_AMP_CTC_LINKS definitions in CTC related topologies.

Signed-off-by: Eddy Hsu <eddyhsu@google.com>
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