[U-Boot] [PATCH] armv8: ls1043a: Extend the size for SPL

[Prabhakar Kushwaha]

Hi Gong,

You have increased SPL size to 124KB. It is very high.
As per my understanding 40K has been reserved for headers to be used during secure boot. 

Have you consider the secure boot case?

Regards,
Prabhakar

> -----Original Message-----
> From: Q.Y. Gong
> Sent: Thursday, September 08, 2016 8:04 AM
> To: york sun <york.sun at nxp.com>; u-boot at lists.denx.de
> Cc: Prabhakar Kushwaha <prabhakar.kushwaha at nxp.com>; Vincent Hu
> <mingkai.hu at nxp.com>
> Subject: RE: [PATCH] armv8: ls1043a: Extend the size for SPL
> 
> Hi York,
> 
> > -----Original Message-----
> > From: york sun
> > Sent: Wednesday, September 07, 2016 11:36 PM
> > To: Q.Y. Gong <qianyu.gong at nxp.com>; u-boot at lists.denx.de
> > Cc: Prabhakar Kushwaha <prabhakar.kushwaha at nxp.com>; Vincent Hu
> > <mingkai.hu at nxp.com>
> > Subject: Re: [PATCH] armv8: ls1043a: Extend the size for SPL
> >
> > On 09/07/2016 03:33 AM, Gong Qianyu wrote:
> > > The SPL images are growing much bigger especially when DEBUG is ON.
> > > So need to fix the values for them.
> > >
> > > Signed-off-by: Gong Qianyu <Qianyu.Gong at nxp.com>
> > > ---
> > >  include/configs/ls1043a_common.h | 25 ++++++++++++++++---------
> > >  1 file changed, 16 insertions(+), 9 deletions(-)
> > >
> > > diff --git a/include/configs/ls1043a_common.h
> > > b/include/configs/ls1043a_common.h
> > > index e55fcb2..fa20e6d 100644
> > > --- a/include/configs/ls1043a_common.h
> > > +++ b/include/configs/ls1043a_common.h
> > > @@ -69,16 +69,22 @@
> > >  #define CONFIG_SPL_SERIAL_SUPPORT
> > >  #define CONFIG_SPL_DRIVERS_MISC_SUPPORT  #define
> > > CONFIG_SPL_MMC_SUPPORT
> > > -#define CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR
> > 	0xf0
> > > +#define CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_SECTOR
> > 	0x110
> > >  #define CONFIG_SYS_U_BOOT_MAX_SIZE_SECTORS		0x500
> > >
> > >  #define CONFIG_SPL_TEXT_BASE		0x10000000
> > > -#define CONFIG_SPL_MAX_SIZE		0x1d000
> > > -#define CONFIG_SPL_STACK		0x1001e000
> > > -#define CONFIG_SPL_PAD_TO		0x1d000
> > > +/*
> > > + * CONFIG_SPL_MAX_SIZE is limited by OCRAM memory size(128 KiB) and
> > > + * a reserved stack size(4 KiB).
> > > + * So notice that even if DEBUG is ON, the SPL
> > > +image(spl/u-boot-spl.bin)
> > > + * should not be > 124 KiB.
> > > + */
> >
> > Qianyu,
> >
> > It is good to see comment here. However, I am concerned about your
> calculation.
> > Beside the image and stack, the early MMU tables are at the beginning of
> OCRAM.
> > Did you count them?
> >
> > York
> 
> No. We don't set up MMU tables in SPL stage on LS1043A/LS1046A.
> Seems so far only LS2080A has done that in SPL.
> 
> 
> Regards,
> Qianyu