-
Notifications
You must be signed in to change notification settings - Fork 322
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
dp: look at all sinks and sources in default "is ready" #8668
Conversation
The default "is ready to process" implementation used to look at a first sink and first source only. This change makes it to look at all sinks and sources. Signed-off-by: Marcin Szkudlinski <[email protected]>
Ready to go, just waiting for the dependency to be merged. |
@marcinszkudlinski do we need to update the dependencies given we have new PRs from @andyross ? |
@lgirdwood I have this patch in my "all my AEC junk" integration branch at https://github.com/andyross/sof/tree/mtl-aec-on-main and it's working great on MTL for me. There are only three cherry picks from mtl-007-drop-stable needed for working audio at this point[1], the other two being pipeline-related. [1] Though there are about a dozen power/clock management patches in there to both SOF and Zephyr that have skewed vs. what merged upstream. That will probably take some work to untangle. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Forgot +1
@lgirdwood it is independent - just need one of the PRs above merged |
One fail https://sof-ci.01.org/sofpr/PR8668/build1397/devicetest/index.html?model=TGLU_RVP_NOCODEC-ipc4&testcase=check-suspend-resume-with-playback but that's a new problem in baseline #8721 |
The default "is ready to process" implementation used to look at a first sink and first source only.
This change makes it to look at all sinks and sources.
dependency:
module: Add support for saving and retrieving the base config extension #8575OR