dts: intel_adsp: align boards to new CPU deferral config - #116943
Conversation
On Intel ADSP platforms startup of secondary cores must be deferred till the IPC communication with the host driver. Previously this was achieved through SMP_BOOT_DELAY option. Recently this option has been removed. This change is aimed at maintaining legacy behavior by using new, more granular dts approach. Signed-off-by: Wojciech Jablonski <wojciech.jablonski@intel.com>
zephyrbot
added
area: Xtensa
|
@kv2019i PR seems fixes same thing but differently thesofproject/sof#11098 (first commit there). What solution is preferred? |
|
I considered the overlay solution but decided that this is more of a permanent/fundamental DTS property that won't be frequently modified and therefore it would be better to keep everything in one place. |
|
Btw @nashif @dcpleung , after more discussion, it turns out CONFIG_SMP_BOOT_DELAY might have been a better fit for us (referring to 7a72151 ). SOF assumes this (only start core0) SMP behaviour for all targets, not just Intel hardware. I think we'll now go with this PR, but inherently this deferred SMP boot it not attribute of these boards, but rather an application level choice (application wants to control CPU power up/down more explicitly). Let's see how other vendors want to approach this when they add their multicore configurations. More discussion in thesofproject/sof#11098 (comment) |
7 participants
On Intel ADSP platforms startup of secondary cores must be deferred till the IPC communication with the host driver. Previously this was achieved through SMP_BOOT_DELAY option. Recently this option has been removed. This change is aimed at maintaining legacy behavior by using new, more granular dts approach.
EDIT: This is the original patch that removed SMP_BOOT_DELAY option: 7a72151