On 11/29/2017 03:58 AM, Tom Rini wrote:
On Wed, Nov 29, 2017 at 03:45:45AM +0100, Marek Vasut wrote:
It is legal to have multiple /memory nodes in a device tree . Currently, fdtdec_setup_memory_size() only supports parsing the first node . This patch extends the function such that if a particular /memory node does no longer have further "reg" entries and CONFIG_NR_DRAM_BANKS still allows for more DRAM banks, the code moves on to the next memory node and checks it's "reg"s. This makes it possible to handle both systems with single memory node with multiple entries and systems with multiple memory nodes with single entry.
Signed-off-by: Marek Vasut marek.vasut+renesas@gmail.com Cc: Tom Rini trini@konsulko.com Cc: Simon Glass sjg@chromium.org
https://www.devicetree.org/downloads/devicetree-specification-v0.1-20160524.... says that multiple nodes can be used, so this is the correct fix here.
I am not a big fan of this kind of practice, but that's how our DTs are, so it cannot be helped and we have to support this :-(
Thanks!
Reviewed-by: Tom Rini trini@konsulko.com
Thanks.