[master/dunfell,V5,10/12] recipes-bsp: Create a bootstrap-initrd package

Message ID 20220322174808.16341-11-nm@ti.com
State Superseded
Headers show
Series recipes-core: Add a bootstrap Linux image | expand

Commit Message

Nishanth Menon March 22, 2022, 5:48 p.m. UTC
Lets create an bootstrap-initrd package that installs the cpio
generated in the correct locations.

Signed-off-by: Nishanth Menon <nm@ti.com>
---
Changes since V4: None

V4: https://lore.kernel.org/yocto-meta-arago/20220319123310.8546-11-nm@ti.com/

 .../bootstrap-initrd/bootstrap-initrd_1.0.bb  | 22 +++++++++++++++++++
 1 file changed, 22 insertions(+)
 create mode 100644 meta-arago-distro/recipes-bsp/bootstrap-initrd/bootstrap-initrd_1.0.bb

Comments

Denys Dmytriyenko March 22, 2022, 10:21 p.m. UTC | #1
On Tue, Mar 22, 2022 at 12:48:06PM -0500, Nishanth Menon wrote:
> Lets create an bootstrap-initrd package that installs the cpio
> generated in the correct locations.
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
> Changes since V4: None
> 
> V4: https://lore.kernel.org/yocto-meta-arago/20220319123310.8546-11-nm@ti.com/
> 
>  .../bootstrap-initrd/bootstrap-initrd_1.0.bb  | 22 +++++++++++++++++++
>  1 file changed, 22 insertions(+)
>  create mode 100644 meta-arago-distro/recipes-bsp/bootstrap-initrd/bootstrap-initrd_1.0.bb
> 
> diff --git a/meta-arago-distro/recipes-bsp/bootstrap-initrd/bootstrap-initrd_1.0.bb b/meta-arago-distro/recipes-bsp/bootstrap-initrd/bootstrap-initrd_1.0.bb
> new file mode 100644
> index 000000000000..2a53efcd9062
> --- /dev/null
> +++ b/meta-arago-distro/recipes-bsp/bootstrap-initrd/bootstrap-initrd_1.0.bb
> @@ -0,0 +1,22 @@
> +SUMMARY = "Prebuilt initramfs with apps for bootstraping new board"
> +
> +LICENSE = "GPLv2"

I was wondering about the same question I raised in another thread about the 
license[1]:

Default license for OpenEmbedded metadata is MIT, not GPLv2. Any specific 
reason for the change?

[1] https://patchwork.yoctoproject.org/project/ti/patch/20220322162528.15298-7-reatmon@ti.com/


> +CLEANBROKEN = "1"
> +
> +PACKAGE_ARCH = "${MACHINE_ARCH}"
> +
> +S = "${WORKDIR}"
> +
> +BOOSTRAP_IMAGE="tisdk-bootstrap-base-image"
> +
> +TARGET = "bootstrap-rootfs-${MACHINE}.cpio"
> +
> +do_install() {
> +	install -d ${D}/boot
> +	install -m 0644 ${DEPLOY_DIR_IMAGE}/${BOOSTRAP_IMAGE}-${MACHINE}.cpio  ${D}/boot/${TARGET}
> +}
> +
> +FILES_${PN} = "/boot"
> +
> +do_install[depends] = "${BOOSTRAP_IMAGE}:do_image_complete"
> -- 
> 2.31.1
>
Nishanth Menon March 23, 2022, 1:10 p.m. UTC | #2
On 18:21-20220322, Denys Dmytriyenko wrote:
> On Tue, Mar 22, 2022 at 12:48:06PM -0500, Nishanth Menon wrote:
> > Lets create an bootstrap-initrd package that installs the cpio
> > generated in the correct locations.
> > 
> > Signed-off-by: Nishanth Menon <nm@ti.com>
> > ---
> > Changes since V4: None
> > 
> > V4: https://lore.kernel.org/yocto-meta-arago/20220319123310.8546-11-nm@ti.com/
> > 
> >  .../bootstrap-initrd/bootstrap-initrd_1.0.bb  | 22 +++++++++++++++++++
> >  1 file changed, 22 insertions(+)
> >  create mode 100644 meta-arago-distro/recipes-bsp/bootstrap-initrd/bootstrap-initrd_1.0.bb
> > 
> > diff --git a/meta-arago-distro/recipes-bsp/bootstrap-initrd/bootstrap-initrd_1.0.bb b/meta-arago-distro/recipes-bsp/bootstrap-initrd/bootstrap-initrd_1.0.bb
> > new file mode 100644
> > index 000000000000..2a53efcd9062
> > --- /dev/null
> > +++ b/meta-arago-distro/recipes-bsp/bootstrap-initrd/bootstrap-initrd_1.0.bb
> > @@ -0,0 +1,22 @@
> > +SUMMARY = "Prebuilt initramfs with apps for bootstraping new board"
> > +
> > +LICENSE = "GPLv2"
> 
> I was wondering about the same question I raised in another thread about the 
> license[1]:
> 
> Default license for OpenEmbedded metadata is MIT, not GPLv2. Any specific 
> reason for the change?
> 
> [1] https://patchwork.yoctoproject.org/project/ti/patch/20220322162528.15298-7-reatmon@ti.com/

Aaah.. Thanks for pointing it out.. no specific reason why I picked
GPLV2 other than that as a kernel dev, that is what comes to mind as
default ;)..

It should follow the standard convention of MIT.

Let me know if I need to send a V6 with the fixup, I am guessing
switching it over to LICENSE = "MIT" should do the job.. (I'd rather be
explicit).

Patch

diff --git a/meta-arago-distro/recipes-bsp/bootstrap-initrd/bootstrap-initrd_1.0.bb b/meta-arago-distro/recipes-bsp/bootstrap-initrd/bootstrap-initrd_1.0.bb
new file mode 100644
index 000000000000..2a53efcd9062
--- /dev/null
+++ b/meta-arago-distro/recipes-bsp/bootstrap-initrd/bootstrap-initrd_1.0.bb
@@ -0,0 +1,22 @@ 
+SUMMARY = "Prebuilt initramfs with apps for bootstraping new board"
+
+LICENSE = "GPLv2"
+
+CLEANBROKEN = "1"
+
+PACKAGE_ARCH = "${MACHINE_ARCH}"
+
+S = "${WORKDIR}"
+
+BOOSTRAP_IMAGE="tisdk-bootstrap-base-image"
+
+TARGET = "bootstrap-rootfs-${MACHINE}.cpio"
+
+do_install() {
+	install -d ${D}/boot
+	install -m 0644 ${DEPLOY_DIR_IMAGE}/${BOOSTRAP_IMAGE}-${MACHINE}.cpio  ${D}/boot/${TARGET}
+}
+
+FILES_${PN} = "/boot"
+
+do_install[depends] = "${BOOSTRAP_IMAGE}:do_image_complete"