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: cavs-cs42l43: Add topology for cs42l43/cs35l56 on MTL #8773
Conversation
Corresponding mtl-match.c file change: https://lore.kernel.org/lkml/20240123113246.75539-1-mstrozek@opensource.cirrus.com/T/#u |
@@ -32,6 +32,12 @@ NUM_HDMIS=0,SDW_SPK_STREAM=Playback-SmartAmp,SDW_SPK_IN_STREAM=Capture-SmartAmp, | |||
SDW_DMIC_STREAM=Capture-SmartMic,SDW_JACK_OUT_STREAM=Playback-SimpleJack,\ | |||
SDW_JACK_IN_STREAM=Capture-SimpleJack,SDW_AMP_FMT_24=true,SDW_JACK_FMT_24=true" | |||
|
|||
"cavs-sdw\;sof-mtl-cs42l43-l0-cs35l56-l23\;PLATFORM=mtl,NUM_SDW_AMP_LINKS=2,SDW_DMIC=1,\ | |||
NUM_HDMIS=0,SDW_AMP_FEEDBACK=false,SDW_SPK_STREAM=Playback-SmartAmp,\ |
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.
please remove NUM_HDMIS=0, this is no longer needed.
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.
we also need to fix line 31 @bardliao, NUM_HDMI=0 is a leftover we need to clean-up
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.
a7a2ff8
to
284b20e
Compare
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.
change id nit pick, sound good otherwise.
The CS35L56 has feedback but it is currently only used for debug. As it isn't currently needed in production it seems safer to leave it off for now. We could turn it on in future. There is no need to remove the feedback from sof-mtl-cs42l43-l0-cs35l56-l12. That is for the EIGHT-C/AIC board, right? Which is a development system so debugging is useful. |
The layout is configured as: - Link0: CS42L43 Jack and mics - Link2: 2x CS35L56 Speaker (amps 3 and 4, right) - Link3: 2x CS35L56 Speaker (amps 1 and 2, left) Signed-off-by: Maciej Strozek <mstrozek@opensource.cirrus.com>
284b20e
to
8d988ab
Compare
The layout is configured as: - Link0: CS42L43 Jack and mics (2ch) - Link2: 2x CS35L56 Speaker (amps 3 and 4, right) - Link3: 2x CS35L56 Speaker (amps 1 and 2, left) Corresponding SOF topology: thesofproject/sof#8773 Signed-off-by: Maciej Strozek <mstrozek@opensource.cirrus.com> Link: https://msgid.link/r/20240123113246.75539-1-mstrozek@opensource.cirrus.com Reviewed-by: Charles Keepax <ckeepax@opensource.cirrus.com> Signed-off-by: Mark Brown <broonie@kernel.org>
The layout is configured as:
- Link0: CS42L43 Jack and mics
- Link2: 2x CS35L56 Speaker (amps 3 and 4, right)
- Link3: 2x CS35L56 Speaker (amps 1 and 2, left)