On Tue, Sep 3, 2019 at 12:09 AM Peng Fan <peng....@nxp.com> wrote: > > > Subject: [PATCH] apalis_imx6: allocate specific region of memory to OP-TEE > > > > OP-TEE uses the memory region defined by the maximum DRAM address > > minus CONFIG_OPTEE_TZDRAM_SIZE, so subtract > > CONFIG_OPTEE_TZDRAM_SIZE from the available DRAM size to avoid > > conflicts. > > > > Signed-off-by: Ricardo Salveti <rica...@foundries.io> > > --- > > board/toradex/apalis_imx6/apalis_imx6.c | 5 +++++ > > 1 file changed, 5 insertions(+) > > > > diff --git a/board/toradex/apalis_imx6/apalis_imx6.c > > b/board/toradex/apalis_imx6/apalis_imx6.c > > index 6421a22c25..fa7fcc8d46 100644 > > --- a/board/toradex/apalis_imx6/apalis_imx6.c > > +++ b/board/toradex/apalis_imx6/apalis_imx6.c > > @@ -75,6 +75,11 @@ int dram_init(void) > > gd->ram_size = get_ram_size((void *)CONFIG_SYS_SDRAM_BASE, > > (ulong)imx_ddr_size()); > > > > + /* Subtract the defined OPTEE runtime firmware length */ #ifdef > > +CONFIG_OPTEE_TZDRAM_SIZE > > + gd->ram_size -= CONFIG_OPTEE_TZDRAM_SIZE; #endif > > + > > Has OPTEE been enabled? I not see that in defconfig.
Not yet enable by default, this is just to make it compatible with OP-TEE. Should we have it enabled by default at apalis_imx6_defconfig? I could also send another patch to add a new config that has secure boot and OP-TEE enabled by default, as done with a few other imx targets. Thanks, -- Ricardo Salveti _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot