Re: [PATCH] imx93_frdm: Add OP-TEE device tree node

["Mathieu Dubois-Briand" <mathieu.dubois-briand@bootlin.com>]

Hi,

On Mon Jul 6, 2026 at 4:14 PM CEST, Peter Robinson wrote:
> On Mon, 6 Jul 2026 at 13:34, Mathieu Dubois-Briand
> <mathieu.dubois-briand@bootlin.com> wrote:
>>
>> Add missing firmware/optee node, so is can be both used and copied into
>> Linux device tree by optee_copy_firmware_node(). This is consistent with
>> other i.MX93 boards.
>
> Shouldn't this be going to the upstream Linux DT at which point get it
> when we resync?

I don't think it should. Or maybe it should, but then we are doing wrong
things for other boards, particularly for the i.MX family.

We have this node in the -u-boot.dtsi file for various other boards:

git grep "linaro,optee-tz" -- '*-u-boot.dtsi'
arch/arm/dts/imx8mm-u-boot.dtsi:                        compatible = "linaro,optee-tz";
arch/arm/dts/imx8mn-u-boot.dtsi:                        compatible = "linaro,optee-tz";
arch/arm/dts/imx8mp-debix-model-a-u-boot.dtsi:                  compatible = "linaro,optee-tz";
arch/arm/dts/imx8mp-navqp-u-boot.dtsi:                  compatible = "linaro,optee-tz";
arch/arm/dts/imx8mp-u-boot.dtsi:                        compatible = "linaro,optee-tz";
arch/arm/dts/imx91-11x11-evk-u-boot.dtsi:                       compatible = "linaro,optee-tz";
arch/arm/dts/imx91-11x11-frdm-u-boot.dtsi:                      compatible = "linaro,optee-tz";
arch/arm/dts/imx91-93-phyboard-segin-common-u-boot.dtsi:                        compatible = "linaro,optee-tz";
arch/arm/dts/imx93-11x11-evk-u-boot.dtsi:                       compatible = "linaro,optee-tz";
arch/arm/dts/imx93-11x11-frdm-u-boot.dtsi:                      compatible = "linaro,optee-tz";
arch/arm/dts/imx93-9x9-qsb-u-boot.dtsi:                 compatible = "linaro,optee-tz";
arch/arm/dts/imx93-kontron-bl-osm-s-u-boot.dtsi:                        compatible = "linaro,optee-tz";
arch/arm/dts/imx93-var-som-symphony-u-boot.dtsi:                        compatible = "linaro,optee-tz";
arch/arm/dts/stm32mp157a-dk1-u-boot.dtsi:                       compatible = "linaro,optee-tz";
arch/arm/dts/stm32mp157c-ed1-u-boot.dtsi:                       compatible = "linaro,optee-tz";

Thanks,
Mathieu

-- 
Mathieu Dubois-Briand, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com