[dunfell/master PATCH 0/6] recipes-core: Lets add a bootstrap Linux image

[dunfell/master PATCH 0/6] recipes-core: Lets add a bootstrap Linux image

[Nishanth Menon]

Hi,

Board bringups are typically painful. however, over the years, folks
know of minimal debug tools that are pretty effective in helping debug
basic peripherals. Lets create a reusable packagegroup balancing the
tools from wisdom from various folks vs the size of the resultant
package.

We would like to base such a system on more simplistic sysvinit (no
offence systemd, you are still our fav), as it has a much smaller
footprint.

The board startup is best done with a minimal u-boot and kernel device
tree configurations.

Nishanth Menon (6):
  tisdk-tiny-image: Split common components into include file
  tisdk-tiny-image: Add some usability notes
  packagegroups: Add a new bootstrap package group
  recipes-core: images: Add a basic bootstrap base image
  recipes-bsp: Create a k3-bootstrap-initrd package
  recipes-core: images: Introduce tisdk-bootstrap-image

 .../k3-bootstrap-initrd_1.0.bb                | 22 ++++++++
 .../recipes-core/images/arago-tiny-image.inc  |  9 +++
 .../images/tisdk-bootstrap-base-image.bb      | 20 +++++++
 .../images/tisdk-bootstrap-image.bb           | 17 ++++++
 .../recipes-core/images/tisdk-tiny-image.bb   | 14 ++---
 .../packagegroup-arago-bootstrap.bb           | 55 +++++++++++++++++++
 6 files changed, 130 insertions(+), 7 deletions(-)
 create mode 100644 meta-arago-distro/recipes-bsp/k3-bootstrap-initrd/k3-bootstrap-initrd_1.0.bb
 create mode 100644 meta-arago-distro/recipes-core/images/arago-tiny-image.inc
 create mode 100644 meta-arago-distro/recipes-core/images/tisdk-bootstrap-base-image.bb
 create mode 100644 meta-arago-distro/recipes-core/images/tisdk-bootstrap-image.bb
 create mode 100644 meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb

-- 
2.31.1

[dunfell/master PATCH 1/6] tisdk-tiny-image: Split common components into include file

[Nishanth Menon]

Lets split the common pieces for creating a tiny image into an include
file This can then be used to incrementally build additional images as
well.

Signed-off-by: Nishanth Menon <nm@ti.com>
---
 .../recipes-core/images/arago-tiny-image.inc          |  9 +++++++++
 .../recipes-core/images/tisdk-tiny-image.bb           | 11 +++--------
 2 files changed, 12 insertions(+), 8 deletions(-)
 create mode 100644 meta-arago-distro/recipes-core/images/arago-tiny-image.inc

diff --git a/meta-arago-distro/recipes-core/images/arago-tiny-image.inc b/meta-arago-distro/recipes-core/images/arago-tiny-image.inc
new file mode 100644
index 000000000000..79c8f6075911
--- /dev/null
+++ b/meta-arago-distro/recipes-core/images/arago-tiny-image.inc
@@ -0,0 +1,9 @@
+ARAGO_SYSVINIT = "1"
+
+require arago-image.inc
+
+IMAGE_FEATURES_remove = "package-management splash"
+
+IMAGE_INSTALL = " \
+	packagegroup-arago-sysvinit-boot \
+"
diff --git a/meta-arago-distro/recipes-core/images/tisdk-tiny-image.bb b/meta-arago-distro/recipes-core/images/tisdk-tiny-image.bb
index c0ef8f247ee7..af04b76bc588 100644
--- a/meta-arago-distro/recipes-core/images/tisdk-tiny-image.bb
+++ b/meta-arago-distro/recipes-core/images/tisdk-tiny-image.bb
@@ -1,15 +1,10 @@
-ARAGO_SYSVINIT = "1"
-
-require arago-image.inc
+ARAGO_TINY_IMAGE_EXTRA_INSTALL ?= ""
 
-IMAGE_FEATURES_remove = "package-management splash"
+require arago-tiny-image.inc
 
 IMAGE_FSTYPES += "cpio cpio.xz"
 
-ARAGO_TINY_IMAGE_EXTRA_INSTALL ?= ""
-
-IMAGE_INSTALL = " \
-	packagegroup-arago-sysvinit-boot \
+IMAGE_INSTALL += " \
 	${ARAGO_TINY_IMAGE_EXTRA_INSTALL} \
 "
 
-- 
2.31.1

[dunfell/master PATCH 2/6] tisdk-tiny-image: Add some usability notes

[Nishanth Menon]

Lets clarify the usage of tiny image as something meant as a building
block and not something that folks can directly use.

Signed-off-by: Nishanth Menon <nm@ti.com>
---
 meta-arago-distro/recipes-core/images/tisdk-tiny-image.bb | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/meta-arago-distro/recipes-core/images/tisdk-tiny-image.bb b/meta-arago-distro/recipes-core/images/tisdk-tiny-image.bb
index af04b76bc588..e9abe31894ff 100644
--- a/meta-arago-distro/recipes-core/images/tisdk-tiny-image.bb
+++ b/meta-arago-distro/recipes-core/images/tisdk-tiny-image.bb
@@ -1,3 +1,8 @@
+# Arago TI SDK super minimal base image for initramfs
+# This will not get a completely usable image since kernel will be missing
+# intent for this image is to build as basic a system to allow further
+# manual addition as needed.
+
 ARAGO_TINY_IMAGE_EXTRA_INSTALL ?= ""
 
 require arago-tiny-image.inc
-- 
2.31.1

[dunfell/master PATCH 3/6] packagegroups: Add a new bootstrap package group

[Nishanth Menon]

Lets try and create a minimal package group for board bringup. Lets
call this basic system necessary for board bringup as "bootstrap".

Signed-off-by: Nishanth Menon <nm@ti.com>
---
 .../packagegroup-arago-bootstrap.bb           | 55 +++++++++++++++++++
 1 file changed, 55 insertions(+)
 create mode 100644 meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb

diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb
new file mode 100644
index 000000000000..e5b670a16b73
--- /dev/null
+++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb
@@ -0,0 +1,55 @@
+DESCRIPTION = "Task to install additional utilities for initial board bringup"
+LICENSE = "MIT"
+PR = "r0"
+
+PACKAGE_ARCH = "${MACHINE_ARCH}"
+
+inherit packagegroup
+
+PACKAGES =+ "${PN}-extra"
+
+UTILS = " \
+	 arm-benchmarks \
+	 evtest \
+	 memtester \
+	 nbench-byte \
+	 pcitest \
+	 serialcheck \
+	 yavta \
+	 hdparm \
+	 i2c-tools \
+	 libdrm-tests \
+	 rt-tests \
+	 iozone3 \
+	 mtd-utils-ubifs-tests \
+	 net-tools \
+	 ethtool  \
+	 pciutils \
+	 rng-tools \
+	 stress-ng \
+	 iperf \
+	 lmbench \
+	 usbutils \
+	 devmem2 \
+	 phytool \
+"
+
+UTILS_append_ti33x = " \
+    omapconf \
+"
+
+UTILS_append_ti43x = " \
+    omapconf \
+"
+
+UTILS_append_omap-a15 = " \
+    omapconf \
+"
+
+UTILS_append_k3 = " \
+    k3conf \
+"
+
+RDEPENDS_${PN} = "\
+    ${UTILS} \
+"
-- 
2.31.1

[dunfell/master PATCH 4/6] recipes-core: images: Add a basic bootstrap base image

[Nishanth Menon]

Since we do not wish to modify the tiny-image from anything but really
tiny, lets use that infrastructure and add the bootstrap package group
to create initramfs (not exactly a completely usable system) which can
be used for basic board bringup.

This generates cpio around 36-40MB which should be usually fine for
even smaller systems (Ofcourse, this is much larger than tiny-image
which is around 6MB size). The packages are organized per size to
allow easier customization if desired.

This image follows the Bring Your Own Kernel model from
tisdk-tiny-image.

Signed-off-by: Nishanth Menon <nm@ti.com>
---
 .../images/tisdk-bootstrap-base-image.bb      | 20 +++++++++++++++++++
 1 file changed, 20 insertions(+)
 create mode 100644 meta-arago-distro/recipes-core/images/tisdk-bootstrap-base-image.bb

diff --git a/meta-arago-distro/recipes-core/images/tisdk-bootstrap-base-image.bb b/meta-arago-distro/recipes-core/images/tisdk-bootstrap-base-image.bb
new file mode 100644
index 000000000000..a18f39602286
--- /dev/null
+++ b/meta-arago-distro/recipes-core/images/tisdk-bootstrap-base-image.bb
@@ -0,0 +1,20 @@
+# Arago TI SDK bootstrap base image for initramfs
+# This will not get a completely usable image since kernel will be missing
+# intent for this image is to build as basic a system with enough basic
+# Board bringup utilities that can then be used for additions as needed.
+
+require arago-tiny-image.inc
+
+IMAGE_FSTYPES += "cpio cpio.xz"
+
+ARAGO_BOOTSTRAP_IMAGE_EXTRA_INSTALL ?= ""
+
+IMAGE_INSTALL += " \
+	packagegroup-arago-bootstrap \
+	${ARAGO_BOOTSTRAP_IMAGE_EXTRA_INSTALL} \
+"
+
+# We do not need kernel Image, dtbs and modules in the initrd
+deltask do_packagedata
+
+export IMAGE_BASENAME = "tisdk-bootstrap-base-image"
-- 
2.31.1

[dunfell/master PATCH 5/6] recipes-bsp: Create a k3-bootstrap-initrd package

[Nishanth Menon]

Lets create an k3-bootstrap-initrd package that installs the cpio
generated in the correct locations.

Signed-off-by: Nishanth Menon <nm@ti.com>
---
 .../k3-bootstrap-initrd_1.0.bb                | 22 +++++++++++++++++++
 1 file changed, 22 insertions(+)
 create mode 100644 meta-arago-distro/recipes-bsp/k3-bootstrap-initrd/k3-bootstrap-initrd_1.0.bb

diff --git a/meta-arago-distro/recipes-bsp/k3-bootstrap-initrd/k3-bootstrap-initrd_1.0.bb b/meta-arago-distro/recipes-bsp/k3-bootstrap-initrd/k3-bootstrap-initrd_1.0.bb
new file mode 100644
index 000000000000..452f88a35346
--- /dev/null
+++ b/meta-arago-distro/recipes-bsp/k3-bootstrap-initrd/k3-bootstrap-initrd_1.0.bb
@@ -0,0 +1,22 @@
+DESCRIPTION = "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"
-- 
2.31.1

[dunfell/master PATCH 6/6] recipes-core: images: Introduce tisdk-bootstrap-image

[Nishanth Menon]

Lets introduce tisdk-bootstrap-image which now consists of a
completely bootable system with bootstrap initramfs that can be used
to boot up and minimally debug a system. We also will create a rootfs
that matches with the initramfs to allow users who use mmc and other
similar boot media to have access to the same debug tools for a new
platform bringup.

This kind of image is usually suited to be used with a minimal
bootloader and kernel devicetree configuration.

Signed-off-by: Nishanth Menon <nm@ti.com>
---
 .../images/tisdk-bootstrap-image.bb             | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)
 create mode 100644 meta-arago-distro/recipes-core/images/tisdk-bootstrap-image.bb

diff --git a/meta-arago-distro/recipes-core/images/tisdk-bootstrap-image.bb b/meta-arago-distro/recipes-core/images/tisdk-bootstrap-image.bb
new file mode 100644
index 000000000000..85584f0cf257
--- /dev/null
+++ b/meta-arago-distro/recipes-core/images/tisdk-bootstrap-image.bb
@@ -0,0 +1,17 @@
+# Arago TI SDK bootstrap image usable for board bringup
+# This consists of an usable initramfs containing board bringup utilities
+# that can then be used. In addition the root filesystem also has the same
+# set of utilities for debug and basic performance checks.
+
+require arago-tiny-image.inc
+
+ARAGO_BOOTSTRAP_IMAGE_EXTRA_INSTALL ?= ""
+
+IMAGE_INSTALL += " \
+	packagegroup-arago-bootstrap \
+	kernel \
+	k3-bootstrap-initrd \
+	${ARAGO_BOOTSTRAP_IMAGE_EXTRA_INSTALL} \
+"
+
+export IMAGE_BASENAME = "tisdk-bootstrap-image"
-- 
2.31.1

Re: [dunfell/master PATCH 2/6] tisdk-tiny-image: Add some usability notes

[Denys Dmytriyenko]

On Tue, Mar 08, 2022 at 08:13:12PM -0600, Nishanth Menon wrote:
> Lets clarify the usage of tiny image as something meant as a building
> block and not something that folks can directly use.

It still can be directly used though - see below.


> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
>  meta-arago-distro/recipes-core/images/tisdk-tiny-image.bb | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/meta-arago-distro/recipes-core/images/tisdk-tiny-image.bb b/meta-arago-distro/recipes-core/images/tisdk-tiny-image.bb
> index af04b76bc588..e9abe31894ff 100644
> --- a/meta-arago-distro/recipes-core/images/tisdk-tiny-image.bb
> +++ b/meta-arago-distro/recipes-core/images/tisdk-tiny-image.bb
> @@ -1,3 +1,8 @@
> +# Arago TI SDK super minimal base image for initramfs
> +# This will not get a completely usable image since kernel will be missing

I'd like to rephrase this a bit. The kernel is not in /boot of the rootfs, but 
it will be deployed and there are still many other ways to load it separately 
(tftp, separate boot partition, etc) and boot this image.


> +# intent for this image is to build as basic a system to allow further
> +# manual addition as needed.
> +
>  ARAGO_TINY_IMAGE_EXTRA_INSTALL ?= ""
>  
>  require arago-tiny-image.inc
> -- 
> 2.31.1
> 

-- 
Regards,
Denys Dmytriyenko <denis@denix.org>
PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964
Fingerprint: 25FC E4A5 8A72 2F69 1186  6D76 4209 0272 9A92 C964

Re: [meta-arago] [dunfell/master PATCH 3/6] packagegroups: Add a new bootstrap package group

[Denys Dmytriyenko]

On Tue, Mar 08, 2022 at 08:13:13PM -0600, Nishanth Menon via lists.yoctoproject.org wrote:
> Lets try and create a minimal package group for board bringup. Lets
> call this basic system necessary for board bringup as "bootstrap".
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
>  .../packagegroup-arago-bootstrap.bb           | 55 +++++++++++++++++++
>  1 file changed, 55 insertions(+)
>  create mode 100644 meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb
> 
> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb
> new file mode 100644
> index 000000000000..e5b670a16b73
> --- /dev/null
> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb
> @@ -0,0 +1,55 @@
> +DESCRIPTION = "Task to install additional utilities for initial board bringup"

Need to start setting SUMMARY by default, not DESCRIPTION. DESCRIPTION is an 
alternative longer explanation for the package, otherwise it's the same as 
SUMMARY. Yes, I know many recipes still use the old convention, but need to be 
fixed.


> +LICENSE = "MIT"
> +PR = "r0"
> +
> +PACKAGE_ARCH = "${MACHINE_ARCH}"
> +
> +inherit packagegroup
> +
> +PACKAGES =+ "${PN}-extra"

I don't see ${PN}-extra being used/populated anywhere here.


> +UTILS = " \
> +	 arm-benchmarks \
> +	 evtest \
> +	 memtester \
> +	 nbench-byte \
> +	 pcitest \
> +	 serialcheck \
> +	 yavta \
> +	 hdparm \
> +	 i2c-tools \
> +	 libdrm-tests \
> +	 rt-tests \
> +	 iozone3 \
> +	 mtd-utils-ubifs-tests \
> +	 net-tools \
> +	 ethtool  \
> +	 pciutils \
> +	 rng-tools \
> +	 stress-ng \
> +	 iperf \
> +	 lmbench \
> +	 usbutils \
> +	 devmem2 \
> +	 phytool \
> +"
> +
> +UTILS_append_ti33x = " \
> +    omapconf \
> +"
> +
> +UTILS_append_ti43x = " \
> +    omapconf \
> +"
> +
> +UTILS_append_omap-a15 = " \
> +    omapconf \
> +"
> +
> +UTILS_append_k3 = " \
> +    k3conf \
> +"
> +
> +RDEPENDS_${PN} = "\
> +    ${UTILS} \
> +"
> -- 
> 2.31.1
> 
> 
> 
> 
> 
> 

-- 
Regards,
Denys Dmytriyenko <denis@denix.org>
PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964
Fingerprint: 25FC E4A5 8A72 2F69 1186  6D76 4209 0272 9A92 C964

Re: [meta-arago] [dunfell/master PATCH 3/6] packagegroups: Add a new bootstrap package group

[Denys Dmytriyenko]

On Tue, Mar 08, 2022 at 08:13:13PM -0600, Nishanth Menon via lists.yoctoproject.org wrote:
> Lets try and create a minimal package group for board bringup. Lets
> call this basic system necessary for board bringup as "bootstrap".
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
>  .../packagegroup-arago-bootstrap.bb           | 55 +++++++++++++++++++
>  1 file changed, 55 insertions(+)
>  create mode 100644 meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb
> 
> diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb
> new file mode 100644
> index 000000000000..e5b670a16b73
> --- /dev/null
> +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb
> @@ -0,0 +1,55 @@
> +DESCRIPTION = "Task to install additional utilities for initial board bringup"

Need to start setting SUMMARY by default, not DESCRIPTION. DESCRIPTION is an 
alternative longer explanation for the package, otherwise it's the same as 
SUMMARY. Yes, I know many recipes still use the old convention, but need to be 
fixed.


> +LICENSE = "MIT"
> +PR = "r0"
> +
> +PACKAGE_ARCH = "${MACHINE_ARCH}"
> +
> +inherit packagegroup
> +
> +PACKAGES =+ "${PN}-extra"

I don't see ${PN}-extra being used/populated anywhere here.


> +UTILS = " \
> +	 arm-benchmarks \
> +	 evtest \
> +	 memtester \
> +	 nbench-byte \
> +	 pcitest \
> +	 serialcheck \
> +	 yavta \
> +	 hdparm \
> +	 i2c-tools \
> +	 libdrm-tests \
> +	 rt-tests \
> +	 iozone3 \
> +	 mtd-utils-ubifs-tests \
> +	 net-tools \
> +	 ethtool  \
> +	 pciutils \
> +	 rng-tools \
> +	 stress-ng \
> +	 iperf \
> +	 lmbench \
> +	 usbutils \
> +	 devmem2 \
> +	 phytool \
> +"
> +
> +UTILS_append_ti33x = " \
> +    omapconf \
> +"
> +
> +UTILS_append_ti43x = " \
> +    omapconf \
> +"
> +
> +UTILS_append_omap-a15 = " \
> +    omapconf \
> +"
> +
> +UTILS_append_k3 = " \
> +    k3conf \
> +"
> +
> +RDEPENDS_${PN} = "\
> +    ${UTILS} \
> +"
> -- 
> 2.31.1
> 
> 
> 
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#13610): https://lists.yoctoproject.org/g/meta-arago/message/13610
> Mute This Topic: https://lists.yoctoproject.org/mt/89653829/3617104
> Group Owner: meta-arago+owner@lists.yoctoproject.org
> Unsubscribe: https://lists.yoctoproject.org/g/meta-arago/unsub [denis@denix.org]
> -=-=-=-=-=-=-=-=-=-=-=-
> 
> 

-- 
Regards,
Denys Dmytriyenko <denis@denix.org>
PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964
Fingerprint: 25FC E4A5 8A72 2F69 1186  6D76 4209 0272 9A92 C964

Re: [dunfell/master PATCH 4/6] recipes-core: images: Add a basic bootstrap base image

[Denys Dmytriyenko]

On Tue, Mar 08, 2022 at 08:13:14PM -0600, Nishanth Menon wrote:
> Since we do not wish to modify the tiny-image from anything but really
> tiny, lets use that infrastructure and add the bootstrap package group
> to create initramfs (not exactly a completely usable system) which can
> be used for basic board bringup.
> 
> This generates cpio around 36-40MB which should be usually fine for
> even smaller systems (Ofcourse, this is much larger than tiny-image
> which is around 6MB size). The packages are organized per size to
> allow easier customization if desired.
> 
> This image follows the Bring Your Own Kernel model from
> tisdk-tiny-image.
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
>  .../images/tisdk-bootstrap-base-image.bb      | 20 +++++++++++++++++++
>  1 file changed, 20 insertions(+)
>  create mode 100644 meta-arago-distro/recipes-core/images/tisdk-bootstrap-base-image.bb
> 
> diff --git a/meta-arago-distro/recipes-core/images/tisdk-bootstrap-base-image.bb b/meta-arago-distro/recipes-core/images/tisdk-bootstrap-base-image.bb
> new file mode 100644
> index 000000000000..a18f39602286
> --- /dev/null
> +++ b/meta-arago-distro/recipes-core/images/tisdk-bootstrap-base-image.bb
> @@ -0,0 +1,20 @@
> +# Arago TI SDK bootstrap base image for initramfs
> +# This will not get a completely usable image since kernel will be missing

Same comment as for earlier patch.


> +# intent for this image is to build as basic a system with enough basic
> +# Board bringup utilities that can then be used for additions as needed.
> +
> +require arago-tiny-image.inc
> +
> +IMAGE_FSTYPES += "cpio cpio.xz"
> +
> +ARAGO_BOOTSTRAP_IMAGE_EXTRA_INSTALL ?= ""
> +
> +IMAGE_INSTALL += " \
> +	packagegroup-arago-bootstrap \
> +	${ARAGO_BOOTSTRAP_IMAGE_EXTRA_INSTALL} \
> +"
> +
> +# We do not need kernel Image, dtbs and modules in the initrd
> +deltask do_packagedata

Huh? Not sure what you are trying to do here, but it certainly isn't right...
You are trying to partially disable generating of IPK packages for the image, 
but images by default have no packages to produce.


> +export IMAGE_BASENAME = "tisdk-bootstrap-base-image"
> -- 
> 2.31.1
> 

-- 
Regards,
Denys Dmytriyenko <denis@denix.org>
PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964
Fingerprint: 25FC E4A5 8A72 2F69 1186  6D76 4209 0272 9A92 C964

Re: [meta-arago] [dunfell/master PATCH 5/6] recipes-bsp: Create a k3-bootstrap-initrd package

[Denys Dmytriyenko]

On Tue, Mar 08, 2022 at 08:13:15PM -0600, Nishanth Menon via lists.yoctoproject.org wrote:
> Lets create an k3-bootstrap-initrd package that installs the cpio
> generated in the correct locations.

This has k3 in the name - is it really k3-specific? Do you want to set 
COMPATIBLE_MACHINE accordingly?


> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
>  .../k3-bootstrap-initrd_1.0.bb                | 22 +++++++++++++++++++
>  1 file changed, 22 insertions(+)
>  create mode 100644 meta-arago-distro/recipes-bsp/k3-bootstrap-initrd/k3-bootstrap-initrd_1.0.bb
> 
> diff --git a/meta-arago-distro/recipes-bsp/k3-bootstrap-initrd/k3-bootstrap-initrd_1.0.bb b/meta-arago-distro/recipes-bsp/k3-bootstrap-initrd/k3-bootstrap-initrd_1.0.bb
> new file mode 100644
> index 000000000000..452f88a35346
> --- /dev/null
> +++ b/meta-arago-distro/recipes-bsp/k3-bootstrap-initrd/k3-bootstrap-initrd_1.0.bb
> @@ -0,0 +1,22 @@
> +DESCRIPTION = "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"
> -- 
> 2.31.1

-- 
Regards,
Denys Dmytriyenko <denis@denix.org>
PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964
Fingerprint: 25FC E4A5 8A72 2F69 1186  6D76 4209 0272 9A92 C964

Re: [meta-arago] [dunfell/master PATCH 4/6] recipes-core: images: Add a basic bootstrap base image

[Denys Dmytriyenko]

On Fri, Mar 11, 2022 at 12:53:38PM -0500, Denys Dmytriyenko wrote:
> On Tue, Mar 08, 2022 at 08:13:14PM -0600, Nishanth Menon wrote:
> > Since we do not wish to modify the tiny-image from anything but really
> > tiny, lets use that infrastructure and add the bootstrap package group
> > to create initramfs (not exactly a completely usable system) which can
> > be used for basic board bringup.
> > 
> > This generates cpio around 36-40MB which should be usually fine for
> > even smaller systems (Ofcourse, this is much larger than tiny-image
> > which is around 6MB size). The packages are organized per size to
> > allow easier customization if desired.
> > 
> > This image follows the Bring Your Own Kernel model from
> > tisdk-tiny-image.
> > 
> > Signed-off-by: Nishanth Menon <nm@ti.com>
> > ---
> >  .../images/tisdk-bootstrap-base-image.bb      | 20 +++++++++++++++++++
> >  1 file changed, 20 insertions(+)
> >  create mode 100644 meta-arago-distro/recipes-core/images/tisdk-bootstrap-base-image.bb
> > 
> > diff --git a/meta-arago-distro/recipes-core/images/tisdk-bootstrap-base-image.bb b/meta-arago-distro/recipes-core/images/tisdk-bootstrap-base-image.bb
> > new file mode 100644
> > index 000000000000..a18f39602286
> > --- /dev/null
> > +++ b/meta-arago-distro/recipes-core/images/tisdk-bootstrap-base-image.bb
> > @@ -0,0 +1,20 @@
> > +# Arago TI SDK bootstrap base image for initramfs
> > +# This will not get a completely usable image since kernel will be missing
> 
> Same comment as for earlier patch.
> 
> 
> > +# intent for this image is to build as basic a system with enough basic
> > +# Board bringup utilities that can then be used for additions as needed.
> > +
> > +require arago-tiny-image.inc
> > +
> > +IMAGE_FSTYPES += "cpio cpio.xz"
> > +
> > +ARAGO_BOOTSTRAP_IMAGE_EXTRA_INSTALL ?= ""
> > +
> > +IMAGE_INSTALL += " \
> > +	packagegroup-arago-bootstrap \
> > +	${ARAGO_BOOTSTRAP_IMAGE_EXTRA_INSTALL} \
> > +"
> > +
> > +# We do not need kernel Image, dtbs and modules in the initrd
> > +deltask do_packagedata
> 
> Huh? Not sure what you are trying to do here, but it certainly isn't right...
> You are trying to partially disable generating of IPK packages for the image, 
> but images by default have no packages to produce.

I think you stepped on this workaround, that is no longer needed:
https://git.yoctoproject.org/meta-arago/commit/?id=a75e468a372d4f5c2c32a179d4d2219899b5190c


> > +export IMAGE_BASENAME = "tisdk-bootstrap-base-image"
> > -- 
> > 2.31.1

-- 
Regards,
Denys Dmytriyenko <denis@denix.org>
PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964
Fingerprint: 25FC E4A5 8A72 2F69 1186  6D76 4209 0272 9A92 C964

Re: [meta-arago] [dunfell/master PATCH 6/6] recipes-core: images: Introduce tisdk-bootstrap-image

[Denys Dmytriyenko]

On Tue, Mar 08, 2022 at 08:13:16PM -0600, Nishanth Menon via lists.yoctoproject.org wrote:
> Lets introduce tisdk-bootstrap-image which now consists of a
> completely bootable system with bootstrap initramfs that can be used
> to boot up and minimally debug a system. We also will create a rootfs
> that matches with the initramfs to allow users who use mmc and other
> similar boot media to have access to the same debug tools for a new
> platform bringup.
> 
> This kind of image is usually suited to be used with a minimal
> bootloader and kernel devicetree configuration.
> 
> Signed-off-by: Nishanth Menon <nm@ti.com>
> ---
>  .../images/tisdk-bootstrap-image.bb             | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
>  create mode 100644 meta-arago-distro/recipes-core/images/tisdk-bootstrap-image.bb
> 
> diff --git a/meta-arago-distro/recipes-core/images/tisdk-bootstrap-image.bb b/meta-arago-distro/recipes-core/images/tisdk-bootstrap-image.bb
> new file mode 100644
> index 000000000000..85584f0cf257
> --- /dev/null
> +++ b/meta-arago-distro/recipes-core/images/tisdk-bootstrap-image.bb
> @@ -0,0 +1,17 @@
> +# Arago TI SDK bootstrap image usable for board bringup
> +# This consists of an usable initramfs containing board bringup utilities
> +# that can then be used. In addition the root filesystem also has the same
> +# set of utilities for debug and basic performance checks.
> +
> +require arago-tiny-image.inc
> +
> +ARAGO_BOOTSTRAP_IMAGE_EXTRA_INSTALL ?= ""
> +
> +IMAGE_INSTALL += " \
> +	packagegroup-arago-bootstrap \
> +	kernel \

This should probably be kernel-base. There's no package called just kernel, 
but rather kernel-<version> which is kernel-base in disguise :)


> +	k3-bootstrap-initrd \
> +	${ARAGO_BOOTSTRAP_IMAGE_EXTRA_INSTALL} \
> +"
> +
> +export IMAGE_BASENAME = "tisdk-bootstrap-image"
> -- 
> 2.31.1

-- 
Regards,
Denys Dmytriyenko <denis@denix.org>
PGP: 0x420902729A92C964 - https://denix.org/0x420902729A92C964
Fingerprint: 25FC E4A5 8A72 2F69 1186  6D76 4209 0272 9A92 C964

Re: [dunfell/master PATCH 2/6] tisdk-tiny-image: Add some usability notes

[Nishanth Menon]

On 12:38-20220311, Denys Dmytriyenko wrote:
> On Tue, Mar 08, 2022 at 08:13:12PM -0600, Nishanth Menon wrote:
> > Lets clarify the usage of tiny image as something meant as a building
> > block and not something that folks can directly use.
> 
> It still can be directly used though - see below.
> 
> 
> > Signed-off-by: Nishanth Menon <nm@ti.com>
> > ---
> >  meta-arago-distro/recipes-core/images/tisdk-tiny-image.bb | 5 +++++
> >  1 file changed, 5 insertions(+)
> > 
> > diff --git a/meta-arago-distro/recipes-core/images/tisdk-tiny-image.bb b/meta-arago-distro/recipes-core/images/tisdk-tiny-image.bb
> > index af04b76bc588..e9abe31894ff 100644
> > --- a/meta-arago-distro/recipes-core/images/tisdk-tiny-image.bb
> > +++ b/meta-arago-distro/recipes-core/images/tisdk-tiny-image.bb
> > @@ -1,3 +1,8 @@
> > +# Arago TI SDK super minimal base image for initramfs
> > +# This will not get a completely usable image since kernel will be missing
> 
> I'd like to rephrase this a bit. The kernel is not in /boot of the rootfs, but 
> it will be deployed and there are still many other ways to load it separately 
> (tftp, separate boot partition, etc) and boot this image.

How about: This image does not package the kernel in the standard /boot
of rootfs, instead, it provides a template for allowing kernel to be
deployed elsewhere (tftp/separate boot partition/jtag log etc..) and
boot the image.


Thinking again, I should probably use a SUMMARY: and DESCRIPTION: tag to
help standardize the comment help sections (which could then be queried
by oe-layer-setup for proper list of images)

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D

Re: [meta-arago] [dunfell/master PATCH 3/6] packagegroups: Add a new bootstrap package group

[Nishanth Menon]

On 12:43-20220311, Denys Dmytriyenko wrote:
> On Tue, Mar 08, 2022 at 08:13:13PM -0600, Nishanth Menon via lists.yoctoproject.org wrote:
> > Lets try and create a minimal package group for board bringup. Lets
> > call this basic system necessary for board bringup as "bootstrap".
> > 
> > Signed-off-by: Nishanth Menon <nm@ti.com>
> > ---
> >  .../packagegroup-arago-bootstrap.bb           | 55 +++++++++++++++++++
> >  1 file changed, 55 insertions(+)
> >  create mode 100644 meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb
> > 
> > diff --git a/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb
> > new file mode 100644
> > index 000000000000..e5b670a16b73
> > --- /dev/null
> > +++ b/meta-arago-distro/recipes-core/packagegroups/packagegroup-arago-bootstrap.bb
> > @@ -0,0 +1,55 @@
> > +DESCRIPTION = "Task to install additional utilities for initial board bringup"
> 
> Need to start setting SUMMARY by default, not DESCRIPTION. DESCRIPTION is an 
> alternative longer explanation for the package, otherwise it's the same as 
> SUMMARY. Yes, I know many recipes still use the old convention, but need to be 
> fixed.

Aaah, thanks.. will fix.

> > +LICENSE = "MIT"
> > +PR = "r0"
> > +
> > +PACKAGE_ARCH = "${MACHINE_ARCH}"
> > +
> > +inherit packagegroup
> > +
> > +PACKAGES =+ "${PN}-extra"
> 
> I don't see ${PN}-extra being used/populated anywhere here.

Will drop - carry over from copy paste.


[...]

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D

Re: [dunfell/master PATCH 4/6] recipes-core: images: Add a basic bootstrap base image

[Nishanth Menon]

On 12:53-20220311, Denys Dmytriyenko wrote:
> On Tue, Mar 08, 2022 at 08:13:14PM -0600, Nishanth Menon wrote:
> > Since we do not wish to modify the tiny-image from anything but really
> > tiny, lets use that infrastructure and add the bootstrap package group
> > to create initramfs (not exactly a completely usable system) which can
> > be used for basic board bringup.
> > 
> > This generates cpio around 36-40MB which should be usually fine for
> > even smaller systems (Ofcourse, this is much larger than tiny-image
> > which is around 6MB size). The packages are organized per size to
> > allow easier customization if desired.
> > 
> > This image follows the Bring Your Own Kernel model from
> > tisdk-tiny-image.
> > 
> > Signed-off-by: Nishanth Menon <nm@ti.com>
> > ---
> >  .../images/tisdk-bootstrap-base-image.bb      | 20 +++++++++++++++++++
> >  1 file changed, 20 insertions(+)
> >  create mode 100644 meta-arago-distro/recipes-core/images/tisdk-bootstrap-base-image.bb
> > 
> > diff --git a/meta-arago-distro/recipes-core/images/tisdk-bootstrap-base-image.bb b/meta-arago-distro/recipes-core/images/tisdk-bootstrap-base-image.bb
> > new file mode 100644
> > index 000000000000..a18f39602286
> > --- /dev/null
> > +++ b/meta-arago-distro/recipes-core/images/tisdk-bootstrap-base-image.bb
> > @@ -0,0 +1,20 @@
> > +# Arago TI SDK bootstrap base image for initramfs
> > +# This will not get a completely usable image since kernel will be missing
> 
> Same comment as for earlier patch.
> 

Will update.

> 
> > +# intent for this image is to build as basic a system with enough basic
> > +# Board bringup utilities that can then be used for additions as needed.
> > +
> > +require arago-tiny-image.inc
> > +
> > +IMAGE_FSTYPES += "cpio cpio.xz"
> > +
> > +ARAGO_BOOTSTRAP_IMAGE_EXTRA_INSTALL ?= ""
> > +
> > +IMAGE_INSTALL += " \
> > +	packagegroup-arago-bootstrap \
> > +	${ARAGO_BOOTSTRAP_IMAGE_EXTRA_INSTALL} \
> > +"
> > +
> > +# We do not need kernel Image, dtbs and modules in the initrd
> > +deltask do_packagedata
> 
> Huh? Not sure what you are trying to do here, but it certainly isn't right...
> You are trying to partially disable generating of IPK packages for the image, 
> but images by default have no packages to produce.
> 

I assumed this was part of the magic stuff we do for tiny-image. I will
try dropping the same - both here and tiny image.


-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D

Re: [meta-arago] [dunfell/master PATCH 5/6] recipes-bsp: Create a k3-bootstrap-initrd package

[Nishanth Menon]

On 13:06-20220311, Denys Dmytriyenko wrote:
> On Tue, Mar 08, 2022 at 08:13:15PM -0600, Nishanth Menon via lists.yoctoproject.org wrote:
> > Lets create an k3-bootstrap-initrd package that installs the cpio
> > generated in the correct locations.
> 
> This has k3 in the name - is it really k3-specific? Do you want to set 
> COMPATIBLE_MACHINE accordingly?

It was a mistake. We should'nt tie this to k3 specific - there is
nothing k3 specific in this.


-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D

Re: [meta-arago] [dunfell/master PATCH 4/6] recipes-core: images: Add a basic bootstrap base image

[Nishanth Menon]

On 13:18-20220311, Denys Dmytriyenko wrote:
[...]

> > > +
> > > +# We do not need kernel Image, dtbs and modules in the initrd
> > > +deltask do_packagedata
> > 
> > Huh? Not sure what you are trying to do here, but it certainly isn't right...
> > You are trying to partially disable generating of IPK packages for the image, 
> > but images by default have no packages to produce.
> 
> I think you stepped on this workaround, that is no longer needed:
> https://git.yoctoproject.org/meta-arago/commit/?id=a75e468a372d4f5c2c32a179d4d2219899b5190c
> 


Aaah.. Thanks. will drop and retest.

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D

Re: [meta-arago] [dunfell/master PATCH 6/6] recipes-core: images: Introduce tisdk-bootstrap-image

[Nishanth Menon]

On 16:34-20220311, Denys Dmytriyenko wrote:
[...]

> > +
> > +IMAGE_INSTALL += " \
> > +	packagegroup-arago-bootstrap \
> > +	kernel \
> 
> This should probably be kernel-base. There's no package called just kernel, 
> but rather kernel-<version> which is kernel-base in disguise :)

Woops.. thanks a bunch. will fix.

-- 
Regards,
Nishanth Menon
Key (0xDDB5849D1736249D) / Fingerprint: F8A2 8693 54EB 8232 17A3  1A34 DDB5 849D 1736 249D