-
Notifications
You must be signed in to change notification settings - Fork 349
zephyr: CMakeLists.txt: add rimage support for the imx95 target #10319
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
44d2c73 to
2f3f036
Compare
|
MX93 build warnings to be expected. It seems like the HAL SAI driver was updated. During the update, the fix that was supposed to silence these warning was removed. The warnings are generated by |
2f3f036 to
34e5ee2
Compare
|
"DNM" for now for a test west.yml change |
34e5ee2 to
74783f7
Compare
good catch, thanks. Forgot to update the hash after zephyrproject-rtos/zephyr#97988 got merged :( |
|
@dbaluta good for you ? |
Daniel is OOO. |
|
@LaurentiuM1234 Can you refresh the PR now that west.yml is updated? I didn't see imx95 fails in #10312 , but in any case (did I just miss the fails), better to merge this ASAP. |
74783f7 to
b842f0c
Compare
Zephyr PR #97988 attempts to drop all of the SOF dependencies from Zephyr for the imx95 and imx93 EVK boards. Because of this, move the cmake statement setting the RIMAGE_TARGET from Zephyr to SOF. Signed-off-by: Laurentiu Mihalcea <laurentiu.mihalcea@nxp.com>
double-checked locally with my HEAD at 7bcdf6f (latest main) and the MX95 build does fail rebased on latest main and dropped the manifest update (already taken care of by #10312) |
|
Ack @LaurentiuM1234 , let's get this merged today on a fast path. |
|
LP64-WIP fail is due to #10344 This PR only affects IMX builds, so no need to wait for the remaining Intel CI jobs to complete. Proceeding with merge. |
Zephyr PR #97988 attempts to drop all of the SOF dependencies from Zephyr for the imx95 and imx93 EVK boards. Because of this, move the cmake statement setting the RIMAGE_TARGET from Zephyr to SOF.
Depends on zephyrproject-rtos/zephyr#97988
This commit also includes a manifest update