| Message ID | 20250315020224.3100249-1-denis@denix.org |
|---|---|
| State | New |
| Headers | show |
| Series | [master/scarthgap] omap-a15.inc: don't include ipu1 firmware in the boot partition | expand |
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"
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"
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] >>> -=-=-=-=-=-=-=-=-=-=-=- >>>
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"
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"
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"
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"
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"
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"
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"
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"
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"