Skip to content

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 <yung-chuan.liao@linux.intel.com>
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 <yung-chuan.liao@linux.intel.com>
Use tab instead of space.

Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
To indicate the SoundWire jack need to use 24 bit format.

Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
@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 <yung-chuan.liao@linux.intel.com>
Add cs42l43 + cs35l56 topology for MTL.

Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
@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