Messages in this thread | | | Date | Mon, 3 Sep 2012 11:09:01 +0800 | From | Shawn Guo <> | Subject | Re: [PATCH v4 1/7] mfd: add syscon driver based on regmap |
| |
On Mon, Sep 03, 2012 at 10:31:03AM +0800, Dong Aisheng wrote: > > > > I think of_node_put should be moved out from here and put into > > syscon_node_to_regmap and syscon_regmap_lookup_by_compatible. > > > I guess no, if you want to move of_node_put into syscon_regmap_lookup_by_phandle, > then syscon_regmap_lookup_by_phandle has the same issue.
I guess not. syscon_regmap_lookup_by_phandle itself calls of_parse_phandle, and that's where the refcount gets incremented, so syscon_regmap_lookup_by_phandle should be responsible for calling of_node_put to get the refcount decremented.
-- Regards, Shawn
| |