Skip to content

Conversation

@kv2019i
Copy link
Collaborator

@kv2019i kv2019i commented Dec 15, 2022

Add build check for Intel 'tgl-h' IPC4 build.

Link: #6710
Signed-off-by: Kai Vehmanen kai.vehmanen@linux.intel.com

Add build check for Intel 'tgl-h' IPC4 build.

Link: thesofproject#6710
Signed-off-by: Kai Vehmanen <kai.vehmanen@linux.intel.com>
# Very few IPC3 platforms support IPC4 too.
-i IPC4 tgl,
# Some IPC3 platforms support IPC4 too.
-i IPC4 tgl, -i IPC4 tgl-h,
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For a shorter list:

Suggested change
-i IPC4 tgl, -i IPC4 tgl-h,
-i IPC4 tgl tgl-h,

I will submit that as a followup patch, it's more urgent to merge this as is.

@marc-hb
Copy link
Collaborator

marc-hb commented Dec 15, 2022

It looks like the COMPILER crashed in https://github.com/thesofproject/sof/actions/runs/3703350921/jobs/6274688053, that's unusual! Re-running it...

[ 60%] Building C object CMakeFiles/sof.dir/src/audio/tdfb/tdfb_hifi3.c.o
[ 61%] Building C object CMakeFiles/sof.dir/src/audio/tdfb/tdfb_direction.c.o
[ 61%] Building C object CMakeFiles/sof.dir/src/audio/mixer/mixer.c.o
make[3]: *** [CMakeFiles/sof.dir/build.make:628: CMakeFiles/sof.dir/src/audio/tdfb/tdfb_hifiep.c.o] Segmentation fault (core dumped)
make[3]: *** Waiting for unfinished jobs....
make[2]: *** [CMakeFiles/Makefile2:1335: CMakeFiles/sof.dir/all] Error 2
make[1]: *** [CMakeFiles/Makefile2:2866: src/arch/xtensa/CMakeFiles/bin.dir/rule] Error 2
make: *** [Makefile:957: bin] Error 2

@marc-hb
Copy link
Collaborator

marc-hb commented Dec 15, 2022

https://github.com/thesofproject/sof/actions/runs/3703350921/jobs/6281224222 is passing now. This one-line PR was totally unrelated to the check that crash above.

We unfortunately don't have time to debug Github containers or compiler crashes, let's ignore unless it comes back.

@ranj063 ranj063 merged commit 543acc1 into thesofproject:main Dec 15, 2022
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.

4 participants