Repository navigation
stable-v2.0: set default value instead of hardcoding Zephyr commit - #5672
Conversation
Align with xtensa-build-all.sh and add a "-v" option to toggle
verbose build log ("west -v build").
Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
(cherry picked from commit b95d995)
Same -u option as the new Python script. We need this for the new sof/stable-v2.1 branch on https://github.com/thesofproject/zephyr while our CI has not entirely switched to the python script yet. Signed-off-by: Marc Herbert <marc.herbert@intel.com> (cherry picked from commit 177e18f)
|
This is a preliminary step before fixing stable-v2.0 compatibility with the latest Zephyr SDK, long story in #5654 |
|
https://github.com/thesofproject/sof/runs/5892208159 shows the expected Zephyr commit fd089b361d8aebbcb49cca989444aaeb0b6fbb4d However https://sof-ci.01.org/sofpr/PR5672/build12674/build/tgl_zph.txt shows the most Zephyr commit c11b7852d15c29 @keqiaozhang can you please restrict your (very useful) "STABLE Zephyr cooldown" to the main SOF branch? It should not apply to other SOF branches. Same problem as with the stable-v2.1 branch, example: https://sof-ci.01.org/sofpr/PR5551/build12426/build/tgl_zph.txt |
|
https://sof-ci.01.org/sof-pr-viewer/#/build/PR5672/build9070849 failed with In https://sof-ci.01.org/sofpr/PR5672/build12674/devicetest/ , ZEPHYR configurations had many failures and non-Zephyr failures had none. This is likely because this was using the most current Zephyr branch instead of fd089b361d8aeb as explained above. Everything else passed. EDIT: same failures in https://sof-ci.01.org/sof-pr-viewer/#/build/PR5672/build9082199 and https://sof-ci.01.org/sofpr/PR5672/build12688/devicetest/ |
This reverts commit b678a4d because it's hardcoding the zephyr version real hard. Instead, merely change the default value. Signed-off-by: Marc Herbert <marc.herbert@intel.com>
41a367e to
42a3c3d
Compare
|
Can one of the admins verify this patch? |
|
and now I'm not "whitelisted" in https://quickbuild.igk.intel.com/build/9120013. Never a dull moment (PTS-38280) https://sof-ci.01.org/sof-pr-viewer/#/build/PR5672/build9086308 failed with That likely means that CI cannot support this stable branch which is too old to have IPC4 |
|
@marc-hb yeah, CI does not fully support and work on older branches. Happy to merge if you see it essential since this is all build. |
|
OK let's merge this then. |
4 participants
2 clean cherry-picks from main + unhardcoding the Zephyr commit. Review commits separately.
Zero functional change when you don't (try to) pass a zephyr commit on the command line.