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.

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

topology: pre-processor: Add support for enum controls #236

Closed
wants to merge 1 commit into from
Closed

topology: pre-processor: Add support for enum controls #236

wants to merge 1 commit into from

Conversation

ranj063
Copy link
Contributor

@ranj063 ranj063 commented Sep 12, 2023

Add support for adding enum controls in the topology pre-processor.

@ranj063
topology: pre-processor: Add support for enum controls
13eb1f4 
Add support for adding enum controls in the topology pre-processor.

Signed-off-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
singalsu
singalsu approved these changes Sep 12, 2023
View reviewed changes
Copy link
Contributor

@singalsu singalsu left a comment

Choose a reason for hiding this comment

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

The enum control in SOF works with this update. Thanks a lot for adding this!

@singalsu
Copy link
Contributor

@perexg Can this update get priority, the SOF topologies build fails without this.

lgirdwood
lgirdwood approved these changes Oct 17, 2023
View reviewed changes
Copy link

@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.

@perexg @tiwai fyi this is needed for enum kcontrols with topology2 and SOF v2.8

@perexg perexg closed this in 68dd547 Oct 17, 2023
@singalsu
Copy link
Contributor

Thanks @perexg !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@singalsu singalsu singalsu approved these changes

@lgirdwood lgirdwood lgirdwood approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ranj063 @singalsu @lgirdwood