From: Russell King - ARM Linux <linux@arm.linux.org.uk>
To: Linus Walleij <linus.walleij@linaro.org>
Cc: "Rafael J. Wysocki" <rjw@sisk.pl>, Rabin Vincent <rabin@rab.in>,
khilman@ti.com, magnus.damm@gmail.com,
LKML <linux-kernel@vger.kernel.org>,
stern@rowland.harvard.edu, linux-i2c@vger.kernel.org,
linux-pm@lists.linux-foundation.org,
linux-arm-kernel <linux-arm-kernel@lists.infradead.org>
Subject: Re: platform/i2c busses: pm runtime and system sleep
Date: Sat, 19 Feb 2011 10:00:17 +0000 [thread overview]
Message-ID: <20110219100017.GA29493@n2100.arm.linux.org.uk> (raw)
In-Reply-To: <AANLkTim2TfG-18XN3r9m3Rq68ZD6QHUY7X82E4-XWH83@mail.gmail.com>
On Sat, Feb 19, 2011 at 10:54:57AM +0100, Linus Walleij wrote:
> 2011/2/18 Russell King - ARM Linux <linux@arm.linux.org.uk>:
>
> > Do we have any pressing need to convert AMBA stuff? I haven't heard any
> > reason yet to convert them to runtime PM - they don't even make any
> > runtime PM calls.
> >
> > Maybe Linus can comment on the PM stuff as he has SoCs with these in.
> > As my boards don't have any sensible PM support, I don't have any
> > visibility of what PM facilities would be required.
>
> Sure, basically I ACK Rabins patch and his reasoning for it. (BTW
> Rabin spends most of his days working on the Ux500 SoCs too.)
>
> The runtime PM we need for Ux500 is to switch off silicon core
> voltage first and foremost. The call I've added to switch of a core
> voltage regulator will need to be called when the silicon is idle.
>
> In spi/amba-pl022.c I take the most brutal approach with a recent
> patch: hammer off this core switch (and clock) whenever the hardware
> is not used. This is simple in this driver since it has no state to preserve
> across transfers, it is written such that the core is loaded with the
> appropriate state for each message.
>
> Continuing this approach we run into two problems with this
> and other drivers:
>
> - Hammering off/on the clock+voltage is causing delays in HW
> so what you want is some hysteresis (usually, wait a few us/ms
> then switch off) - sort of a takeoff/landing effect.
>
> - Modelling voltage domains as regulators is nice, but require
> us to switch on/off from process context, so we cannot do this
> from interrupt handlers.
>
> Both of these problems are solved by elegance if we use runtime
> PM, since it will provide a hysteresis timeout that can be triggered
> from interrupt context and call the idling hooks in process context.
So what's the interdependence with the platform bus that was being talked
about earlier in this thread?
next prev parent reply other threads:[~2011-02-19 10:00 UTC|newest]
Thread overview: 28+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-12-16 18:26 platform/i2c busses: pm runtime and system sleep Rabin Vincent
[not found] ` <AANLkTinyDE3OxKup_aqsN8HJH_r5LcwkP17OtuMRpACx-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2010-12-17 0:09 ` Rafael J. Wysocki
2011-02-17 15:25 ` Rabin Vincent
[not found] ` <AANLkTikRUZRh0YnP8nYTKFnFeUiJbK5xKvHHjn_S+gZE-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-02-18 2:48 ` Rabin Vincent
2011-02-18 15:05 ` Alan Stern
[not found] ` <AANLkTinsPqFcodH0w7LQeFEY+amodNH4CneRCRhhbKaz-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2011-02-18 18:28 ` Rafael J. Wysocki
[not found] ` <201102181928.05911.rjw-KKrjLPT3xs0@public.gmane.org>
2011-02-18 19:25 ` Rabin Vincent
2011-02-18 20:20 ` Rafael J. Wysocki
2011-02-18 20:27 ` Russell King - ARM Linux
[not found] ` <20110218202744.GA19427-l+eeeJia6m9vn6HldHNs0ANdhmdF6hFW@public.gmane.org>
2011-02-18 22:16 ` Mark Brown
2011-02-19 9:54 ` Linus Walleij
2011-02-19 10:00 ` Russell King - ARM Linux [this message]
[not found] ` <20110219100017.GA29493-l+eeeJia6m9vn6HldHNs0ANdhmdF6hFW@public.gmane.org>
2011-02-19 10:16 ` Linus Walleij
2011-02-19 7:24 ` Rabin Vincent
2010-12-17 12:54 ` Mark Brown
[not found] ` <20101217125427.GA29640-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2010-12-17 13:25 ` Rafael J. Wysocki
[not found] ` <201012171425.07775.rjw-KKrjLPT3xs0@public.gmane.org>
2010-12-17 13:34 ` Mark Brown
[not found] ` <20101217133434.GH31453-HF5t3jzXg/6ND3a5+9QAFujbO/Zr0HzV@public.gmane.org>
2010-12-17 13:49 ` Rafael J. Wysocki
[not found] ` <201012171449.26082.rjw-KKrjLPT3xs0@public.gmane.org>
2010-12-17 14:24 ` Mark Brown
[not found] ` <20101217142402.GA19391-HF5t3jzXg/6ND3a5+9QAFujbO/Zr0HzV@public.gmane.org>
2010-12-17 23:01 ` Rafael J. Wysocki
[not found] ` <201012180001.25263.rjw-KKrjLPT3xs0@public.gmane.org>
2010-12-18 1:04 ` Mark Brown
2010-12-18 12:54 ` Rafael J. Wysocki
[not found] ` <201012181354.58077.rjw-KKrjLPT3xs0@public.gmane.org>
2010-12-18 13:20 ` Mark Brown
[not found] ` <20101218132029.GA22273-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org>
2010-12-18 14:59 ` Rafael J. Wysocki
[not found] ` <201012181559.50347.rjw-KKrjLPT3xs0@public.gmane.org>
2010-12-20 15:00 ` Mark Brown
[not found] ` <20101220150051.GI26706-HF5t3jzXg/6ND3a5+9QAFujbO/Zr0HzV@public.gmane.org>
2010-12-20 21:13 ` Rafael J. Wysocki
[not found] ` <201012202213.53902.rjw-KKrjLPT3xs0@public.gmane.org>
2010-12-21 23:51 ` Mark Brown
[not found] ` <20101221235127.GC10081-yzvPICuk2AATkU/dhu1WVueM+bqZidxxQQ4Iyu8u01E@public.gmane.org>
2010-12-22 0:35 ` Rafael J. Wysocki
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20110219100017.GA29493@n2100.arm.linux.org.uk \
--to=linux@arm.linux.org.uk \
--cc=khilman@ti.com \
--cc=linus.walleij@linaro.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-i2c@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@lists.linux-foundation.org \
--cc=magnus.damm@gmail.com \
--cc=rabin@rab.in \
--cc=rjw@sisk.pl \
--cc=stern@rowland.harvard.edu \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).