-
Notifications
You must be signed in to change notification settings - Fork 321
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 cs42l43 + cs35l56 topology for MTL and TGL #8359
topology2: add cs42l43 + cs35l56 topology for MTL and TGL #8359
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.
@RanderWang @ranj063 @jsarha in general we need to make sure we add more comments in our topologies like we do with our C cide, especially around any conditional logic.
70b9412
to
e93e3e4
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 of mind, see inline.
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.
@bardliao So end-of-day summary, if you add the MSB_INTEGER to both true and false cases (for 24/32bit), I'm good to go with this. I now understand we may need to signal both options in tplg.
MSB_INTEGER is required for ALH copier. Signed-off-by: Bard Liao <[email protected]>
To indicate the SoundWire amp need to use 24 bit format. I use a flag instead of a macro like $SDW_AMP_FMT to set valid_bit_depth because bit_depth and valid_bit_depth may not be the same. Signed-off-by: Bard Liao <[email protected]>
Use tab instead of space. Signed-off-by: Bard Liao <[email protected]>
To indicate the SoundWire jack need to use 24 bit format. Signed-off-by: Bard Liao <[email protected]>
e93e3e4
to
331639d
Compare
Add topology2 support for cs42l43 + cs35l56. Signed-off-by: Bard Liao <[email protected]>
Add cs42l43 + cs35l56 topology for MTL. Signed-off-by: Bard Liao <[email protected]>
331639d
to
a644197
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.
Thanks @bardliao , looks good now!
@ranj063 @plbossart Final call on this soon... |
topology2: add cs42l43 + cs35l56 topology for MTL and TGL