Quoting Boris BREZILLON (2013-11-12 13:57:19) > +static const struct clk_ops pll_ops = { > + .prepare = clk_pll_prepare, > + .is_prepared = clk_pll_is_ready, > + .disable = clk_pll_disable, > + .is_enabled = clk_pll_is_ready, > + .recalc_rate = clk_pll_recalc_rate, > + .round_rate = clk_pll_round_rate, > + .set_rate = clk_pll_set_rate, > +}; Hi Boris, It is a bit strange to see only a .prepare and .disable callback populated. What happens if a driver calls clk_disable and then clk_enable? You clock will still be disabled in hardware. Regards, Mike -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html