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

[Menon, Nishanth]

> -----Original Message-----
> From: Menon, Nishanth
> Sent: Monday, May 12, 2008 8:25 AM
> To: 'Sascha Hauer'
> Cc: u-boot-users at lists.sourceforge.net
> Subject: RE: [Patch 1/2] U-Boot-V2: ARM: Makefile cleaup
> 
> Sascha,
> > -----Original Message-----
> > From: Sascha Hauer [mailto:s.hauer at pengutronix.de]
> > Sent: Monday, May 12, 2008 8:22 AM
> > To: Menon, Nishanth
> > Cc: u-boot-users at lists.sourceforge.net
> > Subject: Re: [Patch 1/2] U-Boot-V2: ARM: Makefile cleaup
> >
> > > * Introduce -ffixed-r8 for CFLAGS
> >
> > Why? U-Boot v1 uses to keep a pointer to global data here, but in v2
> > there is no global data anymore.
> Yes, you are right. I missed the fact that fixed r8 does not make sense anymore. Is the rest of the
> patch fine?
> Regards,
> Nishanth Menon

Rev 2 of the patch.

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-12 08:26:57.000000000 -0500
+++ u-boot-v2.git/arch/arm/Makefile	2008-05-12 08:27:24.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
@@ -20,7 +20,8 @@
 TEXT_BASE = $(CONFIG_TEXT_BASE)
 
 CPPFLAGS += -mabi=apcs-gnu -DTEXT_BASE=$(TEXT_BASE) -P
-CFLAGS := -fno-common -msoft-float -Os
+CFLAGS += -msoft-float -Os
+AFLAGS += -msoft-float -Os
 
 ifeq ($(incdir-y),)
 incdir-y := $(machine-y)