[U-Boot] [PATCH] apalis_imx6: allocate specific region of memory to OP-TEE

[Ricardo Salveti]

On Tue, Sep 3, 2019 at 12:09 AM Peng Fan <peng.fan at 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 <ricardo at 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