On 6/18/2018 22:58, Liam Girdwood wrote:
On Mon, 2018-06-18 at 18:53 +0800, Pan, Xiuli wrote:
On 6/14/2018 23:34, Liam Girdwood wrote:
BXT/APL compiler is the same but can be have either apl or bxt suffix.
Signed-off-by: Liam Girdwood liam.r.girdwood@linux.intel.com
scripts/xtensa-build-all.sh | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-)
diff --git a/scripts/xtensa-build-all.sh b/scripts/xtensa-build-all.sh index d873d7e..62b94b2 100755 --- a/scripts/xtensa-build-all.sh +++ b/scripts/xtensa-build-all.sh @@ -96,7 +96,12 @@ do PLATFORM="apollolake" XTENSA_CORE="X4H3I16w2D48w3a_2017_8" ROOT="$pwd/../xtensa-root/xtensa-bxt-elf"
HOST="xtensa-bxt-elf"
if [ -x xtensa-bxt-elf-gcc ]This -x could not work on my machine. Even if I try to -x with vim or bash that was in /usr/bin this could not return true, I think we need find some alternative method to check if we have these compilers.
Can you try this on your PC :-
From 98e48d5cc5f38e4362642e5aa31ccf2dc1032c35 Mon Sep 17 00:00:00 2001 From: Liam Girdwood liam.r.girdwood@linux.intel.com Date: Mon, 18 Jun 2018 15:57:26 +0100 Subject: [PATCH] scripts: xtensa-build: fix alias detection for bxt/apl gcc
use bash builtin type instead of -x (requires full path)
Signed-off-by: Liam Girdwood liam.r.girdwood@linux.intel.com
scripts/xtensa-build-all.sh | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/scripts/xtensa-build-all.sh b/scripts/xtensa-build-all.sh index 62b94b2..7b7a8fa 100755 --- a/scripts/xtensa-build-all.sh +++ b/scripts/xtensa-build-all.sh @@ -96,7 +96,10 @@ do PLATFORM="apollolake" XTENSA_CORE="X4H3I16w2D48w3a_2017_8" ROOT="$pwd/../xtensa-root/xtensa-bxt-elf"
if [ -x xtensa-bxt-elf-gcc ]
# test APL compiler aliasestype xtensa-bxt-elf-gcc
Type command works fine for me. This patch is good for me.
Thanks Xiuli
if [ $? == 0 ]