> -----Original Message-----
> From: Menon, Nishanth
> Sent: Monday, May 12, 2008 8:25 AM
> To: 'Sascha Hauer'
> Cc: [email protected]
> Subject: RE: [Patch 1/2] U-Boot-V2: ARM: Makefile cleaup
> 
> Sascha,
> > -----Original Message-----
> > From: Sascha Hauer [mailto:[EMAIL PROTECTED]
> > Sent: Monday, May 12, 2008 8:22 AM
> > To: Menon, Nishanth
> > Cc: [email protected]
> > 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 <[EMAIL PROTECTED]>

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)

-------------------------------------------------------------------------
This SF.net email is sponsored by the 2008 JavaOne(SM) Conference 
Don't miss this year's exciting event. There's still time to save $100. 
Use priority code J8TL2D2. 
http://ad.doubleclick.net/clk;198757673;13503038;p?http://java.sun.com/javaone
_______________________________________________
U-Boot-Users mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/u-boot-users

Reply via email to