[poky] [PATCH 05/25][v2] u-boot: disable -Os option not supported by gcc

[Darren Hart]

On 01/25/2011 01:09 PM, Adrian Alonso wrote:
> * Disable -Os optspace option not supported by gcc
> * v2 use OPTFLAGS variable, remove meke file patch
> * Bump PR
>
> Signed-off-by: Adrian Alonso<aalonso at secretlab.ca>
> ---
>   recipes-bsp/uboot/u-boot-xilinx_git.bb |    6 ++++--
>   1 files changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/recipes-bsp/uboot/u-boot-xilinx_git.bb b/recipes-bsp/uboot/u-boot-xilinx_git.bb
> index 50d99e3..499ce0b 100644
> --- a/recipes-bsp/uboot/u-boot-xilinx_git.bb
> +++ b/recipes-bsp/uboot/u-boot-xilinx_git.bb
> @@ -1,9 +1,11 @@
>   require ../meta/recipes-bsp/uboot/u-boot.inc
> -PR = "r2"
> +PR = "r3"
>
> +OPTFLAGS=""

Have you tested this build? Looking at the Makefile, it doesn't do a 
conditional assignment, so I would expect OPTFLAGS set here to be 
ignored. This is why I added it to EXTRA_OEMAKE, which overrides 
anything in the Makefil, regardless of how it is assigned. At least, 
that was my understanding of the mechanism.

EXTRA_OEMAKE = "CROSS_COMPILE=${TARGET_PREFIX} OPTFLAGS=''"


Josh, I think you may had something to do with my education in this area 
- can you comment? Am I all wet here?

--
Darren

>   BRANCH="master"
>   TAG="e094f2479ea339d7f48b6826f06f0be4984d9a98"
> -SRC_URI = "git://git.xilinx.com/u-boot-xlnx.git;branch=${BRANCH};protocol=git"
> +SRC_URI = "git://git.xilinx.com/u-boot-xlnx.git;branch=${BRANCH};protocol=git \
> +          "
>   SRCREV = "${TAG}"
>
>   inherit xilinx-boot


-- 
Darren Hart
Intel Open Source Technology Center
Yocto Project - Linux Kernel