Hi Rob, > On Feb 12, 2016, at 01:18 , Rob Herring <robh+dt@xxxxxxxxxx> wrote: > > On Wed, Feb 3, 2016 at 12:09 PM, Amitoj Kaur Chawla > <amitoj1606@xxxxxxxxx> wrote: >> In __of_find_node_by_full_name, add an of_node_get when detecting the >> desired element, to ensure that it ends up with a reference count that is >> one greater than on entering the function. >> >> Also in __of_find_node_by_full_name, add an of_node_put on breaking >> out of the for_each_child_of_node loop, to ensure that the reference >> count of the returned value is not double incremented. This change >> was made using Coccinelle. > > Pantelis, are you convinced this is correct (albeit somewhat pointless)? > This is good enough. It’s a local method and we won’t ever try to use it outside of the resolver. We need to fix the reference counting properly sometime in the future. > Rob Regards — Pantelis -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html