Skip to content

Conversation

@ranj063
Copy link
Collaborator

@ranj063 ranj063 commented May 5, 2021

This reverts commit 2c69afc.
The dynamic pipeline feature is broken with PA enabled. Revert
this change until the issue is sorted.

Signed-off-by: Ranjani Sridharan ranjani.sridharan@linux.intel.com

This reverts commit 2c69afc.
The dynamic pipeline feature is broken with PA enabled. Revert
this change until the issue is sorted.

Signed-off-by: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
@ranj063 ranj063 requested review from lgirdwood and plbossart May 5, 2021 18:58
@ranj063
Copy link
Collaborator Author

ranj063 commented May 5, 2021

@lgirdwood this is critical for 1.8

Copy link
Member

@plbossart plbossart left a comment

Choose a reason for hiding this comment

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

Agree, we clearly have major gaps in our validation with PulseAudio removed from all CI devices.

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.

A link to a bug would be very nice in the git commit (to help understand the issue and later confirm whether the particular issue is fixed). But given this is critical for 1.8 release, ok with the change as-it-is as well. I'm assuming this goes beyond what we are already covering in thesofproject/linux#2888 ..?

@lgirdwood lgirdwood merged commit f2bd51a into thesofproject:main May 6, 2021
@ranj063 ranj063 deleted the fix/hda_dyn_pipes branch May 12, 2021 18:28
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.

4 participants