Repository navigation
[SKIP SOF-TEST] .github: fuzz IPC4 too - #7579
Conversation
Update comments for brand new IPC4 support. No functional change. Signed-off-by: Marc Herbert <marc.herbert@intel.com>
Fix the comment indentation to show it applies only to the older fuzzer. Signed-off-by: Marc Herbert <marc.herbert@intel.com>
Hot on the heels of PR thesofproject#7531. Signed-off-by: Marc Herbert <marc.herbert@intel.com>
|
https://github.com/thesofproject/sof/actions/runs/4929778823/jobs/8809848778?pr=7579 is a known imx8m rename zephyrproject-rtos/zephyr#57084, unrelated. https://github.com/thesofproject/sof/actions/runs/4929778825/jobs/8809847628?pr=7579 is fuzzing IPC4! |
@marc-hb can you please, help me a bit with this failure? I see here that is using Zephyr's revision: Also, the I've compiled SOF with zephyr for imx8m, both with So, what's different here? Am I missing something? Thank you! |
|
"zmain" checks like https://github.com/thesofproject/sof/actions/runs/4929778823/jobs/8809848778?pr=7579 ignore the manifest, they test for compatibility with the zephyr main branch. They should be warnings not errors but Github supports only two colors.
|
|
https://sof-ci.01.org/sof-pr-viewer/#/build/PR7579/build11962909 failed with the network error below. This can be ignored, this pure Github Actions PR does not affect other tests in any way. Lots of yellow on https://www.githubstatus.com/ right now. |
I see in CI we are doing a "select zephyr revision", but I don't see a " I believe we should have a |
We import hal_xtensa indirectly through zephyr/west.yml. When overriding the Zephyr revision in sof/west.yml to test the Zephyr main branch "zmain", we must also update the version of hal_xtensa specified in zephyr/west.yml. Stop doing a manual, single repo git fetch and use a submanifest to perform this correctly. This is exactly why submanifests/ were added in the first place. These fixes the imx8m compilation error spotted and discussed in (unrelated) thesofproject#7579 following the IMX rename in zephyrproject-rtos/zephyr#57084 and zephyrproject-rtos/zephyr#57795 Signed-off-by: Marc Herbert <marc.herbert@intel.com>
We import hal_xtensa indirectly through zephyr/west.yml. When overriding the Zephyr revision in sof/west.yml to test the Zephyr main branch "zmain", we must also update the version of hal_xtensa specified in zephyr/west.yml. Stop doing a manual, single repo git fetch and use a submanifest to perform this correctly. This is exactly why submanifests/ were added in the first place. These fixes the imx8m compilation error spotted and discussed in (unrelated) #7579 following the IMX rename in zephyrproject-rtos/zephyr#57084 and zephyrproject-rtos/zephyr#57795 Signed-off-by: Marc Herbert <marc.herbert@intel.com>
3 commits