Skip to content
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

Merged
merged 6 commits into from
Nov 15, 2023

Conversation

bardliao
Copy link
Collaborator

topology2: add cs42l43 + cs35l56 topology for MTL and TGL

Copy link
Member

@lgirdwood lgirdwood left a 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.

@bardliao bardliao force-pushed the sof-tgl-cs42l43-l3-cs35l56-l01 branch from 70b9412 to e93e3e4 Compare November 6, 2023 01:36
@lgirdwood
Copy link
Member

@kv2019i @ranj063 comments addressed ?

@lgirdwood lgirdwood added this to the v2.8 milestone Nov 10, 2023
Copy link
Collaborator

@kv2019i kv2019i left a 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.

Copy link
Collaborator

@kv2019i kv2019i left a 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]>
@bardliao bardliao force-pushed the sof-tgl-cs42l43-l3-cs35l56-l01 branch from e93e3e4 to 331639d Compare November 14, 2023 07:07
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]>
@bardliao bardliao force-pushed the sof-tgl-cs42l43-l3-cs35l56-l01 branch from 331639d to a644197 Compare November 14, 2023 08:56
Copy link
Collaborator

@kv2019i kv2019i left a 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!

@kv2019i
Copy link
Collaborator

kv2019i commented Nov 14, 2023

@ranj063 @plbossart Final call on this soon...

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