soc: intel_adsp: cavs_v20: fix overlapping .noinit and .cached#40472
soc: intel_adsp: cavs_v20: fix overlapping .noinit and .cached#40472dcpleung wants to merge 1 commit intozephyrproject-rtos:mainfrom
Conversation
|
Again no reason to refuse it, but #39603 contains a unified linker script for all cavs soc's. Continuing to massage this in place instead of trying to get that merged seems like wasted work. |
|
This may be wasted effort but unlike #39603 this is a single commit faster to review/approve/test/merge/cherry-pick/revert/backport/etc. and can quickly address the ICL build regression introduced by #40319 which would allow reverting thesofproject/sof@d4b8a01f206ca7a9 that temporarily disables ICL (bad). Successfully build-Tested-By: @marc-hb with |
|
Nit: the commit message should really reference 2d5c724 |
Previous commit 2d5c724 and commit 566c466) changed the linker script for intel_adsp/cavs_v20 to fix XCC build issues resulted in another problem where .noinit and .cached both had overlapping "cached" addresses. Rimage thus refused to build the firmware. This updates the linker script to align it closer to cavs_v25 which is working with rimage. Signed-off-by: Daniel Leung <daniel.leung@intel.com>
dcpleung
force-pushed
the
intel_adsp_cavs20_linker_script_fix
branch
from
b349a8d to
8149ba4
Compare
|
Updated the commit message to mention the previous commits. There is no code changes. Note that this is just a quick fix to un-break the build and image signing. Will close this if #39603 gets merged before this one. |
|
#39603 is merged, so closing this one. |
6 participants
Previous changes to linker script for intel_adsp/cavs_v20 to fix
XCC build issues resulted in another problem where .noinit
and .cached both had overlapping "cached" addresses. Rimage
thus refused to build the firmware.
This updates the linker script to align it closer to cavs_v25
which is working with rimage.
Signed-off-by: Daniel Leung daniel.leung@intel.com