Hi Bin,
On 20.08.20 11:22, Bin Meng wrote:
Hi Stefan,
On Thu, Aug 20, 2020 at 12:45 PM Stefan Roese <s...@denx.de> wrote:
Remove the bi_memstart / bi_memsize assignment in setup_bdinfo() and
make sure, that bd_dram[] is always configured in the weak default
implementation of dram_init_banksize(), when CONFIG_SYS_SDRAM_BASE is
not set.
Signed-off-by: Stefan Roese <s...@denx.de>
Reviewed-by: Ovidiu Panait <ovidiu.pan...@windriver.com>
---
Changes in v7:
- Add Reviewe-by tag from Ovidiu
Changes in v6:
- Simplify dram_init_banksize() even more as suggested by Daniel
Changes in v4:
- New patch
common/board_f.c | 7 +------
1 file changed, 1 insertion(+), 6 deletions(-)
diff --git a/common/board_f.c b/common/board_f.c
index d1f0f9af08..8a43df97e4 100644
--- a/common/board_f.c
+++ b/common/board_f.c
@@ -237,10 +237,8 @@ static int show_dram_config(void)
__weak int dram_init_banksize(void)
{
-#if defined(CONFIG_SYS_SDRAM_BASE)
- gd->bd->bi_dram[0].start = CONFIG_SYS_SDRAM_BASE;
After this change, I guess CONFIG_SYS_SDRAM_BASE can be dropped in
board's config.h now?
No. Its still referenced in many other places, e.g.:
static int setup_dest_addr(void)
{
...
#ifdef CONFIG_SYS_SDRAM_BASE
gd->ram_base = CONFIG_SYS_SDRAM_BASE;
#endif
Please see Daniel's explanation for this change here:
https://www.mail-archive.com/u-boot@lists.denx.de/msg380426.html
Thanks,
Stefan
