fdt: fix get_next_memory_node()
authorMarek Vasut <marek.vasut@gmail.com>
Sun, 9 Sep 2018 14:30:11 +0000 (16:30 +0200)
committerTom Rini <trini@konsulko.com>
Mon, 10 Sep 2018 13:05:41 +0000 (09:05 -0400)
commitf850371997fb3688885a814902fed07d22f113ef
tree33104232a8e4f5298a940fb265ea5f1f27374f49
parent792b204798453d96b00e0817e8472c19455e92a2
fdt: fix get_next_memory_node()

The get_next_memory_node() always sets mem to -1 , which is incorrect,
because then every iteration of memory bank parsing will start from the
first memory bank instead of the previous one.

On systems with 1 memory bank defined in DT and CONFIG_NR_DRAM_BANKS=4 ,
like ie. r8a77965-salvator-x , this will result in U-Boot incorrectly
reporting four identical memory banks with the same memory configuration.

Fix this by setting mem to startoffset value, which restores the behavior
before the fixed patch was applied.

Signed-off-by: Marek Vasut <marek.vasut+renesas@gmail.com>
Cc: Jens Wiklander <jens.wiklander@linaro.org>
Cc: Simon Glass <sjg@chromium.org>
Cc: Tom Rini <trini@konsulko.com>
Fixes: 452bc121027d ("fdt: fix fdtdec_setup_memory_banksize()")
Tested-by: Michal Simek <michal.simek@xilinx.com> [on ZynqMP}
lib/fdtdec.c