diff options
| author | Darren Hart <dvhart@linux.intel.com> | 2011-02-09 13:26:46 -0800 |
|---|---|---|
| committer | Richard Purdie <rpurdie@linux.intel.com> | 2011-02-09 22:44:52 +0000 |
| commit | 6b76ceed1423480e18844e20fc81f05298bfdd6d (patch) | |
| tree | 86a090c1cf091e3436a4264366785aef04746119 | |
| parent | ba163a43789e2e776737a7da11217796b6c0e362 (diff) | |
| download | openembedded-core-6b76ceed1423480e18844e20fc81f05298bfdd6d.tar.gz openembedded-core-6b76ceed1423480e18844e20fc81f05298bfdd6d.tar.bz2 openembedded-core-6b76ceed1423480e18844e20fc81f05298bfdd6d.zip | |
uboot: build with -O2 on all architectures
The -Os option was disabled due to a bug in gcc building bad binaries for ARM
in an earlier commit:
f2dc7fadd8c6b180c3f985873261216d53f47f0d
This caused problems for powerpc which was resolved by replacing -Os with
-O2 for that architecture:
d0eb6794d964aa5ac938533a222c39bef09fd945
Using -O2 also works for ARM, so there is no need to condition using -O2 on
powerpc. Remove the condition and use -O2 on all architectures.
Signed-off-by: Darren Hart <dvhart@linux.intel.com>
CC: Ilya Yanok <yanok@emcraft.com>
CC: Richard Purdie <richard.purdie@linuxfoundation.org>
| -rw-r--r-- | meta/recipes-bsp/uboot/u-boot.inc | 9 |
1 files changed, 2 insertions, 7 deletions
diff --git a/meta/recipes-bsp/uboot/u-boot.inc b/meta/recipes-bsp/uboot/u-boot.inc index 3a53fbad4d..f60667ee6b 100644 --- a/meta/recipes-bsp/uboot/u-boot.inc +++ b/meta/recipes-bsp/uboot/u-boot.inc @@ -12,7 +12,7 @@ inherit deploy PARALLEL_MAKE="" # GCC 4.5.1 builds unusable binaries using -Os, remove it from OPTFLAGS -EXTRA_OEMAKE = "CROSS_COMPILE=${TARGET_PREFIX} OPTFLAGS=''" +EXTRA_OEMAKE = "CROSS_COMPILE=${TARGET_PREFIX} OPTFLAGS='-O2'" UBOOT_MACHINE ?= "${MACHINE}_config" UBOOT_IMAGE ?= "u-boot-${MACHINE}-${PV}-${PR}.bin" @@ -23,12 +23,7 @@ do_compile () { unset CFLAGS unset CPPFLAGS oe_runmake ${UBOOT_MACHINE} - # -Os is disabled on PowerPC - if [ ${TARGET_ARCH} == "powerpc" ] ; then - oe_runmake OPTFLAGS=-O2 all - else - oe_runmake all - fi + oe_runmake all } do_deploy () { |