Messages in this thread | | | Date | Sun, 23 Oct 2011 22:24:59 +0800 | From | Shawn Guo <> | Subject | Re: [PATCH v2 2/7] clk: Implement clk_set_rate |
| |
On Thu, Sep 22, 2011 at 03:26:57PM -0700, Mike Turquette wrote: > From: Jeremy Kerr <jeremy.kerr@canonical.com> > > Implement clk_set_rate by adding a set_rate callback to clk_hw_ops. > Rates are propagated down the clock tree and recalculated. Also adds a > flag for signaling that parents must change rates to achieve the desired > frequency (upstream propagation). > > TODO: > Upstream propagation is not yet implemented. > Device pre-change and post-change notifications are not implemented, but > are marked up as FIXME comments. > > Signed-off-by: Jeremy Kerr <jeremy.kerr@canonical.com> > Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com> > Signed-off-by: Mike Turquette <mturquette@ti.com> > --- > Changes since v1: > Remove upstream propagation (for now) > Rename CLK_SET_RATE_PROPAGATE to CLK_PARENT_RATE_CHANGE
[...]
> + * @set_rate Change the rate of this clock. If this callback returns > + * CLK_SET_RATE_PROPAGATE, the rate change will be propagated to
s/CLK_SET_RATE_PROPAGATE/CLK_PARENT_RATE_CHANGE, as suggested by your change log above?
> + * the parent clock (which may propagate again). The requested > + * rate of the parent is passed back from the callback in the > + * second 'unsigned long *' argument. > + *
-- Regards, Shawn
| |