Hi, On Tue, 9 Apr 2013 14:56:09 +0800 <[email protected]> wrote: > > From: Tang Yuantian <[email protected]> > > As the function itself says it is caller's responsibility to call the > of_node_put(). So, remove it on success to keep the reference count > correct. > > Signed-off-by: Tang Yuantian <[email protected]> > --- > drivers/of/base.c | 3 --- > 1 file changed, 3 deletions(-) > > diff --git a/drivers/of/base.c b/drivers/of/base.c > index 321d3ef..e8b4c28 100644 > --- a/drivers/of/base.c > +++ b/drivers/of/base.c > @@ -1168,9 +1168,6 @@ static int __of_parse_phandle_with_args(const struct > device_node *np, > out_args->args[i] = > be32_to_cpup(list++); > } > > - /* Found it! return success */ > - if (node) > - of_node_put(node);
Actually, if out_args is NULL, you should do the of_node_put(node), so probably the correct fix is to add an "else" before the above "if" (and move the comment). -- Cheers, Stephen Rothwell [email protected] http://www.canb.auug.org.au/~sfr/
pgp9hiTWgtSqI.pgp
Description: PGP signature
_______________________________________________ devicetree-discuss mailing list [email protected] https://lists.ozlabs.org/listinfo/devicetree-discuss
