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 > <[email protected]> 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
