On 01/29/2014 02:22 AM, Andreas Färber wrote: > Hi, > > Am 29.01.2014 06:42, schrieb Stephen Warren: >> The entries in config.mk were needed so that U-Boot could be built >> with an old version of the Raspberry Pi Foundation's toolchain. Without >> them, the build would error out with: >> >> ...-ld: error: .../libgcc.a(_bswapsi2.o) uses VFP register arguments, >> u-boot does not >> >> However, none of the 3 toolchains in the latest version of their >> tools.git, nor the Ubuntu/Linaro ARM compilers in at least Ubuntu Quantal >> or Saucy, need these options set in order to compile a working U-Boot. >> Hence, remove the options for simplicity. >> >> Reported-by: Tom Rini <tr...@ti.com> >> Signed-off-by: Stephen Warren <swar...@wwwdotorg.org> >> --- >> arch/arm/cpu/arm1176/bcm2835/config.mk | 19 ------------------- >> 1 file changed, 19 deletions(-) >> delete mode 100644 arch/arm/cpu/arm1176/bcm2835/config.mk > > No objection against the patch, but why was arch/arm/config.mk using > -msoft-float in the first place?
It wasn't. It was filtering -msoft-float out of the compiler flags, which then left the ABI up to the toolchain default, which was hardfp for one of the Raspberry Pi Foundation's toolchains - the same one that caused the error messages quoted above unless -msoft-float was removed from the compiler options. _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot