Re: [EXTERNAL] [meta-ti] [master/scarthgap][PATCH 2/4] conf/machine/j721e: some BSPs don't support all SR and HS variants

["Limaye, Aniket" <a-limaye@ti.com>]

[-- Attachment #1: Type: text/plain, Size: 4743 bytes --]

Hi Denys,

On 6/13/2024 10:52 PM, Denys Dmytriyenko wrote:
> From: Denys Dmytriyenko <denys@ konsulko. com> Adjust IMAGE_BOOT_FILES 
> list as some BSPs do not support all combinations of SR1. 1, SR2. 0, 
> HS-FS and HS-SE variants. Signed-off-by: Denys Dmytriyenko 
> <denys@ konsulko. com> --- meta-ti-bsp/conf/machine/include/j721e. inc
> ZjQcmQRYFpfptBannerStart
> From: Denys Dmytriyenko <denys@konsulko.com>
> Adjust IMAGE_BOOT_FILES list as some BSPs do not support all combinations
> of SR1.1, SR2.0, HS-FS and HS-SE variants.
>
> Signed-off-by: Denys Dmytriyenko<denys@konsulko.com>
> ---

Same change is needed for J7200 as well. At the current commit pointed 
by u-boot-ti-mainline [1], j7200 only has support for below:
GP , SR2.0 HS-FS , SR2.0 HS-SE

Hence also suggest below for j7200.inc:

diff --git a/meta-ti-bsp/conf/machine/include/j7200.inc 
b/meta-ti-bsp/conf/machine/include/j7200.inc
index caa79876..2a11e3dd 100644
--- a/meta-ti-bsp/conf/machine/include/j7200.inc
+++ b/meta-ti-bsp/conf/machine/include/j7200.inc
@@ -5,10 +5,14 @@ SOC_FAMILY:append = ":j7200"
  IMAGE_BOOT_FILES += "tiboot3-j7200-gp-evm.bin"

  # Since default tiboot3.bin on J7200 is for GP, add a version for 
SR1.0 HS-FS
-IMAGE_BOOT_FILES += "tiboot3-j7200-hs-fs-evm.bin"
+IMAGE_SR1_HS_FS_FILES = "tiboot3-j7200-hs-fs-evm.bin"
+IMAGE_SR1_HS_FS_FILES:bsp-mainline = ""
+IMAGE_BOOT_FILES += "${IMAGE_SR1_HS_FS_FILES}"

  # Since default tiboot3.bin on J7200 is for GP, add a version for 
SR1.0 HS-SE
-IMAGE_BOOT_FILES += "tiboot3-j7200-hs-evm.bin"
+IMAGE_SR1_HS_SE_FILES = "tiboot3-j7200-hs-evm.bin"
+IMAGE_SR1_HS_SE_FILES:bsp-mainline = ""
+IMAGE_BOOT_FILES += "${IMAGE_SR1_HS_SE_FILES}"

  # Since default tiboot3.bin on J7200 is for GP, add a version for 
SR2.0 HS-FS
  IMAGE_BOOT_FILES += "tiboot3-j7200_sr2-hs-fs-evm.bin"


[1]: 
https://source.denx.de/u-boot/u-boot/-/blob/866ca972d6c3cabeaf6dbac431e8e08bb30b3c8e/arch/arm/dts/k3-j7200-binman.dtsi


>   meta-ti-bsp/conf/machine/include/j721e.inc | 18 ++++++++++--------
>   1 file changed, 10 insertions(+), 8 deletions(-)
>
> diff --git a/meta-ti-bsp/conf/machine/include/j721e.inc b/meta-ti-bsp/conf/machine/include/j721e.inc
> index a54c3f2f..62487aa9 100644
> --- a/meta-ti-bsp/conf/machine/include/j721e.inc
> +++ b/meta-ti-bsp/conf/machine/include/j721e.inc
> @@ -13,20 +13,22 @@ IMAGE_BOOT_FILES += "tiboot3-j721e-gp-evm.bin"
>   IMAGE_BOOT_FILES += "sysfw-j721e-gp-evm.itb"
>   
>   # Since default on J721e is for GP, add a version for SR1.1 HS-FS
> -IMAGE_BOOT_FILES += "tiboot3-j721e_sr1_1-hs-fs-evm.bin"
> -IMAGE_BOOT_FILES += "sysfw-j721e_sr1_1-hs-fs-evm.itb"
> +IMAGE_SR1_1_HS_FS_FILES = "tiboot3-j721e_sr1_1-hs-fs-evm.bin sysfw-j721e_sr1_1-hs-fs-evm.itb"
> +IMAGE_SR1_1_HS_FS_FILES:bsp-mainline = ""
> +IMAGE_BOOT_FILES += "${IMAGE_SR1_1_HS_FS_FILES}"
>   
>   # Since default on J721e is for GP, add a version for SR1.1 HS-SE
> -IMAGE_BOOT_FILES += "tiboot3-j721e_sr1_1-hs-evm.bin"
> -IMAGE_BOOT_FILES += "sysfw-j721e_sr1_1-hs-evm.itb"
> +IMAGE_SR1_1_HS_SE_FILES = "tiboot3-j721e_sr1_1-hs-evm.bin sysfw-j721e_sr1_1-hs-evm.itb"
> +IMAGE_BOOT_FILES += "${IMAGE_SR1_1_HS_SE_FILES}"
>   
>   # Since default on J721e is for GP, add a version for SR2.0 HS-FS
> -IMAGE_BOOT_FILES += "tiboot3-j721e_sr2-hs-fs-evm.bin"
> -IMAGE_BOOT_FILES += "sysfw-j721e_sr2-hs-fs-evm.itb"
> +IMAGE_SR2_HS_FS_FILES = "tiboot3-j721e_sr2-hs-fs-evm.bin sysfw-j721e_sr2-hs-fs-evm.itb"
> +IMAGE_BOOT_FILES += "${IMAGE_SR2_HS_FS_FILES}"
>   
>   # Since default on J721e is for GP, add a version for SR2.0 HS-SE
> -IMAGE_BOOT_FILES += "tiboot3-j721e_sr2-hs-evm.bin"
> -IMAGE_BOOT_FILES += "sysfw-j721e_sr2-hs-evm.itb"
> +IMAGE_SR2_HS_SE_FILES = "tiboot3-j721e_sr2-hs-evm.bin sysfw-j721e_sr2-hs-evm.itb"
> +IMAGE_SR2_HS_SE_FILES:bsp-mainline = ""
> +IMAGE_BOOT_FILES += "${IMAGE_SR2_HS_SE_FILES}"
>   
>   TFA_BOARD = "generic"
>   
> -- 
> 2.25.1
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#17720):https://urldefense.com/v3/__https://lists.yoctoproject.org/g/meta-ti/message/17720__;!!G3vK!ViJQwrQKeC3qqXVZ8kKcUDbfpHLQlIvvjLNyHFQgHl3_TFIS3cLW8FGbQrROSxcrLl16vBx3jw$  
> Mute This Topic:https://urldefense.com/v3/__https://lists.yoctoproject.org/mt/106655976/6607860__;!!G3vK!ViJQwrQKeC3qqXVZ8kKcUDbfpHLQlIvvjLNyHFQgHl3_TFIS3cLW8FGbQrROSxcrLl2K7krThA$  
> Group Owner:meta-ti+owner@lists.yoctoproject.org
> Unsubscribe:https://urldefense.com/v3/__https://lists.yoctoproject.org/g/meta-ti/unsub__;!!G3vK!ViJQwrQKeC3qqXVZ8kKcUDbfpHLQlIvvjLNyHFQgHl3_TFIS3cLW8FGbQrROSxcrLl1EHYbH3A$   [a-limaye@ti.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>

[-- Attachment #2: Type: text/html, Size: 8888 bytes --]