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

[Menon, Nishanth]

Sascha,
> -----Original Message-----
> From: Sascha Hauer [mailto:s.hauer at pengutronix.de]
> Sent: Monday, May 12, 2008 8:44 AM
> To: Menon, Nishanth
> Cc: u-boot-users at lists.sourceforge.net
> Subject: Re: [Patch 1/2] U-Boot-V2: ARM: Makefile cleaup
> 
> > +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.
I agree on the -Os being unused. On -msoft-float without the same in AFLAGS:
arm-none-eabi-ld: ERROR: arch/arm/mach-omap/omap3_clock.o uses hardware FP, whereas arch/arm/mach-omap/built-in.o uses software FP
arm-none-eabi-ld: failed to merge target specific data of file arch/arm/mach-omap/omap3_clock.o
make[1]: *** [arch/arm/mach-omap/built-in.o] Error 1
make: *** [arch/arm/mach-omap] Error 2

Ideally the gcc flags should be set per platform -> Linux kernel has a neat manner of doing it. We can move ahead with h/w FP as long as we allow flags to be set for platform. Currently though, if C files are compiled with -msoft-float, so should AFLAGS. Probably we need some new management of compiler flags?

Regards,
Nishanth Menon