| Message ID | 20240704071935.840815-1-c-shilwant@ti.com |
|---|---|
| State | Accepted |
| Delegated to: | Ryan Eatmon |
| Headers | show |
| Series | [meta-ti,master/scarthgap] conf: machine: Update am62pxx-evm for multi-BSP support | expand |
Hi Chirag, Thanks for the patch. On 04/07/24 12:49, Chirag Shilwant wrote: > - In ti-u-boot-2024.04, the config fragment to disable splashscreen > is now renamed to am62x_evm_prune_splashscreen.config [1] > Hence, update UBOOT_CONFIG_FRAGMENTS for the same. > > - Add UBOOT_CONFIG_FRAGMENTS:bsp-ti-6_1 which ensures that > am62pxx-evm machine configuration still supports Display Cluster usecase [2] > with ti-u-boot-2023.04 > > [1]: https://git.ti.com/cgit/ti-u-boot/ti-u-boot/commit/?h=ti-u-boot-2024.04&id=981eddc83dfdbbf176a4984977f9779dc1d56f87 > > [2]: https://software-dl.ti.com/mcu-plus-sdk/esd/AM62PX/09_01_00_39/exports/docs/api_guide_am62px/EXAMPLES_DRIVERS_DSS_DISPLAY_SHARE.html > > Signed-off-by: Chirag Shilwant <c-shilwant@ti.com> Reviewed-by: Devarsh Thakkar <devarsht@ti.com> On a side-note, The uEnv.txt also needs to be updated as need to prefix ti in name_overlays as otherwise we get below error : [2024-07-04 15:57:12.135] Failed to load '/boot/dtb/k3-am62p5-sk-microtips-mf101hie-panel.dtbo' [2024-07-04 15:57:12.151] Failed to load '/boot/dtb/k3-am62p5-sk-dss-shared-mode.dtbo' [2024-07-04 15:57:12.151] ## Flattened Device Tree blob at 88000000 After setting as below, it works fine : setenv name_overlays ti/k3-am62p5-sk-microtips-mf101hie-panel.dtbo ti/k3-am62p5-sk-dss-shared-mode.dtbo $name_overlay Above can go as a separate patch and there could be multiple platforms affected, please verify. Regards Devarsh > --- > meta-ti-bsp/conf/machine/am62pxx-evm.conf | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/meta-ti-bsp/conf/machine/am62pxx-evm.conf b/meta-ti-bsp/conf/machine/am62pxx-evm.conf > index fab96bd8..8d94c2b4 100644 > --- a/meta-ti-bsp/conf/machine/am62pxx-evm.conf > +++ b/meta-ti-bsp/conf/machine/am62pxx-evm.conf > @@ -22,5 +22,7 @@ UBOOT_MACHINE = "am62px_evm_a53_defconfig" > # along with the base defconfig mentioned in UBOOT_MACHINE. Refer u-boot-mergeconfig.inc > # under meta-ti-bsp/recipes-bsp/u-boot/ for more details. > # For AM62P tisdk-display-cluster image, splash screen is handled by SBL. > -# Hence, disable the A53 based splash screen using the am62px_evm_prune_splashscreen.config fragment present in ti-u-boot tree > -UBOOT_CONFIG_FRAGMENTS = "${@oe.utils.conditional('DISPLAY_CLUSTER_ENABLE', '1', 'am62px_evm_prune_splashscreen.config', '', d)}" > +# Hence, disable the A53 based splash screen using the am62x_evm_prune_splashscreen.config fragment present in ti-u-boot tree > +UBOOT_CONFIG_FRAGMENTS = "${@oe.utils.conditional('DISPLAY_CLUSTER_ENABLE', '1', 'am62x_evm_prune_splashscreen.config', '', d)}" > + > +UBOOT_CONFIG_FRAGMENTS:bsp-ti-6_1 = "${@oe.utils.conditional('DISPLAY_CLUSTER_ENABLE', '1', 'am62px_evm_prune_splashscreen.config', '', d)}"
Hi, On 04/07/24 16:35, Devarsh Thakkar wrote: > Hi Chirag, > > Thanks for the patch. > On 04/07/24 12:49, Chirag Shilwant wrote: >> - In ti-u-boot-2024.04, the config fragment to disable splashscreen >> is now renamed to am62x_evm_prune_splashscreen.config [1] >> Hence, update UBOOT_CONFIG_FRAGMENTS for the same. >>> - Add UBOOT_CONFIG_FRAGMENTS:bsp-ti-6_1 which ensures that >> am62pxx-evm machine configuration still supports Display Cluster usecase [2] >> with ti-u-boot-2023.04 >> >> [1]: https://git.ti.com/cgit/ti-u-boot/ti-u-boot/commit/?h=ti-u-boot-2024.04&id=981eddc83dfdbbf176a4984977f9779dc1d56f87 >> >> [2]: https://software-dl.ti.com/mcu-plus-sdk/esd/AM62PX/09_01_00_39/exports/docs/api_guide_am62px/EXAMPLES_DRIVERS_DSS_DISPLAY_SHARE.html >> >> Signed-off-by: Chirag Shilwant <c-shilwant@ti.com> > Reviewed-by: Devarsh Thakkar <devarsht@ti.com> > > > On a side-note, > The uEnv.txt also needs to be updated as need to prefix ti in name_overlays as > otherwise we get below error : > > [2024-07-04 15:57:12.135] Failed to load > '/boot/dtb/k3-am62p5-sk-microtips-mf101hie-panel.dtbo' > [2024-07-04 15:57:12.151] Failed to load > '/boot/dtb/k3-am62p5-sk-dss-shared-mode.dtbo' > [2024-07-04 15:57:12.151] ## Flattened Device Tree blob at 88000000 > > After setting as below, it works fine : > > setenv name_overlays ti/k3-am62p5-sk-microtips-mf101hie-panel.dtbo > ti/k3-am62p5-sk-dss-shared-mode.dtbo $name_overlay > > Above can go as a separate patch and there could be multiple platforms > affected, please verify. Just a clarification for folks in mailing list, The uEnv.txt only for Display Cluster usecase of AM62P is updated in meta-tisdk [1] & is independent of meta-arago. Will take an action to update the uEnv.txt in meta-tisdk post we have this patch integrated to master/scarthgap. For other platforms, by default the uEnv.txt is empty & it comes from meta-arago [2]. So, rest all platforms aren't affected! [1]: https://git.ti.com/cgit/ti-sdk-linux/meta-tisdk/tree/recipes-tisdk/tisdk-uenv/tisdk-uenv/am62pxx-evm/uEnv-am62p-display-cluster.txt?h=scarthgap [2]: https://git.ti.com/cgit/arago-project/meta-arago/tree/meta-arago-distro/recipes-tisdk/tisdk-uenv/tisdk-uenv/uEnv.txt?h=scarthgap --- Regards, Chirag > Regards > Devarsh > >> --- >> meta-ti-bsp/conf/machine/am62pxx-evm.conf | 6 ++++-- >> 1 file changed, 4 insertions(+), 2 deletions(-) >> >> diff --git a/meta-ti-bsp/conf/machine/am62pxx-evm.conf b/meta-ti-bsp/conf/machine/am62pxx-evm.conf >> index fab96bd8..8d94c2b4 100644 >> --- a/meta-ti-bsp/conf/machine/am62pxx-evm.conf >> +++ b/meta-ti-bsp/conf/machine/am62pxx-evm.conf >> @@ -22,5 +22,7 @@ UBOOT_MACHINE = "am62px_evm_a53_defconfig" >> # along with the base defconfig mentioned in UBOOT_MACHINE. Refer u-boot-mergeconfig.inc >> # under meta-ti-bsp/recipes-bsp/u-boot/ for more details. >> # For AM62P tisdk-display-cluster image, splash screen is handled by SBL. >> -# Hence, disable the A53 based splash screen using the am62px_evm_prune_splashscreen.config fragment present in ti-u-boot tree >> -UBOOT_CONFIG_FRAGMENTS = "${@oe.utils.conditional('DISPLAY_CLUSTER_ENABLE', '1', 'am62px_evm_prune_splashscreen.config', '', d)}" >> +# Hence, disable the A53 based splash screen using the am62x_evm_prune_splashscreen.config fragment present in ti-u-boot tree >> +UBOOT_CONFIG_FRAGMENTS = "${@oe.utils.conditional('DISPLAY_CLUSTER_ENABLE', '1', 'am62x_evm_prune_splashscreen.config', '', d)}" >> + >> +UBOOT_CONFIG_FRAGMENTS:bsp-ti-6_1 = "${@oe.utils.conditional('DISPLAY_CLUSTER_ENABLE', '1', 'am62px_evm_prune_splashscreen.config', '', d)}"
diff --git a/meta-ti-bsp/conf/machine/am62pxx-evm.conf b/meta-ti-bsp/conf/machine/am62pxx-evm.conf index fab96bd8..8d94c2b4 100644 --- a/meta-ti-bsp/conf/machine/am62pxx-evm.conf +++ b/meta-ti-bsp/conf/machine/am62pxx-evm.conf @@ -22,5 +22,7 @@ UBOOT_MACHINE = "am62px_evm_a53_defconfig" # along with the base defconfig mentioned in UBOOT_MACHINE. Refer u-boot-mergeconfig.inc # under meta-ti-bsp/recipes-bsp/u-boot/ for more details. # For AM62P tisdk-display-cluster image, splash screen is handled by SBL. -# Hence, disable the A53 based splash screen using the am62px_evm_prune_splashscreen.config fragment present in ti-u-boot tree -UBOOT_CONFIG_FRAGMENTS = "${@oe.utils.conditional('DISPLAY_CLUSTER_ENABLE', '1', 'am62px_evm_prune_splashscreen.config', '', d)}" +# Hence, disable the A53 based splash screen using the am62x_evm_prune_splashscreen.config fragment present in ti-u-boot tree +UBOOT_CONFIG_FRAGMENTS = "${@oe.utils.conditional('DISPLAY_CLUSTER_ENABLE', '1', 'am62x_evm_prune_splashscreen.config', '', d)}" + +UBOOT_CONFIG_FRAGMENTS:bsp-ti-6_1 = "${@oe.utils.conditional('DISPLAY_CLUSTER_ENABLE', '1', 'am62px_evm_prune_splashscreen.config', '', d)}"
- In ti-u-boot-2024.04, the config fragment to disable splashscreen is now renamed to am62x_evm_prune_splashscreen.config [1] Hence, update UBOOT_CONFIG_FRAGMENTS for the same. - Add UBOOT_CONFIG_FRAGMENTS:bsp-ti-6_1 which ensures that am62pxx-evm machine configuration still supports Display Cluster usecase [2] with ti-u-boot-2023.04 [1]: https://git.ti.com/cgit/ti-u-boot/ti-u-boot/commit/?h=ti-u-boot-2024.04&id=981eddc83dfdbbf176a4984977f9779dc1d56f87 [2]: https://software-dl.ti.com/mcu-plus-sdk/esd/AM62PX/09_01_00_39/exports/docs/api_guide_am62px/EXAMPLES_DRIVERS_DSS_DISPLAY_SHARE.html Signed-off-by: Chirag Shilwant <c-shilwant@ti.com> --- meta-ti-bsp/conf/machine/am62pxx-evm.conf | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-)