From 003166440372810819c16372dfffbff01bdbc55b Mon Sep 17 00:00:00 2001
From: Marc Herbert <marc.herbert@intel.com>
Date: Wed, 12 Jan 2022 18:03:15 +0000
Subject: [PATCH] xtensa-build-all.sh: distinguish default/working platform and
 WIP

The -a option was recently broken by the addition of platform(s) that
don't have a toolchain available in the Docker image. Every platform can
be built by someone but no one can built -a(ll) platforms right now.

Add a new platform array to make the distinction between default
platforms built by the -a option versus work in progress.

Signed-off-by: Marc Herbert <marc.herbert@intel.com>
---
 scripts/xtensa-build-all.sh | 18 ++++++++++++------
 1 file changed, 12 insertions(+), 6 deletions(-)

diff --git a/scripts/xtensa-build-all.sh b/scripts/xtensa-build-all.sh
index b7dd17b05..f331fd970 100755
--- a/scripts/xtensa-build-all.sh
+++ b/scripts/xtensa-build-all.sh
@@ -5,8 +5,15 @@
 # stop on most errors
 set -e
 
-SUPPORTED_PLATFORMS=(byt cht bdw hsw apl skl kbl cnl sue icl jsl \
-                    imx8 imx8x imx8m imx8ulp tgl tgl-h rn mt8195)
+# Platforms with a toolchain available in the latest Docker image and
+# built by the -a option.
+DEFAULT_PLATFORMS=(  byt cht bdw hsw apl skl kbl cnl sue icl jsl \
+                    imx8 imx8x imx8m imx8ulp tgl tgl-h rn mt8195 )
+
+# Work in progress can be added to this "staging area" without breaking
+# the -a option for everyone.
+SUPPORTED_PLATFORMS=( "${DEFAULT_PLATFORMS[@]}" )
+
 BUILD_ROM=no
 BUILD_DEBUG=no
 BUILD_FORCE_UP=no
@@ -51,8 +58,8 @@ https://thesofproject.github.io/latest/developer_guides/firmware/cmake.html
 usage: $0 [options] platform(s)
 
        -r Build rom if available (gcc only)
-       -a Build all platforms
-       -u Force UP ARCH
+       -a Build all platforms which have a toolchain in the latest Docker image
+       -u Force CONFIG_MULTICORE=n
        -d Enable debug build
        -c Interactive menuconfig
        -o arg, copies src/arch/xtensa/configs/override/<arg>.config
@@ -108,7 +115,7 @@ while getopts "rudj:ckvao:m:" OPTION; do
 		k) USE_PRIVATE_KEY=yes ;;
 		o) OVERRIDE_CONFIG=$OPTARG ;;
 		v) BUILD_VERBOSE='VERBOSE=1' ;;
-		a) PLATFORMS=("${SUPPORTED_PLATFORMS[@]}") ;;
+		a) PLATFORMS=("${DEFAULT_PLATFORMS[@]}") ;;
 		m) MEU_TOOL_PATH=$OPTARG ;;
 		*) print_usage; exit 1 ;;
         esac
@@ -416,7 +423,6 @@ do
 		echo "CONFIG_BUILD_VM_ROM=y" >> override.config
 	fi
 
-	# override default ARCH if BUILD_FORCE_UP is set
 	if [ "x$BUILD_FORCE_UP" == "xyes" ]
 	then
 		echo "Force building UP(xtensa)..."