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

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

[Ryan Eatmon]

lzop has disappeared from upstream master, but lzo is there.

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

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

[Denys Dmytriyenko]

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.


> 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

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

[Andrew Davis]

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.

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
>>
>>
>> -=-=-=-=-=-=-=-=-=-=-=-
>> Links: You receive all messages sent to this group.
>> View/Reply Online (#15164): https://lists.yoctoproject.org/g/meta-ti/message/15164
>> Mute This Topic: https://lists.yoctoproject.org/mt/94310494/3619733
>> Group Owner: meta-ti+owner@lists.yoctoproject.org
>> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [afd@ti.com]
>> -=-=-=-=-=-=-=-=-=-=-=-
>>

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

[Ryan Eatmon]

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?


> 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
>>>
>>>
>>> -=-=-=-=-=-=-=-=-=-=-=-
>>> Links: You receive all messages sent to this group.
>>> View/Reply Online (#15164): 
>>> https://lists.yoctoproject.org/g/meta-ti/message/15164
>>> Mute This Topic: https://lists.yoctoproject.org/mt/94310494/3619733
>>> Group Owner: meta-ti+owner@lists.yoctoproject.org
>>> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [afd@ti.com]
>>> -=-=-=-=-=-=-=-=-=-=-=-
>>>

-- 
Ryan Eatmon                reatmon@ti.com
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS

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

[Denys Dmytriyenko]

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