Clang doesn't support the option and even on GCC it's ignored whenver -mabi=aapcs-linux is specified[1]: -mthumb-interwork [..]. In AAPCS configurations this option is meaningless. We can't get rid of the option altogether because that breaks old GCC builds[2], but for now we can make -mno-thumb-interwork optional, which is what this patch does. [1]: https://gcc.gnu.org/onlinedocs/gcc/ARM-Options.html [2]: https://patchwork.kernel.org/patch/10306823/ Signed-off-by: Ahmad Fatoum <ahmad@xxxxxx> --- missed this one the first time, but here it is --- arch/arm/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm/Makefile b/arch/arm/Makefile index 5cb46f661335..ef85f9d72823 100644 --- a/arch/arm/Makefile +++ b/arch/arm/Makefile @@ -52,7 +52,7 @@ ifeq ($(CONFIG_CPU_V8), y) CFLAGS_ABI :=-mabi=lp64 else ifeq ($(CONFIG_AEABI),y) -CFLAGS_ABI :=-mabi=aapcs-linux -mno-thumb-interwork +CFLAGS_ABI :=-mabi=aapcs-linux $(call cc-option,-mno-thumb-interwork) else CFLAGS_ABI :=$(call cc-option,-mapcs-32,-mabi=apcs-gnu) $(call cc-option,-mno-thumb-interwork,) endif -- 2.20.1 _______________________________________________ barebox mailing list barebox@xxxxxxxxxxxxxxxxxxx http://lists.infradead.org/mailman/listinfo/barebox
