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+rene...@gmail.com> >> Cc: Tom Rini <tr...@konsulko.com> >> Cc: Simon Glass <s...@chromium.org> > > https://www.devicetree.org/downloads/devicetree-specification-v0.1-20160524.pdf > 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 <tr...@konsulko.com> Thanks. -- Best regards, Marek Vasut _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
