From cb0d2c8759eeb8aa10587983fdbd45994eb5e186 Mon Sep 17 00:00:00 2001 From: Marc Herbert Date: Tue, 31 Oct 2023 06:14:31 +0000 Subject: [PATCH] app/prj.conf: stop forcing CONFIG_OUTPUT_DISASSEMBLY=y This reverts 8f6018057b64 ("app: enable OUTPUT_DISASSEMBLY for Zephyr builds") The disassembly is useful in some contexts but can't be parallelized and takes an enormous amount of time. On my system OUTPUT_DISASSEMBLY almost doubles the compilation time! Quoting Zephyr commit cc5763344709: > This on-by-default option takes the majority of "link" time. If people > need the disassembly, they can turn it on. Signed-off-by: Marc Herbert --- app/prj.conf | 3 --- scripts/xtensa-build-zephyr.py | 2 +- 2 files changed, 1 insertion(+), 4 deletions(-) diff --git a/app/prj.conf b/app/prj.conf index b3f1016620ea..cd182e5be16c 100644 --- a/app/prj.conf +++ b/app/prj.conf @@ -9,9 +9,6 @@ CONFIG_BUILD_OUTPUT_BIN=n # and usage in xtensa-build-zephyr.py CONFIG_BUILD_OUTPUT_STRIPPED=y -# Generate zephyr.lst file -CONFIG_OUTPUT_DISASSEMBLY=y - CONFIG_HAVE_AGENT=n # Since Zephyr commit f0daf904bb0202c PICOLIBC is the new default. diff --git a/scripts/xtensa-build-zephyr.py b/scripts/xtensa-build-zephyr.py index 2e8f810d9444..66e5d80d780f 100755 --- a/scripts/xtensa-build-zephyr.py +++ b/scripts/xtensa-build-zephyr.py @@ -943,7 +943,7 @@ class InstFile: InstFile("include/generated/sof_versions.h", "sof_versions.h", gzip=False, txt=True), InstFile(BIN_NAME + ".elf"), - InstFile(BIN_NAME + ".lst", txt=True), + InstFile(BIN_NAME + ".lst", txt=True, optional=True), InstFile(BIN_NAME + ".map", txt=True), # CONFIG_BUILD_OUTPUT_STRIPPED