On Mon, 12 Oct 2015, Krzysztof Kozlowski wrote: > 2015-10-10 21:30 GMT+09:00 Julia Lawall <Julia.Lawall@xxxxxxx>: > > > > for_each_child_of_node performs an of_node_get on each iteration, so > > a break out of the loop requires an of_node_put. > > > > The semantic patch that fixes this problem is as follows > > (http://coccinelle.lip6.fr): > > > > // <smpl> > > @@ > > expression root,e; > > local idexpression child; > > @@ > > > > for_each_child_of_node(root, child) { > > ... when != of_node_put(child) > > when != e = child > > ( > > return child; > > | > > + of_node_put(child); > > ? return ...; > > ) > > ... > > } > > // </smpl> > > > > Signed-off-by: Julia Lawall <Julia.Lawall@xxxxxxx> > > > > --- > > drivers/regulator/of_regulator.c | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/drivers/regulator/of_regulator.c b/drivers/regulator/of_regulator.c > > index 499e437..f9d77b4 100644 > > --- a/drivers/regulator/of_regulator.c > > +++ b/drivers/regulator/of_regulator.c > > @@ -274,6 +274,7 @@ int of_regulator_match(struct device *dev, struct device_node *node, > > dev_err(dev, > > "failed to parse DT for regulator %s\n", > > child->name); > > + of_node_put(child); > > This looks good. > > > return -EINVAL; > > } > > match->of_node = of_node_get(child); > > But what about 'break' few lines below? The reference from last > of_get_next_child() should be also dropped because... or we should > remove this of_node_get() call. > > How about fixing also usage of for_each_available_child_of_node() in > regulator_of_get_init_data()? I'll check on all of this and resend. Thanks for the feedback. julia -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
