[PATCH 1/2] arm: dts: k3-am62a-sk: remove GP tiboot3 builds

[Neha Malcom Francis]

Hi Bryan

On 30/07/25 00:06, Bryan Brattlof wrote:
> The AM62Ax SoC family was the last part from TI to support the GP
> security variant, however this security variant was used mostly
> internally and with select early partners and never sold publicly.
> 
> To simplify things and to avoid any confusion of which parts are
> supported in the future, remove the GP tiboot3.bin builds from binman.
> 
> Signed-off-by: Bryan Brattlof <bb at ti.com>
> ---

As per the discussion, in the hope that no more of these are "in the wild":

Reviewed-by: Neha Malcom Francis <n-francis at ti.com>

Is there a Yocto patch that goes along with this?

>  arch/arm/dts/k3-am62a-sk-binman.dtsi | 181 +----------------------------------
>  doc/board/ti/am62ax_sk.rst           |   7 +-
>  2 files changed, 2 insertions(+), 186 deletions(-)
> 
> diff --git a/arch/arm/dts/k3-am62a-sk-binman.dtsi b/arch/arm/dts/k3-am62a-sk-binman.dtsi
> index 0685bdd7e0c78087777e12cfa3d936c63f2fa597..e64c165ecbf1cd82586b850549776ad189b104dc 100644
> --- a/arch/arm/dts/k3-am62a-sk-binman.dtsi
> +++ b/arch/arm/dts/k3-am62a-sk-binman.dtsi
> @@ -98,43 +98,6 @@
>  	};
>  };
>  
> -&binman {
> -	tiboot3-am62ax-gp-evm.bin {
> -		filename = "tiboot3-am62ax-gp-evm.bin";
> -		ti-secure-rom {
> -			content = <&u_boot_spl_unsigned>, <&ti_fs_gp>,
> -				<&combined_tifs_cfg_gp>, <&combined_dm_cfg_gp>;
> -			combined;
> -			dm-data;
> -			content-sbl = <&u_boot_spl_unsigned>;
> -			load = <CONFIG_SPL_TEXT_BASE>;
> -			content-sysfw = <&ti_fs_gp>;
> -			load-sysfw = <0x40000>;
> -			content-sysfw-data = <&combined_tifs_cfg_gp>;
> -			load-sysfw-data = <0x67000>;
> -			content-dm-data = <&combined_dm_cfg_gp>;
> -			load-dm-data = <0x43c3a800>;
> -			sw-rev = <1>;
> -			keyfile = "ti-degenerate-key.pem";
> -		};
> -		u_boot_spl_unsigned: u-boot-spl {
> -			no-expanded;
> -		};
> -		ti_fs_gp: ti-fs-gp.bin {
> -			filename = "ti-sysfw/ti-fs-firmware-am62ax-gp.bin";
> -			type = "blob-ext";
> -		};
> -		combined_tifs_cfg_gp: combined-tifs-cfg-gp.bin {
> -			filename = "combined-tifs-cfg.bin";
> -			type = "blob-ext";
> -		};
> -		combined_dm_cfg_gp: combined-dm-cfg-gp.bin {
> -			filename = "combined-dm-cfg.bin";
> -			type = "blob-ext";
> -		};
> -	};
> -};
> -
>  #endif
>  
>  #ifdef CONFIG_TARGET_AM62A7_A53_EVM
> @@ -181,23 +144,6 @@
>  
>  	};
>  
> -	tifsstub-gp {
> -		filename = "tifsstub.bin_gp";
> -		ti-secure-rom {
> -			content = <&tifsstub_gp>;
> -			core = "secure";
> -			load = <0x60000>;
> -			sw-rev = <CONFIG_K3_X509_SWRV>;
> -			keyfile = "ti-degenerate-key.pem";
> -			tifsstub;
> -		};
> -		tifsstub_gp: tifsstub-gp.bin {
> -			filename = "ti-sysfw/ti-fs-stub-firmware-am62ax-gp.bin";
> -			type = "blob-ext";
> -			optional;
> -		};
> -	};
> -
>  	ti-spl {
>  		insert-template = <&ti_spl_template>;
>  
> @@ -229,18 +175,6 @@
>  					};
>  				};
>  
> -				tifsstub-gp {
> -					description = "TIFSSTUB";
> -					type = "firmware";
> -					arch = "arm32";
> -					compression = "none";
> -					os = "tifsstub-gp";
> -					load = <0x9ca00000>;
> -					entry = <0x9ca00000>;
> -					blob-ext {
> -						filename = "tifsstub.bin_gp";
> -					};
> -				};
>  				dm {
>  					ti-secure {
>  						content = <&dm>;
> @@ -275,7 +209,7 @@
>  					description = "k3-am62a7-sk";
>  					firmware = "atf";
>  					loadables = "tee", "dm", "spl",
> -					"tifsstub-hs", "tifsstub-fs", "tifsstub-gp";
> +					"tifsstub-hs", "tifsstub-fs";
>  					fdt = "fdt-0";
>  				};
>  			};
> @@ -326,117 +260,4 @@
>  	};
>  };
>  
> -&binman {
> -	ti-spl_unsigned {
> -		insert-template = <&ti_spl_unsigned_template>;
> -
> -		fit {
> -			images {
> -				tifsstub-hs {
> -					description = "tifsstub";
> -					type = "firmware";
> -					arch = "arm32";
> -					compression = "none";
> -					os = "tifsstub-hs";
> -					load = <0x9ca00000>;
> -					entry = <0x9ca00000>;
> -					blob-ext {
> -						filename = "tifsstub.bin_hs";
> -					};
> -				};
> -
> -				tifsstub-fs {
> -					description = "tifsstub";
> -					type = "firmware";
> -					arch = "arm32";
> -					compression = "none";
> -					os = "tifsstub-fs";
> -					load = <0x9ca00000>;
> -					entry = <0x9ca00000>;
> -					blob-ext {
> -						filename = "tifsstub.bin_fs";
> -					};
> -				};
> -
> -				tifsstub-gp {
> -					description = "tifsstub";
> -					type = "firmware";
> -					arch = "arm32";
> -					compression = "none";
> -					os = "tifsstub-gp";
> -					load = <0x9ca00000>;
> -					entry = <0x9ca00000>;
> -					blob-ext {
> -						filename = "tifsstub.bin_gp";
> -					};
> -				};
> -				dm {
> -					ti-dm {
> -						filename = "ti-dm/am62axx/ipc_echo_testb_mcu1_0_release_strip.xer5f";
> -					};
> -				};
> -
> -				fdt-0 {
> -					description = "k3-am62a7-sk";
> -					type = "flat_dt";
> -					arch = "arm";
> -					compression = "none";
> -					blob {
> -						filename = SPL_AM62A7_SK_DTB;
> -					};
> -				};
> -			};
> -
> -			configurations {
> -				default = "conf-0";
> -
> -				conf-0 {
> -					description = "k3-am62a7-sk";
> -					firmware = "atf";
> -					loadables = "tee", "dm", "spl",
> -					"tifsstub-hs", "tifsstub-fs", "tifsstub-gp";
> -					fdt = "fdt-0";
> -				};
> -			};
> -		};
> -	};
> -};
> -
> -&binman {
> -	u-boot_unsigned {
> -		insert-template = <&u_boot_unsigned_template>;
> -
> -		fit {
> -			images {
> -				uboot {
> -					description = "U-Boot for AM62Ax Board";
> -				};
> -
> -				fdt-0 {
> -					description = "k3-am62a7-sk";
> -					type = "flat_dt";
> -					arch = "arm";
> -					compression = "none";
> -					blob {
> -						filename = AM62A7_SK_DTB;
> -					};
> -					hash {
> -						algo = "crc32";
> -					};
> -				};
> -			};
> -
> -			configurations {
> -				default = "conf-0";
> -
> -				conf-0 {
> -					description = "k3-am62a7-sk";
> -					firmware = "uboot";
> -					loadables = "uboot";
> -					fdt = "fdt-0";
> -				};
> -			};
> -		};
> -	};
> -};
>  #endif
> diff --git a/doc/board/ti/am62ax_sk.rst b/doc/board/ti/am62ax_sk.rst
> index 43a1db688738d5c08dacdb698f4ac78d919c23ed..4dd998bcce95068f5d1b260a088fdeb8c3422f41 100644
> --- a/doc/board/ti/am62ax_sk.rst
> +++ b/doc/board/ti/am62ax_sk.rst
> @@ -121,12 +121,7 @@ Set the variables corresponding to this platform:
>  Target Images
>  --------------
>  In order to boot we need tiboot3.bin, tispl.bin and u-boot.img.  Each SoC
> -variant (GP, HS-FS, HS-SE) requires a different source for these files.
> -
> - - GP
> -
> -        * tiboot3-am62ax-gp-evm.bin from step 3.1
> -        * tispl.bin_unsigned, u-boot.img_unsigned from step 3.2
> +variant (HS-FS, HS-SE) requires a different source for these files.
>  
>   - HS-FS
>  
> 

-- 
Thanking You
Neha Malcom Francis