Re: [meta-ti][master][PATCH] u-boot: Add lzop dependency

[Denys Dmytriyenko <denis@denix.org>]

On Fri, Dec 02, 2022 at 08:58:17AM -0600, Ryan Eatmon via lists.yoctoproject.org wrote:
> It turns out we do need lzop for building dra7 and am57 in u-boot,
> so add the dependency back in and add a layer dependency on
> meta-oe(openembedded-layer).

Oh, no, please don't do that! This is a heavy dependency and meta-ti BSP has 
stayed away from it since the beginning of times.

It would be better to figure out why lzop is used for those platforms and 
change that, as LZOP is pretty much deprecated these days...


> Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> ---
>  meta-ti-bsp/conf/layer.conf                  | 1 +
>  meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc | 2 +-
>  2 files changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/meta-ti-bsp/conf/layer.conf b/meta-ti-bsp/conf/layer.conf
> index 39e7f9eb..8bafa854 100644
> --- a/meta-ti-bsp/conf/layer.conf
> +++ b/meta-ti-bsp/conf/layer.conf
> @@ -15,6 +15,7 @@ LICENSE_PATH += "${LAYERDIR}/licenses"
>  
>  LAYERDEPENDS_meta-ti-bsp = " \
>      core \
> +    openembedded-layer \
>      meta-arm \
>  "
>  
> diff --git a/meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc b/meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc
> index d87dcbe1..5127e137 100644
> --- a/meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc
> +++ b/meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc
> @@ -23,7 +23,7 @@ SRC_URI = "${UBOOT_GIT_URI};protocol=${UBOOT_GIT_PROTOCOL};branch=${BRANCH}"
>  PV:append = "+git${SRCPV}"
>  
>  # u-boot needs devtree compiler to parse dts files
> -DEPENDS += "dtc-native bc-native flex-native bison-native python3-setuptools-native"
> +DEPENDS += "dtc-native bc-native lzop-native flex-native bison-native python3-setuptools-native"
>  
>  DM_FIRMWARE = "ipc_echo_testb_mcu1_0_release_strip.xer5f"
>  
> -- 
> 2.17.1