[master/scarthgap][PATCH] omap-a15.inc: don't include ipu1 firmware in the boot partition

[master/scarthgap][PATCH] omap-a15.inc: don't include ipu1 firmware in the boot partition

[Denys Dmytriyenko]

From: Denys Dmytriyenko <denys@konsulko.com>

This reverts commit c58af0b172660ef1248a3454af06251d872770b1.

This adds an undesired dependency on meta-ti-extras - more details:
https://lists.yoctoproject.org/g/meta-ti/topic/110003435

Cc: Aniket Limaye <a-limaye@ti.com>
Signed-off-by: Denys Dmytriyenko <denys@konsulko.com>
---
 meta-ti-bsp/conf/machine/include/omap-a15.inc | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/meta-ti-bsp/conf/machine/include/omap-a15.inc b/meta-ti-bsp/conf/machine/include/omap-a15.inc
index a258decd..be086936 100644
--- a/meta-ti-bsp/conf/machine/include/omap-a15.inc
+++ b/meta-ti-bsp/conf/machine/include/omap-a15.inc
@@ -48,6 +48,5 @@ MACHINE_FEATURES = "apm usbgadget usbhost vfat ext2 screen alsa gpu mmip dsp gc3
 
 IMAGE_FSTYPES += "tar.xz wic.xz wic.bmap"
 WKS_FILE ?= "sdimage-2part.wks"
-IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES} ipc/dra7-ipu1-fw.xem4;dra7-ipu1-fw.xem4"
+IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES}"
 do_image_wic[depends] += "virtual/bootloader:do_deploy"
-do_image_wic[depends] += "ti-ipc-rtos:do_deploy"
-- 
2.25.1

Re: [master/scarthgap][PATCH] omap-a15.inc: don't include ipu1 firmware in the boot partition

[Limaye, Aniket]

On 3/15/2025 7:32 AM, Denys Dmytriyenko wrote:
> From: Denys Dmytriyenko <denys@konsulko.com>
>
> This reverts commit c58af0b172660ef1248a3454af06251d872770b1.
>
> This adds an undesired dependency on meta-ti-extras - more details:
> https://lists.yoctoproject.org/g/meta-ti/topic/110003435
>
> Cc: Aniket Limaye <a-limaye@ti.com>
> Signed-off-by: Denys Dmytriyenko <denys@konsulko.com
> ---

The fw binary is actually a requirement for AM57x boards to boot now. 
U-Boot expects
the firmware to be present in the boot partition, and fails if not found.
(Beleswar can correct me if I'm mistaken).

If we don't want to break this board, we need to maybe migrate the 
ti-ipc-rtos recipe from
meta-ti-extras to meta-ti-bsp, since it is anyway a requirement for 
AM57X u-boot recipe.
Denys, I also see that you mentioned that the ti-ipc-rtos recipe 
currently resides in extras
due to the legacy recipes built from sources using old tools like XDC, 
SYS/BIOS and CGT...
which cannot be easily moved to meta-ti-bsp.

A temporary fix might be to somehow add these inside of 
meta-ti-extras/conf/layer.conf?

However, I Ack the revert in case it is needed for now:

Acked-by: Aniket Limaye <a-limaye@ti.com>


>   meta-ti-bsp/conf/machine/include/omap-a15.inc | 3 +--
>   1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/meta-ti-bsp/conf/machine/include/omap-a15.inc b/meta-ti-bsp/conf/machine/include/omap-a15.inc
> index a258decd..be086936 100644
> --- a/meta-ti-bsp/conf/machine/include/omap-a15.inc
> +++ b/meta-ti-bsp/conf/machine/include/omap-a15.inc
> @@ -48,6 +48,5 @@ MACHINE_FEATURES = "apm usbgadget usbhost vfat ext2 screen alsa gpu mmip dsp gc3
>   
>   IMAGE_FSTYPES += "tar.xz wic.xz wic.bmap"
>   WKS_FILE ?= "sdimage-2part.wks"
> -IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES} ipc/dra7-ipu1-fw.xem4;dra7-ipu1-fw.xem4"
> +IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES}"
>   do_image_wic[depends] += "virtual/bootloader:do_deploy"
> -do_image_wic[depends] += "ti-ipc-rtos:do_deploy"

Re: [meta-ti] [master/scarthgap][PATCH] omap-a15.inc: don't include ipu1 firmware in the boot partition

[Denys Dmytriyenko]

On Sat, Mar 15, 2025 at 10:29:18AM +0530, Aniket Limaye via lists.yoctoproject.org wrote:
> 
> On 3/15/2025 7:32 AM, Denys Dmytriyenko wrote:
> >From: Denys Dmytriyenko <denys@konsulko.com>
> >
> >This reverts commit c58af0b172660ef1248a3454af06251d872770b1.
> >
> >This adds an undesired dependency on meta-ti-extras - more details:
> >https://lists.yoctoproject.org/g/meta-ti/topic/110003435
> >
> >Cc: Aniket Limaye <a-limaye@ti.com>
> >Signed-off-by: Denys Dmytriyenko <denys@konsulko.com
> >---
> 
> The fw binary is actually a requirement for AM57x boards to boot
> now. U-Boot expects
> the firmware to be present in the boot partition, and fails if not found.
> (Beleswar can correct me if I'm mistaken).

Huh, interesting - do you know the version where this started?

If it is indeed a hard requirement to boot the platform now, another option 
is to get a prebuilt binary into ti-linux-firmware and pull it from there, 
instead of building it from sources using legacy tools...


> If we don't want to break this board, we need to maybe migrate the
> ti-ipc-rtos recipe from
> meta-ti-extras to meta-ti-bsp, since it is anyway a requirement for
> AM57X u-boot recipe.
> Denys, I also see that you mentioned that the ti-ipc-rtos recipe
> currently resides in extras
> due to the legacy recipes built from sources using old tools like
> XDC, SYS/BIOS and CGT...
> which cannot be easily moved to meta-ti-bsp.
> 
> A temporary fix might be to somehow add these inside of
> meta-ti-extras/conf/layer.conf?
> 
> However, I Ack the revert in case it is needed for now:
> 
> Acked-by: Aniket Limaye <a-limaye@ti.com>
> 
> 
> >  meta-ti-bsp/conf/machine/include/omap-a15.inc | 3 +--
> >  1 file changed, 1 insertion(+), 2 deletions(-)
> >
> >diff --git a/meta-ti-bsp/conf/machine/include/omap-a15.inc b/meta-ti-bsp/conf/machine/include/omap-a15.inc
> >index a258decd..be086936 100644
> >--- a/meta-ti-bsp/conf/machine/include/omap-a15.inc
> >+++ b/meta-ti-bsp/conf/machine/include/omap-a15.inc
> >@@ -48,6 +48,5 @@ MACHINE_FEATURES = "apm usbgadget usbhost vfat ext2 screen alsa gpu mmip dsp gc3
> >  IMAGE_FSTYPES += "tar.xz wic.xz wic.bmap"
> >  WKS_FILE ?= "sdimage-2part.wks"
> >-IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES} ipc/dra7-ipu1-fw.xem4;dra7-ipu1-fw.xem4"
> >+IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES}"
> >  do_image_wic[depends] += "virtual/bootloader:do_deploy"
> >-do_image_wic[depends] += "ti-ipc-rtos:do_deploy"

Re: [meta-ti] [master/scarthgap][PATCH] omap-a15.inc: don't include ipu1 firmware in the boot partition

[Ryan Eatmon]

On 3/18/2025 11:18 AM, Denys Dmytriyenko wrote:
> On Sat, Mar 15, 2025 at 10:29:18AM +0530, Aniket Limaye via lists.yoctoproject.org wrote:
>>
>> On 3/15/2025 7:32 AM, Denys Dmytriyenko wrote:
>>> From: Denys Dmytriyenko <denys@konsulko.com>
>>>
>>> This reverts commit c58af0b172660ef1248a3454af06251d872770b1.
>>>
>>> This adds an undesired dependency on meta-ti-extras - more details:
>>> https://lists.yoctoproject.org/g/meta-ti/topic/110003435
>>>
>>> Cc: Aniket Limaye <a-limaye@ti.com>
>>> Signed-off-by: Denys Dmytriyenko <denys@konsulko.com
>>> ---
>>
>> The fw binary is actually a requirement for AM57x boards to boot
>> now. U-Boot expects
>> the firmware to be present in the boot partition, and fails if not found.
>> (Beleswar can correct me if I'm mistaken).
> 
> Huh, interesting - do you know the version where this started?
> 
> If it is indeed a hard requirement to boot the platform now, another option
> is to get a prebuilt binary into ti-linux-firmware and pull it from there,
> instead of building it from sources using legacy tools...

That's what I was thinking...


> 
>> If we don't want to break this board, we need to maybe migrate the
>> ti-ipc-rtos recipe from
>> meta-ti-extras to meta-ti-bsp, since it is anyway a requirement for
>> AM57X u-boot recipe.
>> Denys, I also see that you mentioned that the ti-ipc-rtos recipe
>> currently resides in extras
>> due to the legacy recipes built from sources using old tools like
>> XDC, SYS/BIOS and CGT...
>> which cannot be easily moved to meta-ti-bsp.
>>
>> A temporary fix might be to somehow add these inside of
>> meta-ti-extras/conf/layer.conf?
>>
>> However, I Ack the revert in case it is needed for now:
>>
>> Acked-by: Aniket Limaye <a-limaye@ti.com>
>>
>>
>>>   meta-ti-bsp/conf/machine/include/omap-a15.inc | 3 +--
>>>   1 file changed, 1 insertion(+), 2 deletions(-)
>>>
>>> diff --git a/meta-ti-bsp/conf/machine/include/omap-a15.inc b/meta-ti-bsp/conf/machine/include/omap-a15.inc
>>> index a258decd..be086936 100644
>>> --- a/meta-ti-bsp/conf/machine/include/omap-a15.inc
>>> +++ b/meta-ti-bsp/conf/machine/include/omap-a15.inc
>>> @@ -48,6 +48,5 @@ MACHINE_FEATURES = "apm usbgadget usbhost vfat ext2 screen alsa gpu mmip dsp gc3
>>>   IMAGE_FSTYPES += "tar.xz wic.xz wic.bmap"
>>>   WKS_FILE ?= "sdimage-2part.wks"
>>> -IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES} ipc/dra7-ipu1-fw.xem4;dra7-ipu1-fw.xem4"
>>> +IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES}"
>>>   do_image_wic[depends] += "virtual/bootloader:do_deploy"
>>> -do_image_wic[depends] += "ti-ipc-rtos:do_deploy"
>>>
>>>
>>> -=-=-=-=-=-=-=-=-=-=-=-
>>> Links: You receive all messages sent to this group.
>>> View/Reply Online (#18400): https://lists.yoctoproject.org/g/meta-ti/message/18400
>>> Mute This Topic: https://lists.yoctoproject.org/mt/111711244/6551054
>>> Group Owner: meta-ti+owner@lists.yoctoproject.org
>>> Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/unsub [reatmon@ti.com]
>>> -=-=-=-=-=-=-=-=-=-=-=-
>>>

-- 
Ryan Eatmon                reatmon@ti.com
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS

Re: [meta-ti] [master/scarthgap][PATCH] omap-a15.inc: don't include ipu1 firmware in the boot partition

[Beleswar Prasad Padhi]

On 18/03/25 21:48, Denys Dmytriyenko wrote:
> On Sat, Mar 15, 2025 at 10:29:18AM +0530, Aniket Limaye via lists.yoctoproject.org wrote:
>> On 3/15/2025 7:32 AM, Denys Dmytriyenko wrote:
>>> From: Denys Dmytriyenko <denys@konsulko.com>
>>>
>>> This reverts commit c58af0b172660ef1248a3454af06251d872770b1.
>>>
>>> This adds an undesired dependency on meta-ti-extras - more details:
>>> https://lists.yoctoproject.org/g/meta-ti/topic/110003435
>>>
>>> Cc: Aniket Limaye <a-limaye@ti.com>
>>> Signed-off-by: Denys Dmytriyenko <denys@konsulko.com
>>> ---
>> The fw binary is actually a requirement for AM57x boards to boot
>> now. U-Boot expects
>> the firmware to be present in the boot partition, and fails if not found.
>> (Beleswar can correct me if I'm mistaken).
> Huh, interesting - do you know the version where this started?
>
> If it is indeed a hard requirement to boot the platform now, another option
> is to get a prebuilt binary into ti-linux-firmware and pull it from there,
> instead of building it from sources using legacy tools...


This is great too. However, meta-ti should still have the deploy rule to 
pick the pre-built firmware and put it into the boot partition.

Thanks,
Beleswar

>
>
>> If we don't want to break this board, we need to maybe migrate the
>> ti-ipc-rtos recipe from
>> meta-ti-extras to meta-ti-bsp, since it is anyway a requirement for
>> AM57X u-boot recipe.
>> Denys, I also see that you mentioned that the ti-ipc-rtos recipe
>> currently resides in extras
>> due to the legacy recipes built from sources using old tools like
>> XDC, SYS/BIOS and CGT...
>> which cannot be easily moved to meta-ti-bsp.
>>
>> A temporary fix might be to somehow add these inside of
>> meta-ti-extras/conf/layer.conf?
>>
>> However, I Ack the revert in case it is needed for now:
>>
>> Acked-by: Aniket Limaye <a-limaye@ti.com>
>>
>>
>>>   meta-ti-bsp/conf/machine/include/omap-a15.inc | 3 +--
>>>   1 file changed, 1 insertion(+), 2 deletions(-)
>>>
>>> diff --git a/meta-ti-bsp/conf/machine/include/omap-a15.inc b/meta-ti-bsp/conf/machine/include/omap-a15.inc
>>> index a258decd..be086936 100644
>>> --- a/meta-ti-bsp/conf/machine/include/omap-a15.inc
>>> +++ b/meta-ti-bsp/conf/machine/include/omap-a15.inc
>>> @@ -48,6 +48,5 @@ MACHINE_FEATURES = "apm usbgadget usbhost vfat ext2 screen alsa gpu mmip dsp gc3
>>>   IMAGE_FSTYPES += "tar.xz wic.xz wic.bmap"
>>>   WKS_FILE ?= "sdimage-2part.wks"
>>> -IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES} ipc/dra7-ipu1-fw.xem4;dra7-ipu1-fw.xem4"
>>> +IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES}"
>>>   do_image_wic[depends] += "virtual/bootloader:do_deploy"
>>> -do_image_wic[depends] += "ti-ipc-rtos:do_deploy"

Re: [meta-ti] [master/scarthgap][PATCH] omap-a15.inc: don't include ipu1 firmware in the boot partition

[Denys Dmytriyenko]

Circling back to this...

On Thu, Mar 20, 2025 at 02:29:14PM +0530, Beleswar Padhi via lists.yoctoproject.org wrote:
> 
> On 18/03/25 21:48, Denys Dmytriyenko wrote:
> >On Sat, Mar 15, 2025 at 10:29:18AM +0530, Aniket Limaye via lists.yoctoproject.org wrote:
> >>On 3/15/2025 7:32 AM, Denys Dmytriyenko wrote:
> >>>From: Denys Dmytriyenko <denys@konsulko.com>
> >>>
> >>>This reverts commit c58af0b172660ef1248a3454af06251d872770b1.
> >>>
> >>>This adds an undesired dependency on meta-ti-extras - more details:
> >>>https://lists.yoctoproject.org/g/meta-ti/topic/110003435
> >>>
> >>>Cc: Aniket Limaye <a-limaye@ti.com>
> >>>Signed-off-by: Denys Dmytriyenko <denys@konsulko.com
> >>>---
> >>The fw binary is actually a requirement for AM57x boards to boot
> >>now. U-Boot expects
> >>the firmware to be present in the boot partition, and fails if not found.
> >>(Beleswar can correct me if I'm mistaken).
> >Huh, interesting - do you know the version where this started?
> 
> The firmware is required for Early boot of remote processors (IPU in
> this case) by U-Boot. Without the firmware, there won't be a system
> boot failure, only rproc boot will fail.
> 
> 
> >If it is indeed a hard requirement to boot the platform now, another option
> >is to get a prebuilt binary into ti-linux-firmware and pull it from there,
> >instead of building it from sources using legacy tools...
> 
> This is great too. However, meta-ti should still have the deploy
> rule to pick the pre-built firmware and put it into the boot
> partition.
> 
> Thanks,
> Beleswar

Thanks, Beleswar.

Ryan, All,

How should this be fixed?


> >>If we don't want to break this board, we need to maybe migrate the
> >>ti-ipc-rtos recipe from
> >>meta-ti-extras to meta-ti-bsp, since it is anyway a requirement for
> >>AM57X u-boot recipe.
> >>Denys, I also see that you mentioned that the ti-ipc-rtos recipe
> >>currently resides in extras
> >>due to the legacy recipes built from sources using old tools like
> >>XDC, SYS/BIOS and CGT...
> >>which cannot be easily moved to meta-ti-bsp.
> >>
> >>A temporary fix might be to somehow add these inside of
> >>meta-ti-extras/conf/layer.conf?
> >>
> >>However, I Ack the revert in case it is needed for now:
> >>
> >>Acked-by: Aniket Limaye <a-limaye@ti.com>
> >>
> >>
> >>>  meta-ti-bsp/conf/machine/include/omap-a15.inc | 3 +--
> >>>  1 file changed, 1 insertion(+), 2 deletions(-)
> >>>
> >>>diff --git a/meta-ti-bsp/conf/machine/include/omap-a15.inc b/meta-ti-bsp/conf/machine/include/omap-a15.inc
> >>>index a258decd..be086936 100644
> >>>--- a/meta-ti-bsp/conf/machine/include/omap-a15.inc
> >>>+++ b/meta-ti-bsp/conf/machine/include/omap-a15.inc
> >>>@@ -48,6 +48,5 @@ MACHINE_FEATURES = "apm usbgadget usbhost vfat ext2 screen alsa gpu mmip dsp gc3
> >>>  IMAGE_FSTYPES += "tar.xz wic.xz wic.bmap"
> >>>  WKS_FILE ?= "sdimage-2part.wks"
> >>>-IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES} ipc/dra7-ipu1-fw.xem4;dra7-ipu1-fw.xem4"
> >>>+IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES}"
> >>>  do_image_wic[depends] += "virtual/bootloader:do_deploy"
> >>>-do_image_wic[depends] += "ti-ipc-rtos:do_deploy"

Re: [meta-ti] [master/scarthgap][PATCH] omap-a15.inc: don't include ipu1 firmware in the boot partition

[Denys Dmytriyenko]

On Fri, Apr 11, 2025 at 05:03:12PM -0400, Denys Dmytriyenko wrote:
> Circling back to this...
> 
> On Thu, Mar 20, 2025 at 02:29:14PM +0530, Beleswar Padhi via lists.yoctoproject.org wrote:
> > 
> > On 18/03/25 21:48, Denys Dmytriyenko wrote:
> > >On Sat, Mar 15, 2025 at 10:29:18AM +0530, Aniket Limaye via lists.yoctoproject.org wrote:
> > >>On 3/15/2025 7:32 AM, Denys Dmytriyenko wrote:
> > >>>From: Denys Dmytriyenko <denys@konsulko.com>
> > >>>
> > >>>This reverts commit c58af0b172660ef1248a3454af06251d872770b1.
> > >>>
> > >>>This adds an undesired dependency on meta-ti-extras - more details:
> > >>>https://lists.yoctoproject.org/g/meta-ti/topic/110003435
> > >>>
> > >>>Cc: Aniket Limaye <a-limaye@ti.com>
> > >>>Signed-off-by: Denys Dmytriyenko <denys@konsulko.com
> > >>>---
> > >>The fw binary is actually a requirement for AM57x boards to boot
> > >>now. U-Boot expects
> > >>the firmware to be present in the boot partition, and fails if not found.
> > >>(Beleswar can correct me if I'm mistaken).
> > >Huh, interesting - do you know the version where this started?
> > 
> > The firmware is required for Early boot of remote processors (IPU in
> > this case) by U-Boot. Without the firmware, there won't be a system
> > boot failure, only rproc boot will fail.
> > 
> > 
> > >If it is indeed a hard requirement to boot the platform now, another option
> > >is to get a prebuilt binary into ti-linux-firmware and pull it from there,
> > >instead of building it from sources using legacy tools...
> > 
> > This is great too. However, meta-ti should still have the deploy
> > rule to pick the pre-built firmware and put it into the boot
> > partition.
> > 
> > Thanks,
> > Beleswar
> 
> Thanks, Beleswar.
> 
> Ryan, All,
> 
> How should this be fixed?

Ping.

Since nobody's working on adding a prebuilt binary to ti-linux-firmware, 
then this should be reverted as is for now, IMHO.


> > >>If we don't want to break this board, we need to maybe migrate the
> > >>ti-ipc-rtos recipe from
> > >>meta-ti-extras to meta-ti-bsp, since it is anyway a requirement for
> > >>AM57X u-boot recipe.
> > >>Denys, I also see that you mentioned that the ti-ipc-rtos recipe
> > >>currently resides in extras
> > >>due to the legacy recipes built from sources using old tools like
> > >>XDC, SYS/BIOS and CGT...
> > >>which cannot be easily moved to meta-ti-bsp.
> > >>
> > >>A temporary fix might be to somehow add these inside of
> > >>meta-ti-extras/conf/layer.conf?
> > >>
> > >>However, I Ack the revert in case it is needed for now:
> > >>
> > >>Acked-by: Aniket Limaye <a-limaye@ti.com>
> > >>
> > >>
> > >>>  meta-ti-bsp/conf/machine/include/omap-a15.inc | 3 +--
> > >>>  1 file changed, 1 insertion(+), 2 deletions(-)
> > >>>
> > >>>diff --git a/meta-ti-bsp/conf/machine/include/omap-a15.inc b/meta-ti-bsp/conf/machine/include/omap-a15.inc
> > >>>index a258decd..be086936 100644
> > >>>--- a/meta-ti-bsp/conf/machine/include/omap-a15.inc
> > >>>+++ b/meta-ti-bsp/conf/machine/include/omap-a15.inc
> > >>>@@ -48,6 +48,5 @@ MACHINE_FEATURES = "apm usbgadget usbhost vfat ext2 screen alsa gpu mmip dsp gc3
> > >>>  IMAGE_FSTYPES += "tar.xz wic.xz wic.bmap"
> > >>>  WKS_FILE ?= "sdimage-2part.wks"
> > >>>-IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES} ipc/dra7-ipu1-fw.xem4;dra7-ipu1-fw.xem4"
> > >>>+IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES}"
> > >>>  do_image_wic[depends] += "virtual/bootloader:do_deploy"
> > >>>-do_image_wic[depends] += "ti-ipc-rtos:do_deploy"

Re: [meta-ti] [master/scarthgap][PATCH] omap-a15.inc: don't include ipu1 firmware in the boot partition

[Kumar, Udit]

On 4/23/2025 7:23 AM, Denys Dmytriyenko wrote:
> On Fri, Apr 11, 2025 at 05:03:12PM -0400, Denys Dmytriyenko wrote:
>> Circling back to this...
>>
>> On Thu, Mar 20, 2025 at 02:29:14PM +0530, Beleswar Padhi via lists.yoctoproject.org wrote:
>>> On 18/03/25 21:48, Denys Dmytriyenko wrote:
>>>> On Sat, Mar 15, 2025 at 10:29:18AM +0530, Aniket Limaye via lists.yoctoproject.org wrote:
>>>>> On 3/15/2025 7:32 AM, Denys Dmytriyenko wrote:
>>>>>> From: Denys Dmytriyenko <denys@konsulko.com>
>>>>>>
>>>>>> This reverts commit c58af0b172660ef1248a3454af06251d872770b1.
>>>>>>
>>>>>> This adds an undesired dependency on meta-ti-extras - more details:
>>>>>> https://lists.yoctoproject.org/g/meta-ti/topic/110003435
>>>>>>
>>>>>> Cc: Aniket Limaye <a-limaye@ti.com>
>>>>>> Signed-off-by: Denys Dmytriyenko <denys@konsulko.com
>>>>>> ---
>>>>> The fw binary is actually a requirement for AM57x boards to boot
>>>>> now. U-Boot expects
>>>>> the firmware to be present in the boot partition, and fails if not found.
>>>>> (Beleswar can correct me if I'm mistaken).
>>>> Huh, interesting - do you know the version where this started?
>>> The firmware is required for Early boot of remote processors (IPU in
>>> this case) by U-Boot. Without the firmware, there won't be a system
>>> boot failure, only rproc boot will fail.
>>>
>>>
>>>> If it is indeed a hard requirement to boot the platform now, another option
>>>> is to get a prebuilt binary into ti-linux-firmware and pull it from there,
>>>> instead of building it from sources using legacy tools...
>>> This is great too. However, meta-ti should still have the deploy
>>> rule to pick the pre-built firmware and put it into the boot
>>> partition.
>>>
>>> Thanks,
>>> Beleswar
>> Thanks, Beleswar.
>>
>> Ryan, All,
>>
>> How should this be fixed?
> Ping.
>
> Since nobody's working on adding a prebuilt binary to ti-linux-firmware,
> then this should be reverted as is for now, IMHO.
>
Until, we don't have this fw in ti-linux-firmware,

need to get it through build :(

Thanks

Udit


>>>>> If we don't want to break this board, we need to maybe migrate the
>>>>> ti-ipc-rtos recipe from
>>>>> meta-ti-extras to meta-ti-bsp, since it is anyway a requirement for
>>>>> AM57X u-boot recipe.
>>>>> Denys, I also see that you mentioned that the ti-ipc-rtos recipe
>>>>> currently resides in extras
>>>>> due to the legacy recipes built from sources using old tools like
>>>>> XDC, SYS/BIOS and CGT...
>>>>> which cannot be easily moved to meta-ti-bsp.
>>>>>
>>>>> A temporary fix might be to somehow add these inside of
>>>>> meta-ti-extras/conf/layer.conf?
>>>>>
>>>>> However, I Ack the revert in case it is needed for now:
>>>>>
>>>>> Acked-by: Aniket Limaye <a-limaye@ti.com>
>>>>>
>>>>>
>>>>>>   meta-ti-bsp/conf/machine/include/omap-a15.inc | 3 +--
>>>>>>   1 file changed, 1 insertion(+), 2 deletions(-)
>>>>>>
>>>>>> diff --git a/meta-ti-bsp/conf/machine/include/omap-a15.inc b/meta-ti-bsp/conf/machine/include/omap-a15.inc
>>>>>> index a258decd..be086936 100644
>>>>>> --- a/meta-ti-bsp/conf/machine/include/omap-a15.inc
>>>>>> +++ b/meta-ti-bsp/conf/machine/include/omap-a15.inc
>>>>>> @@ -48,6 +48,5 @@ MACHINE_FEATURES = "apm usbgadget usbhost vfat ext2 screen alsa gpu mmip dsp gc3
>>>>>>   IMAGE_FSTYPES += "tar.xz wic.xz wic.bmap"
>>>>>>   WKS_FILE ?= "sdimage-2part.wks"
>>>>>> -IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES} ipc/dra7-ipu1-fw.xem4;dra7-ipu1-fw.xem4"
>>>>>> +IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES}"
>>>>>>   do_image_wic[depends] += "virtual/bootloader:do_deploy"
>>>>>> -do_image_wic[depends] += "ti-ipc-rtos:do_deploy"

Re: [meta-ti] [master/scarthgap][PATCH] omap-a15.inc: don't include ipu1 firmware in the boot partition

[Denys Dmytriyenko]

On Wed, Apr 23, 2025 at 06:50:14PM +0530, Kumar, Udit wrote:
> 
> On 4/23/2025 7:23 AM, Denys Dmytriyenko wrote:
> >On Fri, Apr 11, 2025 at 05:03:12PM -0400, Denys Dmytriyenko wrote:
> >>Circling back to this...
> >>
> >>On Thu, Mar 20, 2025 at 02:29:14PM +0530, Beleswar Padhi via lists.yoctoproject.org wrote:
> >>>On 18/03/25 21:48, Denys Dmytriyenko wrote:
> >>>>On Sat, Mar 15, 2025 at 10:29:18AM +0530, Aniket Limaye via lists.yoctoproject.org wrote:
> >>>>>On 3/15/2025 7:32 AM, Denys Dmytriyenko wrote:
> >>>>>>From: Denys Dmytriyenko <denys@konsulko.com>
> >>>>>>
> >>>>>>This reverts commit c58af0b172660ef1248a3454af06251d872770b1.
> >>>>>>
> >>>>>>This adds an undesired dependency on meta-ti-extras - more details:
> >>>>>>https://lists.yoctoproject.org/g/meta-ti/topic/110003435
> >>>>>>
> >>>>>>Cc: Aniket Limaye <a-limaye@ti.com>
> >>>>>>Signed-off-by: Denys Dmytriyenko <denys@konsulko.com
> >>>>>>---
> >>>>>The fw binary is actually a requirement for AM57x boards to boot
> >>>>>now. U-Boot expects
> >>>>>the firmware to be present in the boot partition, and fails if not found.
> >>>>>(Beleswar can correct me if I'm mistaken).
> >>>>Huh, interesting - do you know the version where this started?
> >>>The firmware is required for Early boot of remote processors (IPU in
> >>>this case) by U-Boot. Without the firmware, there won't be a system
> >>>boot failure, only rproc boot will fail.
> >>>
> >>>
> >>>>If it is indeed a hard requirement to boot the platform now, another option
> >>>>is to get a prebuilt binary into ti-linux-firmware and pull it from there,
> >>>>instead of building it from sources using legacy tools...
> >>>This is great too. However, meta-ti should still have the deploy
> >>>rule to pick the pre-built firmware and put it into the boot
> >>>partition.
> >>>
> >>>Thanks,
> >>>Beleswar
> >>Thanks, Beleswar.
> >>
> >>Ryan, All,
> >>
> >>How should this be fixed?
> >Ping.
> >
> >Since nobody's working on adding a prebuilt binary to ti-linux-firmware,
> >then this should be reverted as is for now, IMHO.
> >
> Until, we don't have this fw in ti-linux-firmware,
> 
> need to get it through build :(

As discussed - reverting this won't affect the build, it won't even affect the 
platform boot. It only affects rproc boot, which can be resolved by moving 
this code to a curresponding SDK layer.


> Thanks
> 
> Udit
> 
> 
> >>>>>If we don't want to break this board, we need to maybe migrate the
> >>>>>ti-ipc-rtos recipe from
> >>>>>meta-ti-extras to meta-ti-bsp, since it is anyway a requirement for
> >>>>>AM57X u-boot recipe.
> >>>>>Denys, I also see that you mentioned that the ti-ipc-rtos recipe
> >>>>>currently resides in extras
> >>>>>due to the legacy recipes built from sources using old tools like
> >>>>>XDC, SYS/BIOS and CGT...
> >>>>>which cannot be easily moved to meta-ti-bsp.
> >>>>>
> >>>>>A temporary fix might be to somehow add these inside of
> >>>>>meta-ti-extras/conf/layer.conf?
> >>>>>
> >>>>>However, I Ack the revert in case it is needed for now:
> >>>>>
> >>>>>Acked-by: Aniket Limaye <a-limaye@ti.com>
> >>>>>
> >>>>>
> >>>>>>  meta-ti-bsp/conf/machine/include/omap-a15.inc | 3 +--
> >>>>>>  1 file changed, 1 insertion(+), 2 deletions(-)
> >>>>>>
> >>>>>>diff --git a/meta-ti-bsp/conf/machine/include/omap-a15.inc b/meta-ti-bsp/conf/machine/include/omap-a15.inc
> >>>>>>index a258decd..be086936 100644
> >>>>>>--- a/meta-ti-bsp/conf/machine/include/omap-a15.inc
> >>>>>>+++ b/meta-ti-bsp/conf/machine/include/omap-a15.inc
> >>>>>>@@ -48,6 +48,5 @@ MACHINE_FEATURES = "apm usbgadget usbhost vfat ext2 screen alsa gpu mmip dsp gc3
> >>>>>>  IMAGE_FSTYPES += "tar.xz wic.xz wic.bmap"
> >>>>>>  WKS_FILE ?= "sdimage-2part.wks"
> >>>>>>-IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES} ipc/dra7-ipu1-fw.xem4;dra7-ipu1-fw.xem4"
> >>>>>>+IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES}"
> >>>>>>  do_image_wic[depends] += "virtual/bootloader:do_deploy"
> >>>>>>-do_image_wic[depends] += "ti-ipc-rtos:do_deploy"

Re: [meta-ti] [master/scarthgap][PATCH] omap-a15.inc: don't include ipu1 firmware in the boot partition

[Limaye, Aniket]

On 4/23/2025 10:04 PM, Denys Dmytriyenko wrote:
> On Wed, Apr 23, 2025 at 06:50:14PM +0530, Kumar, Udit wrote:
>>
>> On 4/23/2025 7:23 AM, Denys Dmytriyenko wrote:
>>> On Fri, Apr 11, 2025 at 05:03:12PM -0400, Denys Dmytriyenko wrote:
>>>> Circling back to this...
>>>>
>>>> On Thu, Mar 20, 2025 at 02:29:14PM +0530, Beleswar Padhi via lists.yoctoproject.org wrote:
>>>>> On 18/03/25 21:48, Denys Dmytriyenko wrote:
>>>>>> On Sat, Mar 15, 2025 at 10:29:18AM +0530, Aniket Limaye via lists.yoctoproject.org wrote:
>>>>>>> On 3/15/2025 7:32 AM, Denys Dmytriyenko wrote:
>>>>>>>> From: Denys Dmytriyenko <denys@konsulko.com>
>>>>>>>>
>>>>>>>> This reverts commit c58af0b172660ef1248a3454af06251d872770b1.
>>>>>>>>
>>>>>>>> This adds an undesired dependency on meta-ti-extras - more details:
>>>>>>>> https://lists.yoctoproject.org/g/meta-ti/topic/110003435
>>>>>>>>
>>>>>>>> Cc: Aniket Limaye <a-limaye@ti.com>
>>>>>>>> Signed-off-by: Denys Dmytriyenko <denys@konsulko.com
>>>>>>>> ---
>>>>>>> The fw binary is actually a requirement for AM57x boards to boot
>>>>>>> now. U-Boot expects
>>>>>>> the firmware to be present in the boot partition, and fails if not found.
>>>>>>> (Beleswar can correct me if I'm mistaken).
>>>>>> Huh, interesting - do you know the version where this started?
>>>>> The firmware is required for Early boot of remote processors (IPU in
>>>>> this case) by U-Boot. Without the firmware, there won't be a system
>>>>> boot failure, only rproc boot will fail.
>>>>>
>>>>>
>>>>>> If it is indeed a hard requirement to boot the platform now, another option
>>>>>> is to get a prebuilt binary into ti-linux-firmware and pull it from there,
>>>>>> instead of building it from sources using legacy tools...
>>>>> This is great too. However, meta-ti should still have the deploy
>>>>> rule to pick the pre-built firmware and put it into the boot
>>>>> partition.
>>>>>
>>>>> Thanks,
>>>>> Beleswar
>>>> Thanks, Beleswar.
>>>>
>>>> Ryan, All,
>>>>
>>>> How should this be fixed?
>>> Ping.
>>>
>>> Since nobody's working on adding a prebuilt binary to ti-linux-firmware,
>>> then this should be reverted as is for now, IMHO.
>>>
>> Until, we don't have this fw in ti-linux-firmware,
>>
>> need to get it through build :(
> 
> As discussed - reverting this won't affect the build, it won't even affect the
> platform boot. It only affects rproc boot, which can be resolved by moving
> this code to a curresponding SDK layer.
> 

Yeah I can work on making these updates in SDK layer for the time being.

While long term fix being to push prebuilt firmware to 
ti-linux-firmware... such that ti-ipc-rtos recipe can be moved to 
meta-ti-bsp & we can re-implement this in meta-ti-bsp too.

Denys,

One question:
The IMAGE_BOOT_FILES can be easily conditionally appended for omap-a15 
but is there a simple way to add the do_image_wic task dependency 
conditionally in the SDK layer?

Since override syntax does not work with Variable flags, I imagine we 
will need to copy-paste the entire am57xx-evm.conf in the SDK layer with 
this addition?

do_image_wic[depends] += "ti-ipc-rtos:do_deploy"

> 
>> Thanks
>>
>> Udit
>>
>>
>>>>>>> If we don't want to break this board, we need to maybe migrate the
>>>>>>> ti-ipc-rtos recipe from
>>>>>>> meta-ti-extras to meta-ti-bsp, since it is anyway a requirement for
>>>>>>> AM57X u-boot recipe.
>>>>>>> Denys, I also see that you mentioned that the ti-ipc-rtos recipe
>>>>>>> currently resides in extras
>>>>>>> due to the legacy recipes built from sources using old tools like
>>>>>>> XDC, SYS/BIOS and CGT...
>>>>>>> which cannot be easily moved to meta-ti-bsp.
>>>>>>>
>>>>>>> A temporary fix might be to somehow add these inside of
>>>>>>> meta-ti-extras/conf/layer.conf?
>>>>>>>
>>>>>>> However, I Ack the revert in case it is needed for now:
>>>>>>>
>>>>>>> Acked-by: Aniket Limaye <a-limaye@ti.com>
>>>>>>>
>>>>>>>
>>>>>>>>   meta-ti-bsp/conf/machine/include/omap-a15.inc | 3 +--
>>>>>>>>   1 file changed, 1 insertion(+), 2 deletions(-)
>>>>>>>>
>>>>>>>> diff --git a/meta-ti-bsp/conf/machine/include/omap-a15.inc b/meta-ti-bsp/conf/machine/include/omap-a15.inc
>>>>>>>> index a258decd..be086936 100644
>>>>>>>> --- a/meta-ti-bsp/conf/machine/include/omap-a15.inc
>>>>>>>> +++ b/meta-ti-bsp/conf/machine/include/omap-a15.inc
>>>>>>>> @@ -48,6 +48,5 @@ MACHINE_FEATURES = "apm usbgadget usbhost vfat ext2 screen alsa gpu mmip dsp gc3
>>>>>>>>   IMAGE_FSTYPES += "tar.xz wic.xz wic.bmap"
>>>>>>>>   WKS_FILE ?= "sdimage-2part.wks"
>>>>>>>> -IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES} ipc/dra7-ipu1-fw.xem4;dra7-ipu1-fw.xem4"
>>>>>>>> +IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES}"
>>>>>>>>   do_image_wic[depends] += "virtual/bootloader:do_deploy"
>>>>>>>> -do_image_wic[depends] += "ti-ipc-rtos:do_deploy"

Re: [meta-ti] [master/scarthgap][PATCH] omap-a15.inc: don't include ipu1 firmware in the boot partition

[Denys Dmytriyenko]

On Fri, Apr 25, 2025 at 02:29:31AM +0530, Limaye, Aniket wrote:
> 
> 
> On 4/23/2025 10:04 PM, Denys Dmytriyenko wrote:
> >On Wed, Apr 23, 2025 at 06:50:14PM +0530, Kumar, Udit wrote:
> >>
> >>On 4/23/2025 7:23 AM, Denys Dmytriyenko wrote:
> >>>On Fri, Apr 11, 2025 at 05:03:12PM -0400, Denys Dmytriyenko wrote:
> >>>>Circling back to this...
> >>>>
> >>>>On Thu, Mar 20, 2025 at 02:29:14PM +0530, Beleswar Padhi via lists.yoctoproject.org wrote:
> >>>>>On 18/03/25 21:48, Denys Dmytriyenko wrote:
> >>>>>>On Sat, Mar 15, 2025 at 10:29:18AM +0530, Aniket Limaye via lists.yoctoproject.org wrote:
> >>>>>>>On 3/15/2025 7:32 AM, Denys Dmytriyenko wrote:
> >>>>>>>>From: Denys Dmytriyenko <denys@konsulko.com>
> >>>>>>>>
> >>>>>>>>This reverts commit c58af0b172660ef1248a3454af06251d872770b1.
> >>>>>>>>
> >>>>>>>>This adds an undesired dependency on meta-ti-extras - more details:
> >>>>>>>>https://lists.yoctoproject.org/g/meta-ti/topic/110003435
> >>>>>>>>
> >>>>>>>>Cc: Aniket Limaye <a-limaye@ti.com>
> >>>>>>>>Signed-off-by: Denys Dmytriyenko <denys@konsulko.com
> >>>>>>>>---
> >>>>>>>The fw binary is actually a requirement for AM57x boards to boot
> >>>>>>>now. U-Boot expects
> >>>>>>>the firmware to be present in the boot partition, and fails if not found.
> >>>>>>>(Beleswar can correct me if I'm mistaken).
> >>>>>>Huh, interesting - do you know the version where this started?
> >>>>>The firmware is required for Early boot of remote processors (IPU in
> >>>>>this case) by U-Boot. Without the firmware, there won't be a system
> >>>>>boot failure, only rproc boot will fail.
> >>>>>
> >>>>>
> >>>>>>If it is indeed a hard requirement to boot the platform now, another option
> >>>>>>is to get a prebuilt binary into ti-linux-firmware and pull it from there,
> >>>>>>instead of building it from sources using legacy tools...
> >>>>>This is great too. However, meta-ti should still have the deploy
> >>>>>rule to pick the pre-built firmware and put it into the boot
> >>>>>partition.
> >>>>>
> >>>>>Thanks,
> >>>>>Beleswar
> >>>>Thanks, Beleswar.
> >>>>
> >>>>Ryan, All,
> >>>>
> >>>>How should this be fixed?
> >>>Ping.
> >>>
> >>>Since nobody's working on adding a prebuilt binary to ti-linux-firmware,
> >>>then this should be reverted as is for now, IMHO.
> >>>
> >>Until, we don't have this fw in ti-linux-firmware,
> >>
> >>need to get it through build :(
> >
> >As discussed - reverting this won't affect the build, it won't even affect the
> >platform boot. It only affects rproc boot, which can be resolved by moving
> >this code to a curresponding SDK layer.
> >
> 
> Yeah I can work on making these updates in SDK layer for the time being.
> 
> While long term fix being to push prebuilt firmware to
> ti-linux-firmware... such that ti-ipc-rtos recipe can be moved to
> meta-ti-bsp & we can re-implement this in meta-ti-bsp too.
> 
> Denys,
> 
> One question:
> The IMAGE_BOOT_FILES can be easily conditionally appended for
> omap-a15 but is there a simple way to add the do_image_wic task
> dependency conditionally in the SDK layer?
> 
> Since override syntax does not work with Variable flags, I imagine
> we will need to copy-paste the entire am57xx-evm.conf in the SDK
> layer with this addition?
> 
> do_image_wic[depends] += "ti-ipc-rtos:do_deploy"

I was thinking something along the lines (not tested):

WIC_EXTRA_RTOS_DEPENDS = ""
WIC_EXTRA_RTOS_DEPENDS:omap-a15 = "ti-ipc-rtos:do_deploy"
do_image_wic[depends] += "${WIC_EXTRA_RTOS_DEPENDS}"

IMAGE_BOOT_FILES:append:omap-a15 = " ipc/dra7-ipu1-fw.xem4;dra7-ipu1-fw.xem4"

This has to be done in a global namespace, i.e. in one of the .conf files, not 
in a recipe.


> >>>>>>>If we don't want to break this board, we need to maybe migrate the
> >>>>>>>ti-ipc-rtos recipe from
> >>>>>>>meta-ti-extras to meta-ti-bsp, since it is anyway a requirement for
> >>>>>>>AM57X u-boot recipe.
> >>>>>>>Denys, I also see that you mentioned that the ti-ipc-rtos recipe
> >>>>>>>currently resides in extras
> >>>>>>>due to the legacy recipes built from sources using old tools like
> >>>>>>>XDC, SYS/BIOS and CGT...
> >>>>>>>which cannot be easily moved to meta-ti-bsp.
> >>>>>>>
> >>>>>>>A temporary fix might be to somehow add these inside of
> >>>>>>>meta-ti-extras/conf/layer.conf?
> >>>>>>>
> >>>>>>>However, I Ack the revert in case it is needed for now:
> >>>>>>>
> >>>>>>>Acked-by: Aniket Limaye <a-limaye@ti.com>
> >>>>>>>
> >>>>>>>
> >>>>>>>>  meta-ti-bsp/conf/machine/include/omap-a15.inc | 3 +--
> >>>>>>>>  1 file changed, 1 insertion(+), 2 deletions(-)
> >>>>>>>>
> >>>>>>>>diff --git a/meta-ti-bsp/conf/machine/include/omap-a15.inc b/meta-ti-bsp/conf/machine/include/omap-a15.inc
> >>>>>>>>index a258decd..be086936 100644
> >>>>>>>>--- a/meta-ti-bsp/conf/machine/include/omap-a15.inc
> >>>>>>>>+++ b/meta-ti-bsp/conf/machine/include/omap-a15.inc
> >>>>>>>>@@ -48,6 +48,5 @@ MACHINE_FEATURES = "apm usbgadget usbhost vfat ext2 screen alsa gpu mmip dsp gc3
> >>>>>>>>  IMAGE_FSTYPES += "tar.xz wic.xz wic.bmap"
> >>>>>>>>  WKS_FILE ?= "sdimage-2part.wks"
> >>>>>>>>-IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES} ipc/dra7-ipu1-fw.xem4;dra7-ipu1-fw.xem4"
> >>>>>>>>+IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES}"
> >>>>>>>>  do_image_wic[depends] += "virtual/bootloader:do_deploy"
> >>>>>>>>-do_image_wic[depends] += "ti-ipc-rtos:do_deploy"

Re: [meta-ti] [master/scarthgap][PATCH] omap-a15.inc: don't include ipu1 firmware in the boot partition

[Limaye, Aniket]

On 4/25/2025 6:27 AM, Denys Dmytriyenko wrote:
> On Fri, Apr 25, 2025 at 02:29:31AM +0530, Limaye, Aniket wrote:
>>
>>
>> On 4/23/2025 10:04 PM, Denys Dmytriyenko wrote:
>>> On Wed, Apr 23, 2025 at 06:50:14PM +0530, Kumar, Udit wrote:
>>>>
>>>> On 4/23/2025 7:23 AM, Denys Dmytriyenko wrote:
>>>>> On Fri, Apr 11, 2025 at 05:03:12PM -0400, Denys Dmytriyenko wrote:
>>>>>> Circling back to this...
>>>>>>
>>>>>> On Thu, Mar 20, 2025 at 02:29:14PM +0530, Beleswar Padhi via lists.yoctoproject.org wrote:
>>>>>>> On 18/03/25 21:48, Denys Dmytriyenko wrote:
>>>>>>>> On Sat, Mar 15, 2025 at 10:29:18AM +0530, Aniket Limaye via lists.yoctoproject.org wrote:
>>>>>>>>> On 3/15/2025 7:32 AM, Denys Dmytriyenko wrote:
>>>>>>>>>> From: Denys Dmytriyenko <denys@konsulko.com>
>>>>>>>>>>
>>>>>>>>>> This reverts commit c58af0b172660ef1248a3454af06251d872770b1.
>>>>>>>>>>
>>>>>>>>>> This adds an undesired dependency on meta-ti-extras - more details:
>>>>>>>>>> https://lists.yoctoproject.org/g/meta-ti/topic/110003435
>>>>>>>>>>
>>>>>>>>>> Cc: Aniket Limaye <a-limaye@ti.com>
>>>>>>>>>> Signed-off-by: Denys Dmytriyenko <denys@konsulko.com
>>>>>>>>>> ---
>>>>>>>>> The fw binary is actually a requirement for AM57x boards to boot
>>>>>>>>> now. U-Boot expects
>>>>>>>>> the firmware to be present in the boot partition, and fails if not found.
>>>>>>>>> (Beleswar can correct me if I'm mistaken).
>>>>>>>> Huh, interesting - do you know the version where this started?
>>>>>>> The firmware is required for Early boot of remote processors (IPU in
>>>>>>> this case) by U-Boot. Without the firmware, there won't be a system
>>>>>>> boot failure, only rproc boot will fail.
>>>>>>>
>>>>>>>
>>>>>>>> If it is indeed a hard requirement to boot the platform now, another option
>>>>>>>> is to get a prebuilt binary into ti-linux-firmware and pull it from there,
>>>>>>>> instead of building it from sources using legacy tools...
>>>>>>> This is great too. However, meta-ti should still have the deploy
>>>>>>> rule to pick the pre-built firmware and put it into the boot
>>>>>>> partition.
>>>>>>>
>>>>>>> Thanks,
>>>>>>> Beleswar
>>>>>> Thanks, Beleswar.
>>>>>>
>>>>>> Ryan, All,
>>>>>>
>>>>>> How should this be fixed?
>>>>> Ping.
>>>>>
>>>>> Since nobody's working on adding a prebuilt binary to ti-linux-firmware,
>>>>> then this should be reverted as is for now, IMHO.
>>>>>
>>>> Until, we don't have this fw in ti-linux-firmware,
>>>>
>>>> need to get it through build :(
>>>
>>> As discussed - reverting this won't affect the build, it won't even affect the
>>> platform boot. It only affects rproc boot, which can be resolved by moving
>>> this code to a curresponding SDK layer.
>>>
>>
>> Yeah I can work on making these updates in SDK layer for the time being.
>>
>> While long term fix being to push prebuilt firmware to
>> ti-linux-firmware... such that ti-ipc-rtos recipe can be moved to
>> meta-ti-bsp & we can re-implement this in meta-ti-bsp too.
>>
>> Denys,
>>
>> One question:
>> The IMAGE_BOOT_FILES can be easily conditionally appended for
>> omap-a15 but is there a simple way to add the do_image_wic task
>> dependency conditionally in the SDK layer?
>>
>> Since override syntax does not work with Variable flags, I imagine
>> we will need to copy-paste the entire am57xx-evm.conf in the SDK
>> layer with this addition?
>>
>> do_image_wic[depends] += "ti-ipc-rtos:do_deploy"
> 
> I was thinking something along the lines (not tested):
> 
> WIC_EXTRA_RTOS_DEPENDS = ""
> WIC_EXTRA_RTOS_DEPENDS:omap-a15 = "ti-ipc-rtos:do_deploy"
> do_image_wic[depends] += "${WIC_EXTRA_RTOS_DEPENDS}"

Ahhh of course... Yeah this should work

I can add these to the SDK layer once this is removed from here. 
Shouldn't be a problem

Once prebuilt firmware is available at ti-linux-firmware, we can add 
this back to meta-ti-bsp, along with moving the ti-ipc-rtos recipe to 
meta-ti-bsp.

Thanks,
Aniket

> 
> IMAGE_BOOT_FILES:append:omap-a15 = " ipc/dra7-ipu1-fw.xem4;dra7-ipu1-fw.xem4"
> 
> This has to be done in a global namespace, i.e. in one of the .conf files, not
> in a recipe.>
> 
>>>>>>>>> If we don't want to break this board, we need to maybe migrate the
>>>>>>>>> ti-ipc-rtos recipe from
>>>>>>>>> meta-ti-extras to meta-ti-bsp, since it is anyway a requirement for
>>>>>>>>> AM57X u-boot recipe.
>>>>>>>>> Denys, I also see that you mentioned that the ti-ipc-rtos recipe
>>>>>>>>> currently resides in extras
>>>>>>>>> due to the legacy recipes built from sources using old tools like
>>>>>>>>> XDC, SYS/BIOS and CGT...
>>>>>>>>> which cannot be easily moved to meta-ti-bsp.
>>>>>>>>>
>>>>>>>>> A temporary fix might be to somehow add these inside of
>>>>>>>>> meta-ti-extras/conf/layer.conf?
>>>>>>>>>
>>>>>>>>> However, I Ack the revert in case it is needed for now:
>>>>>>>>>
>>>>>>>>> Acked-by: Aniket Limaye <a-limaye@ti.com>
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>>   meta-ti-bsp/conf/machine/include/omap-a15.inc | 3 +--
>>>>>>>>>>   1 file changed, 1 insertion(+), 2 deletions(-)
>>>>>>>>>>
>>>>>>>>>> diff --git a/meta-ti-bsp/conf/machine/include/omap-a15.inc b/meta-ti-bsp/conf/machine/include/omap-a15.inc
>>>>>>>>>> index a258decd..be086936 100644
>>>>>>>>>> --- a/meta-ti-bsp/conf/machine/include/omap-a15.inc
>>>>>>>>>> +++ b/meta-ti-bsp/conf/machine/include/omap-a15.inc
>>>>>>>>>> @@ -48,6 +48,5 @@ MACHINE_FEATURES = "apm usbgadget usbhost vfat ext2 screen alsa gpu mmip dsp gc3
>>>>>>>>>>   IMAGE_FSTYPES += "tar.xz wic.xz wic.bmap"
>>>>>>>>>>   WKS_FILE ?= "sdimage-2part.wks"
>>>>>>>>>> -IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES} ipc/dra7-ipu1-fw.xem4;dra7-ipu1-fw.xem4"
>>>>>>>>>> +IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES}"
>>>>>>>>>>   do_image_wic[depends] += "virtual/bootloader:do_deploy"
>>>>>>>>>> -do_image_wic[depends] += "ti-ipc-rtos:do_deploy"

Re: [meta-ti] [master/scarthgap][PATCH] omap-a15.inc: don't include ipu1 firmware in the boot partition

[Denys Dmytriyenko]

Ryan,

Ping. This slipped through the cracs.


On Mon, Apr 28, 2025 at 01:36:48AM +0530, Aniket Limaye via lists.yoctoproject.org wrote:
> 
> 
> On 4/25/2025 6:27 AM, Denys Dmytriyenko wrote:
> >On Fri, Apr 25, 2025 at 02:29:31AM +0530, Limaye, Aniket wrote:
> >>
> >>
> >>On 4/23/2025 10:04 PM, Denys Dmytriyenko wrote:
> >>>On Wed, Apr 23, 2025 at 06:50:14PM +0530, Kumar, Udit wrote:
> >>>>
> >>>>On 4/23/2025 7:23 AM, Denys Dmytriyenko wrote:
> >>>>>On Fri, Apr 11, 2025 at 05:03:12PM -0400, Denys Dmytriyenko wrote:
> >>>>>>Circling back to this...
> >>>>>>
> >>>>>>On Thu, Mar 20, 2025 at 02:29:14PM +0530, Beleswar Padhi via lists.yoctoproject.org wrote:
> >>>>>>>On 18/03/25 21:48, Denys Dmytriyenko wrote:
> >>>>>>>>On Sat, Mar 15, 2025 at 10:29:18AM +0530, Aniket Limaye via lists.yoctoproject.org wrote:
> >>>>>>>>>On 3/15/2025 7:32 AM, Denys Dmytriyenko wrote:
> >>>>>>>>>>From: Denys Dmytriyenko <denys@konsulko.com>
> >>>>>>>>>>
> >>>>>>>>>>This reverts commit c58af0b172660ef1248a3454af06251d872770b1.
> >>>>>>>>>>
> >>>>>>>>>>This adds an undesired dependency on meta-ti-extras - more details:
> >>>>>>>>>>https://lists.yoctoproject.org/g/meta-ti/topic/110003435
> >>>>>>>>>>
> >>>>>>>>>>Cc: Aniket Limaye <a-limaye@ti.com>
> >>>>>>>>>>Signed-off-by: Denys Dmytriyenko <denys@konsulko.com
> >>>>>>>>>>---
> >>>>>>>>>The fw binary is actually a requirement for AM57x boards to boot
> >>>>>>>>>now. U-Boot expects
> >>>>>>>>>the firmware to be present in the boot partition, and fails if not found.
> >>>>>>>>>(Beleswar can correct me if I'm mistaken).
> >>>>>>>>Huh, interesting - do you know the version where this started?
> >>>>>>>The firmware is required for Early boot of remote processors (IPU in
> >>>>>>>this case) by U-Boot. Without the firmware, there won't be a system
> >>>>>>>boot failure, only rproc boot will fail.
> >>>>>>>
> >>>>>>>
> >>>>>>>>If it is indeed a hard requirement to boot the platform now, another option
> >>>>>>>>is to get a prebuilt binary into ti-linux-firmware and pull it from there,
> >>>>>>>>instead of building it from sources using legacy tools...
> >>>>>>>This is great too. However, meta-ti should still have the deploy
> >>>>>>>rule to pick the pre-built firmware and put it into the boot
> >>>>>>>partition.
> >>>>>>>
> >>>>>>>Thanks,
> >>>>>>>Beleswar
> >>>>>>Thanks, Beleswar.
> >>>>>>
> >>>>>>Ryan, All,
> >>>>>>
> >>>>>>How should this be fixed?
> >>>>>Ping.
> >>>>>
> >>>>>Since nobody's working on adding a prebuilt binary to ti-linux-firmware,
> >>>>>then this should be reverted as is for now, IMHO.
> >>>>>
> >>>>Until, we don't have this fw in ti-linux-firmware,
> >>>>
> >>>>need to get it through build :(
> >>>
> >>>As discussed - reverting this won't affect the build, it won't even affect the
> >>>platform boot. It only affects rproc boot, which can be resolved by moving
> >>>this code to a curresponding SDK layer.
> >>>
> >>
> >>Yeah I can work on making these updates in SDK layer for the time being.
> >>
> >>While long term fix being to push prebuilt firmware to
> >>ti-linux-firmware... such that ti-ipc-rtos recipe can be moved to
> >>meta-ti-bsp & we can re-implement this in meta-ti-bsp too.
> >>
> >>Denys,
> >>
> >>One question:
> >>The IMAGE_BOOT_FILES can be easily conditionally appended for
> >>omap-a15 but is there a simple way to add the do_image_wic task
> >>dependency conditionally in the SDK layer?
> >>
> >>Since override syntax does not work with Variable flags, I imagine
> >>we will need to copy-paste the entire am57xx-evm.conf in the SDK
> >>layer with this addition?
> >>
> >>do_image_wic[depends] += "ti-ipc-rtos:do_deploy"
> >
> >I was thinking something along the lines (not tested):
> >
> >WIC_EXTRA_RTOS_DEPENDS = ""
> >WIC_EXTRA_RTOS_DEPENDS:omap-a15 = "ti-ipc-rtos:do_deploy"
> >do_image_wic[depends] += "${WIC_EXTRA_RTOS_DEPENDS}"
> 
> Ahhh of course... Yeah this should work
> 
> I can add these to the SDK layer once this is removed from here.
> Shouldn't be a problem
> 
> Once prebuilt firmware is available at ti-linux-firmware, we can add
> this back to meta-ti-bsp, along with moving the ti-ipc-rtos recipe
> to meta-ti-bsp.
> 
> Thanks,
> Aniket
> 
> >
> >IMAGE_BOOT_FILES:append:omap-a15 = " ipc/dra7-ipu1-fw.xem4;dra7-ipu1-fw.xem4"
> >
> >This has to be done in a global namespace, i.e. in one of the .conf files, not
> >in a recipe.>
> >
> >>>>>>>>>If we don't want to break this board, we need to maybe migrate the
> >>>>>>>>>ti-ipc-rtos recipe from
> >>>>>>>>>meta-ti-extras to meta-ti-bsp, since it is anyway a requirement for
> >>>>>>>>>AM57X u-boot recipe.
> >>>>>>>>>Denys, I also see that you mentioned that the ti-ipc-rtos recipe
> >>>>>>>>>currently resides in extras
> >>>>>>>>>due to the legacy recipes built from sources using old tools like
> >>>>>>>>>XDC, SYS/BIOS and CGT...
> >>>>>>>>>which cannot be easily moved to meta-ti-bsp.
> >>>>>>>>>
> >>>>>>>>>A temporary fix might be to somehow add these inside of
> >>>>>>>>>meta-ti-extras/conf/layer.conf?
> >>>>>>>>>
> >>>>>>>>>However, I Ack the revert in case it is needed for now:
> >>>>>>>>>
> >>>>>>>>>Acked-by: Aniket Limaye <a-limaye@ti.com>
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>>  meta-ti-bsp/conf/machine/include/omap-a15.inc | 3 +--
> >>>>>>>>>>  1 file changed, 1 insertion(+), 2 deletions(-)
> >>>>>>>>>>
> >>>>>>>>>>diff --git a/meta-ti-bsp/conf/machine/include/omap-a15.inc b/meta-ti-bsp/conf/machine/include/omap-a15.inc
> >>>>>>>>>>index a258decd..be086936 100644
> >>>>>>>>>>--- a/meta-ti-bsp/conf/machine/include/omap-a15.inc
> >>>>>>>>>>+++ b/meta-ti-bsp/conf/machine/include/omap-a15.inc
> >>>>>>>>>>@@ -48,6 +48,5 @@ MACHINE_FEATURES = "apm usbgadget usbhost vfat ext2 screen alsa gpu mmip dsp gc3
> >>>>>>>>>>  IMAGE_FSTYPES += "tar.xz wic.xz wic.bmap"
> >>>>>>>>>>  WKS_FILE ?= "sdimage-2part.wks"
> >>>>>>>>>>-IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES} ipc/dra7-ipu1-fw.xem4;dra7-ipu1-fw.xem4"
> >>>>>>>>>>+IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES}"
> >>>>>>>>>>  do_image_wic[depends] += "virtual/bootloader:do_deploy"
> >>>>>>>>>>-do_image_wic[depends] += "ti-ipc-rtos:do_deploy"

Re: [meta-ti] [master/scarthgap][PATCH] omap-a15.inc: don't include ipu1 firmware in the boot partition

[Kumar, Udit]

Hi Denys

On 6/28/2025 2:45 AM, Denys Dmytriyenko wrote:
> Ryan,
>
> Ping. This slipped through the cracs.

This is WIP ,

we are working to push firmware first to ti firmware repo


>
> On Mon, Apr 28, 2025 at 01:36:48AM +0530, Aniket Limaye via lists.yoctoproject.org wrote:
>>
>> On 4/25/2025 6:27 AM, Denys Dmytriyenko wrote:
>>> On Fri, Apr 25, 2025 at 02:29:31AM +0530, Limaye, Aniket wrote:
>>>>
>>>> On 4/23/2025 10:04 PM, Denys Dmytriyenko wrote:
>>>>> On Wed, Apr 23, 2025 at 06:50:14PM +0530, Kumar, Udit wrote:
>>>>>> On 4/23/2025 7:23 AM, Denys Dmytriyenko wrote:
>>>>>>> On Fri, Apr 11, 2025 at 05:03:12PM -0400, Denys Dmytriyenko wrote:
>>>>>>>> Circling back to this...
>>>>>>>>
>>>>>>>> On Thu, Mar 20, 2025 at 02:29:14PM +0530, Beleswar Padhi via lists.yoctoproject.org wrote:
>>>>>>>>> On 18/03/25 21:48, Denys Dmytriyenko wrote:
>>>>>>>>>> On Sat, Mar 15, 2025 at 10:29:18AM +0530, Aniket Limaye via lists.yoctoproject.org wrote:
>>>>>>>>>>> On 3/15/2025 7:32 AM, Denys Dmytriyenko wrote:
>>>>>>>>>>>> From: Denys Dmytriyenko <denys@konsulko.com>
>>>>>>>>>>>>
>>>>>>>>>>>> This reverts commit c58af0b172660ef1248a3454af06251d872770b1.
>>>>>>>>>>>>
>>>>>>>>>>>> This adds an undesired dependency on meta-ti-extras - more details:
>>>>>>>>>>>> https://lists.yoctoproject.org/g/meta-ti/topic/110003435
>>>>>>>>>>>>
>>>>>>>>>>>> Cc: Aniket Limaye <a-limaye@ti.com>
>>>>>>>>>>>> Signed-off-by: Denys Dmytriyenko <denys@konsulko.com
>>>>>>>>>>>> ---
>>>>>>>>>>> The fw binary is actually a requirement for AM57x boards to boot
>>>>>>>>>>> now. U-Boot expects
>>>>>>>>>>> the firmware to be present in the boot partition, and fails if not found.
>>>>>>>>>>> (Beleswar can correct me if I'm mistaken).
>>>>>>>>>> Huh, interesting - do you know the version where this started?
>>>>>>>>> The firmware is required for Early boot of remote processors (IPU in
>>>>>>>>> this case) by U-Boot. Without the firmware, there won't be a system
>>>>>>>>> boot failure, only rproc boot will fail.
>>>>>>>>>
>>>>>>>>>
>>>>>>>>>> If it is indeed a hard requirement to boot the platform now, another option
>>>>>>>>>> is to get a prebuilt binary into ti-linux-firmware and pull it from there,
>>>>>>>>>> instead of building it from sources using legacy tools...
>>>>>>>>> This is great too. However, meta-ti should still have the deploy
>>>>>>>>> rule to pick the pre-built firmware and put it into the boot
>>>>>>>>> partition.
>>>>>>>>>
>>>>>>>>> Thanks,
>>>>>>>>> Beleswar
>>>>>>>> Thanks, Beleswar.
>>>>>>>>
>>>>>>>> Ryan, All,
>>>>>>>>
>>>>>>>> How should this be fixed?
>>>>>>> Ping.
>>>>>>>
>>>>>>> Since nobody's working on adding a prebuilt binary to ti-linux-firmware,
>>>>>>> then this should be reverted as is for now, IMHO.
>>>>>>>
>>>>>> Until, we don't have this fw in ti-linux-firmware,
>>>>>>
>>>>>> need to get it through build :(
>>>>> As discussed - reverting this won't affect the build, it won't even affect the
>>>>> platform boot. It only affects rproc boot, which can be resolved by moving
>>>>> this code to a curresponding SDK layer.
>>>>>
>>>> Yeah I can work on making these updates in SDK layer for the time being.
>>>>
>>>> While long term fix being to push prebuilt firmware to
>>>> ti-linux-firmware... such that ti-ipc-rtos recipe can be moved to
>>>> meta-ti-bsp & we can re-implement this in meta-ti-bsp too.
>>>>
>>>> Denys,
>>>>
>>>> One question:
>>>> The IMAGE_BOOT_FILES can be easily conditionally appended for
>>>> omap-a15 but is there a simple way to add the do_image_wic task
>>>> dependency conditionally in the SDK layer?
>>>>
>>>> Since override syntax does not work with Variable flags, I imagine
>>>> we will need to copy-paste the entire am57xx-evm.conf in the SDK
>>>> layer with this addition?
>>>>
>>>> do_image_wic[depends] += "ti-ipc-rtos:do_deploy"
>>> I was thinking something along the lines (not tested):
>>>
>>> WIC_EXTRA_RTOS_DEPENDS = ""
>>> WIC_EXTRA_RTOS_DEPENDS:omap-a15 = "ti-ipc-rtos:do_deploy"
>>> do_image_wic[depends] += "${WIC_EXTRA_RTOS_DEPENDS}"
>> Ahhh of course... Yeah this should work
>>
>> I can add these to the SDK layer once this is removed from here.
>> Shouldn't be a problem
>>
>> Once prebuilt firmware is available at ti-linux-firmware, we can add
>> this back to meta-ti-bsp, along with moving the ti-ipc-rtos recipe
>> to meta-ti-bsp.
>>
>> Thanks,
>> Aniket
>>
>>> IMAGE_BOOT_FILES:append:omap-a15 = " ipc/dra7-ipu1-fw.xem4;dra7-ipu1-fw.xem4"
>>>
>>> This has to be done in a global namespace, i.e. in one of the .conf files, not
>>> in a recipe.>
>>>
>>>>>>>>>>> If we don't want to break this board, we need to maybe migrate the
>>>>>>>>>>> ti-ipc-rtos recipe from
>>>>>>>>>>> meta-ti-extras to meta-ti-bsp, since it is anyway a requirement for
>>>>>>>>>>> AM57X u-boot recipe.
>>>>>>>>>>> Denys, I also see that you mentioned that the ti-ipc-rtos recipe
>>>>>>>>>>> currently resides in extras
>>>>>>>>>>> due to the legacy recipes built from sources using old tools like
>>>>>>>>>>> XDC, SYS/BIOS and CGT...
>>>>>>>>>>> which cannot be easily moved to meta-ti-bsp.
>>>>>>>>>>>
>>>>>>>>>>> A temporary fix might be to somehow add these inside of
>>>>>>>>>>> meta-ti-extras/conf/layer.conf?
>>>>>>>>>>>
>>>>>>>>>>> However, I Ack the revert in case it is needed for now:
>>>>>>>>>>>
>>>>>>>>>>> Acked-by: Aniket Limaye <a-limaye@ti.com>
>>>>>>>>>>>
>>>>>>>>>>>
>>>>>>>>>>>>   meta-ti-bsp/conf/machine/include/omap-a15.inc | 3 +--
>>>>>>>>>>>>   1 file changed, 1 insertion(+), 2 deletions(-)
>>>>>>>>>>>>
>>>>>>>>>>>> diff --git a/meta-ti-bsp/conf/machine/include/omap-a15.inc b/meta-ti-bsp/conf/machine/include/omap-a15.inc
>>>>>>>>>>>> index a258decd..be086936 100644
>>>>>>>>>>>> --- a/meta-ti-bsp/conf/machine/include/omap-a15.inc
>>>>>>>>>>>> +++ b/meta-ti-bsp/conf/machine/include/omap-a15.inc
>>>>>>>>>>>> @@ -48,6 +48,5 @@ MACHINE_FEATURES = "apm usbgadget usbhost vfat ext2 screen alsa gpu mmip dsp gc3
>>>>>>>>>>>>   IMAGE_FSTYPES += "tar.xz wic.xz wic.bmap"
>>>>>>>>>>>>   WKS_FILE ?= "sdimage-2part.wks"
>>>>>>>>>>>> -IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES} ipc/dra7-ipu1-fw.xem4;dra7-ipu1-fw.xem4"
>>>>>>>>>>>> +IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES}"
>>>>>>>>>>>>   do_image_wic[depends] += "virtual/bootloader:do_deploy"
>>>>>>>>>>>> -do_image_wic[depends] += "ti-ipc-rtos:do_deploy"

Re: [meta-ti] [master/scarthgap][PATCH] omap-a15.inc: don't include ipu1 firmware in the boot partition

[Denys Dmytriyenko]

On Sat, Jun 28, 2025 at 06:41:43AM +0530, Kumar, Udit wrote:
> Hi Denys
> 
> On 6/28/2025 2:45 AM, Denys Dmytriyenko wrote:
> >Ryan,
> >
> >Ping. This slipped through the cracs.
> 
> This is WIP ,
> 
> we are working to push firmware first to ti firmware repo

Thanks for the update.


> >On Mon, Apr 28, 2025 at 01:36:48AM +0530, Aniket Limaye via lists.yoctoproject.org wrote:
> >>
> >>On 4/25/2025 6:27 AM, Denys Dmytriyenko wrote:
> >>>On Fri, Apr 25, 2025 at 02:29:31AM +0530, Limaye, Aniket wrote:
> >>>>
> >>>>On 4/23/2025 10:04 PM, Denys Dmytriyenko wrote:
> >>>>>On Wed, Apr 23, 2025 at 06:50:14PM +0530, Kumar, Udit wrote:
> >>>>>>On 4/23/2025 7:23 AM, Denys Dmytriyenko wrote:
> >>>>>>>On Fri, Apr 11, 2025 at 05:03:12PM -0400, Denys Dmytriyenko wrote:
> >>>>>>>>Circling back to this...
> >>>>>>>>
> >>>>>>>>On Thu, Mar 20, 2025 at 02:29:14PM +0530, Beleswar Padhi via lists.yoctoproject.org wrote:
> >>>>>>>>>On 18/03/25 21:48, Denys Dmytriyenko wrote:
> >>>>>>>>>>On Sat, Mar 15, 2025 at 10:29:18AM +0530, Aniket Limaye via lists.yoctoproject.org wrote:
> >>>>>>>>>>>On 3/15/2025 7:32 AM, Denys Dmytriyenko wrote:
> >>>>>>>>>>>>From: Denys Dmytriyenko <denys@konsulko.com>
> >>>>>>>>>>>>
> >>>>>>>>>>>>This reverts commit c58af0b172660ef1248a3454af06251d872770b1.
> >>>>>>>>>>>>
> >>>>>>>>>>>>This adds an undesired dependency on meta-ti-extras - more details:
> >>>>>>>>>>>>https://lists.yoctoproject.org/g/meta-ti/topic/110003435
> >>>>>>>>>>>>
> >>>>>>>>>>>>Cc: Aniket Limaye <a-limaye@ti.com>
> >>>>>>>>>>>>Signed-off-by: Denys Dmytriyenko <denys@konsulko.com
> >>>>>>>>>>>>---
> >>>>>>>>>>>The fw binary is actually a requirement for AM57x boards to boot
> >>>>>>>>>>>now. U-Boot expects
> >>>>>>>>>>>the firmware to be present in the boot partition, and fails if not found.
> >>>>>>>>>>>(Beleswar can correct me if I'm mistaken).
> >>>>>>>>>>Huh, interesting - do you know the version where this started?
> >>>>>>>>>The firmware is required for Early boot of remote processors (IPU in
> >>>>>>>>>this case) by U-Boot. Without the firmware, there won't be a system
> >>>>>>>>>boot failure, only rproc boot will fail.
> >>>>>>>>>
> >>>>>>>>>
> >>>>>>>>>>If it is indeed a hard requirement to boot the platform now, another option
> >>>>>>>>>>is to get a prebuilt binary into ti-linux-firmware and pull it from there,
> >>>>>>>>>>instead of building it from sources using legacy tools...
> >>>>>>>>>This is great too. However, meta-ti should still have the deploy
> >>>>>>>>>rule to pick the pre-built firmware and put it into the boot
> >>>>>>>>>partition.
> >>>>>>>>>
> >>>>>>>>>Thanks,
> >>>>>>>>>Beleswar
> >>>>>>>>Thanks, Beleswar.
> >>>>>>>>
> >>>>>>>>Ryan, All,
> >>>>>>>>
> >>>>>>>>How should this be fixed?
> >>>>>>>Ping.
> >>>>>>>
> >>>>>>>Since nobody's working on adding a prebuilt binary to ti-linux-firmware,
> >>>>>>>then this should be reverted as is for now, IMHO.
> >>>>>>>
> >>>>>>Until, we don't have this fw in ti-linux-firmware,
> >>>>>>
> >>>>>>need to get it through build :(
> >>>>>As discussed - reverting this won't affect the build, it won't even affect the
> >>>>>platform boot. It only affects rproc boot, which can be resolved by moving
> >>>>>this code to a curresponding SDK layer.
> >>>>>
> >>>>Yeah I can work on making these updates in SDK layer for the time being.
> >>>>
> >>>>While long term fix being to push prebuilt firmware to
> >>>>ti-linux-firmware... such that ti-ipc-rtos recipe can be moved to
> >>>>meta-ti-bsp & we can re-implement this in meta-ti-bsp too.
> >>>>
> >>>>Denys,
> >>>>
> >>>>One question:
> >>>>The IMAGE_BOOT_FILES can be easily conditionally appended for
> >>>>omap-a15 but is there a simple way to add the do_image_wic task
> >>>>dependency conditionally in the SDK layer?
> >>>>
> >>>>Since override syntax does not work with Variable flags, I imagine
> >>>>we will need to copy-paste the entire am57xx-evm.conf in the SDK
> >>>>layer with this addition?
> >>>>
> >>>>do_image_wic[depends] += "ti-ipc-rtos:do_deploy"
> >>>I was thinking something along the lines (not tested):
> >>>
> >>>WIC_EXTRA_RTOS_DEPENDS = ""
> >>>WIC_EXTRA_RTOS_DEPENDS:omap-a15 = "ti-ipc-rtos:do_deploy"
> >>>do_image_wic[depends] += "${WIC_EXTRA_RTOS_DEPENDS}"
> >>Ahhh of course... Yeah this should work
> >>
> >>I can add these to the SDK layer once this is removed from here.
> >>Shouldn't be a problem
> >>
> >>Once prebuilt firmware is available at ti-linux-firmware, we can add
> >>this back to meta-ti-bsp, along with moving the ti-ipc-rtos recipe
> >>to meta-ti-bsp.
> >>
> >>Thanks,
> >>Aniket
> >>
> >>>IMAGE_BOOT_FILES:append:omap-a15 = " ipc/dra7-ipu1-fw.xem4;dra7-ipu1-fw.xem4"
> >>>
> >>>This has to be done in a global namespace, i.e. in one of the .conf files, not
> >>>in a recipe.>
> >>>
> >>>>>>>>>>>If we don't want to break this board, we need to maybe migrate the
> >>>>>>>>>>>ti-ipc-rtos recipe from
> >>>>>>>>>>>meta-ti-extras to meta-ti-bsp, since it is anyway a requirement for
> >>>>>>>>>>>AM57X u-boot recipe.
> >>>>>>>>>>>Denys, I also see that you mentioned that the ti-ipc-rtos recipe
> >>>>>>>>>>>currently resides in extras
> >>>>>>>>>>>due to the legacy recipes built from sources using old tools like
> >>>>>>>>>>>XDC, SYS/BIOS and CGT...
> >>>>>>>>>>>which cannot be easily moved to meta-ti-bsp.
> >>>>>>>>>>>
> >>>>>>>>>>>A temporary fix might be to somehow add these inside of
> >>>>>>>>>>>meta-ti-extras/conf/layer.conf?
> >>>>>>>>>>>
> >>>>>>>>>>>However, I Ack the revert in case it is needed for now:
> >>>>>>>>>>>
> >>>>>>>>>>>Acked-by: Aniket Limaye <a-limaye@ti.com>
> >>>>>>>>>>>
> >>>>>>>>>>>
> >>>>>>>>>>>>  meta-ti-bsp/conf/machine/include/omap-a15.inc | 3 +--
> >>>>>>>>>>>>  1 file changed, 1 insertion(+), 2 deletions(-)
> >>>>>>>>>>>>
> >>>>>>>>>>>>diff --git a/meta-ti-bsp/conf/machine/include/omap-a15.inc b/meta-ti-bsp/conf/machine/include/omap-a15.inc
> >>>>>>>>>>>>index a258decd..be086936 100644
> >>>>>>>>>>>>--- a/meta-ti-bsp/conf/machine/include/omap-a15.inc
> >>>>>>>>>>>>+++ b/meta-ti-bsp/conf/machine/include/omap-a15.inc
> >>>>>>>>>>>>@@ -48,6 +48,5 @@ MACHINE_FEATURES = "apm usbgadget usbhost vfat ext2 screen alsa gpu mmip dsp gc3
> >>>>>>>>>>>>  IMAGE_FSTYPES += "tar.xz wic.xz wic.bmap"
> >>>>>>>>>>>>  WKS_FILE ?= "sdimage-2part.wks"
> >>>>>>>>>>>>-IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES} ipc/dra7-ipu1-fw.xem4;dra7-ipu1-fw.xem4"
> >>>>>>>>>>>>+IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} ${UBOOT_EXTLINUX_BOOT_FILES}"
> >>>>>>>>>>>>  do_image_wic[depends] += "virtual/bootloader:do_deploy"
> >>>>>>>>>>>>-do_image_wic[depends] += "ti-ipc-rtos:do_deploy"

Re: [master/scarthgap][PATCH] omap-a15.inc: don't include ipu1 firmware in the boot partition

[Beleswar Prasad Padhi]

On 15/03/25 10:29, Limaye, Aniket wrote:
>
> On 3/15/2025 7:32 AM, Denys Dmytriyenko wrote:
>> From: Denys Dmytriyenko <denys@konsulko.com>
>>
>> This reverts commit c58af0b172660ef1248a3454af06251d872770b1.
>>
>> This adds an undesired dependency on meta-ti-extras - more details:
>> https://lists.yoctoproject.org/g/meta-ti/topic/110003435
>>
>> Cc: Aniket Limaye <a-limaye@ti.com>
>> Signed-off-by: Denys Dmytriyenko <denys@konsulko.com
>> ---
>
> The fw binary is actually a requirement for AM57x boards to boot now. 
> U-Boot expects
> the firmware to be present in the boot partition, and fails if not found.
> (Beleswar can correct me if I'm mistaken).


The firmware is required for Early boot of remote processors (IPU in 
this case) by U-Boot. Without the firmware, there won't be a system boot 
failure, only rproc boot will fail.

>
> If we don't want to break this board, we need to maybe migrate the 
> ti-ipc-rtos recipe from
> meta-ti-extras to meta-ti-bsp, since it is anyway a requirement for 
> AM57X u-boot recipe.
> Denys, I also see that you mentioned that the ti-ipc-rtos recipe 
> currently resides in extras
> due to the legacy recipes built from sources using old tools like XDC, 
> SYS/BIOS and CGT...
> which cannot be easily moved to meta-ti-bsp.
>
> A temporary fix might be to somehow add these inside of 
> meta-ti-extras/conf/layer.conf?
>
> However, I Ack the revert in case it is needed for now:
>
> Acked-by: Aniket Limaye <a-limaye@ti.com>
>
>
>> meta-ti-bsp/conf/machine/include/omap-a15.inc | 3 +--
>>   1 file changed, 1 insertion(+), 2 deletions(-)
>>
>> diff --git a/meta-ti-bsp/conf/machine/include/omap-a15.inc 
>> b/meta-ti-bsp/conf/machine/include/omap-a15.inc
>> index a258decd..be086936 100644
>> --- a/meta-ti-bsp/conf/machine/include/omap-a15.inc
>> +++ b/meta-ti-bsp/conf/machine/include/omap-a15.inc
>> @@ -48,6 +48,5 @@ MACHINE_FEATURES = "apm usbgadget usbhost vfat ext2 
>> screen alsa gpu mmip dsp gc3
>>     IMAGE_FSTYPES += "tar.xz wic.xz wic.bmap"
>>   WKS_FILE ?= "sdimage-2part.wks"
>> -IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} 
>> ${UBOOT_EXTLINUX_BOOT_FILES} ipc/dra7-ipu1-fw.xem4;dra7-ipu1-fw.xem4"
>> +IMAGE_BOOT_FILES ?= "${SPL_BINARY} u-boot.${UBOOT_SUFFIX} 
>> ${UBOOT_EXTLINUX_BOOT_FILES}"
>>   do_image_wic[depends] += "virtual/bootloader:do_deploy"
>> -do_image_wic[depends] += "ti-ipc-rtos:do_deploy"