Patchwork [PATCHv2,1/3] u-boot: Don't make the -Os removal part of global settings.

login
register
mail settings
Submitter Paul Gortmaker
Date July 2, 2012, 5:44 a.m.
Message ID <1341207885-9463-2-git-send-email-paul.gortmaker@windriver.com>
Download mbox | patch
Permalink /patch/30973/
State Accepted
Commit 5e3f777f9fcc986ae66203651830b5765bca49fd
Headers show

Comments

Paul Gortmaker - July 2, 2012, 5:44 a.m.
We don't want to force everyone to be stripping the -Os
flags from their u-boot builds.  Remove it, since it pertains
to an old toolchain issue that is no longer relevant, and it
breaks the powerpc mpc8315.

Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>
---
 meta/recipes-bsp/u-boot/u-boot.inc |    3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
Otavio Salvador - July 2, 2012, 11:23 a.m.
On Mon, Jul 2, 2012 at 2:44 AM, Paul Gortmaker
<paul.gortmaker@windriver.com> wrote:
> We don't want to force everyone to be stripping the -Os
> flags from their u-boot builds.  Remove it, since it pertains
> to an old toolchain issue that is no longer relevant, and it
> breaks the powerpc mpc8315.
>
> Signed-off-by: Paul Gortmaker <paul.gortmaker@windriver.com>

Acked-by: Otavio Salvador <otavio@ossystems.com.br>

Patch

diff --git a/meta/recipes-bsp/u-boot/u-boot.inc b/meta/recipes-bsp/u-boot/u-boot.inc
index a13eddc..5de3583 100644
--- a/meta/recipes-bsp/u-boot/u-boot.inc
+++ b/meta/recipes-bsp/u-boot/u-boot.inc
@@ -7,8 +7,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='-O2'"
+EXTRA_OEMAKE = "CROSS_COMPILE=${TARGET_PREFIX}"
 
 python () {
 	if not d.getVar("UBOOT_MACHINE", True):