Audio: add CTC to PTL rt5682 path.#10696
Open
eddy1021 wants to merge 1 commit intothesofproject:mainfrom
Open
Conversation
johnylin76
reviewed
Apr 13, 2026
lgirdwood
reviewed
Apr 13, 2026
Member
lgirdwood
left a comment
lgirdwood
left a comment
There was a problem hiding this comment.
Looks good, just the opens already mentioned.
eddy1021
force-pushed
the
ctc_ptl_rt5682_2
branch
from
1d53704 to
63efc72
Compare
lgirdwood
reviewed
Apr 17, 2026
Member
lgirdwood
left a comment
lgirdwood
left a comment
There was a problem hiding this comment.
There a lot of change here including in common files, can you add more context in the commit message especially around the changes in any common files.
eddy1021
force-pushed
the
ctc_ptl_rt5682_2
branch
from
63efc72 to
c74ac5a
Compare
Author
|
Added more details in the commit message now. Thanks! |
johnylin76
reviewed
Apr 20, 2026
Include CTC component in PTL with rt5682. Introduce `USE_CTC_SPK` in `cavs-mixin-mixout-ctc-ssp.conf` and `cavs-mixin-mixout-dax-ssp.conf` to conditionally include CTC component in the pipeline. Signed-off-by: Eddy Hsu <eddyhsu@google.com>
eddy1021
force-pushed
the
ctc_ptl_rt5682_2
branch
from
c74ac5a to
12be0cd
Compare
lgirdwood
reviewed
Apr 21, 2026
Comment on lines
+128
to
+135
| { | ||
| source "dai-copier.SSP.$HEADSET_CODEC_NAME.capture" | ||
| sink "host-copier.$HEADSET_PCM_ID.capture" | ||
| } | ||
| { | ||
| source "host-copier.$HEADSET_PCM_ID.playback" | ||
| sink "gain.1.1" | ||
| } |
Member
There was a problem hiding this comment.
is there any reason we need to add this here and ...
Comment on lines
-345
to
-352
| { | ||
| source "host-copier.$HEADSET_PCM_ID.playback" | ||
| sink "gain.1.1" | ||
| } | ||
| { | ||
| source "host-copier.$SPEAKER_PCM_ID.playback" | ||
| sink "gain.3.1" | ||
| } |
Member
There was a problem hiding this comment.
...delete this here, wont this impact users who are not using CTC/DAX with this codec ?
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Include CTC component in PTL with rt5682.