Hi Prabhakar, Does secure boot enable SPL on LS1043A? I only see secure boot for NOR boot.
As there won't be enough space for SPL soon, I don't think it could support secure boot and SPL at the same time. Regards, Qianyu > -----Original Message----- > From: Prabhakar Kushwaha > Sent: Thursday, September 08, 2016 2:35 PM > To: Q.Y. Gong <qianyu.g...@nxp.com>; york sun <york....@nxp.com>; Ruchika > Gupta <ruchika.gu...@nxp.com>; u-boot@lists.denx.de > Cc: Vincent Hu <mingkai...@nxp.com> > Subject: RE: [PATCH] armv8: ls1043a: Extend the size for SPL > > 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....@nxp.com>; u-boot@lists.denx.de > > Cc: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>; Vincent Hu > > <mingkai...@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.g...@nxp.com>; u-boot@lists.denx.de > > > Cc: Prabhakar Kushwaha <prabhakar.kushw...@nxp.com>; Vincent Hu > > > <mingkai...@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.g...@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 _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot