Hi Krzysztof

> > +           prev = of_get_child_by_name(parent, "ports");
(snip)
> > +           if (!prev) {
> > +                   prev = of_node_get(parent);
> > +
> > +                   /* check wether it has port node */
> > +                   struct device_node *port __free(device_node) =
> > +                           of_get_child_by_name(prev, "port");
> > +
> > +                   if (!port)
> > +                           prev = NULL;
> 
> It looks like you leak here "prev".

Oops, yes ineed.
Thank you for pointing it, will fix it in v4

Best regards
---
Kuninori Morimoto

Reply via email to