Re: [PATCH v4 1/2] ARM: keystone: pm: switch to use generic pm domains

[santosh shilimkar]

Grygorii,

On 11/25/2014 6:53 AM, Grygorii Strashko wrote:
> Hi Russell,
>
> On 11/25/2014 04:04 PM, Russell King - ARM Linux wrote:
> > On Tue, Nov 25, 2014 at 03:30:20PM +0200, Grygorii Strashko wrote:
> > > On 11/25/2014 02:09 PM, Arnd Bergmann wrote:
> > > > It might be possible to do this implicitly if the driver calls clk_get(),
> > > > basically doing clk_get() (or another call if necessary) would prevent the
> > > > simple pmdomain from turning it off during suspend.
> > >
> > > Unfortunately, clk_get() will not work, because drivers still need to use it
> > > to get functional clocks even if they are not going to control them explicitly.
> > > For example, if it need to know clock's rate.
> >
> > If you don't want a clock to be turned off, then clk_get() it, then
> > clk_prepare() it, and finally clk_enable() it.
> >
> > Even if someone else gets it, the only time that a clock is turned off
> > is when _all_ users of it mutually agree that it can be turned off - by
> > every user disabling (and possibly unpreparing) it.
> >
> > So, if the PM domain code gets a clock, prepares and enables it, then
> > a driver gets the same clock, prepares and enables it also, it won't
> > be disabled until _both_ the PM domain code _and_ the driver disable
> > and unprepare the clock.
>
> All 100% true :)
>
> But the question here is how prevent pm_clk domain (clock_ops.c) from
> getting the control on clock if this particular clock is optional from driver's
> perspective. So, only driver should control it. As opposite, all other clocks
> should be controlled by pm-domain (in case of GPD from .start/stop callbacks).
I know there has been lot of back and forth, we have done on getting
multiple clock handling and from the thread looks like we are converging
with simple power domain approach. Since we have already made several
different attempt to solve the problem and some how stumbled, I suggest
you to do a new RFC with new $subject so that we can get rest of the
folks eyes on it.

The reason I say that because may be just because of $subject, some
folks might ignore the thread ;-) and later raise objections.

Thanks for persisting with it.

Regards,
Santosh
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/