[U-Boot-Users] [patch u-boot git 2/3] better ARM9 compiler options

[Ulf Samuelsson]

> Update some ARM9 builds to pass the exact CPU to GCC, when the compiler
> supports that specification.  This stops pessimizing codegen, and removes
> voluminous warnings about (no) interworking support from EABI compilers.
> (Like CodeSourcery's ARM 2007q3 release.)
> 
> Signed-off-by: David Brownell <dbrownell at users.sourceforge.net>
> 
> --- u-boot.orig/cpu/arm920t/config.mk 2008-01-13 10:42:55.000000000 -0800
> +++ u-boot/cpu/arm920t/config.mk 2008-01-13 11:05:02.000000000 -0800
> @@ -24,11 +24,11 @@
> PLATFORM_RELFLAGS += -fno-strict-aliasing  -fno-common -ffixed-r8 \
>  -msoft-float

Why not remove the "-msoft-float" as well.
U-Boot does not use soft float and why then the requirement to 
use a compiler which supports soft-float???

> 
> -PLATFORM_CPPFLAGS += -march=armv4
> # =========================================================================
> #
> # Supply options according to compiler version
> #
> # =========================================================================
> +PLATFORM_CPPFLAGS +=$(call cc-option,-mcpu=arm920t,-march=armv4)
> PLATFORM_CPPFLAGS +=$(call cc-option,-mapcs-32,-mabi=apcs-gnu)
> PLATFORM_RELFLAGS +=$(call cc-option,-mshort-load-bytes,$(call cc-option,-malignment-traps,))
> --- u-boot.orig/cpu/arm926ejs/config.mk 2008-01-13 10:42:55.000000000 -0800
> +++ u-boot/cpu/arm926ejs/config.mk 2008-01-13 11:04:56.000000000 -0800
> @@ -24,11 +24,11 @@
> PLATFORM_RELFLAGS += -fno-strict-aliasing  -fno-common -ffixed-r8 \
>  -msoft-float
> 
> -PLATFORM_CPPFLAGS += -march=armv4
> # =========================================================================
> #
> # Supply options according to compiler version
> #
> # =========================================================================
> +PLATFORM_CPPFLAGS +=$(call cc-option,-mcpu=arm926ej-s,-march=armv4)
> PLATFORM_CPPFLAGS +=$(call cc-option,-mapcs-32,-mabi=apcs-gnu)
> PLATFORM_RELFLAGS +=$(call cc-option,-mshort-load-bytes,$(call cc-option,-malignment-traps,))
> 


Is this correct?
ARM926EJ-S is ARMV5 architecture, not ARMV4.


Best Regards
Ulf Samuelsson