* [meta-ti][master][PATCH v2] u-boot: Change depends from lzop to lzo @ 2022-10-13 18:26 Ryan Eatmon 2022-10-13 19:24 ` Denys Dmytriyenko 0 siblings, 1 reply; 5+ messages in thread From: Ryan Eatmon @ 2022-10-13 18:26 UTC (permalink / raw) To: Praneeth Bajjuri, Denys Dmytriyenko, meta-ti 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 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [meta-ti][master][PATCH v2] u-boot: Change depends from lzop to lzo 2022-10-13 18:26 [meta-ti][master][PATCH v2] u-boot: Change depends from lzop to lzo Ryan Eatmon @ 2022-10-13 19:24 ` Denys Dmytriyenko 2022-10-13 20:10 ` Andrew Davis 0 siblings, 1 reply; 5+ messages in thread From: Denys Dmytriyenko @ 2022-10-13 19:24 UTC (permalink / raw) To: reatmon; +Cc: Praneeth Bajjuri, meta-ti 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 ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [meta-ti][master][PATCH v2] u-boot: Change depends from lzop to lzo 2022-10-13 19:24 ` Denys Dmytriyenko @ 2022-10-13 20:10 ` Andrew Davis 2022-10-13 20:15 ` Ryan Eatmon 0 siblings, 1 reply; 5+ messages in thread From: Andrew Davis @ 2022-10-13 20:10 UTC (permalink / raw) To: Denys Dmytriyenko, reatmon; +Cc: Praneeth Bajjuri, meta-ti 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] >> -=-=-=-=-=-=-=-=-=-=-=- >> ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [meta-ti][master][PATCH v2] u-boot: Change depends from lzop to lzo 2022-10-13 20:10 ` Andrew Davis @ 2022-10-13 20:15 ` Ryan Eatmon 2022-10-13 20:38 ` Denys Dmytriyenko 0 siblings, 1 reply; 5+ messages in thread From: Ryan Eatmon @ 2022-10-13 20:15 UTC (permalink / raw) To: Andrew Davis, Denys Dmytriyenko; +Cc: Praneeth Bajjuri, meta-ti 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 ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [meta-ti][master][PATCH v2] u-boot: Change depends from lzop to lzo 2022-10-13 20:15 ` Ryan Eatmon @ 2022-10-13 20:38 ` Denys Dmytriyenko 0 siblings, 0 replies; 5+ messages in thread From: Denys Dmytriyenko @ 2022-10-13 20:38 UTC (permalink / raw) To: Ryan Eatmon; +Cc: Andrew Davis, Praneeth Bajjuri, meta-ti 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 ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-10-13 20:38 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2022-10-13 18:26 [meta-ti][master][PATCH v2] u-boot: Change depends from lzop to lzo Ryan Eatmon 2022-10-13 19:24 ` Denys Dmytriyenko 2022-10-13 20:10 ` Andrew Davis 2022-10-13 20:15 ` Ryan Eatmon 2022-10-13 20:38 ` Denys Dmytriyenko
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.