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