Re: [U-Boot] [PATCH 5/5] boot: fdt: fixup the memory dt nodes falcon boot

Hi Lukasz

...

/*

• Weak default function for board specific cleanup/preparation

before

• Linux boot. Some boards/platforms might not need it, so just

provide @@ -323,6 +359,9 @@ void board_init_r(gd_t *dummy1, ulong dummy2) debug(">>spl:board_init_r()\n"); gd->bd = &bdata; +#ifdef CONFIG_SPL_OS_BOOT

• dram_init_banksize();

What is the purpose of this function? In this point the SDRAM should be already configured.

Lokesh can comment more on this.

AFAIK, the dram_init_bankszie() is platform specific function to populate the dram banks (start address & size). For dra7xx its defined in board/ti/dra7xx/evm.c.

In patch 2/5 you already configured TI's specific "board_init_f" to add some memory related information to gd struct.

I think board_init_f() is for u-boot right ?

Regards Ravi