diff mbox series

[meta-ti,kirkstone] meta-ti-bsp: Add bootsplash image file for AM62P

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

Commit Message

Devarsh Thakkar Jan. 24, 2024, 2:46 p.m. UTC
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(+)

Comments

Ryan Eatmon Jan. 24, 2024, 6:23 p.m. UTC | #1
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}
> +}
Ryan Eatmon Jan. 25, 2024, 8:57 p.m. UTC | #2
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]
> -=-=-=-=-=-=-=-=-=-=-=-
>
Chirag Shilwant Jan. 26, 2024, 5:47 a.m. UTC | #3
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 mbox series

Patch

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}
+}