[ELDK] [PATCH v2 2/2] META-ELDK: u-boot: unbreak target U-Boot build

[Marek Vasut]

On Tuesday, April 22, 2014 at 04:06:27 PM, Gerhard Sittig wrote:
> add a build dependency on openssl and openssl-native, since boot loaders
> may need it depending on their configuration, and image conversion tools
> unconditionally need it (and don't use external u-boot-*-native
> packages)
> 
> pass additional EXTRA_OEMAKE parameters, to unbreak the build for both
> image conversion tools that are run at compile time on the host, as well
> as for target objects that get built from sources that were created from
> binary resources at compile time (logo bitmaps)
> 
> Signed-off-by: Gerhard Sittig <gsi at denx.de>
> ---
> 
> changes in v2:
> - introduced the U-Boot (boot loader) fix
> 
>  meta-eldk/recipes-bsp/uboot/u-boot_2014.01.bb |   16 ++++++++++++++++
>  1 file changed, 16 insertions(+)
> 
> diff --git a/meta-eldk/recipes-bsp/uboot/u-boot_2014.01.bb
> b/meta-eldk/recipes-bsp/uboot/u-boot_2014.01.bb index
> 7010cd38347c..b16ceaf0ae59 100644
> --- a/meta-eldk/recipes-bsp/uboot/u-boot_2014.01.bb
> +++ b/meta-eldk/recipes-bsp/uboot/u-boot_2014.01.bb
> @@ -28,6 +28,22 @@ SRC_URI_append_m53evk = " \
>  			file://0003-ARM-m53evk-Update-default-environment.patch 
\
>  			"
> 
> +# bootloader code might depend on OpenSSL (depending on config options),
> +# image handling related user space tools unconditionally depend on
> OpenSSL +DEPENDS += " openssl openssl-native "

This is slightly weird, do you really need the non-native OpenSSL ?

> +# provide additional make parameters to help build the user space tools;
> +# note that the native.bbclass approach does not work here, the phrase
> +# HOSTCC="${CCACHE}${HOST_PREFIX}gcc ${HOST_CC_ARCH}" results in target
> specs; +# note as well that U-Boot v2014.01 requires the CFLAGS in the
> HOSTCC spec, +# passing separate HOST_CFLAGS only became operational in
> version v2014.04 +EXTRA_OEMAKE += ' \
> +	HOST_TOOLS_ALL=n \
> +	HOSTCC="gcc ${BUILD_CFLAGS}" \
> +	HOSTLD="ld" \
> +	HOSTLDFLAGS="${BUILD_LDFLAGS}" \
> +	HOSTSTRIP=true '
> +

Why this "HOST_TOOLS_ALL=n" ?

Best regards,
Marek Vasut