Skip to content

Added sof-ptl-cs42l43-agg-l3-cs35l56-l2-dax.tplg for PTL platform #10411

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.

Sign up for GitHub

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

Jump to bottom
Merged
lgirdwood merged 1 commit into from
Dec 22, 2025
Merged

Added sof-ptl-cs42l43-agg-l3-cs35l56-l2-dax.tplg for PTL platform #10411

lgirdwood merged 1 commit into from
Dec 22, 2025

Conversation

@checkupup
Copy link
Contributor

@checkupup checkupup commented Dec 3, 2025

Added sof-ptl-cs42l43-agg-l3-cs35l56-l2-dax.tplg for PTL platform.

@checkupup checkupup changed the title Adapted to PTL platform Added sof-ptl-cs42l43-agg-l3-cs35l56-l2-dax.tplg for PTL platform Dec 3, 2025
@sofci
Copy link
Collaborator

sofci commented Dec 3, 2025

Can one of the admins verify this patch?

reply test this please to run this test once

@johnylin76
Copy link
Contributor

johnylin76 commented Dec 4, 2025

I guess you could un-draft this PR to start the review then

@checkupup checkupup marked this pull request as ready for review December 4, 2025 08:18
@kv2019i kv2019i requested a review from bardliao December 4, 2025 13:44
kv2019i
kv2019i approved these changes Dec 4, 2025
View reviewed changes
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 @checkupup , looks good! Some comments inline, but nothing really that would block this PR.

checkupup
checkupup commented Dec 5, 2025
View reviewed changes
Copy link
Contributor Author

@checkupup checkupup left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems like this PR is ready for merging

@lgirdwood
Copy link
Member

lgirdwood commented Dec 5, 2025

Seems like this PR is ready for merging

@johnylin76 if your testing on your end we can wait or we can merge and fix any issue incrementally ?

johnylin76
johnylin76 reviewed Dec 7, 2025
View reviewed changes
johnylin76
johnylin76 reviewed Dec 8, 2025
View reviewed changes
@checkupup checkupup force-pushed the dax_ptl_tplg branch 2 times, most recently from c9b756c to a1a1a41 Compare December 9, 2025 01:51
@johnylin76
Copy link
Contributor

johnylin76 commented Dec 18, 2025

I think we are all good and this PR is ready to merge, thanks

@kv2019i
Copy link
Collaborator

kv2019i commented Dec 18, 2025

test this please

@kv2019i
Copy link
Collaborator

kv2019i commented Dec 18, 2025

Above message to triggering a build test on this. We can then merge.

@kv2019i
Copy link
Collaborator

kv2019i commented Dec 19, 2025

SOFCI TEST

@lgirdwood
Copy link
Member

lgirdwood commented Dec 19, 2025

@checkupup I think we are good to go - just a conflict to resolve and then we can merge after CI run. Thanks !

@checkupup
tools: topology: add dax module to ptl topologies
f27ddcc 
Added sof-ptl-cs42l43-agg-l3-cs35l56-l2-dax.tplg for
PTL platform.

Signed-off-by: Jun Lai <jun.lai@dolby.com>
@checkupup
Copy link
Contributor Author

checkupup commented Dec 22, 2025

@checkupup I think we are good to go - just a conflict to resolve and then we can merge after CI run. Thanks !

@lgirdwood conflict has been resolved. Please rerun CI TEST, thanks.

@bardliao
Copy link
Collaborator

bardliao commented Dec 22, 2025

test this please

@lgirdwood lgirdwood merged commit 42d9472 into thesofproject:main Dec 22, 2025
42 of 45 checks passed
@checkupup checkupup deleted the dax_ptl_tplg branch December 23, 2025 01:53
@lyakh lyakh mentioned this pull request Dec 23, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lgirdwood lgirdwood lgirdwood approved these changes

@johnylin76 johnylin76 johnylin76 approved these changes

@kv2019i kv2019i kv2019i approved these changes

@ranj063 ranj063 Awaiting requested review from ranj063 ranj063 is a code owner

@jsarha jsarha Awaiting requested review from jsarha jsarha is a code owner

@plbossart plbossart Awaiting requested review from plbossart plbossart is a code owner

@mmaka1 mmaka1 Awaiting requested review from mmaka1 mmaka1 is a code owner

@lbetlej lbetlej Awaiting requested review from lbetlej lbetlej is a code owner

@dbaluta dbaluta Awaiting requested review from dbaluta dbaluta is a code owner

@bardliao bardliao Awaiting requested review from bardliao

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@checkupup @sofci @johnylin76 @lgirdwood @kv2019i @bardliao @jsarha