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

[FEATURE] add alsa-utils and alsa-lib to SOF manifest #9791

Open
kv2019i opened this issue Jan 22, 2025 · 3 comments
Open

[FEATURE] add alsa-utils and alsa-lib to SOF manifest #9791

kv2019i opened this issue Jan 22, 2025 · 3 comments
Assignees
Labels
enhancement New feature or request topology Topology issues
Milestone

Comments

@kv2019i
Copy link
Collaborator

kv2019i commented Jan 22, 2025

Is your feature request related to a problem? Please describe.
The SOF tools build uses alsatplg tool to build the binary tplg files from sources in SOF git.

Currently making changes that require change in alsatplg tool, need to be first merged to upstream alsa-utils,
then do an update of the SOF CI docker image, and only then a PR to SOF topology content can be submitted and tested.

Describe the solution you'd like
Add alsa-utils and alsa-lib to west manifest directly allowing to test draft version sof alsa-lib/alsa-utils changes and ensure correct version of alsatplg is used in topology builds. Original proposal from @lgirdwood in #9763 (comment)

Describe alternatives you've considered
Current BKM is to track docker updates with separate tickets, but this has proven to be slow and error prone:
#9790

Given the resulting binary tplg files do not encoding tool versioning information, it is very easy to have integration issues in case some alsatplg patch is missing in the tplg build.

Additional context

@kv2019i kv2019i added enhancement New feature or request topology Topology issues labels Jan 22, 2025
@kv2019i
Copy link
Collaborator Author

kv2019i commented Jan 22, 2025

FYI to @singalsu @cgturner1 @ranj063 @udaymb

@lgirdwood lgirdwood added this to the v2.13 milestone Feb 12, 2025
@teburd
Copy link
Contributor

teburd commented Feb 12, 2025

I think this is ok given the issue described. I've been thinking a lot more about west and dependency management as a whole lately as I've I think ran into some corners where west and the ideas behind it (repo) just aren't workable. This case though seems like it'd be ok to me.

@lgirdwood
Copy link
Member

I think this is ok given the issue described. I've been thinking a lot more about west and dependency management as a whole lately as I've I think ran into some corners where west and the ideas behind it (repo) just aren't workable. This case though seems like it'd be ok to me.

Thanks @teburd - I will give this a try and discuss with you next time we chat.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request topology Topology issues
Projects
None yet
Development

No branches or pull requests

3 participants