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

Tan, Ley Foon ley.foon.tan at intel.com
Mon Mar 2 08:20:33 CET 2020



> -----Original Message-----
> From: Marek Vasut <marex at denx.de>
> Sent: Sunday, March 1, 2020 11:12 PM
> To: Tan, Ley Foon <ley.foon.tan at intel.com>; u-boot at lists.denx.de
> Cc: Simon Goldschmidt <simon.k.r.goldschmidt at gmail.com>; Ley Foon Tan
> <lftan.linux 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 2/26/20 8:01 PM, Ley Foon Tan wrote:
> [...]
> > +#define BOOTROM_SHARED_MEM_ADDR
> 	(CONFIG_SYS_INIT_RAM_ADDR + 0x40000 \
> > +					 - 0x800)
> > +#define RST_STATUS_SHARED_ADDR
> 	(BOOTROM_SHARED_MEM_ADDR + 0x438)
> 
> Are all these magic values needed or is there some more descriptive macro
> name available for them ?
0x40000 is onchip ram size and 0x800 is 2KB size.
I can convert these to 2 macos.
> 
> > +u32 rst_mgr_status __section(.data);
> 
> static u32 ... ?
Other files might want to use this.

Thanks.

Regards
Ley Foon


More information about the U-Boot mailing list