On Tue, Mar 13, 2012 at 2:48 PM, Rob Herring <robherri...@gmail.com> 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 <mturque...@linaro.org> >> Signed-off-by: Mike Turquette <mturque...@ti.com> >> Cc: Russell King <li...@arm.linux.org.uk> >> Cc: Jeremy Kerr <jeremy.k...@canonical.com> >> Cc: Thomas Gleixner <t...@linutronix.de> >> Cc: Arnd Bergman <arnd.bergm...@linaro.org> >> Cc: Paul Walmsley <p...@pwsan.com> >> Cc: Shawn Guo <shawn....@freescale.com> >> Cc: Sascha Hauer <s.ha...@pengutronix.de> >> Cc: Richard Zhao <richard.z...@linaro.org> >> Cc: Saravana Kannan <skan...@codeaurora.org> >> Cc: Magnus Damm <magnus.d...@gmail.com> >> Cc: Rob Herring <rob.herr...@calxeda.com> >> Cc: Mark Brown <broo...@opensource.wolfsonmicro.com> >> Cc: Linus Walleij <linus.wall...@stericsson.com> >> Cc: Stephen Boyd <sb...@codeaurora.org> >> Cc: Amit Kucheria <amit.kuche...@linaro.org> >> Cc: Deepak Saxena <dsax...@linaro.org> >> Cc: Grant Likely <grant.lik...@secretlab.ca> >> Cc: Andrew Lunn <and...@lunn.ch> > > 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 > linaro-dev@lists.linaro.org > http://lists.linaro.org/mailman/listinfo/linaro-dev _______________________________________________ linaro-dev mailing list linaro-dev@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-dev