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: sof-tgl-rt712: remove SDWx from stream name #9008

Merged
merged 1 commit into from
Apr 9, 2024

Conversation

bardliao
Copy link
Collaborator

@bardliao bardliao commented Apr 8, 2024

The widget name size will exceed the character limit and lead to below error

failed to find module info for widget
alh-copier.SDW0-Playback-SimpleJack.0 with UUID
00000000-0000-0000-0000-000000000000

Shorten the stream name to fix it.

The widget name size will exceed the character limit and lead to below
error

failed to find module info for widget
alh-copier.SDW0-Playback-SimpleJack.0 with UUID
00000000-0000-0000-0000-000000000000

Shorten the stream name to fix it.

Signed-off-by: Bard Liao <yung-chuan.liao@linux.intel.com>
@bardliao bardliao requested a review from ranj063 April 8, 2024 05:53
@bardliao bardliao requested a review from jsarha as a code owner April 8, 2024 05:53
@kv2019i
Copy link
Collaborator

kv2019i commented Apr 8, 2024

@jsarha @ranj063 Can we have a tplg buildtime check for the stream name sizes? That would be a very useful sanity check to have in place.

Copy link
Contributor

@jsarha jsarha left a comment

Choose a reason for hiding this comment

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

Unfortunately I can not easily find from where this limitation comes from, but in any case this a necessary fix.

@ranj063
Copy link
Collaborator

ranj063 commented Apr 8, 2024

The widget name size will exceed the character limit and lead to below error

failed to find module info for widget alh-copier.SDW0-Playback-SimpleJack.0 with UUID 00000000-0000-0000-0000-000000000000

Shorten the stream name to fix it.

@bardliao "alh-copier.SDW0-Playback-SimpleJack.0" is 38 characters and AFAIK the character limit is 44. What am I missing? And you're fixing the stream name but mention that the widget name is too long. very confusing.

@ranj063
Copy link
Collaborator

ranj063 commented Apr 8, 2024

@jsarha @ranj063 Can we have a tplg buildtime check for the stream name sizes? That would be a very useful sanity check to have in place.

@kv2019i I'll take a look

@jsarha
Copy link
Contributor

jsarha commented Apr 8, 2024

@jsarha @ranj063 Can we have a tplg buildtime check for the stream name sizes? That would be a very useful sanity check to have in place.

@kv2019i I'll take a look

So limiting factor is SNDRV_CTL_ELEM_ID_NAME_MAXLEN so and its defined 44, right? Bit confusing that CTL ELEM limit applies to other widgets too, or is it all widgets? But in any case, a check somewhere in alsatplg pre-process code should be relatively easy to add. My first instinct would be putting it in the end of tplg_construct_object_name(), but I am not sure about it yet.

However, its still a mystery why a widget with 38 char name gets 00000000-0000-0000-0000-000000000000 UUID.

@bardliao
Copy link
Collaborator Author

bardliao commented Apr 9, 2024

I don't know what happened exactly. But the PR does fix the "failed to find module info for widget alh-copier.SDW0-Playback-SimpleJack.0 with UUID 00000000-0000-0000-0000-000000000000" issue.

@bardliao
Copy link
Collaborator Author

bardliao commented Apr 9, 2024

I can confirm that alh-copier.DW0-Playback-SimpleJack.0 (length = 36) works but alh-copier.SDW0-Playback-SimpleJack.0 (length = 37) doesn't. I added some trace in kernel. And found that we can't find any array with type = SND_SOC_TPLG_TUPLE_TYPE_UUID in sof_parse_token_sets() in the not working case. Maybe something went wrong in topology?

@kv2019i
Copy link
Collaborator

kv2019i commented Apr 9, 2024

Filed enhancement to track this #9011

@kv2019i kv2019i merged commit 2e71793 into thesofproject:main Apr 9, 2024
44 of 45 checks passed
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