From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chee, Tien Fong Date: Thu, 7 Mar 2019 08:32:31 +0000 Subject: [U-Boot] [PATCH v11 9/9] ARM: socfpga: Increase Malloc pool size to support FAT filesystem in SPL In-Reply-To: References: <20190305162310.1396-1-tien.fong.chee@intel.com> <20190305162310.1396-10-tien.fong.chee@intel.com> <1551942475.9744.3.camel@intel.com> Message-ID: <1551947550.9744.40.camel@intel.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: u-boot@lists.denx.de On Thu, 2019-03-07 at 09:10 +0100, Simon Goldschmidt wrote: > On Thu, Mar 7, 2019 at 8:08 AM Chee, Tien Fong com> wrote: > > > > > > On Tue, 2019-03-05 at 21:05 +0100, Simon Goldschmidt wrote: > > > > > > Am 05.03.2019 um 17:23 schrieb tien.fong.chee at intel.com: > > > > > > > > > > > > From: Tien Fong Chee > > > > > > > > After some series of patches to maximise reusable of memory > > > > pool, > > > > here come > > > > to result of reasonable size required for whole SDMMC boot > > > > working > > > > on A10 > > > > SoCDK. Size required come from default max cluster(0x10000) + > > > > others(0x2000) + additional memory for headroom(0x3000). > > > > > > > > Signed-off-by: Tien Fong Chee > > > > > > > > --- > > > > > > > > changes for v7 > > > > - Added 0x3000 for memory headroom. > > > > --- > > > >   include/configs/socfpga_common.h | 4 ++-- > > > >   1 file changed, 2 insertions(+), 2 deletions(-) > > > > > > > > diff --git a/include/configs/socfpga_common.h > > > > b/include/configs/socfpga_common.h > > > > index 4551cb29bc..548b458e78 100644 > > > > --- a/include/configs/socfpga_common.h > > > > +++ b/include/configs/socfpga_common.h > > > > @@ -1,6 +1,6 @@ > > > >   /* SPDX-License-Identifier: GPL-2.0+ */ > > > >   /* > > > > - * Copyright (C) 2012 Altera Corporation > > > > + * Copyright (C) 2012-2019 Altera Corporation > > > >    */ > > > >   #ifndef __CONFIG_SOCFPGA_COMMON_H__ > > > >   #define __CONFIG_SOCFPGA_COMMON_H__ > > > > @@ -258,7 +258,7 @@ unsigned int > > > > cm_get_qspi_controller_clk_hz(void); > > > >   #if defined(CONFIG_TARGET_SOCFPGA_ARRIA10) > > > >   /* SPL memory allocation configuration, this is for FAT > > > > implementation */ > > > >   #ifndef CONFIG_SYS_SPL_MALLOC_START > > > > -#define CONFIG_SYS_SPL_MALLOC_SIZE 0x00010000 > > > > +#define CONFIG_SYS_SPL_MALLOC_SIZE 0x00015000 > > > This will clash with my series here: > > > https://patchwork.ozlabs.org/patch/1051451/ > > > > > > Any chance you could test that on A10? I only have a cyclone 5. > > > > > I have tested the series, and checking the failure from the codes, > > SPL_SYS_MALLOC_SIMPLE and SPL_SYS_MALLOC_F_LEN are not working in > > A10, > > because FPGA loading and distro boot from FAT FS require full > > malloc > > mechanism. > OK, thanks for testing. Seems like I was missing that the difference > between > CONFIG_SYS_SPL_MALLOC_START/_SIZE and SYS_MALLOC_F_LEN is > full malloc vs. simple malloc. > > I'll work on that. Great!! Thanks a lot. TF. > > > > > > > Thanks, > > TF. > > > > > > > > > > > > > > > > > > > > >   #define > > > > CONFIG_SYS_SPL_MALLOC_START       (CONFIG_SYS_INIT_RAM_S > > > > IZE - \ > > > >                                      CONFIG_SYS_SPL_MALLOC_SI > > > > ZE + \ > > > >                                      CONFIG_SYS_INIT_RAM_ADDR > > > > ) > > > >