[PATCH] arm: socfpga: arria10: Add save_boot_params()

Tan, Ley Foon ley.foon.tan at intel.com
Fri Mar 6 01:33:11 CET 2020



> -----Original Message-----
> From: Marek Vasut <marex at denx.de>
> Sent: Thursday, March 5, 2020 8:23 PM
> To: Ley Foon Tan <lftan.linux at gmail.com>
> Cc: Tan, Ley Foon <ley.foon.tan at intel.com>; u-boot at lists.denx.de; Simon
> Goldschmidt <simon.k.r.goldschmidt at gmail.com>; See, Chin Liang
> <chin.liang.see at intel.com>; Chee, Tien Fong <tien.fong.chee at intel.com>
> Subject: Re: [PATCH] arm: socfpga: arria10: Add save_boot_params()
> 
> On 3/5/20 9:56 AM, Ley Foon Tan wrote:
> Hi,
> 
> [...]
> 
> >>>>>> Aren't those already defined in include/configs/socfpga_common.h ?
> >>>>> socfpga_common.h have this:
> >>>>> #define CONFIG_SYS_INIT_RAM_SIZE        (0x40000 -
> >>>> CONFIG_SYS_SPL_MALLOC_SIZE)
> >>>>>
> >>>>> But, we can't use it here.
> >>>>
> >>>> Why ?
> >>> CONFIG_SYS_INIT_RAM_SIZE is minus with
> CONFIG_SYS_SPL_MALLOC_SIZE, not 256KB (0x40000).
> >>
> >> Then define the init ram size macro somewhere in
> >> arch/arm/mach-socfpga/include and use it in both the configs/ and
> >> your code ? :)
> > Look at existing header files under arch/arm/mach-socfpga/include/,
> > can add to base_addr_a10.h.
> 
> Isn't that base address common to A10 and CV/AV ?
No, CV/AV uses base_addr_ac5.h.

Regards
Ley Foon


More information about the U-Boot mailing list