Message ID | 20240124144617.327734-1-devarsht@ti.com |
---|---|
State | Superseded |
Delegated to: | Ryan Eatmon |
Headers | show |
Series | [meta-ti,kirkstone] meta-ti-bsp: Add bootsplash image file for AM62P | expand |
On 1/24/2024 8:46 AM, Devarsh Thakkar wrote: > Add TI logo image as bootsplash image file for AM62P and install it in > boot directory from where u-boot can read from and display during > boot-up time. I'm going to send another patch that updates the u-boot mainline recipe to 2024.01 and move the logo code out of the .bb files and into the common .inc file. Can you rebase this patch on that patch once I send it and make this a v2? This will reduce the number of places that we need to maintain the logo install/deploy code. I'll let you know once these changes are in place for you to rebase off of. > Signed-off-by: Devarsh Thakkar <devarsht@ti.com> > --- > meta-ti-bsp/conf/machine/include/am62pxx.inc | 3 +++ > .../recipes-bsp/u-boot/u-boot-ti-staging_2023.04.bb | 10 ++++++++++ > 2 files changed, 13 insertions(+) > > diff --git a/meta-ti-bsp/conf/machine/include/am62pxx.inc b/meta-ti-bsp/conf/machine/include/am62pxx.inc > index bac68253..20e0771e 100644 > --- a/meta-ti-bsp/conf/machine/include/am62pxx.inc > +++ b/meta-ti-bsp/conf/machine/include/am62pxx.inc > @@ -9,6 +9,9 @@ PREFERRED_PROVIDER_virtual/gpudriver ?= "ti-img-rogue-driver" > # Default tiboot3.bin on AM62Px is for HS-FS > IMAGE_BOOT_FILES += "tiboot3-am62px-hs-fs-evm.bin" > > +# Bitmap image tarball for early splashscreen > +IMAGE_BOOT_FILES += "ti_logo_414x97_32bpp.bmp.gz" > + > TFA_BOARD = "lite" > TFA_K3_SYSTEM_SUSPEND = "1" > > 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 4fe1218a..c98cbe61 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 > @@ -17,3 +17,13 @@ do_deploy:append:am62xx() { > install -d ${DEPLOYDIR} > install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz ${DEPLOYDIR} > } > + > +do_install:append:am62pxx() { > + install -d ${D}/boot > + install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz ${D}/boot > +} > + > +do_deploy:append:am62pxx() { > + install -d ${DEPLOYDIR} > + install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz ${DEPLOYDIR} > +}
On 1/24/2024 12:23 PM, Ryan Eatmon via lists.yoctoproject.org wrote: > > > On 1/24/2024 8:46 AM, Devarsh Thakkar wrote: >> Add TI logo image as bootsplash image file for AM62P and install it in >> boot directory from where u-boot can read from and display during >> boot-up time. > > I'm going to send another patch that updates the u-boot mainline recipe > to 2024.01 and move the logo code out of the .bb files and into the > common .inc file. Can you rebase this patch on that patch once I send > it and make this a v2? This will reduce the number of places that we > need to maintain the logo install/deploy code. > > I'll let you know once these changes are in place for you to rebase off of. Ok. If you want to redo your changes on top of kirkstone-wip you should be good to go. > >> Signed-off-by: Devarsh Thakkar <devarsht@ti.com> >> --- >> meta-ti-bsp/conf/machine/include/am62pxx.inc | 3 +++ >> .../recipes-bsp/u-boot/u-boot-ti-staging_2023.04.bb | 10 ++++++++++ >> 2 files changed, 13 insertions(+) >> >> diff --git a/meta-ti-bsp/conf/machine/include/am62pxx.inc >> b/meta-ti-bsp/conf/machine/include/am62pxx.inc >> index bac68253..20e0771e 100644 >> --- a/meta-ti-bsp/conf/machine/include/am62pxx.inc >> +++ b/meta-ti-bsp/conf/machine/include/am62pxx.inc >> @@ -9,6 +9,9 @@ PREFERRED_PROVIDER_virtual/gpudriver ?= >> "ti-img-rogue-driver" >> # Default tiboot3.bin on AM62Px is for HS-FS >> IMAGE_BOOT_FILES += "tiboot3-am62px-hs-fs-evm.bin" >> +# Bitmap image tarball for early splashscreen >> +IMAGE_BOOT_FILES += "ti_logo_414x97_32bpp.bmp.gz" >> + >> TFA_BOARD = "lite" >> TFA_K3_SYSTEM_SUSPEND = "1" >> 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 4fe1218a..c98cbe61 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 >> @@ -17,3 +17,13 @@ do_deploy:append:am62xx() { >> install -d ${DEPLOYDIR} >> install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz >> ${DEPLOYDIR} >> } >> + >> +do_install:append:am62pxx() { >> + install -d ${D}/boot >> + install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz >> ${D}/boot >> +} >> + >> +do_deploy:append:am62pxx() { >> + install -d ${DEPLOYDIR} >> + install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz >> ${DEPLOYDIR} >> +} > > > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > View/Reply Online (#17413): https://lists.yoctoproject.org/g/meta-ti/message/17413 > Mute This Topic: https://lists.yoctoproject.org/mt/103932848/6551054 > Group Owner: meta-ti+owner@lists.yoctoproject.org > Unsubscribe: https://lists.yoctoproject.org/g/meta-ti/leave/10828724/6551054/1815494134/xyzzy [reatmon@ti.com] > -=-=-=-=-=-=-=-=-=-=-=- >
Hi Devarsh, On 24/01/24 20:16, Devarsh Thakkar via lists.yoctoproject.org wrote: > Add TI logo image as bootsplash image file for AM62P and install it in > boot directory from where u-boot can read from and display during > boot-up time. Signed-off-by: Devarsh Thakkar <devarsht@ ti. com> --- > meta-ti-bsp/conf/machine/include/am62pxx. inc > ZjQcmQRYFpfptBannerStart > This message was sent from outside of Texas Instruments. > Do not click links or open attachments unless you recognize the source > of this email and know the content is safe. > ZjQcmQRYFpfptBannerEnd > Add TI logo image as bootsplash image file for AM62P and install it in > boot directory from where u-boot can read from and display during > boot-up time. In u-boot tree, have we added the equivalent config fragment in-order to disable A53 splash which was planned to be used for tisdk-display-cluster image? Integrating this patch without a config fragment in u-boot tree might unnecessarily break the tisdk-display-cluster image! > > Signed-off-by: Devarsh Thakkar <devarsht@ti.com> > --- > meta-ti-bsp/conf/machine/include/am62pxx.inc | 3 +++ > .../recipes-bsp/u-boot/u-boot-ti-staging_2023.04.bb | 10 ++++++++++ > 2 files changed, 13 insertions(+) > > diff --git a/meta-ti-bsp/conf/machine/include/am62pxx.inc b/meta-ti-bsp/conf/machine/include/am62pxx.inc > index bac68253..20e0771e 100644 > --- a/meta-ti-bsp/conf/machine/include/am62pxx.inc > +++ b/meta-ti-bsp/conf/machine/include/am62pxx.inc > @@ -9,6 +9,9 @@ PREFERRED_PROVIDER_virtual/gpudriver ?= "ti-img-rogue-driver" > # Default tiboot3.bin on AM62Px is for HS-FS > IMAGE_BOOT_FILES += "tiboot3-am62px-hs-fs-evm.bin" > > +# Bitmap image tarball for early splashscreen > +IMAGE_BOOT_FILES += "ti_logo_414x97_32bpp.bmp.gz" > + > TFA_BOARD = "lite" > TFA_K3_SYSTEM_SUSPEND = "1" > > 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 4fe1218a..c98cbe61 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 > @@ -17,3 +17,13 @@ do_deploy:append:am62xx() { > install -d ${DEPLOYDIR} > install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz ${DEPLOYDIR} > } > + > +do_install:append:am62pxx() { > + install -d ${D}/boot > + install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz ${D}/boot > +} > + > +do_deploy:append:am62pxx() { > + install -d ${DEPLOYDIR} > + install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz ${DEPLOYDIR} > +} > -- > 2.34.1 > > > -=-=-=-=-=-=-=-=-=-=-=- > Links: You receive all messages sent to this group. > View/Reply Online (#17412): https://urldefense.com/v3/__https://lists.yoctoproject.org/g/meta-ti/message/17412__;!!G3vK!SAjBqsYWt8FxWuH8q6YfBKxCfvqOjkX08Mt0sMB_0-PNFr1Ctdjp3WCUXZyWCcEFwFLoQeFVfluLPCQ2GQdFHCYOOH1JtDnl94g$ > Mute This Topic: https://urldefense.com/v3/__https://lists.yoctoproject.org/mt/103932848/7030289__;!!G3vK!SAjBqsYWt8FxWuH8q6YfBKxCfvqOjkX08Mt0sMB_0-PNFr1Ctdjp3WCUXZyWCcEFwFLoQeFVfluLPCQ2GQdFHCYOOH1JI-qC-0g$ > Group Owner: meta-ti+owner@lists.yoctoproject.org > Unsubscribe: https://urldefense.com/v3/__https://lists.yoctoproject.org/g/meta-ti/unsub__;!!G3vK!SAjBqsYWt8FxWuH8q6YfBKxCfvqOjkX08Mt0sMB_0-PNFr1Ctdjp3WCUXZyWCcEFwFLoQeFVfluLPCQ2GQdFHCYOOH1Jo26-_Iw$ [c-shilwant@ti.com] > -=-=-=-=-=-=-=-=-=-=-=- >
diff --git a/meta-ti-bsp/conf/machine/include/am62pxx.inc b/meta-ti-bsp/conf/machine/include/am62pxx.inc index bac68253..20e0771e 100644 --- a/meta-ti-bsp/conf/machine/include/am62pxx.inc +++ b/meta-ti-bsp/conf/machine/include/am62pxx.inc @@ -9,6 +9,9 @@ PREFERRED_PROVIDER_virtual/gpudriver ?= "ti-img-rogue-driver" # Default tiboot3.bin on AM62Px is for HS-FS IMAGE_BOOT_FILES += "tiboot3-am62px-hs-fs-evm.bin" +# Bitmap image tarball for early splashscreen +IMAGE_BOOT_FILES += "ti_logo_414x97_32bpp.bmp.gz" + TFA_BOARD = "lite" TFA_K3_SYSTEM_SUSPEND = "1" 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 4fe1218a..c98cbe61 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 @@ -17,3 +17,13 @@ do_deploy:append:am62xx() { install -d ${DEPLOYDIR} install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz ${DEPLOYDIR} } + +do_install:append:am62pxx() { + install -d ${D}/boot + install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz ${D}/boot +} + +do_deploy:append:am62pxx() { + install -d ${DEPLOYDIR} + install -m 0644 ${S}/tools/logos/ti_logo_414x97_32bpp.bmp.gz ${DEPLOYDIR} +}
Add TI logo image as bootsplash image file for AM62P and install it in boot directory from where u-boot can read from and display during boot-up time. Signed-off-by: Devarsh Thakkar <devarsht@ti.com> --- meta-ti-bsp/conf/machine/include/am62pxx.inc | 3 +++ .../recipes-bsp/u-boot/u-boot-ti-staging_2023.04.bb | 10 ++++++++++ 2 files changed, 13 insertions(+)