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.
