On Wed, Jun 14, 2023 at 01:16:13PM -0500, Ryan Eatmon wrote: > > > On 6/14/2023 11:57 AM, Devarsh Thakkar wrote: > > > > > >On 14/06/23 21:21, Denys Dmytriyenko wrote: > >>On Wed, Jun 14, 2023 at 08:53:07PM +0530, Devarsh Thakkar wrote: > >>>Hi Denys, > >>> > >>>Thanks for the review, > >>> > >>>On 14/06/23 20:24, Denys Dmytriyenko wrote: > >>>>On Wed, Jun 14, 2023 at 08:03:05PM +0530, Devarsh Thakkar > >>>>via lists.yoctoproject.org wrote: > >>>>>Early splash screen support for AM62x has been enabled in u-boot > >>>>>and it requires bitmap tarball to be present in boot partition > >>>>>as per the default environment settings done in u-boot [1]. > >>>>> > >>>>>Copy the ti logo bitmap tarball to boot partition of wic > >>>>>image in order > >>>>>to enable out of box early display on AM62x. > >>>>> > >>>>>[1] > >>>>>https://git.ti.com/cgit/ti-u-boot/ti-u-boot/tree/board/ti/am62x/am62x.env > >>>>> > >>>>>Signed-off-by: Devarsh Thakkar <devar...@ti.com> > >>>>>--- > >>>>>V2: Rename ti logo tarball as per suggested changes (same is being > >>>>> done in ti-u-boot too) > >>>>>--- > >>>>> .../recipes-bsp/u-boot/u-boot-ti-staging_2023.04.bb > >>>>>| 9 +++++++++ > >>>>> 1 file changed, 9 insertions(+) > >>>>> > >>>>>diff --git > >>>>>a/meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti-staging_2023.04.bb > >>>>>b/meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti-staging_2023.04.bb > >>>>>index e6da84cb..c948e1ed 100644 > >>>>>--- a/meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti-staging_2023.04.bb > >>>>>+++ b/meta-ti-bsp/recipes-bsp/u-boot/u-boot-ti-staging_2023.04.bb > >>>>>@@ -8,3 +8,12 @@ BRANCH = "ti-u-boot-2023.04" > >>>>> SRCREV = "1f510931dcc90e6f28f7fed757ad5000d4382e22" > >>>>>+do_install:append:am62xx() { > >>>>>+ install -d ${D}/boot > >>>>>+ install -m 0644 > >>>>>${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz ${D}/boot > >>>>>+} > >>>>>+ > >>>>>+do_deploy:append:am62xx() { > >>>>>+ install -d ${DEPLOYDIR} > >>>>>+ install -m 0644 > >>>>>${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz ${DEPLOYDIR} > >>>> > >>>>Any specific reason you also need to deploy this loose file? > >>>> > >>> > >>>I think the boot partition script which is ran as part of > >>>do_image_wic expects > >>>IMAGE_BOOTFILES to be present in deploy directory, I remember > >>>getting below > >>>error without doing so : > >>> > >>>| ERROR: _exec_cmd: install -m 0644 -D > >>>/home/devarsht/ti/yocto_sdk_am62/oe-layersetup/build/arago-tmp-default-glibc/deploy/images/am62xx-evm/ti.gz > >>>/home/devarsht/ti/yocto_sdk_am62/oe-layersetup/build/arago-tmp-default-glibc/work/am62xx_evm-oe-linux/tisdk-default-image/1.0-r0/tmp-wic/hdd/boot/ti.gz > >>>returned '1' instead of 0 > >>>| output: install: cannot stat > >>>'/home/devarsht/ti/yocto_sdk_am62/oe-layersetup/build/arago-tmp-default-glibc/deploy/images/am62xx-evm/ti.gz': > >>>No such file or directory > >>>| WARNING: > >>>/home/devarsht/ti/yocto_sdk_am62/oe-layersetup/build/arago-tmp-default-glibc/work/am62xx_evm-oe-linux/tisdk-default-image/1.0-r0/temp/run.do_image_wic.1581619:162 > >>>exit 1 from > >>>'BUILDDIR="/home/devarsht/ti/yocto_sdk_am62/oe-layersetup/build" > >>>PSEUDO_UNLOAD=1 wic create "$wks" --vars > >>>"/home/devarsht/ti/yocto_sdk_am62/oe-layersetup/build/arago-tmp-default-glibc/sysroots/am62xx-evm/imgdata/" > >>>-e "tisdk-default-image" -o "$build_wic/" -w "$tmp_wic"' > >> > >>So, is this logo expected by U-boot to reside in the boot partition of SD > >>card, or in the /boot directory of the rootfs? > >>The first is handled by do_deploy and IMAGE_BOOTFILES, while the > >>second is > >>handled by do_install. I wonder if doing both is going to be confusing... > >> > >> > > > >Yes, this logo is expected to reside in boot partition of SD only > >per current implementation in ti-u-boot. > > So then it only requires do_deploy and IMAGE_BOOTFILES since that > puts items into the boot partition... Do we need the do_install?
Well, thinking a bit more about this, we do the same for all other binaries from U-boot build in u-boot-ti.inc and even upstream u-boot.inc - we install and deploy them. The installed files go into /boot directory and end up in corresponding IPK package, but we do not install it, hence the files are not in the rootfs. So this should be fine as is... > >>>One more reason to do so was to allow user ease of use, to > >>>copy the logo too > >>>from deploy folder along with other boot files (if not flashing from wic > >>>image) since the same is set in am62x.env and if logo is not > >>>present in boot > >>>partition then it throws some error message although the > >>>device still boots.
-=-=-=-=-=-=-=-=-=-=-=- Links: You receive all messages sent to this group. View/Reply Online (#16728): https://lists.yoctoproject.org/g/meta-ti/message/16728 Mute This Topic: https://lists.yoctoproject.org/mt/99528771/21656 Group Owner: meta-ti+ow...@lists.yoctoproject.org Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/leave/6695321/21656/1393940836/xyzzy [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-