Message ID | 20231101092855.101794-2-sabiya.d@ti.com |
---|---|
State | Rejected |
Delegated to: | Ryan Eatmon |
Headers | show |
Series | [meta-arago,master/kirkstone] packagegroup-arago-initramfs: Add e2fsprogs-mke2fs and dosfstools | expand |
On 11/1/2023 4:28 AM, sabiya.d@mistralsolutions.com wrote: > From: Dasnavis Sabiya <sabiya.d@ti.com> > > Add e2fsprogs-mke2fs and dosfstools for mkfs.ext4 and mkfs.vfat Right now the only image that is using this package group is the tiny image. And part of the goal of the tiny image is to remain as small as possible. The intent of the tiny image is to support the initial wakeup efforts. During this time the various hardware settings are not fully known and so external memory and other things do not work. The initramfs is meant to provide the bare minimal OS needed to fit into the on-chip SRAM to figure out those settings. Once those settings are figured out, we never go back to the tiny image. So what is the need for the mkfs tools in the tiny image? Why not use a bigger image that has these tools in it? > Signed-off-by: Dasnavis Sabiya <sabiya.d@ti.com> > --- > .../packagegroups/packagegroup-arago-initramfs.bb | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb > index 380cc0e3..10412784 100644 > --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb > +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb > @@ -1,4 +1,13 @@ > > require recipes-core/packagegroups/packagegroup-core-boot.bb > > +ARAGO_FSTOOLS = "\ > + e2fsprogs-mke2fs \ > + dosfstools \ > + " > + > RDEPENDS:${PN}:remove = "grub-efi kernel" > + > +RDEPENDS:${PN}:append = "\ > + ${ARAGO_FSTOOLS} \ > + "
Hi Ryan, On Wed, Nov 1, 2023 at 7:08 PM Ryan Eatmon <reatmon@ti.com> wrote: > > > > On 11/1/2023 4:28 AM, sabiya.d@mistralsolutions.com wrote: > > From: Dasnavis Sabiya <sabiya.d@ti.com> > > > > Add e2fsprogs-mke2fs and dosfstools for mkfs.ext4 and mkfs.vfat > > > Right now the only image that is using this package group is the tiny > image. And part of the goal of the tiny image is to remain as small as > possible. > > The intent of the tiny image is to support the initial wakeup efforts. > During this time the various hardware settings are not fully known and > so external memory and other things do not work. The initramfs is meant > to provide the bare minimal OS needed to fit into the on-chip SRAM to > figure out those settings. Once those settings are figured out, we > never go back to the tiny image. > > So what is the need for the mkfs tools in the tiny image? Why not use a > bigger image that has these tools in it? > There is a customer requirement to provide production programming solution to use arago-tiny-image to partition and program the boot media like eMMC. We need these utilities to format the eMMC. Please refer https://jira.itg.ti.com/browse/LCPD-34783 > > > > Signed-off-by: Dasnavis Sabiya <sabiya.d@ti.com> > > --- > > .../packagegroups/packagegroup-arago-initramfs.bb | 9 +++++++++ > > 1 file changed, 9 insertions(+) > > > > diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb > > index 380cc0e3..10412784 100644 > > --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb > > +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb > > @@ -1,4 +1,13 @@ > > > > require recipes-core/packagegroups/packagegroup-core-boot.bb > > > > +ARAGO_FSTOOLS = "\ > > + e2fsprogs-mke2fs \ > > + dosfstools \ > > + " > > + > > RDEPENDS:${PN}:remove = "grub-efi kernel" > > + > > +RDEPENDS:${PN}:append = "\ > > + ${ARAGO_FSTOOLS} \ > > + " > > -- > Ryan Eatmon reatmon@ti.com > ----------------------------------------- > Texas Instruments, Inc. - LCPD - MGTS Regards, Dasnavis Sabiya
Sabiya, On 02/11/23 12:26, Dasnavis Sabiya via lists.yoctoproject.org wrote: > Hi Ryan, > > On Wed, Nov 1, 2023 at 7:08 PM Ryan Eatmon <reatmon@ti.com> wrote: >> >> >> On 11/1/2023 4:28 AM, sabiya.d@mistralsolutions.com wrote: >>> From: Dasnavis Sabiya <sabiya.d@ti.com> >>> >>> Add e2fsprogs-mke2fs and dosfstools for mkfs.ext4 and mkfs.vfat >> >> Right now the only image that is using this package group is the tiny >> image. And part of the goal of the tiny image is to remain as small as >> possible. >> >> The intent of the tiny image is to support the initial wakeup efforts. >> During this time the various hardware settings are not fully known and >> so external memory and other things do not work. The initramfs is meant >> to provide the bare minimal OS needed to fit into the on-chip SRAM to >> figure out those settings. Once those settings are figured out, we >> never go back to the tiny image. >> >> So what is the need for the mkfs tools in the tiny image? Why not use a >> bigger image that has these tools in it? >> > There is a customer requirement to provide production programming > solution to use arago-tiny-image to partition and program > the boot media like eMMC. We need these utilities to format the eMMC. > Please refer https://jira.itg.ti.com/browse/LCPD-34783 This is upstream mailing list. Please keep the TI internal jira-ids away from it. Regards, Chirag >> >>> Signed-off-by: Dasnavis Sabiya <sabiya.d@ti.com> >>> --- >>> .../packagegroups/packagegroup-arago-initramfs.bb | 9 +++++++++ >>> 1 file changed, 9 insertions(+) >>> >>> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb >>> index 380cc0e3..10412784 100644 >>> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb >>> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb >>> @@ -1,4 +1,13 @@ >>> >>> require recipes-core/packagegroups/packagegroup-core-boot.bb >>> >>> +ARAGO_FSTOOLS = "\ >>> + e2fsprogs-mke2fs \ >>> + dosfstools \ >>> + " >>> + >>> RDEPENDS:${PN}:remove = "grub-efi kernel" >>> + >>> +RDEPENDS:${PN}:append = "\ >>> + ${ARAGO_FSTOOLS} \ >>> + " >> -- >> Ryan Eatmon reatmon@ti.com >> ----------------------------------------- >> Texas Instruments, Inc. - LCPD - MGTS > Regards, > Dasnavis Sabiya > > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > View/Reply Online (#14987): https://lists.yoctoproject.org/g/meta-arago/message/14987 > Mute This Topic: https://lists.yoctoproject.org/mt/102315218/7030289 > Group Owner: meta-arago+owner@lists.yoctoproject.org > Unsubscribe: https://lists.yoctoproject.org/g/meta-arago/unsub [c-shilwant@ti.com] > -=-=-=-=-=-=-=-=-=-=-=- > >
On 11/2/2023 7:05 AM, Chirag Shilwant wrote: > Sabiya, > > On 02/11/23 12:26, Dasnavis Sabiya via lists.yoctoproject.org wrote: >> Hi Ryan, >> >> On Wed, Nov 1, 2023 at 7:08 PM Ryan Eatmon <reatmon@ti.com> wrote: >>> >>> >>> On 11/1/2023 4:28 AM, sabiya.d@mistralsolutions.com wrote: >>>> From: Dasnavis Sabiya <sabiya.d@ti.com> >>>> >>>> Add e2fsprogs-mke2fs and dosfstools for mkfs.ext4 and mkfs.vfat >>> >>> Right now the only image that is using this package group is the tiny >>> image. And part of the goal of the tiny image is to remain as small as >>> possible. >>> >>> The intent of the tiny image is to support the initial wakeup efforts. >>> During this time the various hardware settings are not fully known and >>> so external memory and other things do not work. The initramfs is meant >>> to provide the bare minimal OS needed to fit into the on-chip SRAM to >>> figure out those settings. Once those settings are figured out, we >>> never go back to the tiny image. >>> >>> So what is the need for the mkfs tools in the tiny image? Why not use a >>> bigger image that has these tools in it? >>> >> There is a customer requirement to provide production programming >> solution to use arago-tiny-image to partition and program >> the boot media like eMMC. We need these utilities to format the eMMC. > > > This is upstream mailing list. Please keep the TI internal jira-ids away > from it. > > Regards, > Chirag The purpose of tiny, as stated, is to serve the wakeup efforts of figuring out hardware settings for the kernel/uboot so that we can move to a "real" image that has these kinds of tools. If there is a customer need, they are free to add their own layer to configure their own package-groups or their own images. We are not going to accept this patch. >>> >>>> Signed-off-by: Dasnavis Sabiya <sabiya.d@ti.com> >>>> --- >>>> .../packagegroups/packagegroup-arago-initramfs.bb | 9 >>>> +++++++++ >>>> 1 file changed, 9 insertions(+) >>>> >>>> diff --git >>>> a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb >>>> index 380cc0e3..10412784 100644 >>>> --- >>>> a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb >>>> +++ >>>> b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb >>>> @@ -1,4 +1,13 @@ >>>> >>>> require recipes-core/packagegroups/packagegroup-core-boot.bb >>>> >>>> +ARAGO_FSTOOLS = "\ >>>> + e2fsprogs-mke2fs \ >>>> + dosfstools \ >>>> + " >>>> + >>>> RDEPENDS:${PN}:remove = "grub-efi kernel" >>>> + >>>> +RDEPENDS:${PN}:append = "\ >>>> + ${ARAGO_FSTOOLS} \ >>>> + " >>> -- >>> Ryan Eatmon reatmon@ti.com >>> ----------------------------------------- >>> Texas Instruments, Inc. - LCPD - MGTS >> Regards, >> Dasnavis Sabiya >> >> >> -=-=-=-=-=-=-=-=-=-=-=- >> Links: You receive all messages sent to this group. >> View/Reply Online (#14987): >> https://lists.yoctoproject.org/g/meta-arago/message/14987 >> Mute This Topic: https://lists.yoctoproject.org/mt/102315218/7030289 >> Group Owner: meta-arago+owner@lists.yoctoproject.org >> Unsubscribe: https://lists.yoctoproject.org/g/meta-arago/unsub >> [c-shilwant@ti.com] >> -=-=-=-=-=-=-=-=-=-=-=- >> >>
On 11/2/23 11:10 AM, Ryan Eatmon via lists.yoctoproject.org wrote: > > > On 11/2/2023 7:05 AM, Chirag Shilwant wrote: >> Sabiya, >> >> On 02/11/23 12:26, Dasnavis Sabiya via lists.yoctoproject.org wrote: >>> Hi Ryan, >>> >>> On Wed, Nov 1, 2023 at 7:08 PM Ryan Eatmon <reatmon@ti.com> wrote: >>>> >>>> >>>> On 11/1/2023 4:28 AM, sabiya.d@mistralsolutions.com wrote: >>>>> From: Dasnavis Sabiya <sabiya.d@ti.com> >>>>> >>>>> Add e2fsprogs-mke2fs and dosfstools for mkfs.ext4 and mkfs.vfat >>>> >>>> Right now the only image that is using this package group is the tiny >>>> image. And part of the goal of the tiny image is to remain as small as >>>> possible. >>>> >>>> The intent of the tiny image is to support the initial wakeup efforts. >>>> During this time the various hardware settings are not fully known and >>>> so external memory and other things do not work. The initramfs is meant >>>> to provide the bare minimal OS needed to fit into the on-chip SRAM to >>>> figure out those settings. Once those settings are figured out, we >>>> never go back to the tiny image. >>>> >>>> So what is the need for the mkfs tools in the tiny image? Why not use a >>>> bigger image that has these tools in it? >>>> >>> There is a customer requirement to provide production programming >>> solution to use arago-tiny-image to partition and program >>> the boot media like eMMC. We need these utilities to format the eMMC. >> >> >> This is upstream mailing list. Please keep the TI internal jira-ids away from it. >> >> Regards, >> Chirag > > The purpose of tiny, as stated, is to serve the wakeup efforts of figuring out hardware settings for the kernel/uboot so that we can move to a "real" image that has these kinds of tools. If there is a customer need, they are free to add their own layer to configure their own package-groups or their own images. > > We are not going to accept this patch. > > Also if a customer wants to flash their board we usually recommend doing that from U-Boot[0]. Which can be easily network/DFU/JTAG booted and already can create partitions and flash filesystem images. Booting all the way into some minimal Linux and manually creating the filesystem during a flashing/production flow sounds flawed to begin with so we shouldn't encourage it with an image like this. Andrew [0] https://www.ti.com/lit/an/spracy5/spracy5.pdf >>>> >>>>> Signed-off-by: Dasnavis Sabiya <sabiya.d@ti.com> >>>>> --- >>>>> .../packagegroups/packagegroup-arago-initramfs.bb | 9 +++++++++ >>>>> 1 file changed, 9 insertions(+) >>>>> >>>>> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb >>>>> index 380cc0e3..10412784 100644 >>>>> --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb >>>>> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb >>>>> @@ -1,4 +1,13 @@ >>>>> >>>>> require recipes-core/packagegroups/packagegroup-core-boot.bb >>>>> >>>>> +ARAGO_FSTOOLS = "\ >>>>> + e2fsprogs-mke2fs \ >>>>> + dosfstools \ >>>>> + " >>>>> + >>>>> RDEPENDS:${PN}:remove = "grub-efi kernel" >>>>> + >>>>> +RDEPENDS:${PN}:append = "\ >>>>> + ${ARAGO_FSTOOLS} \ >>>>> + " >>>> -- >>>> Ryan Eatmon reatmon@ti.com >>>> ----------------------------------------- >>>> Texas Instruments, Inc. - LCPD - MGTS >>> Regards, >>> Dasnavis Sabiya >>> >>> >>> >>> >>> >
diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb index 380cc0e3..10412784 100644 --- a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-initramfs.bb @@ -1,4 +1,13 @@ require recipes-core/packagegroups/packagegroup-core-boot.bb +ARAGO_FSTOOLS = "\ + e2fsprogs-mke2fs \ + dosfstools \ + " + RDEPENDS:${PN}:remove = "grub-efi kernel" + +RDEPENDS:${PN}:append = "\ + ${ARAGO_FSTOOLS} \ + "