Re: [U-Boot] [PATCH 1/1] arm: fix -march for ARM11

2019-11-22 Thread Linus Walleij
On Tue, Nov 19, 2019 at 11:16 PM Heinrich Schuchardt  wrote:

> -march v5 is invalid for GCC 9.2.1. ARM11 is an armv6 implementation. So
> change the architecture flag for the compiler to armv6.
> Cf. https://gcc.gnu.org/onlinedocs/gcc/ARM-Options.html
>
> Suggested-by: Fabio Estevam 
> Signed-off-by: Heinrich Schuchardt 

After I read up on it I would add something to the commit saying
that the first -march=armv5 must have been wrong all the time
since there are no known implementations of
armv5, only armv5t, armv5te and armv5tej. Maybe add a link
to this:
https://gcc.gnu.org/gcc-9/changes.html

Yours,
Linus Walleij
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


Re: [U-Boot] [PATCH 1/1] arm: fix -march for ARM11

2019-11-20 Thread Linus Walleij
On Tue, Nov 19, 2019 at 11:16 PM Heinrich Schuchardt  wrote:

> -march v5 is invalid for GCC 9.2.1. ARM11 is an armv6 implementation. So
> change the architecture flag for the compiler to armv6.
> Cf. https://gcc.gnu.org/onlinedocs/gcc/ARM-Options.html
>
> Suggested-by: Fabio Estevam 
> Signed-off-by: Heinrich Schuchardt 

Reviewed-by: Linus Walleij 

Yours,
Linus Walleij
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


[U-Boot] [PATCH 1/1] arm: fix -march for ARM11

2019-11-19 Thread Heinrich Schuchardt
-march v5 is invalid for GCC 9.2.1. ARM11 is an armv6 implementation. So
change the architecture flag for the compiler to armv6.
Cf. https://gcc.gnu.org/onlinedocs/gcc/ARM-Options.html

Suggested-by: Fabio Estevam 
Signed-off-by: Heinrich Schuchardt 
---
 arch/arm/Makefile | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/arch/arm/Makefile b/arch/arm/Makefile
index 5384981c17..60af7e3199 100644
--- a/arch/arm/Makefile
+++ b/arch/arm/Makefile
@@ -12,8 +12,8 @@ arch-$(CONFIG_CPU_ARM926EJS)  =-march=armv5te
 arch-$(CONFIG_CPU_ARM946ES)=-march=armv5te
 arch-$(CONFIG_CPU_SA1100)  =-march=armv4
 arch-$(CONFIG_CPU_PXA) =
-arch-$(CONFIG_CPU_ARM1136) =-march=armv5
-arch-$(CONFIG_CPU_ARM1176) =-march=armv5t
+arch-$(CONFIG_CPU_ARM1136) =-march=armv6
+arch-$(CONFIG_CPU_ARM1176) =-march=armv6
 arch-$(CONFIG_CPU_V7A) =$(call cc-option, -march=armv7-a, \
 $(call cc-option, -march=armv7))
 arch-$(CONFIG_CPU_V7M) =-march=armv7-m
--
2.24.0

___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot