On Fri, 20 Jul 2018 at 15:31, Ard Biesheuvel <[email protected]> wrote: > > On 20 July 2018 at 16:12, Sumit Garg <[email protected]> wrote: > > Cc: Ard Biesheuvel <[email protected]> > > Cc: Leif Lindholm <[email protected]> > > Contributed-under: TianoCore Contribution Agreement 1.1 > > Signed-off-by: Sumit Garg <[email protected]> > > --- > > Silicon/Socionext/SynQuacer/DeviceTree/SynQuacer.dtsi | 7 +++++++ > > 1 file changed, 7 insertions(+) > > > > diff --git a/Silicon/Socionext/SynQuacer/DeviceTree/SynQuacer.dtsi > > b/Silicon/Socionext/SynQuacer/DeviceTree/SynQuacer.dtsi > > index 37d642e4b237..d109a5742793 100644 > > --- a/Silicon/Socionext/SynQuacer/DeviceTree/SynQuacer.dtsi > > +++ b/Silicon/Socionext/SynQuacer/DeviceTree/SynQuacer.dtsi > > @@ -574,6 +574,13 @@ > > #address-cells = <1>; > > #size-cells = <0>; > > }; > > + > > + firmware { > > + optee { > > + compatible = "linaro,optee-tz"; > > + method = "smc"; > > + }; > > + }; > > }; > > > > #include "SynQuacerCaches.dtsi" > > Hello Sumit, > > Is it safe to provide this node when optee is not running?
Yes it is safe. If optee is not running then Linux TEE driver exits gracefully with below info: [ 1.976021] optee: probing for conduit method from DT. [ 1.976033] optee: api uid mismatch -Sumit > > If not, this should be made dependent on that, so we should probably > add status = 'disabled', and remove it in PlatformDxe if the DIP > switch is not set, and/or if there are other ways we might know that > optee is not running. _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
