west.yml: update zephyr to 958057176c7

[sneha-voona]

Update west.yml to bring hal_xtensa headers in zephyr for AMD ACP 7X.

[sofci]

Can one of the admins verify this patch?

reply test this please to run this test once

[lgirdwood]

@sneha-voona do we need a later west commit ID as its failing on NXP platform AFAICT: @dbaluta fyi.

g/drivers/esai/fsl_esai.c
/__w/sof/sof/modules/hal/nxp/mcux/mcux-sdk-ng/drivers/esai/fsl_esai.c:1242:2: error: #endif without #if
 1242 | #endif
      |  ^~~~~
[297/358] Building C object modules/xtensa/CMakeFiles/modules_xtensa_hal.dir/src/hal/coherence.c.obj
[298/358] Building ASM object modules/xtensa/CMakeFiles/modules_xtensa_hal.dir/src/hal/clock.S.obj
[299/358] Building C object modules/hal_nxp/CMakeFiles/modules__hal_nxp.dir/__w/sof/sof/modules/hal/nxp/mcux/mcux-sdk-ng/drivers/common/fsl_common_dsp.c.obj
[300/358] Building C object modules/hal_nxp/CMakeFiles/modules__hal_nxp.dir/__w/sof/sof/modules/hal/nxp/mcux/mcux-sdk-ng/drivers/edma_rev2/fsl_edma_rev2.c.obj
[301/358] Building C object modules/hal_nxp/CMakeFiles/modules__hal_nxp.dir/__w/sof/sof/modules/hal/nxp/mcux/mcux-sdk-ng/drivers/sai/fsl_sai.c.obj
ninja: build stopped: subcommand failed.
FATAL ERROR: command exited with status 1: /opt/python/venv/bin/cmake --build /__w/sof/sof/build-imx8
Traceback (most recent call last):
  File "/__w/sof/sof/./sof/scripts/xtensa-build-zephyr.py", line 1419, in <module>
    main()
  File "/__w/sof/sof/./sof/scripts/xtensa-build-zephyr.py", line 1411, in main
    build_platforms()
  File "/__w/sof/sof/./sof/scripts/xtensa-build-zephyr.py", line 1013, in build_platforms
    raise cpe
  File "/__w/sof/sof/./sof/scripts/xtensa-build-zephyr.py", line 1007, in build_platforms
    execute_command(build_cmd, cwd=west_top, env=platf_build_environ, sof_log_env=True)
  File "/__w/sof/sof/./sof/scripts/xtensa-build-zephyr.py", line 488, in execute_command
    return subprocess.run(*run_args, **run_kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/subprocess.py", line 571, in run
    raise CalledProcessError(retcode, process.args,
subprocess.CalledProcessError: Command '['west', 'build', '--build-dir', 'build-imx8', '--board', 'imx8qm_mek/mimx8qm6/adsp', '/__w/sof/sof/sof/app', '--', '-DEXTRA_CFLAGS=-Werror', '-DEXTRA_CXXFLAGS=-Werror', '-DEXTRA_AFLAGS=-Werror -Wa,--fatal-warnings', '--warn-uninitialized', '-DEXTRA_CONF_FILE=/__w/sof/sof/sof/app/configs/repro-build.conf']' returned non-zero exit status 1.
Error: Process completed with exit code 1.

[sneha-voona]

@lgirdwood
The hal_xtensa header changes I needed are also present in newer Zephyr commits.
I can update this to a later commit to see if it fixes the NXP issue—does that sound good to you?

[lgirdwood]

@lgirdwood The hal_xtensa header changes I needed are also present in newer Zephyr commits. I can update this to a later commit to see if it fixes the NXP issue—does that sound good to you?

Yes, it sometimes a bit of trial and error, it could also be that zephyrs own west has not been updated for the NXP hal. @dbaluta any insights here ?

[dbaluta]

@lgirdwood added a fix for this here: zephyrproject-rtos/hal_nxp#755

@sneha-voona It will take few days until it propagates so if updating zephyr to an older hash works for you go ahead.

Later edit: Zephyr PR here: zephyrproject-rtos/zephyr#111877