On June 18, 2024 thus sayeth Dhruva Gole:
> Add support for packaging the TIFS Stub as it's required for basic Low
> Power Modes like Deep Sleep.
> 
> Acked-by: Neha Malcom Francis <n-fran...@ti.com>
> Signed-off-by: Dhruva Gole <d-g...@ti.com>
> ---
> 
> No changes from v1, just picked Neha's ack and rebased on master again.
> Link to v1:
> https://lore.kernel.org/u-boot/20240612062351.3690091-1-d-g...@ti.com/
> 
>  arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi | 33 +++++++++++++++++++-
>  1 file changed, 32 insertions(+), 1 deletion(-)
> 
> diff --git a/arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi 
> b/arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi
> index fb2032068d1c..5e2248a4a668 100644
> --- a/arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi
> +++ b/arch/arm/dts/k3-am625-beagleplay-u-boot.dtsi
> @@ -69,6 +69,23 @@
>               };
>       };
>  
> +     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-am62x-gp.bin";
> +                     type = "blob-ext";
> +                     optional;
> +             };
> +     };
> +
>       ti-spl_unsigned {
>               filename = "tispl.bin_unsigned";
>               pad-byte = <0xff>;
> @@ -105,6 +122,19 @@
>                                       };
>                               };
>  
> +                             tifsstub-gp {
> +                                     description = "tifsstub";
> +                                     type = "firmware";
> +                                     arch = "arm32";
> +                                     compression = "none";
> +                                     os = "tifsstub-gp";
> +                                     load = <0x9dc00000>;
> +                                     entry = <0x9dc00000>;

Is this stub position independent? Or is this address compiled in at 
build time? We have some variants of 62x that don't have these addresses 
backed by DRAM.

~Bryan

> +                                     blob-ext {
> +                                             filename = "tifsstub.bin_gp";
> +                                     };
> +                             };
> +
>                               dm {
>                                       description = "DM binary";
>                                       type = "firmware";
> @@ -148,7 +178,8 @@
>                               conf-0 {
>                                       description = "k3-am625-beagleplay";
>                                       firmware = "atf";
> -                                     loadables = "tee", "dm", "spl";
> +                                     loadables = "tee", "dm", "spl",
> +                                     "tifsstub-gp";
>                                       fdt = "fdt-0";
>                               };
>                       };
> 
> base-commit: 16324b43db3f2b4fbbc3b701893fcfc4104f33fb
> -- 
> 2.34.1
> 

Reply via email to