Skip to content

Conversation

@LaurentiuM1234
Copy link
Contributor

@LaurentiuM1234 LaurentiuM1234 commented Oct 21, 2025

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

@LaurentiuM1234 LaurentiuM1234 force-pushed the fix/drop_sof_deps branch 2 times, most recently from 44d2c73 to 2f3f036 Compare October 21, 2025 12:18
@LaurentiuM1234
Copy link
Contributor Author

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 SAI_GetInstance().

@LaurentiuM1234 LaurentiuM1234 changed the title zephyr: CMakeLists.txt: add rimage support for the imx95 target [DO NOT MERGE] zephyr: CMakeLists.txt: add rimage support for the imx95 target Oct 22, 2025
@LaurentiuM1234 LaurentiuM1234 marked this pull request as ready for review October 22, 2025 09:25
@LaurentiuM1234
Copy link
Contributor Author

zephyrproject-rtos/zephyr#97988 merged

@LaurentiuM1234 LaurentiuM1234 changed the title [DO NOT MERGE] zephyr: CMakeLists.txt: add rimage support for the imx95 target zephyr: CMakeLists.txt: add rimage support for the imx95 target Oct 28, 2025
@lyakh lyakh added the DNM Do Not Merge tag label Oct 29, 2025
@lyakh
Copy link
Collaborator

lyakh commented Oct 29, 2025

"DNM" for now for a test west.yml change

@LaurentiuM1234
Copy link
Contributor Author

"DNM" for now for a test west.yml change

good catch, thanks. Forgot to update the hash after zephyrproject-rtos/zephyr#97988 got merged :(

@kv2019i
Copy link
Collaborator

kv2019i commented Oct 29, 2025

This will probably fail due to regression in upstream Zephyr with SOF (see details in #10312 ). The fix in Zephyr side was just merged, so we could now try #10312 again, and after that, this PR should be able to proceed without a manifest update.

@lyakh lyakh removed the DNM Do Not Merge tag label Oct 30, 2025
@lgirdwood
Copy link
Member

@dbaluta good for you ?

@iuliana-prodan
Copy link
Contributor

@dbaluta good for you ?

Daniel is OOO.
So, yes, this is good for us.
Actually this is needed for #10312 otherwise, for imx95 will get:

[351/351] Linking C executable zephyr/zephyr.elf
Memory region         Used Size  Region Size  %age Used
           FLASH:          0 GB         0 GB
             RAM:      634552 B         4 MB     15.13%
        IDT_LIST:          0 GB        32 KB      0.00%
Generating files from /sof/build-imx95/zephyr/zephyr.elf for board: imx95_evk
Traceback (most recent call last):
  File "./scripts/xtensa-build-zephyr.py", line 1366, in <module>
    main()
  File "./scripts/xtensa-build-zephyr.py", line 1358, in main
    build_platforms()
  File "./scripts/xtensa-build-zephyr.py", line 977, in build_platforms
    install_platform(platform, sof_platform_output_dir, platf_build_environ, platform_wcfg)
  File "./scripts/xtensa-build-zephyr.py", line 1148, in install_platform
    shutil.copy2(abs_build_dir / "zephyr.ri", install_key_dir / output_fwname)
  File "/usr/lib/python3.8/shutil.py", line 435, in copy2
    copyfile(src, dst, follow_symlinks=follow_symlinks)
  File "/usr/lib/python3.8/shutil.py", line 264, in copyfile
    with open(src, 'rb') as fsrc, open(dst, 'wb') as fdst:
FileNotFoundError: [Errno 2] No such file or directory: '/sof/build-imx95/zephyr/zephyr.ri'

@kv2019i
Copy link
Collaborator

kv2019i commented Oct 31, 2025

@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.

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>
@LaurentiuM1234
Copy link
Contributor Author

LaurentiuM1234 commented Oct 31, 2025

@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.

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)

@kv2019i
Copy link
Collaborator

kv2019i commented Oct 31, 2025

Ack @LaurentiuM1234 , let's get this merged today on a fast path.

@kv2019i
Copy link
Collaborator

kv2019i commented Oct 31, 2025

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.

@kv2019i kv2019i merged commit cf3a0d2 into thesofproject:main Oct 31, 2025
35 of 41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants