Update Zephyr to include CONFIG_MODULES support #8732
Conversation
lyakh
requested review from
dbaluta,
kv2019i,
lbetlej,
lgirdwood,
mmaka1 and
plbossart
as code owners
|
Rebased commit? |
Rebased yes, the final one: zephyrproject-rtos/zephyr@c3a6274bf5e4 |
|
Newer attempt that includes @lyakh 's fix for the DSP panics: |
lyakh
changed the title
|
@lyakh lot of red on the CI, do you think this is the Zephyr upgrade ? |
@lgirdwood wow... that is... a huge number of failures. And single-core configurations failed even earlier than the multicore nocodec topology. Yes, this must be because of a Zephyr upgrade. There's nothing else in this PR. I suspect HDA DMA changes in Zephyr |
This changes the secondary core power up routine to use the newly
introduced k_smp_cpu_start() and k_smp_cpu_resume(). This removes
the need to mirror part of the SMP start up code from Zephyr, and
no longer need to call into Zephyr private kernel code.
West update includes :
eefaeee061c8 kernel: smp: introduce k_smp_cpu_resume
042cb6ac4e00 soc: intel_adsp: enable DfTTS-based time stamping
on ACE platforms
6a0b1da158a4 soc: intel_adsp: call framework callback function for restore
e7217925c93e ace: use a 'switch' statement in pm_state_set()
c99a604bbf2c ace: remove superfluous variable initialisation
a0ac2faf9bde intel_adsp: ace: enable power domain
4204ca9bcb3f ace: fix DSP panic during startup (fixes c3a6274bf5e4)
d4b0273ab0c4 cmake: sparse.template: add COMMAND_ERROR_IS_FATAL
ca12fd13c6d3 xtensa: intel_adsp: fix a cache handling error
0ee1e28a2f5f xtensa: polish doxygen and add to missing doc
035c8d8ceb4b xtensa: remove sys_define_gpr_with_alias()
a64eec6aaeec xtensa: remove XTENSA_ERR_NORET
Signed-off-by: Daniel Leung <daniel.leung@intel.com>
Signed-off-by: Rander Wang <rander.wang@intel.com>
[guennadi.liakhovetski@linux.intel.com: update Zephyr hversion]
Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
z_soc_uncached_ptr() / z_soc_cached_ptr() have been removed from Zephyr and replaced with sys_cache_uncached_ptr_get() and sys_cache_cached_ptr_get() respectively. Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
|
"Internal Intel CI" results look good for both cAVS and ACE, but QB caught a DSP panic... |
|
Hooray! Some successful Zephyr update at last. I see there are still a couple hacks and the west.yml manifest still points at a personal branch but the latest test results look very promising:
|
lyakh
force-pushed
the
zephyr
branch
2 times, most recently
from
080d54e to
1dcbd41
Compare
|
SOFCI TEST |
|
The fails all seem to relate to errors returned by HDA-DMA driver's dma_reload(). I suspect these are now triggered by @lyakh your commit zephyrproject-rtos/zephyr#68945 . That -EIO return is visible in multiple failure case FW logs. |
Test a recent Zephyr version with additional patches to address multiple multicore issues. Signed-off-by: Guennadi Liakhovetski <guennadi.liakhovetski@linux.intel.com>
|
Excellent, now the mandatory tests are passing! I will mark this with DNM for now, as this points to unmerged Zephyr PRs, and this is not evident in the title. Please remove when ready. |
5 participants
Update to a Zephyr version, including CONFIG_MODULES support. This is needed for #8180, but is now also a general long overdue Zephyr upgrade
cc: