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@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