| Message ID | 20250728183922.580652-2-afd@ti.com |
|---|---|
| State | Under Review |
| Delegated to: | Ryan Eatmon |
| Headers | show |
| Series | [meta-arago,scarthgap/master,1/2] tisdk-uenv: Do not deploy to prebuilt images dir from recipe | expand |
On Mon, Jul 28, 2025 at 2:39 PM Andrew Davis via lists.yoctoproject.org <afd=ti.com@lists.yoctoproject.org> wrote: > > This recipe only now exists to add "tisdk-uenv" to tisdk-core-bundle > images so uEnv.txt can be packaged into prebuilt-images. We only need > to package the uEnv.txt as a "prebuilt-image" if some bundled image > includes it, no need to manually add this for all images. Does this mean the tisdk-base/default images will no longer include the example empty uEnv.txt file? > > Signed-off-by: Andrew Davis <afd@ti.com> > --- > .../recipes-core/images/tisdk-core-bundle.inc | 4 ---- > .../packagegroup-arago-tisdk-amsdk-sdk-host.bb | 16 ---------------- > 2 files changed, 20 deletions(-) > delete mode 100644 meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb > > diff --git a/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc b/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc > index 43b1ef5f..5f72e7b0 100644 > --- a/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc > +++ b/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc > @@ -33,7 +33,3 @@ TARGET_IMAGES = "tisdk-base-image tisdk-default-image tisdk-thinlinux-image tisd > > TISDK_TOOLCHAIN = "meta-toolchain-arago-tisdk" > TOOLCHAIN_SUFFIX = "-tisdk" > - > -IMAGE_INSTALL = "\ > - packagegroup-arago-tisdk-amsdk-sdk-host \ > -" > diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb > deleted file mode 100644 > index 9562310f..00000000 > --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb > +++ /dev/null > @@ -1,16 +0,0 @@ > -SUMMARY = "Task to install additional scripts and applications into the SDK" > -LICENSE = "MIT" > -PR = "r30" > - > -PACKAGE_ARCH = "${MACHINE_ARCH}" > - > -inherit packagegroup > - > -EXTRA_FILES = "" > -EXTRA_FILES:ti43x = "tisdk-uenv" > -EXTRA_FILES:omap-a15 = "tisdk-uenv" > -EXTRA_FILES:k3 = "tisdk-uenv" > - > -RDEPENDS:${PN} = "\ > - ${EXTRA_FILES} \ > -" > -- > 2.39.2 > > > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > View/Reply Online (#16334): https://lists.yoctoproject.org/g/meta-arago/message/16334 > Mute This Topic: https://lists.yoctoproject.org/mt/114418842/7902621 > Group Owner: meta-arago+owner@lists.yoctoproject.org > Unsubscribe: https://lists.yoctoproject.org/g/meta-arago/unsub [jcormier@criticallink.com] > -=-=-=-=-=-=-=-=-=-=-=- > >
On Tue, Jul 29, 2025 at 1:23 PM Jon Cormier via lists.yoctoproject.org <jcormier=criticallink.com@lists.yoctoproject.org> wrote: > > On Mon, Jul 28, 2025 at 2:39 PM Andrew Davis via > lists.yoctoproject.org <afd=ti.com@lists.yoctoproject.org> wrote: > > > > This recipe only now exists to add "tisdk-uenv" to tisdk-core-bundle > > images so uEnv.txt can be packaged into prebuilt-images. We only need > > to package the uEnv.txt as a "prebuilt-image" if some bundled image > > includes it, no need to manually add this for all images. > Does this mean the tisdk-base/default images will no longer include > the example empty uEnv.txt file? Specifically the wic images. The other image formats don't include the boot files. > > > > Signed-off-by: Andrew Davis <afd@ti.com> > > --- > > .../recipes-core/images/tisdk-core-bundle.inc | 4 ---- > > .../packagegroup-arago-tisdk-amsdk-sdk-host.bb | 16 ---------------- > > 2 files changed, 20 deletions(-) > > delete mode 100644 meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb > > > > diff --git a/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc b/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc > > index 43b1ef5f..5f72e7b0 100644 > > --- a/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc > > +++ b/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc > > @@ -33,7 +33,3 @@ TARGET_IMAGES = "tisdk-base-image tisdk-default-image tisdk-thinlinux-image tisd > > > > TISDK_TOOLCHAIN = "meta-toolchain-arago-tisdk" > > TOOLCHAIN_SUFFIX = "-tisdk" > > - > > -IMAGE_INSTALL = "\ > > - packagegroup-arago-tisdk-amsdk-sdk-host \ > > -" > > diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb > > deleted file mode 100644 > > index 9562310f..00000000 > > --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb > > +++ /dev/null > > @@ -1,16 +0,0 @@ > > -SUMMARY = "Task to install additional scripts and applications into the SDK" > > -LICENSE = "MIT" > > -PR = "r30" > > - > > -PACKAGE_ARCH = "${MACHINE_ARCH}" > > - > > -inherit packagegroup > > - > > -EXTRA_FILES = "" > > -EXTRA_FILES:ti43x = "tisdk-uenv" > > -EXTRA_FILES:omap-a15 = "tisdk-uenv" > > -EXTRA_FILES:k3 = "tisdk-uenv" > > - > > -RDEPENDS:${PN} = "\ > > - ${EXTRA_FILES} \ > > -" > > -- > > 2.39.2 > > > > > > > > > > > > > > > -- > Jonathan Cormier > Senior Software Engineer > > Voice: 315.425.4045 x222 > > http://www.CriticalLink.com > 6712 Brooklawn Parkway, Syracuse, NY 13211 > > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > View/Reply Online (#16336): https://lists.yoctoproject.org/g/meta-arago/message/16336 > Mute This Topic: https://lists.yoctoproject.org/mt/114418842/7902621 > Group Owner: meta-arago+owner@lists.yoctoproject.org > Unsubscribe: https://lists.yoctoproject.org/g/meta-arago/unsub [jcormier@criticallink.com] > -=-=-=-=-=-=-=-=-=-=-=- > >
On 7/29/25 12:26 PM, Jon Cormier wrote: > On Tue, Jul 29, 2025 at 1:23 PM Jon Cormier via lists.yoctoproject.org > <jcormier=criticallink.com@lists.yoctoproject.org> wrote: >> >> On Mon, Jul 28, 2025 at 2:39 PM Andrew Davis via >> lists.yoctoproject.org <afd=ti.com@lists.yoctoproject.org> wrote: >>> >>> This recipe only now exists to add "tisdk-uenv" to tisdk-core-bundle >>> images so uEnv.txt can be packaged into prebuilt-images. We only need >>> to package the uEnv.txt as a "prebuilt-image" if some bundled image >>> includes it, no need to manually add this for all images. >> Does this mean the tisdk-base/default images will no longer include >> the example empty uEnv.txt file? > Specifically the wic images. The other image formats don't include > the boot files. For most images, adding that example uEnv.txt file is done with this code here in arago.conf[0]. So the wic images will still have this file added even after this patchset, for now.. I say "for now" as I'm not a fan of how these get added to the images unconditionally in a .conf file, vs added explicitly in the image's .bb files. Even longer term, U-Boot is moving more towards boot script files (boot.scr), which are more flexible and standard (uEnv.txt is rather TI specific). Andrew [0] https://git.yoctoproject.org/meta-arago/tree/meta-arago-distro/conf/distro/arago.conf#n35 >>> >>> Signed-off-by: Andrew Davis <afd@ti.com> >>> --- >>> .../recipes-core/images/tisdk-core-bundle.inc | 4 ---- >>> .../packagegroup-arago-tisdk-amsdk-sdk-host.bb | 16 ---------------- >>> 2 files changed, 20 deletions(-) >>> delete mode 100644 meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb >>> >>> diff --git a/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc b/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc >>> index 43b1ef5f..5f72e7b0 100644 >>> --- a/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc >>> +++ b/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc >>> @@ -33,7 +33,3 @@ TARGET_IMAGES = "tisdk-base-image tisdk-default-image tisdk-thinlinux-image tisd >>> >>> TISDK_TOOLCHAIN = "meta-toolchain-arago-tisdk" >>> TOOLCHAIN_SUFFIX = "-tisdk" >>> - >>> -IMAGE_INSTALL = "\ >>> - packagegroup-arago-tisdk-amsdk-sdk-host \ >>> -" >>> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb >>> deleted file mode 100644 >>> index 9562310f..00000000 >>> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb >>> +++ /dev/null >>> @@ -1,16 +0,0 @@ >>> -SUMMARY = "Task to install additional scripts and applications into the SDK" >>> -LICENSE = "MIT" >>> -PR = "r30" >>> - >>> -PACKAGE_ARCH = "${MACHINE_ARCH}" >>> - >>> -inherit packagegroup >>> - >>> -EXTRA_FILES = "" >>> -EXTRA_FILES:ti43x = "tisdk-uenv" >>> -EXTRA_FILES:omap-a15 = "tisdk-uenv" >>> -EXTRA_FILES:k3 = "tisdk-uenv" >>> - >>> -RDEPENDS:${PN} = "\ >>> - ${EXTRA_FILES} \ >>> -" >>> -- >>> 2.39.2 >>> >>> >>> >>> >>> >>> >> >> >> -- >> Jonathan Cormier >> Senior Software Engineer >> >> Voice: 315.425.4045 x222 >> >> http://www.CriticalLink.com >> 6712 Brooklawn Parkway, Syracuse, NY 13211 >> >> >> -=-=-=-=-=-=-=-=-=-=-=- >> Links: You receive all messages sent to this group. >> View/Reply Online (#16336): https://lists.yoctoproject.org/g/meta-arago/message/16336 >> Mute This Topic: https://lists.yoctoproject.org/mt/114418842/7902621 >> Group Owner: meta-arago+owner@lists.yoctoproject.org >> Unsubscribe: https://lists.yoctoproject.org/g/meta-arago/unsub [jcormier@criticallink.com] >> -=-=-=-=-=-=-=-=-=-=-=- >> >> > >
On Tue, Jul 29, 2025 at 1:47 PM Andrew Davis <afd@ti.com> wrote: > > > > On 7/29/25 12:26 PM, Jon Cormier wrote: > > On Tue, Jul 29, 2025 at 1:23 PM Jon Cormier via lists.yoctoproject.org > > <jcormier=criticallink.com@lists.yoctoproject.org> wrote: > >> > >> On Mon, Jul 28, 2025 at 2:39 PM Andrew Davis via > >> lists.yoctoproject.org <afd=ti.com@lists.yoctoproject.org> wrote: > >>> > >>> This recipe only now exists to add "tisdk-uenv" to tisdk-core-bundle > >>> images so uEnv.txt can be packaged into prebuilt-images. We only need > >>> to package the uEnv.txt as a "prebuilt-image" if some bundled image > >>> includes it, no need to manually add this for all images. > >> Does this mean the tisdk-base/default images will no longer include > >> the example empty uEnv.txt file? > > Specifically the wic images. The other image formats don't include > > the boot files. > > For most images, adding that example uEnv.txt file is done with this > code here in arago.conf[0]. So the wic images will still have this > file added even after this patchset, for now.. > > I say "for now" as I'm not a fan of how these get added to the images > unconditionally in a .conf file, vs added explicitly in the image's .bb > files. This is fair. It does make it tricky to remove the uEnv if a customer doesn't want it. Or if they want to replace it with their own recipe. Currently are forced to bbappend the ti recipe. > Even longer term, U-Boot is moving more towards boot script > files (boot.scr), which are more flexible and standard (uEnv.txt is > rather TI specific). Hmm at least for development, having the config file being text is pretty desirable. > > Andrew > > [0] https://git.yoctoproject.org/meta-arago/tree/meta-arago-distro/conf/distro/arago.conf#n35 > > >>> > >>> Signed-off-by: Andrew Davis <afd@ti.com> > >>> --- > >>> .../recipes-core/images/tisdk-core-bundle.inc | 4 ---- > >>> .../packagegroup-arago-tisdk-amsdk-sdk-host.bb | 16 ---------------- > >>> 2 files changed, 20 deletions(-) > >>> delete mode 100644 meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb > >>> > >>> diff --git a/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc b/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc > >>> index 43b1ef5f..5f72e7b0 100644 > >>> --- a/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc > >>> +++ b/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc > >>> @@ -33,7 +33,3 @@ TARGET_IMAGES = "tisdk-base-image tisdk-default-image tisdk-thinlinux-image tisd > >>> > >>> TISDK_TOOLCHAIN = "meta-toolchain-arago-tisdk" > >>> TOOLCHAIN_SUFFIX = "-tisdk" > >>> - > >>> -IMAGE_INSTALL = "\ > >>> - packagegroup-arago-tisdk-amsdk-sdk-host \ > >>> -" > >>> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb > >>> deleted file mode 100644 > >>> index 9562310f..00000000 > >>> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb > >>> +++ /dev/null > >>> @@ -1,16 +0,0 @@ > >>> -SUMMARY = "Task to install additional scripts and applications into the SDK" > >>> -LICENSE = "MIT" > >>> -PR = "r30" > >>> - > >>> -PACKAGE_ARCH = "${MACHINE_ARCH}" > >>> - > >>> -inherit packagegroup > >>> - > >>> -EXTRA_FILES = "" > >>> -EXTRA_FILES:ti43x = "tisdk-uenv" > >>> -EXTRA_FILES:omap-a15 = "tisdk-uenv" > >>> -EXTRA_FILES:k3 = "tisdk-uenv" > >>> - > >>> -RDEPENDS:${PN} = "\ > >>> - ${EXTRA_FILES} \ > >>> -" > >>> -- > >>> 2.39.2 > >>> > >>> > >>> > >>> > >>> > >>> > >> > >> > >> -- > >> Jonathan Cormier > >> Senior Software Engineer > >> > >> Voice: 315.425.4045 x222 > >> > >> http://www.CriticalLink.com > >> 6712 Brooklawn Parkway, Syracuse, NY 13211 > >> > >> > >> -=-=-=-=-=-=-=-=-=-=-=- > >> Links: You receive all messages sent to this group. > >> View/Reply Online (#16336): https://lists.yoctoproject.org/g/meta-arago/message/16336 > >> Mute This Topic: https://lists.yoctoproject.org/mt/114418842/7902621 > >> Group Owner: meta-arago+owner@lists.yoctoproject.org > >> Unsubscribe: https://lists.yoctoproject.org/g/meta-arago/unsub [jcormier@criticallink.com] > >> -=-=-=-=-=-=-=-=-=-=-=- > >> > >> > > > > >
On 7/29/25 12:57 PM, Jon Cormier wrote: > On Tue, Jul 29, 2025 at 1:47 PM Andrew Davis <afd@ti.com> wrote: >> >> >> >> On 7/29/25 12:26 PM, Jon Cormier wrote: >>> On Tue, Jul 29, 2025 at 1:23 PM Jon Cormier via lists.yoctoproject.org >>> <jcormier=criticallink.com@lists.yoctoproject.org> wrote: >>>> >>>> On Mon, Jul 28, 2025 at 2:39 PM Andrew Davis via >>>> lists.yoctoproject.org <afd=ti.com@lists.yoctoproject.org> wrote: >>>>> >>>>> This recipe only now exists to add "tisdk-uenv" to tisdk-core-bundle >>>>> images so uEnv.txt can be packaged into prebuilt-images. We only need >>>>> to package the uEnv.txt as a "prebuilt-image" if some bundled image >>>>> includes it, no need to manually add this for all images. >>>> Does this mean the tisdk-base/default images will no longer include >>>> the example empty uEnv.txt file? >>> Specifically the wic images. The other image formats don't include >>> the boot files. >> >> For most images, adding that example uEnv.txt file is done with this >> code here in arago.conf[0]. So the wic images will still have this >> file added even after this patchset, for now.. >> >> I say "for now" as I'm not a fan of how these get added to the images >> unconditionally in a .conf file, vs added explicitly in the image's .bb >> files. > This is fair. It does make it tricky to remove the uEnv if a customer > doesn't want it. Or if they want to replace it with their own recipe. > Currently are forced to bbappend the ti recipe. >> Even longer term, U-Boot is moving more towards boot script >> files (boot.scr), which are more flexible and standard (uEnv.txt is >> rather TI specific). > Hmm at least for development, having the config file being text is > pretty desirable. boot.scr files are text, you might be getting that confused with the binary environment files produced by "env save", usually called <board_name>.env or similar. The difference between uEnv.txt and boot.scr is that the latter is run as a series of commands instead of environment variable definitions like uEnv.txt. The commands can also set env vars just like uEnv.txt by using 'env set xxx` commands, but also allows much more like variable evaluation and conditional logic. Andrew >> >> Andrew >> >> [0] https://git.yoctoproject.org/meta-arago/tree/meta-arago-distro/conf/distro/arago.conf#n35 >> >>>>> >>>>> Signed-off-by: Andrew Davis <afd@ti.com> >>>>> --- >>>>> .../recipes-core/images/tisdk-core-bundle.inc | 4 ---- >>>>> .../packagegroup-arago-tisdk-amsdk-sdk-host.bb | 16 ---------------- >>>>> 2 files changed, 20 deletions(-) >>>>> delete mode 100644 meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb >>>>> >>>>> diff --git a/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc b/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc >>>>> index 43b1ef5f..5f72e7b0 100644 >>>>> --- a/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc >>>>> +++ b/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc >>>>> @@ -33,7 +33,3 @@ TARGET_IMAGES = "tisdk-base-image tisdk-default-image tisdk-thinlinux-image tisd >>>>> >>>>> TISDK_TOOLCHAIN = "meta-toolchain-arago-tisdk" >>>>> TOOLCHAIN_SUFFIX = "-tisdk" >>>>> - >>>>> -IMAGE_INSTALL = "\ >>>>> - packagegroup-arago-tisdk-amsdk-sdk-host \ >>>>> -" >>>>> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb >>>>> deleted file mode 100644 >>>>> index 9562310f..00000000 >>>>> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb >>>>> +++ /dev/null >>>>> @@ -1,16 +0,0 @@ >>>>> -SUMMARY = "Task to install additional scripts and applications into the SDK" >>>>> -LICENSE = "MIT" >>>>> -PR = "r30" >>>>> - >>>>> -PACKAGE_ARCH = "${MACHINE_ARCH}" >>>>> - >>>>> -inherit packagegroup >>>>> - >>>>> -EXTRA_FILES = "" >>>>> -EXTRA_FILES:ti43x = "tisdk-uenv" >>>>> -EXTRA_FILES:omap-a15 = "tisdk-uenv" >>>>> -EXTRA_FILES:k3 = "tisdk-uenv" >>>>> - >>>>> -RDEPENDS:${PN} = "\ >>>>> - ${EXTRA_FILES} \ >>>>> -" >>>>> -- >>>>> 2.39.2 >>>>> >>>>> >>>>> >>>>> >>>>> >>>>> >>>> >>>> >>>> -- >>>> Jonathan Cormier >>>> Senior Software Engineer >>>> >>>> Voice: 315.425.4045 x222 >>>> >>>> http://www.CriticalLink.com >>>> 6712 Brooklawn Parkway, Syracuse, NY 13211 >>>> >>>> >>>> -=-=-=-=-=-=-=-=-=-=-=- >>>> Links: You receive all messages sent to this group. >>>> View/Reply Online (#16336): https://lists.yoctoproject.org/g/meta-arago/message/16336 >>>> Mute This Topic: https://lists.yoctoproject.org/mt/114418842/7902621 >>>> Group Owner: meta-arago+owner@lists.yoctoproject.org >>>> Unsubscribe: https://lists.yoctoproject.org/g/meta-arago/unsub [jcormier@criticallink.com] >>>> -=-=-=-=-=-=-=-=-=-=-=- >>>> >>>> >>> >>> >> > >
On Tue, Jul 29, 2025 at 01:12:25PM -0500, Andrew Davis via lists.yoctoproject.org wrote: > On 7/29/25 12:57 PM, Jon Cormier wrote: > >On Tue, Jul 29, 2025 at 1:47 PM Andrew Davis <afd@ti.com> wrote: > >> > >> > >> > >>On 7/29/25 12:26 PM, Jon Cormier wrote: > >>>On Tue, Jul 29, 2025 at 1:23 PM Jon Cormier via lists.yoctoproject.org > >>><jcormier=criticallink.com@lists.yoctoproject.org> wrote: > >>>> > >>>>On Mon, Jul 28, 2025 at 2:39 PM Andrew Davis via > >>>>lists.yoctoproject.org <afd=ti.com@lists.yoctoproject.org> wrote: > >>>>> > >>>>>This recipe only now exists to add "tisdk-uenv" to tisdk-core-bundle > >>>>>images so uEnv.txt can be packaged into prebuilt-images. We only need > >>>>>to package the uEnv.txt as a "prebuilt-image" if some bundled image > >>>>>includes it, no need to manually add this for all images. > >>>>Does this mean the tisdk-base/default images will no longer include > >>>>the example empty uEnv.txt file? > >>>Specifically the wic images. The other image formats don't include > >>>the boot files. > >> > >>For most images, adding that example uEnv.txt file is done with this > >>code here in arago.conf[0]. So the wic images will still have this > >>file added even after this patchset, for now.. > >> > >>I say "for now" as I'm not a fan of how these get added to the images > >>unconditionally in a .conf file, vs added explicitly in the image's .bb > >>files. > >This is fair. It does make it tricky to remove the uEnv if a customer > >doesn't want it. Or if they want to replace it with their own recipe. > >Currently are forced to bbappend the ti recipe. > >>Even longer term, U-Boot is moving more towards boot script > >>files (boot.scr), which are more flexible and standard (uEnv.txt is > >>rather TI specific). > >Hmm at least for development, having the config file being text is > >pretty desirable. > > boot.scr files are text, you might be getting that confused with the > binary environment files produced by "env save", usually called > <board_name>.env or similar. Usually, the source boot.cmd script is a text, which you need to feed to mkimage with "-T script" parameter to process it into boot.scr, which is not exactly a text file any more... > The difference between uEnv.txt and boot.scr is that the latter is > run as a series of commands instead of environment variable definitions > like uEnv.txt. The commands can also set env vars just like uEnv.txt > by using 'env set xxx` commands, but also allows much more like > variable evaluation and conditional logic.
diff --git a/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc b/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc index 43b1ef5f..5f72e7b0 100644 --- a/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc +++ b/meta-arago-distro/recipes-core/images/tisdk-core-bundle.inc @@ -33,7 +33,3 @@ TARGET_IMAGES = "tisdk-base-image tisdk-default-image tisdk-thinlinux-image tisd TISDK_TOOLCHAIN = "meta-toolchain-arago-tisdk" TOOLCHAIN_SUFFIX = "-tisdk" - -IMAGE_INSTALL = "\ - packagegroup-arago-tisdk-amsdk-sdk-host \ -" diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb deleted file mode 100644 index 9562310f..00000000 --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb +++ /dev/null @@ -1,16 +0,0 @@ -SUMMARY = "Task to install additional scripts and applications into the SDK" -LICENSE = "MIT" -PR = "r30" - -PACKAGE_ARCH = "${MACHINE_ARCH}" - -inherit packagegroup - -EXTRA_FILES = "" -EXTRA_FILES:ti43x = "tisdk-uenv" -EXTRA_FILES:omap-a15 = "tisdk-uenv" -EXTRA_FILES:k3 = "tisdk-uenv" - -RDEPENDS:${PN} = "\ - ${EXTRA_FILES} \ -"
This recipe only now exists to add "tisdk-uenv" to tisdk-core-bundle images so uEnv.txt can be packaged into prebuilt-images. We only need to package the uEnv.txt as a "prebuilt-image" if some bundled image includes it, no need to manually add this for all images. Signed-off-by: Andrew Davis <afd@ti.com> --- .../recipes-core/images/tisdk-core-bundle.inc | 4 ---- .../packagegroup-arago-tisdk-amsdk-sdk-host.bb | 16 ---------------- 2 files changed, 20 deletions(-) delete mode 100644 meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-tisdk-amsdk-sdk-host.bb