[U-Boot-Users] [Patch 1/2] U-Boot-V2: ARM: Makefile cleaup

[Sascha Hauer]

On Mon, May 12, 2008 at 07:20:59AM -0500, Menon, Nishanth wrote:
> Does the following:
> * Remove duplicated global flags -Wall, -nostdinc, -fno-common
> * Introduce -ffixed-r8 for CFLAGS
> * Add AFLAGS
> 
> Signed-off-by: Nishanth Menon <x0nishan at ti.com>
> 
> Index: u-boot-v2.git/arch/arm/Makefile
> ===================================================================
> --- u-boot-v2.git.orig/arch/arm/Makefile	2008-05-08 17:36:23.000000000 -0500
> +++ u-boot-v2.git/arch/arm/Makefile	2008-05-08 17:41:31.000000000 -0500
> @@ -1,6 +1,6 @@
>  
> -CPPFLAGS += -fno-builtin -ffreestanding -nostdinc -Wall 	\
> -	-pipe -D __ARM__ -fno-strict-aliasing
> +CPPFLAGS += -fno-builtin -ffreestanding\
> +	-pipe -D__ARM__
>  
>  
>  machine-$(CONFIG_ARCH_IMX)	   := imx
> @@ -19,8 +19,9 @@
>  
>  TEXT_BASE = $(CONFIG_TEXT_BASE)
>  
> -CPPFLAGS += -mabi=apcs-gnu -DTEXT_BASE=$(TEXT_BASE) -P
> -CFLAGS := -fno-common -msoft-float -Os
> +CPPFLAGS += -ffixed-r8 -mabi=apcs-gnu -DTEXT_BASE=$(TEXT_BASE) -P
> +CFLAGS += -msoft-float -Os
> +AFLAGS += -msoft-float -Os

Assemblers don't know about soft-float and size optimisation. This patch
only works because AFLAGS is unused. Besides, you shouldn't use floating
point througout U-Boot anyway.

Sascha

-- 
Pengutronix e.K. - Linux Solutions for Science and Industry
-----------------------------------------------------------
Kontakt-Informationen finden Sie im Header dieser Mail oder
auf der Webseite -> http://www.pengutronix.de/impressum/ <-