-
Notifications
You must be signed in to change notification settings - Fork 321
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
xtensa-build-zephyr.py: fix error message. Simplify and clarify cadence environment variables #8890
Conversation
Like most error handling, the exception was never tested and the error message was wrong. Signed-off-by: Marc Herbert <[email protected]>
Remove unnecessary complexities in the Python code. Document all variables in the comments. Signed-off-by: Marc Herbert <[email protected]>
Pure formatting simplification, zero logical change. Signed-off-by: Marc Herbert <[email protected]>
This has always been mysterious, try to shed a bit of light. Signed-off-by: Marc Herbert <[email protected]>
https://sof-ci.01.org/sof-pr-viewer/#/build/PR8890/build13647991 is green. CAVS https://sof-ci.01.org/sofpr/PR8890/build2992/devicetest/index.html has only one suspend/resume failure. ACE https://sof-ci.01.org/sofpr/PR8890/build2991/devicetest/index.html?model=MTLP_SDW_AIOC&testcase=check-kmod-load-unload has only one failure on Very strange but totally unrelated to this pure xtensa-build-zephyr.py change. |
This comment was marked as outdated.
This comment was marked as outdated.
The errors did not actually matter. What matters is: pipewire keeps |
This comment was marked as outdated.
This comment was marked as outdated.
Single, unrelated "signal overflow" in ACE https://sof-ci.01.org/sofpr/PR8890/build2994/devicetest/index.html?model=MTLP_RVP_NOCODEC&testcase=check-alsabat-nocodec-32bits-599. Everything else green. 1 suspend/resume failure in CAVS https://sof-ci.01.org/sofpr/PR8890/build2995/devicetest/index.html , everything else green. https://sof-ci.01.org/sof-pr-viewer/#/build/PR8890/build13647991 is still green. |
4 commits, see commit messages.