Re: [U-Boot] [PATCH 03/10] ddr: altera: s10: Add multiple memory banks support

On 3/12/19 9:31 AM, Ley Foon Tan wrote:

Setup bank start address and size based on total SDRAM memory size calculated from hardware. Update sdram_size_check() to support multiple banks.

Stratix10 supports up to 2 memory banks.

Bank 0: Address 0, size 2GB Bank 1: Address 0x100000000, size 124GB

Signed-off-by: Dalon Westergreen dalon.westergreen@intel.com Signed-off-by: Ley Foon Tan ley.foon.tan@intel.com

Shouldn't Quartus generate some sort of DT which gives you all the DRAM layout information ? Then you won't need any of this CONFIG_NR_DRAM_BANKS stuff, but you would be able to extract such info from DT and apply the sanity check only on DRAM you know exists.