* [master/scarthgap][PATCH v1 1/3] am62: apply u-boot config fragment on ti-falcon
@ 2025-04-15 9:36 Anshul Dalal
2025-04-15 9:36 ` [master/scarthgap][PATCH v1 2/3] bsp: conf: machine: add ti-falcon.inc Anshul Dalal
2025-04-15 9:36 ` [master/scarthgap][PATCH v1 3/3] bsp: u-boot: deploy tispl_falcon.bin to rootfs Anshul Dalal
0 siblings, 2 replies; 14+ messages in thread
From: Anshul Dalal @ 2025-04-15 9:36 UTC (permalink / raw)
To: meta-ti; +Cc: Anshul Dalal, vigneshr
The am62x_r5_falcon.config fragment in u-boot enables falcon boot
support for R5 SPL, this patch enables the config fragment for am62a,
62p, 62x and 62lp evms based on ti-falcon distro override.
Signed-off-by: Anshul Dalal <anshuld@ti.com>
---
meta-ti-bsp/conf/machine/am62axx-evm-k3r5.conf | 2 ++
meta-ti-bsp/conf/machine/am62pxx-evm-k3r5.conf | 2 ++
meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf | 2 ++
meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf | 2 ++
4 files changed, 8 insertions(+)
diff --git a/meta-ti-bsp/conf/machine/am62axx-evm-k3r5.conf b/meta-ti-bsp/conf/machine/am62axx-evm-k3r5.conf
index 2af3317e..ab14feef 100644
--- a/meta-ti-bsp/conf/machine/am62axx-evm-k3r5.conf
+++ b/meta-ti-bsp/conf/machine/am62axx-evm-k3r5.conf
@@ -9,3 +9,5 @@ SYSFW_CONFIG = "evm"
SYSFW_SUFFIX = "hs-fs"
UBOOT_MACHINE = "am62ax_evm_r5_defconfig"
+
+UBOOT_CONFIG_FRAGMENTS:ti-falcon = "am62x_r5_falcon.config"
diff --git a/meta-ti-bsp/conf/machine/am62pxx-evm-k3r5.conf b/meta-ti-bsp/conf/machine/am62pxx-evm-k3r5.conf
index 36915381..849d6625 100644
--- a/meta-ti-bsp/conf/machine/am62pxx-evm-k3r5.conf
+++ b/meta-ti-bsp/conf/machine/am62pxx-evm-k3r5.conf
@@ -10,3 +10,5 @@ SYSFW_CONFIG = "evm"
SYSFW_SUFFIX = "hs-fs"
UBOOT_MACHINE = "am62px_evm_r5_defconfig"
+
+UBOOT_CONFIG_FRAGMENTS:ti-falcon = "am62x_r5_falcon.config"
diff --git a/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf b/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf
index 548369ca..a624a35d 100644
--- a/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf
+++ b/meta-ti-bsp/conf/machine/am62xx-evm-k3r5.conf
@@ -10,3 +10,5 @@ SYSFW_CONFIG = "evm"
SYSFW_SUFFIX = "hs-fs"
UBOOT_MACHINE = "am62x_evm_r5_defconfig"
+
+UBOOT_CONFIG_FRAGMENTS:ti-falcon = "am62x_r5_falcon.config"
diff --git a/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf b/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf
index 52b69a72..851935db 100644
--- a/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf
+++ b/meta-ti-bsp/conf/machine/am62xx-lp-evm-k3r5.conf
@@ -10,3 +10,5 @@ SYSFW_CONFIG = "evm"
SYSFW_SUFFIX = "hs-fs"
UBOOT_MACHINE = "am62x_lpsk_r5_defconfig"
+
+UBOOT_CONFIG_FRAGMENTS:ti-falcon = "am62x_r5_falcon.config"
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [master/scarthgap][PATCH v1 2/3] bsp: conf: machine: add ti-falcon.inc
2025-04-15 9:36 [master/scarthgap][PATCH v1 1/3] am62: apply u-boot config fragment on ti-falcon Anshul Dalal
@ 2025-04-15 9:36 ` Anshul Dalal
2025-04-16 0:13 ` [meta-ti] " Denys Dmytriyenko
2025-04-15 9:36 ` [master/scarthgap][PATCH v1 3/3] bsp: u-boot: deploy tispl_falcon.bin to rootfs Anshul Dalal
1 sibling, 1 reply; 14+ messages in thread
From: Anshul Dalal @ 2025-04-15 9:36 UTC (permalink / raw)
To: meta-ti; +Cc: Anshul Dalal, vigneshr
ti-falcon.inc configures the following things for enabling falcon boot:
1. Change fitImage class to kernel-fitimage-legacyhs
2. Update kernel and dtb address for falcon boot
3. Disable kernel compression
Signed-off-by: Anshul Dalal <anshuld@ti.com>
---
meta-ti-bsp/conf/machine/include/k3.inc | 4 ++++
meta-ti-bsp/conf/machine/include/ti-falcon.inc | 5 +++++
2 files changed, 9 insertions(+)
create mode 100644 meta-ti-bsp/conf/machine/include/ti-falcon.inc
diff --git a/meta-ti-bsp/conf/machine/include/k3.inc b/meta-ti-bsp/conf/machine/include/k3.inc
index ea6a5028..329ac037 100644
--- a/meta-ti-bsp/conf/machine/include/k3.inc
+++ b/meta-ti-bsp/conf/machine/include/k3.inc
@@ -56,3 +56,7 @@ TI_WKS_BOOTLOADER_APPEND ?= ""
do_image_wic[depends] += "virtual/bootloader:do_deploy"
SERIAL_CONSOLES = "115200;ttyS0 115200;ttyS2"
+
+FALCON_INCLUDE:ti-falcon = "conf/machine/include/ti-falcon.inc"
+
+require ${FALCON_INCLUDE}
diff --git a/meta-ti-bsp/conf/machine/include/ti-falcon.inc b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
new file mode 100644
index 00000000..b571c04d
--- /dev/null
+++ b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
@@ -0,0 +1,5 @@
+KERNEL_CLASSES:remove = "kernel-fitimage"
+KERNEL_CLASSES += "kernel-fitimage-legacyhs"
+UBOOT_DTB_LOADADDRESS = "0x82000000"
+UBOOT_LOADADDRESS = "0x82200000"
+FIT_KERNEL_COMP_ALG = "none"
^ permalink raw reply related [flat|nested] 14+ messages in thread
* [master/scarthgap][PATCH v1 3/3] bsp: u-boot: deploy tispl_falcon.bin to rootfs
2025-04-15 9:36 [master/scarthgap][PATCH v1 1/3] am62: apply u-boot config fragment on ti-falcon Anshul Dalal
2025-04-15 9:36 ` [master/scarthgap][PATCH v1 2/3] bsp: conf: machine: add ti-falcon.inc Anshul Dalal
@ 2025-04-15 9:36 ` Anshul Dalal
2025-04-16 0:11 ` [meta-ti] " Denys Dmytriyenko
1 sibling, 1 reply; 14+ messages in thread
From: Anshul Dalal @ 2025-04-15 9:36 UTC (permalink / raw)
To: meta-ti; +Cc: Anshul Dalal, vigneshr
Falcon boot requires tispl_falcon.bin to be on the same filesystem
partition as the fitImage.
This patch creates a new u-boot package called
'u-boot-ti-staging-falcon' that is included in IMAGE_INSTALL when
ti-falcon is enabled which allows us to deploy the required files to
rootfs.
Signed-off-by: Anshul Dalal <anshuld@ti.com>
---
meta-ti-bsp/conf/machine/include/ti-falcon.inc | 1 +
meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc | 11 +++++++++++
2 files changed, 12 insertions(+)
diff --git a/meta-ti-bsp/conf/machine/include/ti-falcon.inc b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
index b571c04d..9f93c683 100644
--- a/meta-ti-bsp/conf/machine/include/ti-falcon.inc
+++ b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
@@ -1,3 +1,4 @@
+IMAGE_INSTALL:append = " u-boot-ti-staging-falcon"
KERNEL_CLASSES:remove = "kernel-fitimage"
KERNEL_CLASSES += "kernel-fitimage-legacyhs"
UBOOT_DTB_LOADADDRESS = "0x82000000"
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 41d4abdd..76803ae3 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
@@ -14,6 +14,8 @@ require u-boot-mergeconfig.inc
FILESEXTRAPATHS:prepend := "${THISDIR}/u-boot:"
+FILES:${PN}-falcon = "boot/tispl_falcon.bin"
+
SUMMARY = "u-boot bootloader for TI devices"
LICENSE = "GPL-2.0-or-later"
@@ -373,4 +375,13 @@ do_deploy:append:am62pxx() {
install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz ${DEPLOYDIR}
}
+do_install:append:ti-falcon() {
+ install -m 0644 ${B}/tispl_falcon.bin ${D}/boot
+}
+
+PACKAGES:prepend:am62xx-evm = "${PN}-falcon "
+PACKAGES:prepend:am62axx-evm = "${PN}-falcon "
+PACKAGES:prepend:am62pxx-evm = "${PN}-falcon "
+PACKAGES:prepend:am62xx-lp-evm = "${PN}-falcon "
+
TOOLCHAIN = "gcc"
^ permalink raw reply related [flat|nested] 14+ messages in thread
* Re: [meta-ti] [master/scarthgap][PATCH v1 3/3] bsp: u-boot: deploy tispl_falcon.bin to rootfs
2025-04-15 9:36 ` [master/scarthgap][PATCH v1 3/3] bsp: u-boot: deploy tispl_falcon.bin to rootfs Anshul Dalal
@ 2025-04-16 0:11 ` Denys Dmytriyenko
2025-04-16 2:59 ` Anshul Dalal
0 siblings, 1 reply; 14+ messages in thread
From: Denys Dmytriyenko @ 2025-04-16 0:11 UTC (permalink / raw)
To: anshuld; +Cc: meta-ti, vigneshr
On Tue, Apr 15, 2025 at 03:06:44PM +0530, Anshul Dalal via lists.yoctoproject.org wrote:
> Falcon boot requires tispl_falcon.bin to be on the same filesystem
> partition as the fitImage.
>
> This patch creates a new u-boot package called
> 'u-boot-ti-staging-falcon' that is included in IMAGE_INSTALL when
> ti-falcon is enabled which allows us to deploy the required files to
> rootfs.
>
> Signed-off-by: Anshul Dalal <anshuld@ti.com>
> ---
> meta-ti-bsp/conf/machine/include/ti-falcon.inc | 1 +
> meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc | 11 +++++++++++
> 2 files changed, 12 insertions(+)
>
> diff --git a/meta-ti-bsp/conf/machine/include/ti-falcon.inc b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
> index b571c04d..9f93c683 100644
> --- a/meta-ti-bsp/conf/machine/include/ti-falcon.inc
> +++ b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
> @@ -1,3 +1,4 @@
> +IMAGE_INSTALL:append = " u-boot-ti-staging-falcon"
Why += cannot be used here?
> KERNEL_CLASSES:remove = "kernel-fitimage"
> KERNEL_CLASSES += "kernel-fitimage-legacyhs"
> UBOOT_DTB_LOADADDRESS = "0x82000000"
> 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 41d4abdd..76803ae3 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
> @@ -14,6 +14,8 @@ require u-boot-mergeconfig.inc
>
> FILESEXTRAPATHS:prepend := "${THISDIR}/u-boot:"
>
> +FILES:${PN}-falcon = "boot/tispl_falcon.bin"
> +
> SUMMARY = "u-boot bootloader for TI devices"
>
> LICENSE = "GPL-2.0-or-later"
> @@ -373,4 +375,13 @@ do_deploy:append:am62pxx() {
> install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz ${DEPLOYDIR}
> }
>
> +do_install:append:ti-falcon() {
> + install -m 0644 ${B}/tispl_falcon.bin ${D}/boot
> +}
> +
> +PACKAGES:prepend:am62xx-evm = "${PN}-falcon "
> +PACKAGES:prepend:am62axx-evm = "${PN}-falcon "
> +PACKAGES:prepend:am62pxx-evm = "${PN}-falcon "
> +PACKAGES:prepend:am62xx-lp-evm = "${PN}-falcon "
Shouldn't these be specific to ti-falcon only?
> +
> TOOLCHAIN = "gcc"
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [meta-ti] [master/scarthgap][PATCH v1 2/3] bsp: conf: machine: add ti-falcon.inc
2025-04-15 9:36 ` [master/scarthgap][PATCH v1 2/3] bsp: conf: machine: add ti-falcon.inc Anshul Dalal
@ 2025-04-16 0:13 ` Denys Dmytriyenko
2025-04-16 3:01 ` Anshul Dalal
0 siblings, 1 reply; 14+ messages in thread
From: Denys Dmytriyenko @ 2025-04-16 0:13 UTC (permalink / raw)
To: anshuld; +Cc: meta-ti, vigneshr
On Tue, Apr 15, 2025 at 03:06:43PM +0530, Anshul Dalal via lists.yoctoproject.org wrote:
> ti-falcon.inc configures the following things for enabling falcon boot:
>
> 1. Change fitImage class to kernel-fitimage-legacyhs
> 2. Update kernel and dtb address for falcon boot
> 3. Disable kernel compression
>
> Signed-off-by: Anshul Dalal <anshuld@ti.com>
> ---
> meta-ti-bsp/conf/machine/include/k3.inc | 4 ++++
> meta-ti-bsp/conf/machine/include/ti-falcon.inc | 5 +++++
> 2 files changed, 9 insertions(+)
> create mode 100644 meta-ti-bsp/conf/machine/include/ti-falcon.inc
>
> diff --git a/meta-ti-bsp/conf/machine/include/k3.inc b/meta-ti-bsp/conf/machine/include/k3.inc
> index ea6a5028..329ac037 100644
> --- a/meta-ti-bsp/conf/machine/include/k3.inc
> +++ b/meta-ti-bsp/conf/machine/include/k3.inc
> @@ -56,3 +56,7 @@ TI_WKS_BOOTLOADER_APPEND ?= ""
> do_image_wic[depends] += "virtual/bootloader:do_deploy"
>
> SERIAL_CONSOLES = "115200;ttyS0 115200;ttyS2"
> +
> +FALCON_INCLUDE:ti-falcon = "conf/machine/include/ti-falcon.inc"
Where and how ti-falcon override is being defined/set?
> +require ${FALCON_INCLUDE}
> diff --git a/meta-ti-bsp/conf/machine/include/ti-falcon.inc b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
> new file mode 100644
> index 00000000..b571c04d
> --- /dev/null
> +++ b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
> @@ -0,0 +1,5 @@
> +KERNEL_CLASSES:remove = "kernel-fitimage"
> +KERNEL_CLASSES += "kernel-fitimage-legacyhs"
> +UBOOT_DTB_LOADADDRESS = "0x82000000"
> +UBOOT_LOADADDRESS = "0x82200000"
> +FIT_KERNEL_COMP_ALG = "none"
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [meta-ti] [master/scarthgap][PATCH v1 3/3] bsp: u-boot: deploy tispl_falcon.bin to rootfs
2025-04-16 0:11 ` [meta-ti] " Denys Dmytriyenko
@ 2025-04-16 2:59 ` Anshul Dalal
2025-04-16 14:33 ` Denys Dmytriyenko
0 siblings, 1 reply; 14+ messages in thread
From: Anshul Dalal @ 2025-04-16 2:59 UTC (permalink / raw)
To: Denys Dmytriyenko; +Cc: meta-ti, vigneshr
On Wed Apr 16, 2025 at 5:41 AM IST, Denys Dmytriyenko wrote:
> On Tue, Apr 15, 2025 at 03:06:44PM +0530, Anshul Dalal via lists.yoctoproject.org wrote:
>> Falcon boot requires tispl_falcon.bin to be on the same filesystem
>> partition as the fitImage.
>>
>> This patch creates a new u-boot package called
>> 'u-boot-ti-staging-falcon' that is included in IMAGE_INSTALL when
>> ti-falcon is enabled which allows us to deploy the required files to
>> rootfs.
>>
>> Signed-off-by: Anshul Dalal <anshuld@ti.com>
>> ---
>> meta-ti-bsp/conf/machine/include/ti-falcon.inc | 1 +
>> meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc | 11 +++++++++++
>> 2 files changed, 12 insertions(+)
>>
>> diff --git a/meta-ti-bsp/conf/machine/include/ti-falcon.inc b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
>> index b571c04d..9f93c683 100644
>> --- a/meta-ti-bsp/conf/machine/include/ti-falcon.inc
>> +++ b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
>> @@ -1,3 +1,4 @@
>> +IMAGE_INSTALL:append = " u-boot-ti-staging-falcon"
>
> Why += cannot be used here?
>
>
>> KERNEL_CLASSES:remove = "kernel-fitimage"
>> KERNEL_CLASSES += "kernel-fitimage-legacyhs"
>> UBOOT_DTB_LOADADDRESS = "0x82000000"
>> 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 41d4abdd..76803ae3 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
>> @@ -14,6 +14,8 @@ require u-boot-mergeconfig.inc
>>
>> FILESEXTRAPATHS:prepend := "${THISDIR}/u-boot:"
>>
>> +FILES:${PN}-falcon = "boot/tispl_falcon.bin"
>> +
>> SUMMARY = "u-boot bootloader for TI devices"
>>
>> LICENSE = "GPL-2.0-or-later"
>> @@ -373,4 +375,13 @@ do_deploy:append:am62pxx() {
>> install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz ${DEPLOYDIR}
>> }
>>
>> +do_install:append:ti-falcon() {
>> + install -m 0644 ${B}/tispl_falcon.bin ${D}/boot
>> +}
>> +
>> +PACKAGES:prepend:am62xx-evm = "${PN}-falcon "
>> +PACKAGES:prepend:am62axx-evm = "${PN}-falcon "
>> +PACKAGES:prepend:am62pxx-evm = "${PN}-falcon "
>> +PACKAGES:prepend:am62xx-lp-evm = "${PN}-falcon "
>
> Shouldn't these be specific to ti-falcon only?
>
>
These are the only supported platforms at the moment, depending on
machine specific overrides allows us to throw an error at the recipe
parsing stage instead of having to wait till the do_install task fails
to copy the non-existent tispl_falcon.bin in case of unsupported
platforms.
As support for more platforms is added, the list can be expanded until
all platforms are supported when we can just have a ti-falcon override.
>> +
>> TOOLCHAIN = "gcc"
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [meta-ti] [master/scarthgap][PATCH v1 2/3] bsp: conf: machine: add ti-falcon.inc
2025-04-16 0:13 ` [meta-ti] " Denys Dmytriyenko
@ 2025-04-16 3:01 ` Anshul Dalal
0 siblings, 0 replies; 14+ messages in thread
From: Anshul Dalal @ 2025-04-16 3:01 UTC (permalink / raw)
To: Denys Dmytriyenko; +Cc: meta-ti, vigneshr
On Wed Apr 16, 2025 at 5:43 AM IST, Denys Dmytriyenko wrote:
> On Tue, Apr 15, 2025 at 03:06:43PM +0530, Anshul Dalal via lists.yoctoproject.org wrote:
>> ti-falcon.inc configures the following things for enabling falcon boot:
>>
>> 1. Change fitImage class to kernel-fitimage-legacyhs
>> 2. Update kernel and dtb address for falcon boot
>> 3. Disable kernel compression
>>
>> Signed-off-by: Anshul Dalal <anshuld@ti.com>
>> ---
>> meta-ti-bsp/conf/machine/include/k3.inc | 4 ++++
>> meta-ti-bsp/conf/machine/include/ti-falcon.inc | 5 +++++
>> 2 files changed, 9 insertions(+)
>> create mode 100644 meta-ti-bsp/conf/machine/include/ti-falcon.inc
>>
>> diff --git a/meta-ti-bsp/conf/machine/include/k3.inc b/meta-ti-bsp/conf/machine/include/k3.inc
>> index ea6a5028..329ac037 100644
>> --- a/meta-ti-bsp/conf/machine/include/k3.inc
>> +++ b/meta-ti-bsp/conf/machine/include/k3.inc
>> @@ -56,3 +56,7 @@ TI_WKS_BOOTLOADER_APPEND ?= ""
>> do_image_wic[depends] += "virtual/bootloader:do_deploy"
>>
>> SERIAL_CONSOLES = "115200;ttyS0 115200;ttyS2"
>> +
>> +FALCON_INCLUDE:ti-falcon = "conf/machine/include/ti-falcon.inc"
>
> Where and how ti-falcon override is being defined/set?
>
It's a DISTROOVERRIDE set in local.conf as follows:
DISTROOVERRIDES:append = ":ti-falcon"
>
>> +require ${FALCON_INCLUDE}
>> diff --git a/meta-ti-bsp/conf/machine/include/ti-falcon.inc b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
>> new file mode 100644
>> index 00000000..b571c04d
>> --- /dev/null
>> +++ b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
>> @@ -0,0 +1,5 @@
>> +KERNEL_CLASSES:remove = "kernel-fitimage"
>> +KERNEL_CLASSES += "kernel-fitimage-legacyhs"
>> +UBOOT_DTB_LOADADDRESS = "0x82000000"
>> +UBOOT_LOADADDRESS = "0x82200000"
>> +FIT_KERNEL_COMP_ALG = "none"
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [meta-ti] [master/scarthgap][PATCH v1 3/3] bsp: u-boot: deploy tispl_falcon.bin to rootfs
2025-04-16 2:59 ` Anshul Dalal
@ 2025-04-16 14:33 ` Denys Dmytriyenko
2025-04-17 9:48 ` Anshul Dalal
0 siblings, 1 reply; 14+ messages in thread
From: Denys Dmytriyenko @ 2025-04-16 14:33 UTC (permalink / raw)
To: Anshul Dalal; +Cc: meta-ti, vigneshr
On Wed, Apr 16, 2025 at 08:29:13AM +0530, Anshul Dalal wrote:
> On Wed Apr 16, 2025 at 5:41 AM IST, Denys Dmytriyenko wrote:
> > On Tue, Apr 15, 2025 at 03:06:44PM +0530, Anshul Dalal via lists.yoctoproject.org wrote:
> >> Falcon boot requires tispl_falcon.bin to be on the same filesystem
> >> partition as the fitImage.
> >>
> >> This patch creates a new u-boot package called
> >> 'u-boot-ti-staging-falcon' that is included in IMAGE_INSTALL when
> >> ti-falcon is enabled which allows us to deploy the required files to
> >> rootfs.
> >>
> >> Signed-off-by: Anshul Dalal <anshuld@ti.com>
> >> ---
> >> meta-ti-bsp/conf/machine/include/ti-falcon.inc | 1 +
> >> meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc | 11 +++++++++++
> >> 2 files changed, 12 insertions(+)
> >>
> >> diff --git a/meta-ti-bsp/conf/machine/include/ti-falcon.inc b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
> >> index b571c04d..9f93c683 100644
> >> --- a/meta-ti-bsp/conf/machine/include/ti-falcon.inc
> >> +++ b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
> >> @@ -1,3 +1,4 @@
> >> +IMAGE_INSTALL:append = " u-boot-ti-staging-falcon"
> >
> > Why += cannot be used here?
> >
> >
> >> KERNEL_CLASSES:remove = "kernel-fitimage"
> >> KERNEL_CLASSES += "kernel-fitimage-legacyhs"
> >> UBOOT_DTB_LOADADDRESS = "0x82000000"
> >> 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 41d4abdd..76803ae3 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
> >> @@ -14,6 +14,8 @@ require u-boot-mergeconfig.inc
> >>
> >> FILESEXTRAPATHS:prepend := "${THISDIR}/u-boot:"
> >>
> >> +FILES:${PN}-falcon = "boot/tispl_falcon.bin"
> >> +
> >> SUMMARY = "u-boot bootloader for TI devices"
> >>
> >> LICENSE = "GPL-2.0-or-later"
> >> @@ -373,4 +375,13 @@ do_deploy:append:am62pxx() {
> >> install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz ${DEPLOYDIR}
> >> }
> >>
> >> +do_install:append:ti-falcon() {
> >> + install -m 0644 ${B}/tispl_falcon.bin ${D}/boot
> >> +}
> >> +
> >> +PACKAGES:prepend:am62xx-evm = "${PN}-falcon "
> >> +PACKAGES:prepend:am62axx-evm = "${PN}-falcon "
> >> +PACKAGES:prepend:am62pxx-evm = "${PN}-falcon "
> >> +PACKAGES:prepend:am62xx-lp-evm = "${PN}-falcon "
> >
> > Shouldn't these be specific to ti-falcon only?
> >
> >
>
> These are the only supported platforms at the moment, depending on
> machine specific overrides allows us to throw an error at the recipe
> parsing stage instead of having to wait till the do_install task fails
> to copy the non-existent tispl_falcon.bin in case of unsupported
> platforms.
What happens when you build for am62xx-evm and ti-falcon is not enabled?
> As support for more platforms is added, the list can be expanded until
> all platforms are supported when we can just have a ti-falcon override.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [meta-ti] [master/scarthgap][PATCH v1 3/3] bsp: u-boot: deploy tispl_falcon.bin to rootfs
2025-04-16 14:33 ` Denys Dmytriyenko
@ 2025-04-17 9:48 ` Anshul Dalal
2025-04-17 18:37 ` Denys Dmytriyenko
0 siblings, 1 reply; 14+ messages in thread
From: Anshul Dalal @ 2025-04-17 9:48 UTC (permalink / raw)
To: Denys Dmytriyenko; +Cc: meta-ti, vigneshr
On Wed Apr 16, 2025 at 8:03 PM IST, Denys Dmytriyenko wrote:
> On Wed, Apr 16, 2025 at 08:29:13AM +0530, Anshul Dalal wrote:
>> On Wed Apr 16, 2025 at 5:41 AM IST, Denys Dmytriyenko wrote:
>> > On Tue, Apr 15, 2025 at 03:06:44PM +0530, Anshul Dalal via lists.yoctoproject.org wrote:
>> >> Falcon boot requires tispl_falcon.bin to be on the same filesystem
>> >> partition as the fitImage.
>> >>
>> >> This patch creates a new u-boot package called
>> >> 'u-boot-ti-staging-falcon' that is included in IMAGE_INSTALL when
>> >> ti-falcon is enabled which allows us to deploy the required files to
>> >> rootfs.
>> >>
>> >> Signed-off-by: Anshul Dalal <anshuld@ti.com>
>> >> ---
>> >> meta-ti-bsp/conf/machine/include/ti-falcon.inc | 1 +
>> >> meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc | 11 +++++++++++
>> >> 2 files changed, 12 insertions(+)
>> >>
>> >> diff --git a/meta-ti-bsp/conf/machine/include/ti-falcon.inc b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
>> >> index b571c04d..9f93c683 100644
>> >> --- a/meta-ti-bsp/conf/machine/include/ti-falcon.inc
>> >> +++ b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
>> >> @@ -1,3 +1,4 @@
>> >> +IMAGE_INSTALL:append = " u-boot-ti-staging-falcon"
>> >
>> > Why += cannot be used here?
>> >
>> >
>> >> KERNEL_CLASSES:remove = "kernel-fitimage"
>> >> KERNEL_CLASSES += "kernel-fitimage-legacyhs"
>> >> UBOOT_DTB_LOADADDRESS = "0x82000000"
>> >> 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 41d4abdd..76803ae3 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
>> >> @@ -14,6 +14,8 @@ require u-boot-mergeconfig.inc
>> >>
>> >> FILESEXTRAPATHS:prepend := "${THISDIR}/u-boot:"
>> >>
>> >> +FILES:${PN}-falcon = "boot/tispl_falcon.bin"
>> >> +
>> >> SUMMARY = "u-boot bootloader for TI devices"
>> >>
>> >> LICENSE = "GPL-2.0-or-later"
>> >> @@ -373,4 +375,13 @@ do_deploy:append:am62pxx() {
>> >> install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz ${DEPLOYDIR}
>> >> }
>> >>
>> >> +do_install:append:ti-falcon() {
>> >> + install -m 0644 ${B}/tispl_falcon.bin ${D}/boot
>> >> +}
>> >> +
>> >> +PACKAGES:prepend:am62xx-evm = "${PN}-falcon "
>> >> +PACKAGES:prepend:am62axx-evm = "${PN}-falcon "
>> >> +PACKAGES:prepend:am62pxx-evm = "${PN}-falcon "
>> >> +PACKAGES:prepend:am62xx-lp-evm = "${PN}-falcon "
>> >
>> > Shouldn't these be specific to ti-falcon only?
>> >
>> >
>>
>> These are the only supported platforms at the moment, depending on
>> machine specific overrides allows us to throw an error at the recipe
>> parsing stage instead of having to wait till the do_install task fails
>> to copy the non-existent tispl_falcon.bin in case of unsupported
>> platforms.
>
> What happens when you build for am62xx-evm and ti-falcon is not enabled?
>
The ${PN}-falcon package is added to the list of packages but the
do_package task is never run for u-boot since it's not in
IMAGES_INSTALL.
So, the net effect is no change to the u-boot's working directory during
the build process. It's only with ti-falcon that the respective
packages-split directory is created with u-boot-ti-staging-falcon
sub-directory.
>
>> As support for more platforms is added, the list can be expanded until
>> all platforms are supported when we can just have a ti-falcon override.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [meta-ti] [master/scarthgap][PATCH v1 3/3] bsp: u-boot: deploy tispl_falcon.bin to rootfs
2025-04-17 9:48 ` Anshul Dalal
@ 2025-04-17 18:37 ` Denys Dmytriyenko
2025-04-21 9:01 ` Anshul Dalal
0 siblings, 1 reply; 14+ messages in thread
From: Denys Dmytriyenko @ 2025-04-17 18:37 UTC (permalink / raw)
To: Anshul Dalal; +Cc: meta-ti, vigneshr
On Thu, Apr 17, 2025 at 03:18:03PM +0530, Anshul Dalal wrote:
> On Wed Apr 16, 2025 at 8:03 PM IST, Denys Dmytriyenko wrote:
> > On Wed, Apr 16, 2025 at 08:29:13AM +0530, Anshul Dalal wrote:
> >> On Wed Apr 16, 2025 at 5:41 AM IST, Denys Dmytriyenko wrote:
> >> > On Tue, Apr 15, 2025 at 03:06:44PM +0530, Anshul Dalal via lists.yoctoproject.org wrote:
> >> >> Falcon boot requires tispl_falcon.bin to be on the same filesystem
> >> >> partition as the fitImage.
> >> >>
> >> >> This patch creates a new u-boot package called
> >> >> 'u-boot-ti-staging-falcon' that is included in IMAGE_INSTALL when
> >> >> ti-falcon is enabled which allows us to deploy the required files to
> >> >> rootfs.
> >> >>
> >> >> Signed-off-by: Anshul Dalal <anshuld@ti.com>
> >> >> ---
> >> >> meta-ti-bsp/conf/machine/include/ti-falcon.inc | 1 +
> >> >> meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc | 11 +++++++++++
> >> >> 2 files changed, 12 insertions(+)
> >> >>
> >> >> diff --git a/meta-ti-bsp/conf/machine/include/ti-falcon.inc b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
> >> >> index b571c04d..9f93c683 100644
> >> >> --- a/meta-ti-bsp/conf/machine/include/ti-falcon.inc
> >> >> +++ b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
> >> >> @@ -1,3 +1,4 @@
> >> >> +IMAGE_INSTALL:append = " u-boot-ti-staging-falcon"
> >> >
> >> > Why += cannot be used here?
> >> >
> >> >
> >> >> KERNEL_CLASSES:remove = "kernel-fitimage"
> >> >> KERNEL_CLASSES += "kernel-fitimage-legacyhs"
> >> >> UBOOT_DTB_LOADADDRESS = "0x82000000"
> >> >> 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 41d4abdd..76803ae3 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
> >> >> @@ -14,6 +14,8 @@ require u-boot-mergeconfig.inc
> >> >>
> >> >> FILESEXTRAPATHS:prepend := "${THISDIR}/u-boot:"
> >> >>
> >> >> +FILES:${PN}-falcon = "boot/tispl_falcon.bin"
> >> >> +
> >> >> SUMMARY = "u-boot bootloader for TI devices"
> >> >>
> >> >> LICENSE = "GPL-2.0-or-later"
> >> >> @@ -373,4 +375,13 @@ do_deploy:append:am62pxx() {
> >> >> install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz ${DEPLOYDIR}
> >> >> }
> >> >>
> >> >> +do_install:append:ti-falcon() {
> >> >> + install -m 0644 ${B}/tispl_falcon.bin ${D}/boot
> >> >> +}
> >> >> +
> >> >> +PACKAGES:prepend:am62xx-evm = "${PN}-falcon "
> >> >> +PACKAGES:prepend:am62axx-evm = "${PN}-falcon "
> >> >> +PACKAGES:prepend:am62pxx-evm = "${PN}-falcon "
> >> >> +PACKAGES:prepend:am62xx-lp-evm = "${PN}-falcon "
> >> >
> >> > Shouldn't these be specific to ti-falcon only?
> >> >
> >> >
> >>
> >> These are the only supported platforms at the moment, depending on
> >> machine specific overrides allows us to throw an error at the recipe
> >> parsing stage instead of having to wait till the do_install task fails
> >> to copy the non-existent tispl_falcon.bin in case of unsupported
> >> platforms.
> >
> > What happens when you build for am62xx-evm and ti-falcon is not enabled?
> >
>
> The ${PN}-falcon package is added to the list of packages but the
> do_package task is never run for u-boot since it's not in
> IMAGES_INSTALL.
All our platforms have u-boot in EXTRA_IMAGEDEPENDS, which means u-boot gets
built and packaged all the time. Moreover, it's common to "bitbake u-boot"
> So, the net effect is no change to the u-boot's working directory during
> the build process. It's only with ti-falcon that the respective
> packages-split directory is created with u-boot-ti-staging-falcon
> sub-directory.
>
> >
> >> As support for more platforms is added, the list can be expanded until
> >> all platforms are supported when we can just have a ti-falcon override.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [meta-ti] [master/scarthgap][PATCH v1 3/3] bsp: u-boot: deploy tispl_falcon.bin to rootfs
2025-04-17 18:37 ` Denys Dmytriyenko
@ 2025-04-21 9:01 ` Anshul Dalal
2025-04-22 19:53 ` Denys Dmytriyenko
0 siblings, 1 reply; 14+ messages in thread
From: Anshul Dalal @ 2025-04-21 9:01 UTC (permalink / raw)
To: Denys Dmytriyenko; +Cc: meta-ti, vigneshr
On Fri Apr 18, 2025 at 12:07 AM IST, Denys Dmytriyenko wrote:
> On Thu, Apr 17, 2025 at 03:18:03PM +0530, Anshul Dalal wrote:
>> On Wed Apr 16, 2025 at 8:03 PM IST, Denys Dmytriyenko wrote:
>> > On Wed, Apr 16, 2025 at 08:29:13AM +0530, Anshul Dalal wrote:
>> >> On Wed Apr 16, 2025 at 5:41 AM IST, Denys Dmytriyenko wrote:
>> >> > On Tue, Apr 15, 2025 at 03:06:44PM +0530, Anshul Dalal via lists.yoctoproject.org wrote:
>> >> >> Falcon boot requires tispl_falcon.bin to be on the same filesystem
>> >> >> partition as the fitImage.
>> >> >>
>> >> >> This patch creates a new u-boot package called
>> >> >> 'u-boot-ti-staging-falcon' that is included in IMAGE_INSTALL when
>> >> >> ti-falcon is enabled which allows us to deploy the required files to
>> >> >> rootfs.
>> >> >>
>> >> >> Signed-off-by: Anshul Dalal <anshuld@ti.com>
>> >> >> ---
>> >> >> meta-ti-bsp/conf/machine/include/ti-falcon.inc | 1 +
>> >> >> meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc | 11 +++++++++++
>> >> >> 2 files changed, 12 insertions(+)
>> >> >>
>> >> >> diff --git a/meta-ti-bsp/conf/machine/include/ti-falcon.inc b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
>> >> >> index b571c04d..9f93c683 100644
>> >> >> --- a/meta-ti-bsp/conf/machine/include/ti-falcon.inc
>> >> >> +++ b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
>> >> >> @@ -1,3 +1,4 @@
>> >> >> +IMAGE_INSTALL:append = " u-boot-ti-staging-falcon"
>> >> >
>> >> > Why += cannot be used here?
>> >> >
>> >> >
>> >> >> KERNEL_CLASSES:remove = "kernel-fitimage"
>> >> >> KERNEL_CLASSES += "kernel-fitimage-legacyhs"
>> >> >> UBOOT_DTB_LOADADDRESS = "0x82000000"
>> >> >> 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 41d4abdd..76803ae3 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
>> >> >> @@ -14,6 +14,8 @@ require u-boot-mergeconfig.inc
>> >> >>
>> >> >> FILESEXTRAPATHS:prepend := "${THISDIR}/u-boot:"
>> >> >>
>> >> >> +FILES:${PN}-falcon = "boot/tispl_falcon.bin"
>> >> >> +
>> >> >> SUMMARY = "u-boot bootloader for TI devices"
>> >> >>
>> >> >> LICENSE = "GPL-2.0-or-later"
>> >> >> @@ -373,4 +375,13 @@ do_deploy:append:am62pxx() {
>> >> >> install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz ${DEPLOYDIR}
>> >> >> }
>> >> >>
>> >> >> +do_install:append:ti-falcon() {
>> >> >> + install -m 0644 ${B}/tispl_falcon.bin ${D}/boot
>> >> >> +}
>> >> >> +
>> >> >> +PACKAGES:prepend:am62xx-evm = "${PN}-falcon "
>> >> >> +PACKAGES:prepend:am62axx-evm = "${PN}-falcon "
>> >> >> +PACKAGES:prepend:am62pxx-evm = "${PN}-falcon "
>> >> >> +PACKAGES:prepend:am62xx-lp-evm = "${PN}-falcon "
>> >> >
>> >> > Shouldn't these be specific to ti-falcon only?
>> >> >
>> >> >
>> >>
>> >> These are the only supported platforms at the moment, depending on
>> >> machine specific overrides allows us to throw an error at the recipe
>> >> parsing stage instead of having to wait till the do_install task fails
>> >> to copy the non-existent tispl_falcon.bin in case of unsupported
>> >> platforms.
>> >
>> > What happens when you build for am62xx-evm and ti-falcon is not enabled?
>> >
>>
>> The ${PN}-falcon package is added to the list of packages but the
>> do_package task is never run for u-boot since it's not in
>> IMAGES_INSTALL.
>
> All our platforms have u-boot in EXTRA_IMAGEDEPENDS, which means u-boot gets
> built and packaged all the time. Moreover, it's common to "bitbake u-boot"
>
Recipes in EXTRA_IMAGEDEPENDS don't provide any packages to rootfs[1],
hence why u-boot has no `package` directory when ti-falcon is not set or
even before this patch set.
Therefore having the falcon u-boot package in PACKAGES does not change
the build time behaviour until it's added to IMAGES_INSTALL (which is
only done when ti-falcon is set).
In my testing, there is no difference between the work directory for
u-boot with ti-falcon unset vs the work directory before this patch
set[2].
[1]:
https://docs.yoctoproject.org/ref-manual/variables.html#term-EXTRA_IMAGEDEPENDS
[2]:
U-boot's work directories:
# before patch set:
build image recipe-sysroot-native temp
deploy-source-date-epoch license-destdir source-date-epoch
deploy-u-boot-ti-staging pseudo srcipk-staging
git recipe-sysroot sysroot-destdir
# without ti-falcon:
build image recipe-sysroot-native temp
deploy-source-date-epoch license-destdir source-date-epoch
deploy-u-boot-ti-staging pseudo srcipk-staging
git recipe-sysroot sysroot-destdir
# with ti-falcon:
build package recipe-sysroot-native
deploy-ipks packages-split source-date-epoch
deploy-source-date-epoch pkgdata spdx
deploy-u-boot-ti-staging pkgdata-pdata-input srcipk-staging
git pkgdata-sysroot sysroot-destdir
image pseudo temp
license-destdir recipe-sysroot
---
Regards,
Anshul
>
>> So, the net effect is no change to the u-boot's working directory during
>> the build process. It's only with ti-falcon that the respective
>> packages-split directory is created with u-boot-ti-staging-falcon
>> sub-directory.
>>
>> >
>> >> As support for more platforms is added, the list can be expanded until
>> >> all platforms are supported when we can just have a ti-falcon override.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [meta-ti] [master/scarthgap][PATCH v1 3/3] bsp: u-boot: deploy tispl_falcon.bin to rootfs
2025-04-21 9:01 ` Anshul Dalal
@ 2025-04-22 19:53 ` Denys Dmytriyenko
2025-04-23 8:12 ` Anshul Dalal
0 siblings, 1 reply; 14+ messages in thread
From: Denys Dmytriyenko @ 2025-04-22 19:53 UTC (permalink / raw)
To: Anshul Dalal; +Cc: meta-ti, vigneshr
On Mon, Apr 21, 2025 at 02:31:47PM +0530, Anshul Dalal wrote:
> On Fri Apr 18, 2025 at 12:07 AM IST, Denys Dmytriyenko wrote:
> > On Thu, Apr 17, 2025 at 03:18:03PM +0530, Anshul Dalal wrote:
> >> On Wed Apr 16, 2025 at 8:03 PM IST, Denys Dmytriyenko wrote:
> >> > On Wed, Apr 16, 2025 at 08:29:13AM +0530, Anshul Dalal wrote:
> >> >> On Wed Apr 16, 2025 at 5:41 AM IST, Denys Dmytriyenko wrote:
> >> >> > On Tue, Apr 15, 2025 at 03:06:44PM +0530, Anshul Dalal via lists.yoctoproject.org wrote:
> >> >> >> Falcon boot requires tispl_falcon.bin to be on the same filesystem
> >> >> >> partition as the fitImage.
> >> >> >>
> >> >> >> This patch creates a new u-boot package called
> >> >> >> 'u-boot-ti-staging-falcon' that is included in IMAGE_INSTALL when
> >> >> >> ti-falcon is enabled which allows us to deploy the required files to
> >> >> >> rootfs.
> >> >> >>
> >> >> >> Signed-off-by: Anshul Dalal <anshuld@ti.com>
> >> >> >> ---
> >> >> >> meta-ti-bsp/conf/machine/include/ti-falcon.inc | 1 +
> >> >> >> meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc | 11 +++++++++++
> >> >> >> 2 files changed, 12 insertions(+)
> >> >> >>
> >> >> >> diff --git a/meta-ti-bsp/conf/machine/include/ti-falcon.inc b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
> >> >> >> index b571c04d..9f93c683 100644
> >> >> >> --- a/meta-ti-bsp/conf/machine/include/ti-falcon.inc
> >> >> >> +++ b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
> >> >> >> @@ -1,3 +1,4 @@
> >> >> >> +IMAGE_INSTALL:append = " u-boot-ti-staging-falcon"
> >> >> >
> >> >> > Why += cannot be used here?
> >> >> >
> >> >> >
> >> >> >> KERNEL_CLASSES:remove = "kernel-fitimage"
> >> >> >> KERNEL_CLASSES += "kernel-fitimage-legacyhs"
> >> >> >> UBOOT_DTB_LOADADDRESS = "0x82000000"
> >> >> >> 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 41d4abdd..76803ae3 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
> >> >> >> @@ -14,6 +14,8 @@ require u-boot-mergeconfig.inc
> >> >> >>
> >> >> >> FILESEXTRAPATHS:prepend := "${THISDIR}/u-boot:"
> >> >> >>
> >> >> >> +FILES:${PN}-falcon = "boot/tispl_falcon.bin"
> >> >> >> +
> >> >> >> SUMMARY = "u-boot bootloader for TI devices"
> >> >> >>
> >> >> >> LICENSE = "GPL-2.0-or-later"
> >> >> >> @@ -373,4 +375,13 @@ do_deploy:append:am62pxx() {
> >> >> >> install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz ${DEPLOYDIR}
> >> >> >> }
> >> >> >>
> >> >> >> +do_install:append:ti-falcon() {
> >> >> >> + install -m 0644 ${B}/tispl_falcon.bin ${D}/boot
> >> >> >> +}
> >> >> >> +
> >> >> >> +PACKAGES:prepend:am62xx-evm = "${PN}-falcon "
> >> >> >> +PACKAGES:prepend:am62axx-evm = "${PN}-falcon "
> >> >> >> +PACKAGES:prepend:am62pxx-evm = "${PN}-falcon "
> >> >> >> +PACKAGES:prepend:am62xx-lp-evm = "${PN}-falcon "
> >> >> >
> >> >> > Shouldn't these be specific to ti-falcon only?
> >> >> >
> >> >> >
> >> >>
> >> >> These are the only supported platforms at the moment, depending on
> >> >> machine specific overrides allows us to throw an error at the recipe
> >> >> parsing stage instead of having to wait till the do_install task fails
> >> >> to copy the non-existent tispl_falcon.bin in case of unsupported
> >> >> platforms.
> >> >
> >> > What happens when you build for am62xx-evm and ti-falcon is not enabled?
> >> >
> >>
> >> The ${PN}-falcon package is added to the list of packages but the
> >> do_package task is never run for u-boot since it's not in
> >> IMAGES_INSTALL.
> >
> > All our platforms have u-boot in EXTRA_IMAGEDEPENDS, which means u-boot gets
> > built and packaged all the time. Moreover, it's common to "bitbake u-boot"
> >
>
> Recipes in EXTRA_IMAGEDEPENDS don't provide any packages to rootfs[1],
> hence why u-boot has no `package` directory when ti-falcon is not set or
> even before this patch set.
>
> Therefore having the falcon u-boot package in PACKAGES does not change
> the build time behaviour until it's added to IMAGES_INSTALL (which is
> only done when ti-falcon is set).
>
> In my testing, there is no difference between the work directory for
> u-boot with ti-falcon unset vs the work directory before this patch
> set[2].
>
> [1]:
> https://docs.yoctoproject.org/ref-manual/variables.html#term-EXTRA_IMAGEDEPENDS
>
> [2]:
> U-boot's work directories:
>
> # before patch set:
> build image recipe-sysroot-native temp
> deploy-source-date-epoch license-destdir source-date-epoch
> deploy-u-boot-ti-staging pseudo srcipk-staging
> git recipe-sysroot sysroot-destdir
>
> # without ti-falcon:
> build image recipe-sysroot-native temp
> deploy-source-date-epoch license-destdir source-date-epoch
> deploy-u-boot-ti-staging pseudo srcipk-staging
> git recipe-sysroot sysroot-destdir
>
> # with ti-falcon:
> build package recipe-sysroot-native
> deploy-ipks packages-split source-date-epoch
> deploy-source-date-epoch pkgdata spdx
> deploy-u-boot-ti-staging pkgdata-pdata-input srcipk-staging
> git pkgdata-sysroot sysroot-destdir
> image pseudo temp
> license-destdir recipe-sysroot
You should not rely on this behavior expecting that U-boot will be built, but
not packaged!
In scarthgap, this may be true:
$ MACHINE=am62xx-evm bitbake tisdk-base-image -g
$ grep '\-> "u-boot-ti-staging.do_package' task-depends.dot
$
But in walnascar/master, SPDX 3.x functionality for SW Bill of Materials is
enabled by default and operates on packages, hence do_package() task will be
called for U-boot:
$ MACHINE=am62xx-evm bitbake tisdk-base-image -g
$ grep u-boot-ti-staging.do_package task-depends.dot
"u-boot-ti-staging.do_create_spdx" -> "u-boot-ti-staging.do_package"
"u-boot-ti-staging.do_create_spdx" -> "u-boot-ti-staging.do_packagedata"
"u-boot-ti-staging.do_packagedata" -> "u-boot-ti-staging.do_package"
$
And indeed, the work directory contains both package and packages-split
directories:
$ ls -1 arago-tmp-default/work/am62xx_evm-oe-linux/u-boot-ti-staging/2025.01+git/
build
deploy-source-date-epoch
deploy-u-boot-ti-staging
git
image
license-destdir
package
packages-split
pkgdata
pkgdata-pdata-input
pkgdata-sysroot
pseudo
recipe-sysroot
recipe-sysroot-native
source-date-epoch
sources-unpack
spdx
sysroot-destdir
temp
$ ls -1 arago-tmp-default/work/am62xx_evm-oe-linux/u-boot-ti-staging/2025.01+git/packages-split/
u-boot-ti-staging
u-boot-ti-staging-dbg
u-boot-ti-staging-dev
u-boot-ti-staging-doc
u-boot-ti-staging-env
u-boot-ti-staging-extlinux
u-boot-ti-staging-locale
u-boot-ti-staging-src
u-boot-ti-staging-staticdev
> >> So, the net effect is no change to the u-boot's working directory during
> >> the build process. It's only with ti-falcon that the respective
> >> packages-split directory is created with u-boot-ti-staging-falcon
> >> sub-directory.
> >>
> >> >
> >> >> As support for more platforms is added, the list can be expanded until
> >> >> all platforms are supported when we can just have a ti-falcon override.
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [meta-ti] [master/scarthgap][PATCH v1 3/3] bsp: u-boot: deploy tispl_falcon.bin to rootfs
2025-04-22 19:53 ` Denys Dmytriyenko
@ 2025-04-23 8:12 ` Anshul Dalal
2025-04-23 16:17 ` Denys Dmytriyenko
0 siblings, 1 reply; 14+ messages in thread
From: Anshul Dalal @ 2025-04-23 8:12 UTC (permalink / raw)
To: Denys Dmytriyenko; +Cc: meta-ti, vigneshr
On Wed Apr 23, 2025 at 1:23 AM IST, Denys Dmytriyenko wrote:
> On Mon, Apr 21, 2025 at 02:31:47PM +0530, Anshul Dalal wrote:
>> On Fri Apr 18, 2025 at 12:07 AM IST, Denys Dmytriyenko wrote:
>> > On Thu, Apr 17, 2025 at 03:18:03PM +0530, Anshul Dalal wrote:
>> >> On Wed Apr 16, 2025 at 8:03 PM IST, Denys Dmytriyenko wrote:
>> >> > On Wed, Apr 16, 2025 at 08:29:13AM +0530, Anshul Dalal wrote:
>> >> >> On Wed Apr 16, 2025 at 5:41 AM IST, Denys Dmytriyenko wrote:
>> >> >> > On Tue, Apr 15, 2025 at 03:06:44PM +0530, Anshul Dalal via lists.yoctoproject.org wrote:
>> >> >> >> Falcon boot requires tispl_falcon.bin to be on the same filesystem
>> >> >> >> partition as the fitImage.
>> >> >> >>
>> >> >> >> This patch creates a new u-boot package called
>> >> >> >> 'u-boot-ti-staging-falcon' that is included in IMAGE_INSTALL when
>> >> >> >> ti-falcon is enabled which allows us to deploy the required files to
>> >> >> >> rootfs.
>> >> >> >>
>> >> >> >> Signed-off-by: Anshul Dalal <anshuld@ti.com>
>> >> >> >> ---
>> >> >> >> meta-ti-bsp/conf/machine/include/ti-falcon.inc | 1 +
>> >> >> >> meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc | 11 +++++++++++
>> >> >> >> 2 files changed, 12 insertions(+)
>> >> >> >>
>> >> >> >> diff --git a/meta-ti-bsp/conf/machine/include/ti-falcon.inc b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
>> >> >> >> index b571c04d..9f93c683 100644
>> >> >> >> --- a/meta-ti-bsp/conf/machine/include/ti-falcon.inc
>> >> >> >> +++ b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
>> >> >> >> @@ -1,3 +1,4 @@
>> >> >> >> +IMAGE_INSTALL:append = " u-boot-ti-staging-falcon"
>> >> >> >
>> >> >> > Why += cannot be used here?
>> >> >> >
>> >> >> >
>> >> >> >> KERNEL_CLASSES:remove = "kernel-fitimage"
>> >> >> >> KERNEL_CLASSES += "kernel-fitimage-legacyhs"
>> >> >> >> UBOOT_DTB_LOADADDRESS = "0x82000000"
>> >> >> >> 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 41d4abdd..76803ae3 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
>> >> >> >> @@ -14,6 +14,8 @@ require u-boot-mergeconfig.inc
>> >> >> >>
>> >> >> >> FILESEXTRAPATHS:prepend := "${THISDIR}/u-boot:"
>> >> >> >>
>> >> >> >> +FILES:${PN}-falcon = "boot/tispl_falcon.bin"
>> >> >> >> +
>> >> >> >> SUMMARY = "u-boot bootloader for TI devices"
>> >> >> >>
>> >> >> >> LICENSE = "GPL-2.0-or-later"
>> >> >> >> @@ -373,4 +375,13 @@ do_deploy:append:am62pxx() {
>> >> >> >> install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz ${DEPLOYDIR}
>> >> >> >> }
>> >> >> >>
>> >> >> >> +do_install:append:ti-falcon() {
>> >> >> >> + install -m 0644 ${B}/tispl_falcon.bin ${D}/boot
>> >> >> >> +}
>> >> >> >> +
>> >> >> >> +PACKAGES:prepend:am62xx-evm = "${PN}-falcon "
>> >> >> >> +PACKAGES:prepend:am62axx-evm = "${PN}-falcon "
>> >> >> >> +PACKAGES:prepend:am62pxx-evm = "${PN}-falcon "
>> >> >> >> +PACKAGES:prepend:am62xx-lp-evm = "${PN}-falcon "
>> >> >> >
>> >> >> > Shouldn't these be specific to ti-falcon only?
>> >> >> >
>> >> >> >
>> >> >>
>> >> >> These are the only supported platforms at the moment, depending on
>> >> >> machine specific overrides allows us to throw an error at the recipe
>> >> >> parsing stage instead of having to wait till the do_install task fails
>> >> >> to copy the non-existent tispl_falcon.bin in case of unsupported
>> >> >> platforms.
>> >> >
>> >> > What happens when you build for am62xx-evm and ti-falcon is not enabled?
>> >> >
>> >>
>> >> The ${PN}-falcon package is added to the list of packages but the
>> >> do_package task is never run for u-boot since it's not in
>> >> IMAGES_INSTALL.
>> >
>> > All our platforms have u-boot in EXTRA_IMAGEDEPENDS, which means u-boot gets
>> > built and packaged all the time. Moreover, it's common to "bitbake u-boot"
>> >
>>
>> Recipes in EXTRA_IMAGEDEPENDS don't provide any packages to rootfs[1],
>> hence why u-boot has no `package` directory when ti-falcon is not set or
>> even before this patch set.
>>
>> Therefore having the falcon u-boot package in PACKAGES does not change
>> the build time behaviour until it's added to IMAGES_INSTALL (which is
>> only done when ti-falcon is set).
>>
>> In my testing, there is no difference between the work directory for
>> u-boot with ti-falcon unset vs the work directory before this patch
>> set[2].
>>
[snip]
>
> You should not rely on this behavior expecting that U-boot will be built, but
> not packaged!
>
> In scarthgap, this may be true:
>
> $ MACHINE=am62xx-evm bitbake tisdk-base-image -g
> $ grep '\-> "u-boot-ti-staging.do_package' task-depends.dot
> $
>
> But in walnascar/master, SPDX 3.x functionality for SW Bill of Materials is
> enabled by default and operates on packages, hence do_package() task will be
> called for U-boot:
>
Well in that case, we can use nested overrides using ti-falcon as well
as the machine specific config. Like below:
PACKAGES:prepend:ti-append:am62xx-evm = "${PN}-falcon "
PACKAGES:prepend:ti-append:am62axx-evm = "${PN}-falcon "
PACKAGES:prepend:ti-append:am62pxx-evm = "${PN}-falcon "
PACKAGES:prepend:ti-append:am62xx-lp-evm = "${PN}-falcon "
[snip]
^ permalink raw reply [flat|nested] 14+ messages in thread
* Re: [meta-ti] [master/scarthgap][PATCH v1 3/3] bsp: u-boot: deploy tispl_falcon.bin to rootfs
2025-04-23 8:12 ` Anshul Dalal
@ 2025-04-23 16:17 ` Denys Dmytriyenko
0 siblings, 0 replies; 14+ messages in thread
From: Denys Dmytriyenko @ 2025-04-23 16:17 UTC (permalink / raw)
To: Anshul Dalal; +Cc: meta-ti, vigneshr
On Wed, Apr 23, 2025 at 01:42:53PM +0530, Anshul Dalal wrote:
> On Wed Apr 23, 2025 at 1:23 AM IST, Denys Dmytriyenko wrote:
> > On Mon, Apr 21, 2025 at 02:31:47PM +0530, Anshul Dalal wrote:
> >> On Fri Apr 18, 2025 at 12:07 AM IST, Denys Dmytriyenko wrote:
> >> > On Thu, Apr 17, 2025 at 03:18:03PM +0530, Anshul Dalal wrote:
> >> >> On Wed Apr 16, 2025 at 8:03 PM IST, Denys Dmytriyenko wrote:
> >> >> > On Wed, Apr 16, 2025 at 08:29:13AM +0530, Anshul Dalal wrote:
> >> >> >> On Wed Apr 16, 2025 at 5:41 AM IST, Denys Dmytriyenko wrote:
> >> >> >> > On Tue, Apr 15, 2025 at 03:06:44PM +0530, Anshul Dalal via lists.yoctoproject.org wrote:
> >> >> >> >> Falcon boot requires tispl_falcon.bin to be on the same filesystem
> >> >> >> >> partition as the fitImage.
> >> >> >> >>
> >> >> >> >> This patch creates a new u-boot package called
> >> >> >> >> 'u-boot-ti-staging-falcon' that is included in IMAGE_INSTALL when
> >> >> >> >> ti-falcon is enabled which allows us to deploy the required files to
> >> >> >> >> rootfs.
> >> >> >> >>
> >> >> >> >> Signed-off-by: Anshul Dalal <anshuld@ti.com>
> >> >> >> >> ---
> >> >> >> >> meta-ti-bsp/conf/machine/include/ti-falcon.inc | 1 +
> >> >> >> >> meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti.inc | 11 +++++++++++
> >> >> >> >> 2 files changed, 12 insertions(+)
> >> >> >> >>
> >> >> >> >> diff --git a/meta-ti-bsp/conf/machine/include/ti-falcon.inc b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
> >> >> >> >> index b571c04d..9f93c683 100644
> >> >> >> >> --- a/meta-ti-bsp/conf/machine/include/ti-falcon.inc
> >> >> >> >> +++ b/meta-ti-bsp/conf/machine/include/ti-falcon.inc
> >> >> >> >> @@ -1,3 +1,4 @@
> >> >> >> >> +IMAGE_INSTALL:append = " u-boot-ti-staging-falcon"
> >> >> >> >
> >> >> >> > Why += cannot be used here?
> >> >> >> >
> >> >> >> >
> >> >> >> >> KERNEL_CLASSES:remove = "kernel-fitimage"
> >> >> >> >> KERNEL_CLASSES += "kernel-fitimage-legacyhs"
> >> >> >> >> UBOOT_DTB_LOADADDRESS = "0x82000000"
> >> >> >> >> 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 41d4abdd..76803ae3 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
> >> >> >> >> @@ -14,6 +14,8 @@ require u-boot-mergeconfig.inc
> >> >> >> >>
> >> >> >> >> FILESEXTRAPATHS:prepend := "${THISDIR}/u-boot:"
> >> >> >> >>
> >> >> >> >> +FILES:${PN}-falcon = "boot/tispl_falcon.bin"
> >> >> >> >> +
> >> >> >> >> SUMMARY = "u-boot bootloader for TI devices"
> >> >> >> >>
> >> >> >> >> LICENSE = "GPL-2.0-or-later"
> >> >> >> >> @@ -373,4 +375,13 @@ do_deploy:append:am62pxx() {
> >> >> >> >> install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz ${DEPLOYDIR}
> >> >> >> >> }
> >> >> >> >>
> >> >> >> >> +do_install:append:ti-falcon() {
> >> >> >> >> + install -m 0644 ${B}/tispl_falcon.bin ${D}/boot
> >> >> >> >> +}
> >> >> >> >> +
> >> >> >> >> +PACKAGES:prepend:am62xx-evm = "${PN}-falcon "
> >> >> >> >> +PACKAGES:prepend:am62axx-evm = "${PN}-falcon "
> >> >> >> >> +PACKAGES:prepend:am62pxx-evm = "${PN}-falcon "
> >> >> >> >> +PACKAGES:prepend:am62xx-lp-evm = "${PN}-falcon "
> >> >> >> >
> >> >> >> > Shouldn't these be specific to ti-falcon only?
> >> >> >> >
> >> >> >> >
> >> >> >>
> >> >> >> These are the only supported platforms at the moment, depending on
> >> >> >> machine specific overrides allows us to throw an error at the recipe
> >> >> >> parsing stage instead of having to wait till the do_install task fails
> >> >> >> to copy the non-existent tispl_falcon.bin in case of unsupported
> >> >> >> platforms.
> >> >> >
> >> >> > What happens when you build for am62xx-evm and ti-falcon is not enabled?
> >> >> >
> >> >>
> >> >> The ${PN}-falcon package is added to the list of packages but the
> >> >> do_package task is never run for u-boot since it's not in
> >> >> IMAGES_INSTALL.
> >> >
> >> > All our platforms have u-boot in EXTRA_IMAGEDEPENDS, which means u-boot gets
> >> > built and packaged all the time. Moreover, it's common to "bitbake u-boot"
> >> >
> >>
> >> Recipes in EXTRA_IMAGEDEPENDS don't provide any packages to rootfs[1],
> >> hence why u-boot has no `package` directory when ti-falcon is not set or
> >> even before this patch set.
> >>
> >> Therefore having the falcon u-boot package in PACKAGES does not change
> >> the build time behaviour until it's added to IMAGES_INSTALL (which is
> >> only done when ti-falcon is set).
> >>
> >> In my testing, there is no difference between the work directory for
> >> u-boot with ti-falcon unset vs the work directory before this patch
> >> set[2].
> >>
> [snip]
> >
> > You should not rely on this behavior expecting that U-boot will be built, but
> > not packaged!
> >
> > In scarthgap, this may be true:
> >
> > $ MACHINE=am62xx-evm bitbake tisdk-base-image -g
> > $ grep '\-> "u-boot-ti-staging.do_package' task-depends.dot
> > $
> >
> > But in walnascar/master, SPDX 3.x functionality for SW Bill of Materials is
> > enabled by default and operates on packages, hence do_package() task will be
> > called for U-boot:
> >
>
> Well in that case, we can use nested overrides using ti-falcon as well
> as the machine specific config. Like below:
>
> PACKAGES:prepend:ti-append:am62xx-evm = "${PN}-falcon "
> PACKAGES:prepend:ti-append:am62axx-evm = "${PN}-falcon "
> PACKAGES:prepend:ti-append:am62pxx-evm = "${PN}-falcon "
> PACKAGES:prepend:ti-append:am62xx-lp-evm = "${PN}-falcon "
First of all, there's a typo there in the overrides ^^^
Second, consider making it more obvious and deterministic by splitting
overrides:
FALCON_PKG = ""
FALCON_PKG:ti-falcon = "${PN}-falcon"
PACKAGES:prepend:am62xx-evm = "${FALCON_PKG} "
...
^ permalink raw reply [flat|nested] 14+ messages in thread
end of thread, other threads:[~2025-04-23 16:17 UTC | newest]
Thread overview: 14+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-04-15 9:36 [master/scarthgap][PATCH v1 1/3] am62: apply u-boot config fragment on ti-falcon Anshul Dalal
2025-04-15 9:36 ` [master/scarthgap][PATCH v1 2/3] bsp: conf: machine: add ti-falcon.inc Anshul Dalal
2025-04-16 0:13 ` [meta-ti] " Denys Dmytriyenko
2025-04-16 3:01 ` Anshul Dalal
2025-04-15 9:36 ` [master/scarthgap][PATCH v1 3/3] bsp: u-boot: deploy tispl_falcon.bin to rootfs Anshul Dalal
2025-04-16 0:11 ` [meta-ti] " Denys Dmytriyenko
2025-04-16 2:59 ` Anshul Dalal
2025-04-16 14:33 ` Denys Dmytriyenko
2025-04-17 9:48 ` Anshul Dalal
2025-04-17 18:37 ` Denys Dmytriyenko
2025-04-21 9:01 ` Anshul Dalal
2025-04-22 19:53 ` Denys Dmytriyenko
2025-04-23 8:12 ` Anshul Dalal
2025-04-23 16:17 ` 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.