[PATCH] arm: dts: k3-binman: Make optee optional as requirement

[PATCH] arm: dts: k3-binman: Make optee optional as requirement

[Michael Trimarchi]

Boards can use the ti_spl_template but avoid to define a tee
node to be loaded. This is true form board with 512Mb or less
memory. We can limit this in configuation removing the tee
node

configurations {
	default = "conf-0";

	conf-0 {
		description = "k3-am62_ccm_m3";
		firmware = "atf";
		loadables = "dm", "spl";
		fdt = "fdt-0";
	};
};

Signed-off-by: Michael Trimarchi <michael@amarulasolutions.com>
---
 arch/arm/dts/k3-binman.dtsi | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/arch/arm/dts/k3-binman.dtsi b/arch/arm/dts/k3-binman.dtsi
index 758c8bf6ea..5ef5af315a 100644
--- a/arch/arm/dts/k3-binman.dtsi
+++ b/arch/arm/dts/k3-binman.dtsi
@@ -293,6 +293,7 @@
 						keyfile = "custMpk.pem";
 					};
 					tee: tee-os {
+						optional;
 					};
 				};
 
@@ -360,6 +361,7 @@
 					entry = <0x9e800000>;
 					tee-os {
 						filename = "tee-raw.bin";
+						optional;
 					};
 				};
 
-- 
2.40.1

Re: [PATCH] arm: dts: k3-binman: Make optee optional as requirement

[Neha Malcom Francis]

Hi Michael

+ Vignesh

On 25/02/24 23:08, Michael Trimarchi wrote:
> Boards can use the ti_spl_template but avoid to define a tee
> node to be loaded. This is true form board with 512Mb or less

s/form board/for boards? I am guessing that is what was meant. I am not really 
understanding the commit message... does <512MB memory always mean no using 
OPTEE? (AM62 SIP would be an exception) The commit message is putting out a 
misleading reason why we don't need OPTEE in some cases, I don't think memory is 
the primary reason.

> memory. We can limit this in configuation removing the tee

s/configuation/configuration

> node
> 
> configurations {
> 	default = "conf-0";
> 
> 	conf-0 {
> 		description = "k3-am62_ccm_m3";
> 		firmware = "atf";
> 		loadables = "dm", "spl";
> 		fdt = "fdt-0";
> 	};
> };
> 
> Signed-off-by: Michael Trimarchi <michael@amarulasolutions.com>
> ---
>   arch/arm/dts/k3-binman.dtsi | 2 ++
>   1 file changed, 2 insertions(+)
> 
> diff --git a/arch/arm/dts/k3-binman.dtsi b/arch/arm/dts/k3-binman.dtsi
> index 758c8bf6ea..5ef5af315a 100644
> --- a/arch/arm/dts/k3-binman.dtsi
> +++ b/arch/arm/dts/k3-binman.dtsi
> @@ -293,6 +293,7 @@
>   						keyfile = "custMpk.pem";
>   					};
>   					tee: tee-os {
> +						optional;
>   					};
>   				};
>   
> @@ -360,6 +361,7 @@
>   					entry = <0x9e800000>;
>   					tee-os {
>   						filename = "tee-raw.bin";
> +						optional;
>   					};
>   				};
>   

The patch excluding the commit message LGTM.

-- 
Thanking You
Neha Malcom Francis

Re: [PATCH] arm: dts: k3-binman: Make optee optional as requirement

[Michael Nazzareno Trimarchi]

Hi

On Mon, Feb 26, 2024 at 4:17 AM Neha Malcom Francis <n-francis@ti.com> wrote:
>
> Hi Michael
>
> + Vignesh
>
> On 25/02/24 23:08, Michael Trimarchi wrote:
> > Boards can use the ti_spl_template but avoid to define a tee
> > node to be loaded. This is true form board with 512Mb or less
>
> s/form board/for boards? I am guessing that is what was meant. I am not really
> understanding the commit message... does <512MB memory always mean no using
> OPTEE? (AM62 SIP would be an exception) The commit message is putting out a
> misleading reason why we don't need OPTEE in some cases, I don't think memory is
> the primary reason.
>

I will adjust commit message. In general when you have module with 256
Mb of memory you
try to arrange the available memory to put the essential components.
Anyway having still not
submitted board I found out that is convenient to use the template but
in the same time remove
from my configuration what is not mandatory and let binman to warning
and not to fail

Michael

> > memory. We can limit this in configuation removing the tee
>
> s/configuation/configuration
>
> > node
> >
> > configurations {
> >       default = "conf-0";
> >
> >       conf-0 {
> >               description = "k3-am62_ccm_m3";
> >               firmware = "atf";
> >               loadables = "dm", "spl";
> >               fdt = "fdt-0";
> >       };
> > };
> >
> > Signed-off-by: Michael Trimarchi <michael@amarulasolutions.com>
> > ---
> >   arch/arm/dts/k3-binman.dtsi | 2 ++
> >   1 file changed, 2 insertions(+)
> >
> > diff --git a/arch/arm/dts/k3-binman.dtsi b/arch/arm/dts/k3-binman.dtsi
> > index 758c8bf6ea..5ef5af315a 100644
> > --- a/arch/arm/dts/k3-binman.dtsi
> > +++ b/arch/arm/dts/k3-binman.dtsi
> > @@ -293,6 +293,7 @@
> >                                               keyfile = "custMpk.pem";
> >                                       };
> >                                       tee: tee-os {
> > +                                             optional;
> >                                       };
> >                               };
> >
> > @@ -360,6 +361,7 @@
> >                                       entry = <0x9e800000>;
> >                                       tee-os {
> >                                               filename = "tee-raw.bin";
> > +                                             optional;
> >                                       };
> >                               };
> >
>
> The patch excluding the commit message LGTM.
>
> --
> Thanking You
> Neha Malcom Francis



-- 
Michael Nazzareno Trimarchi
Co-Founder & Chief Executive Officer
M. +39 347 913 2170
michael@amarulasolutions.com
__________________________________

Amarula Solutions BV
Joop Geesinkweg 125, 1114 AB, Amsterdam, NL
T. +31 (0)85 111 9172
info@amarulasolutions.com
www.amarulasolutions.com