Re: [PATCH 2/3] imx8m: Automatically add the optee firmware node to the FDT

Hi Alban

On Mon, May 16, 2022 at 10:23 AM Alban Bedel alban.bedel@aerq.com wrote:

If optee is running add the firmware node to the FDT to allow the kernel to use a more generic device tree.

Signed-off-by: Alban Bedel alban.bedel@aerq.com

arch/arm/mach-imx/imx8m/soc.c | 9 +++++++++ 1 file changed, 9 insertions(+)

diff --git a/arch/arm/mach-imx/imx8m/soc.c b/arch/arm/mach-imx/imx8m/soc.c index 8e23e6da326f..2a78cb6a0952 100644 --- a/arch/arm/mach-imx/imx8m/soc.c +++ b/arch/arm/mach-imx/imx8m/soc.c @@ -30,6 +30,7 @@ #include <fsl_wdog.h> #include <imx_sip.h> #include <linux/bitops.h> +#include <tee/optee.h>

DECLARE_GLOBAL_DATA_PTR;

@@ -1347,6 +1348,14 @@ usb_modify_speed: #endif

    cleanup_nodes_for_efi(blob);

•   if (rom_pointer[1]) {
  

•           int err = optee_add_firmware_node(blob, "linaro,optee-tz",
  

•                                             "smc");
  

•           if (err)
  

•                   return err;
  

•   }
  

•   return 0;
  

}

#endif

Adding the node is not sufficient. I think that we need reserved memory node on top of it. The nice scenario will be only to define the size of such area and use always the last part of the memory to reserve it. Do you know if we have some api to pass information to the atf and optee?

Michael

2.34.1