[PATCH] lib: fdt: Fix fdtdec_setup_mem..() conversion to livetree API
Tom Rini
trini at konsulko.com
Tue Sep 15 16:01:48 CEST 2020
On Tue, Sep 15, 2020 at 03:58:43PM +0200, Michal Simek wrote:
>
>
> On 12. 09. 20 12:42, Marek Vasut wrote:
> > Repair incorrectly negated condition in the original patch which broke
> > DT memory node parsing on everything which has more than one DT memory
> > node, e.g. R-Car3.
> >
> > In case multiple valid memory nodes are present in the DT, the original
> > patch would complete parsing cycle for the first memory node, then move
> > on to the next one, identify it as a valid, and end the parsing. The fix
> > is to invert the condition, to make the code behave as it did before the
> > livetree conversion, so it would continue parsing the subsequent memory
> > nodes as well.
> >
> > Fixes: c2f0950c33 ("lib: fdt: Convert fdtdes_setup_mem..() to livetree API")
> > Signed-off-by: Marek Vasut <marek.vasut+renesas at gmail.com>
> > Cc: Michal Simek <michal.simek at xilinx.com>
> > Cc: Simon Glass <sjg at chromium.org>
> > Cc: Tom Rini <trini at konsulko.com>
> > ---
> > lib/fdtdec.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/lib/fdtdec.c b/lib/fdtdec.c
> > index d3b22ec323..5f41f58a63 100644
> > --- a/lib/fdtdec.c
> > +++ b/lib/fdtdec.c
> > @@ -1101,7 +1101,7 @@ int fdtdec_setup_memory_banksize(void)
> > if (ret < 0) {
> > reg = 0;
> > mem = get_next_memory_node(mem);
> > - if (ofnode_valid(mem))
> > + if (!ofnode_valid(mem))
> > break;
> >
> > ret = ofnode_read_resource(mem, reg++, &res);
> > @@ -1146,7 +1146,7 @@ int fdtdec_setup_mem_size_base_lowest(void)
> > if (ret < 0) {
> > reg = 0;
> > mem = get_next_memory_node(mem);
> > - if (ofnode_valid(mem))
> > + if (!ofnode_valid(mem))
> > break;
> >
> > ret = ofnode_read_resource(mem, reg++, &res);
> >
>
> We are not using multiple memory node. We use multiple tuples inside one
> memory node that's why it didn't come up. Good catch.
>
> Reviewed-by: Michal Simek <michal.simek at xilinx.com>
>
> Tom: Can you please take it to your tree?
Will do, thanks!
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20200915/7933c93e/attachment.sig>
More information about the U-Boot
mailing list