Re: [meta-ti][master][PATCH v2] u-boot: Change depends from lzop to lzo

[Denys Dmytriyenko <denis@denix.org>]

On Thu, Oct 13, 2022 at 03:15:19PM -0500, Ryan Eatmon wrote:
> 
> 
> On 10/13/2022 3:10 PM, Andrew Davis wrote:
> >On 10/13/22 2:24 PM, Denys Dmytriyenko wrote:
> >>On Thu, Oct 13, 2022 at 01:26:02PM -0500, Ryan Eatmon via
> >>lists.yoctoproject.org wrote:
> >>>lzop has disappeared from upstream master, but lzo is there.
> >>
> >>lzop is actually a compression tool, while lzo is just a library.
> >>so, I don't believe this change is correct/complete.
> >>
> >
> >Do we use this? Might be better to just drop the DEPENDS.
> 
> I was wondering the same thing.  lzop is clearly gone from master
> upstream.  So we either need to find a replacement, or we need to
> remove it.
> 
> 
> Here is the commit that removed it:
> 
> commit dea5e8863792dc7bb3324b543e04da4c94a060aa
> Author: Ross Burton <ross.burton@arm.com>
> Date:   Tue May 24 16:23:58 2022 +0100
> 
>     lzop: remove recipe from oe-core
> 
>     There are no users of lzop in oe-core, and there hasn't been a
> release of
>     lzop since 2017.
> 
>     Signed-off-by: Ross Burton <ross.burton@arm.com>
>     Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
> 
> 
> Denys, as the person who added the lzop-native requirement in u-boot
> in the first place, can you think of anything?

lzop has been moved from oe-core to meta-oe, so it's not completely gone:

https://git.openembedded.org/meta-openembedded/commit/?id=90ff53b8df1e3259cbc201c658a4f3f4dddf3aa8

Even oe-core still has some conditional dependencies on lzop-native, assuming 
you use meta-oe layer. But normally it is not enabled by default.

When FIT image gets generated, it can compress the kernel image using either 
the default gzip compression algorithm, or lzo one. It is selected by 
FIT_KERNEL_COMP_ALG variable:

https://git.openembedded.org/openembedded-core/tree/meta/classes-recipe/kernel-uboot.bbclass

So, unless you specifically change that variable to "lzo" for FIT image based 
HS platforms, you don't need a dependency on lzop-native. Otherwise, you need 
meta-oe layer in your stack.


> >Andrew
> >
> >>
> >>>Signed-off-by: Ryan Eatmon <reatmon@ti.com>
> >>>---
> >>>v2: Fixed typo lzpo -> lzop
> >>>
> >>>  meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc | 2 +-
> >>>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>>
> >>>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 d29c80cf..88018541 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 lzop-native flex-native
> >>>bison-native python3-setuptools-native"
> >>>+DEPENDS += "dtc-native bc-native lzo-native flex-native
> >>>bison-native python3-setuptools-native"
> >>>  DM_FIRMWARE = "ipc_echo_testb_mcu1_0_release_strip.xer5f"
> >>>-- 
> >>>2.17.1