Re: [PATCH 2/3] clk: Add devm_clk_{register,unregister}()

2012-09-22 Thread Stephen Boyd
On 9/21/2012 6:07 PM, Mike Turquette wrote > I'm not taking any more changes for 3.7, so in the interest of me being > lazy can you resend with the fixup? Sure. I'll pick up Mark's acks too and send the whole series again. -- Sent by an employee of the Qualcomm Innovation Center, Inc. The Qualco

Re: [PATCH 2/3] clk: Add devm_clk_{register,unregister}()

2012-09-20 Thread Stephen Boyd
On 09/18/12 23:05, Stephen Boyd wrote: > +void devm_clk_unregister(struct device *dev, struct clk *clk) > +{ > + WARN_ON(devres_destroy(dev, devm_clk_release, devm_clk_match, clk)); Hm... I guess this needs to be devres_release() instead of destroy. Can you squash this in or should I resend fo

[PATCH 2/3] clk: Add devm_clk_{register,unregister}()

2012-09-18 Thread Stephen Boyd
Some clock drivers can be simplified if devres takes care of unregistering any registered clocks along error paths. Introduce devm_clk_register() so that clock drivers get unregistration for free along with simplified error paths. Signed-off-by: Stephen Boyd --- drivers/clk/clk.c| 11