zephyr: samples: audio: sof: compilation issue, include file not found.

Describe the bug While running automatic tests on STM32 test bench, samples/audio/sof build fails to build. This appends on all STM32 boards but could also be seen on NON STM32 (frdm_k64f)

This is a new sample introduced by #30336

To Reproduce Steps to reproduce the behavior:

scripts/twister --ninja --runtime-artifact-cleanup -b -p nucleo_f429zi -T samples/audio/sof/ --outdir ./log_db_audio
See error

Expected behavior Build successful or sample not build for those boards

Logs and console output

In file included from /home/osboxes/zephyr/zephyr_project/modules/audio/sof/zephyr/../src/arch/xtensa/include/arch/drivers/timer.h:13,
                 from /home/osboxes/zephyr/zephyr_project/modules/audio/sof/zephyr/../src/include/sof/drivers/timer.h:11,
                 from /home/osboxes/zephyr/zephyr_project/modules/audio/sof/src/lib/clk.c:10:
/home/osboxes/zephyr/zephyr_project/modules/audio/sof/zephyr/../src/include/sof/drivers/interrupt.h:11:10: fatal error: platform/drivers/interrupt.h: No such file or directory
   11 | #include <platform/drivers/interrupt.h>

Environment (please complete the following information):

OS: Linux
Toolchain Zephyr SDK
Commit SHA: 536f0f5acbce4c974709d5c5ce224eaac6b08022

Additional context In sample.yaml there is:

    integration_platforms:
      - intel_adsp_cavs15

So is it normal that twister try to build for other platforms ? (I don’t catch exactly the difference between integration_platforms and platform_allow)

About this issue

Original URL
State: closed
Created 3 years ago
Comments: 16 (6 by maintainers)

Commits related to this issue

rimage: Put rimage termporarily in zephyr/ext While we wait for resolution of issue with submodules, put rimage in zephyr/ and make it part of the module. See https://github.com/zephyrproject-rtos/z... — committed to zephyrproject-rtos/sof by nashif 3 years ago

Most upvoted comments

depends on SOC_FAMILY_INTEL_ADSP

we should not add such dependency, the idea is to make SOF also work on non Intel SoCs…

lets just limit what platforms this can run on using explicit platform filters.

nashif on Jan 7, 2021

@lgirdwood I think something like:

diff --git a/modules/Kconfig.sof b/modules/Kconfig.sof
index 17e40df34b..fb5b5c125e 100644
--- a/modules/Kconfig.sof
+++ b/modules/Kconfig.sof
@@ -3,5 +3,6 @@
 
 config SOF
        bool "Sound Open Firmware (SOF)"
+       depends on SOC_FAMILY_INTEL_ADSP
        help
          Build Sound Open Firmware (SOF) support.
diff --git a/samples/audio/sof/sample.yaml b/samples/audio/sof/sample.yaml
index 32ba0df254..c726df2fc4 100644
--- a/samples/audio/sof/sample.yaml
+++ b/samples/audio/sof/sample.yaml
@@ -7,5 +7,6 @@ tests:
   sample.audio.sof:
     tags: sof
     build_only: true
+    filter: ( CONFIG_SOF )
     integration_platforms:
       - intel_adsp_cavs15

as a start.

galak on Jan 7, 2021