Skip to content

Conversation

@macchian
Copy link

If SDCA property is not present in the DisCo table, do not skip codec dai endpoints. This ensures that dai links can still be
created from codec_info_list instead of being ignored.
By enabling the default SoundWire driver and best_effort tplg on PTL, functional topologies work on RT721 and multi-lane amplifier on link 3.

.link_mask = BIT(3),
.links = ptl_rt721_l3,
.drv_name = "sof_sdw",
.sof_tplg_filename = "sof-ptl-rt721.tplg",
Copy link
Collaborator

Choose a reason for hiding this comment

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

sof-ptl-rt721.tplg is already in https://github.com/thesofproject/sof-bin/tree/main/v2.14.x/sof-ipc4-tplg-v2.14. It will use function topology even if we don't remove the item. Not sure if we want to remove this. @ujfalusi @kv2019i What do you think?

Copy link
Author

Choose a reason for hiding this comment

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

@bardliao , in the match table, sof-ptl-rt721.tplg has high priority to use, isn't it ? Or did I miss any related patch?

Copy link
Collaborator

Choose a reason for hiding this comment

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

No, sof-ptl-rt721.tplg is a fallback when not all required function topologies are found in the file system.

If SDCA property is not present in the DisCo table, do not skip
codec dai endpoints. This ensures that dai links can still be
created from codec_info_list instead of being ignored.

Signed-off-by: Mac Chiang <mac.chiang@intel.com>
By enabling the default SoundWire driver and best_effort tplg on PTL,
functional topologies work on RT721 and multi-lane amplifier on link 3.

Signed-off-by: Mac Chiang <mac.chiang@intel.com>
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.

2 participants