On 6/15/2023 8:58 AM, Devarsh Thakkar wrote:


On 15/06/23 19:06, Ryan Eatmon wrote:


On 6/14/2023 9:33 AM, Devarsh Thakkar 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}
+}


Denys,

In the longer term, should this be in the u-boot.inc file and not in the
recipe itself?  That's were all of the other do_* appends are...


This I think is specific to u-boot 2023.04 version only as the logos were
recently added and are not available in previous versions of u-boot and hence
I thought to have the changes in u-boot-ti-staging_2023.04.bb

But they will be there going forward and the only other version is mainline which is meant to track the latest of u-boot which theoretically will contain the logos.

Maybe the best plan would be to create a ti-logos.inc and put these statements in there and add an include to that file in the recipe. Just trying to keep the spirit of the recipe. We can then not include the logos in mainline if they are not working.


Regards
Devarsh



--
Ryan Eatmon                reat...@ti.com
-----------------------------------------
Texas Instruments, Inc.  -  LCPD  -  MGTS
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#16732): 
https://lists.yoctoproject.org/g/meta-ti/message/16732
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]
-=-=-=-=-=-=-=-=-=-=-=-

Reply via email to