On Tue, Mar 13, 2012 at 2:48 PM, Rob Herring <[email protected]> wrote: > Mike, > > On 03/10/2012 01:54 AM, Mike Turquette wrote: >> The common clock framework defines a common struct clk useful across >> most platforms as well as an implementation of the clk api that drivers >> can use safely for managing clocks. >> >> The net result is consolidation of many different struct clk definitions >> and platform-specific clock framework implementations. >> >> This patch introduces the common struct clk, struct clk_ops and an >> implementation of the well-known clock api in include/clk/clk.h. >> Platforms may define their own hardware-specific clock structure and >> their own clock operation callbacks, so long as it wraps an instance of >> struct clk_hw. >> >> See Documentation/clk.txt for more details. >> >> This patch is based on the work of Jeremy Kerr, which in turn was based >> on the work of Ben Herrenschmidt. >> >> Signed-off-by: Mike Turquette <[email protected]> >> Signed-off-by: Mike Turquette <[email protected]> >> Cc: Russell King <[email protected]> >> Cc: Jeremy Kerr <[email protected]> >> Cc: Thomas Gleixner <[email protected]> >> Cc: Arnd Bergman <[email protected]> >> Cc: Paul Walmsley <[email protected]> >> Cc: Shawn Guo <[email protected]> >> Cc: Sascha Hauer <[email protected]> >> Cc: Richard Zhao <[email protected]> >> Cc: Saravana Kannan <[email protected]> >> Cc: Magnus Damm <[email protected]> >> Cc: Rob Herring <[email protected]> >> Cc: Mark Brown <[email protected]> >> Cc: Linus Walleij <[email protected]> >> Cc: Stephen Boyd <[email protected]> >> Cc: Amit Kucheria <[email protected]> >> Cc: Deepak Saxena <[email protected]> >> Cc: Grant Likely <[email protected]> >> Cc: Andrew Lunn <[email protected]> > > snip > >> + >> + /* >> + * walk the list of orphan clocks and reparent any that are children of >> + * this clock >> + */ >> + hlist_for_each_entry(orphan, tmp, &clk_orphan_list, child_node) > > In __clk_init, this needs to be hlist_for_each_entry_safe as entries can > be removed.
Thanks for the catch Rob. I'll take this in. Regards, Mike > > Rob > > _______________________________________________ > linaro-dev mailing list > [email protected] > http://lists.linaro.org/mailman/listinfo/linaro-dev _______________________________________________ linaro-dev mailing list [email protected] http://lists.linaro.org/mailman/listinfo/linaro-dev
