[U-Boot] [PATCH 1/2] armv8: layerscape: Allocate 66 MB DDR for secure memory

[Sumit Garg]

> -----Original Message-----
> From: York Sun
> Sent: Tuesday, October 24, 2017 2:36 AM
> To: Sumit Garg <sumit.garg at nxp.com>; u-boot at lists.denx.de
> Cc: Ruchika Gupta <ruchika.gupta at nxp.com>; Prabhakar Kushwaha
> <prabhakar.kushwaha at nxp.com>; Z.q. Hou <zhiqiang.hou at nxp.com>; Pankaj
> Gupta <pankaj.gupta at nxp.com>; Arun Pathak <arun.pathak at nxp.com>; Sahil
> Malhotra <sahil.malhotra at nxp.com>
> Subject: Re: [PATCH 1/2] armv8: layerscape: Allocate 66 MB DDR for secure
> memory
> 
> On 09/01/2017 01:24 AM, Sumit Garg wrote:
> > Change DDR allocated for secure memory from 2 MB to 66 MB. This
> > additional
> > 64 MB secure memory is required for trusted OS running in Trusted
> > Execution Environment using ARMv8 TrustZone.
> >
> > Signed-off-by: Sumit Garg <sumit.garg at nxp.com>
> > ---
> >  arch/arm/include/asm/arch-fsl-layerscape/config.h | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/arch/arm/include/asm/arch-fsl-layerscape/config.h
> > b/arch/arm/include/asm/arch-fsl-layerscape/config.h
> > index a7098be..95e2791 100644
> > --- a/arch/arm/include/asm/arch-fsl-layerscape/config.h
> > +++ b/arch/arm/include/asm/arch-fsl-layerscape/config.h
> > @@ -16,7 +16,7 @@
> >   * Reserve secure memory
> >   * To be aligned with MMU block size
> >   */
> > -#define CONFIG_SYS_MEM_RESERVE_SECURE	(2048 * 1024)	/* 2MB
> */
> > +#define CONFIG_SYS_MEM_RESERVE_SECURE	(66 * 1024 * 1024)
> 	/* 66MB */
> >  #define SPL_TLB_SETBACK	0x1000000	/* 16MB under effective
> memory top */
> >
> >  #ifdef CONFIG_ARCH_LS2080A
> >
> 
> Sumit,
> 
> I believe this patch is no longer needed after the merge of
> 
> commit 710d0cd79ede129d5203325c2a188c6f93c61ca3
> Author: Sumit Garg <sumit.garg at nxp.com>
> Date:   Wed Oct 4 03:20:49 2017 +0530
> 
>     armv8: fsl-layerscape: Allocate Secure memory from first ddr region
> 
> York
 
No, this patch is needed. The patch you mentioned is just to allocate Secure memory
from first ddr region. But this patch is meant to increase Secure memory allocated from
2MB -> 66MB.

Sumit