On Wed, Oct 12, 2011 at 07:59:19AM -0700, Turquette, Mike wrote: > On Tue, Oct 11, 2011 at 11:46 PM, Richard Zhao > <richard.z...@freescale.com> wrote: > > On Thu, Sep 22, 2011 at 03:26:59PM -0700, Mike Turquette wrote: > >> From: Jeremy Kerr <jeremy.k...@canonical.com> > >> > >> Signed-off-by: Jeremy Kerr <jeremy.k...@canonical.com> > >> Signed-off-by: Mark Brown <broo...@opensource.wolfsonmicro.com> > >> Signed-off-by: Jamie Iles <ja...@jamieiles.com> > >> Signed-off-by: Mike Turquette <mturque...@ti.com> > >> --- > >> Changes since v1: > >> Add copyright header > >> Fold in Jamie's patch for set-to-disable clks > >> Use BIT macro instead of shift > >> > >> drivers/clk/Kconfig | 4 ++ > >> drivers/clk/Makefile | 1 + > >> drivers/clk/clk-gate.c | 78 > >> ++++++++++++++++++++++++++++++++++++++++++++++++ > >> include/linux/clk.h | 13 ++++++++ > >> 4 files changed, 96 insertions(+), 0 deletions(-) > >> create mode 100644 drivers/clk/clk-gate.c > > > > I feel hard to tell the tree the clk parent, at register/init time. For the > > simple gate clk, the only way is to set .get_parent. But normally, for clk > > without any divider we set .get_parent to NULL. Maybe we can put .parent to > > struct clk_hw? > > For non-mux clocks, whose parent is *always* going to be the same, you > should create a duplicate .parent in the clk_hw_* structure and then > have .get_parent return clk_hw_*->parent.
Maybe I do not understand what you mean here, but I think there is something missing in the gate. > > This is analogous to the way clk_hw_fixed returns clk_hw_fixed->rate > when .recalc is called on it. > > >> + > >> +static unsigned long clk_gate_get_rate(struct clk_hw *clk) > >> +{ > >> + return clk_get_rate(clk_get_parent(clk->clk)); > >> +} clk_get_parent goes down to clk_gate_set_enable_ops.get_parent below... > >> + > >> + > >> +struct clk_hw_ops clk_gate_set_enable_ops = { > >> + .recalc_rate = clk_gate_get_rate, > >> + .enable = clk_gate_enable_set, > >> + .disable = clk_gate_disable_clear, > >> +}; ...but this does not have a get_parent pointer, so clk_get_parent() for a gate always returns NULL which means that a gate never has a valid rate. Sascha -- Pengutronix e.K. | | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | _______________________________________________ linaro-dev mailing list linaro-dev@lists.linaro.org http://lists.linaro.org/mailman/listinfo/linaro-dev