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

Marek Vasut marex at denx.de
Fri Mar 6 01:43:01 CET 2020


On 3/6/20 1:33 AM, Tan, Ley Foon wrote:
[...]
>>>>>>>> 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.

Then it might make sense to define the macro in both.


More information about the U-Boot mailing list