On 08/29/2017 12:45 PM, tien.fong.c...@intel.com wrote: > From: Tien Fong Chee <tien.fong.c...@intel.com> > > Add function for both multiple DRAM bank and single DRAM bank size > initialization. This common functionality could be used by every single > SOCFPGA board. > > Signed-off-by: Tien Fong Chee <tien.fong.c...@intel.com> > --- > arch/arm/mach-socfpga/board.c | 12 ++++++++++++ > include/configs/socfpga_common.h | 1 + > 2 files changed, 13 insertions(+), 0 deletions(-) > > diff --git a/arch/arm/mach-socfpga/board.c b/arch/arm/mach-socfpga/board.c > index a41d089..3f2e30a 100644 > --- a/arch/arm/mach-socfpga/board.c > +++ b/arch/arm/mach-socfpga/board.c > @@ -29,6 +29,18 @@ int board_init(void) > return 0; > } > > +int dram_init_banksize(void) > +{ > +#if !defined(CONFIG_SYS_SDRAM_BASE) && !defined(CONFIG_SYS_SDRAM_SIZE) > + fdtdec_setup_memory_banksize();
This should work on al boards, so why do you need this macro ? Also, this should go into arch/arm/ , not board/ > +#else > + gd->bd->bi_dram[0].start = CONFIG_SYS_SDRAM_BASE; > + gd->bd->bi_dram[0].size = CONFIG_SYS_SDRAM_SIZE; > +#endif > + > + return 0; > +} > + > #ifdef CONFIG_USB_GADGET > struct dwc2_plat_otg_data socfpga_otg_data = { > .usb_gusbcfg = 0x1417, > diff --git a/include/configs/socfpga_common.h > b/include/configs/socfpga_common.h > index c15d244..05b03bd 100644 > --- a/include/configs/socfpga_common.h > +++ b/include/configs/socfpga_common.h > @@ -47,6 +47,7 @@ > (CONFIG_SYS_INIT_RAM_ADDR + CONFIG_SYS_INIT_SP_OFFSET) > > #define CONFIG_SYS_SDRAM_BASE PHYS_SDRAM_1 > +#define CONFIG_SYS_SDRAM_SIZE PHYS_SDRAM_1_SIZE > #ifdef CONFIG_SOCFPGA_VIRTUAL_TARGET > #define CONFIG_SYS_TEXT_BASE 0x08000040 > #else > -- Best regards, Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot