From: khilman@kernel•org (Kevin Hilman)
To: linux-arm-kernel@lists•infradead.org
Subject: [PATCH v3] driver core / PM: Add PM domain callbacks for device setup/cleanup
Date: Fri, 20 Mar 2015 17:09:30 -0700 [thread overview]
Message-ID: <7hzj77i539.fsf@deeprootsystems.com> (raw)
In-Reply-To: <3289026.ADmXXipokh@vostro.rjw.lan> (Rafael J. Wysocki's message of "Fri, 20 Mar 2015 13:59:27 +0100")
"Rafael J. Wysocki" <rjw@rjwysocki•net> writes:
> From: Rafael J. Wysocki <rafael.j.wysocki@intel•com>
>
> If PM domains are in use, it may be necessary to prepare the code
> handling a PM domain for driver probing. For example, in some
> cases device drivers rely on the ability to power on the devices
> with the help of the IO runtime PM framework and the PM domain
> code needs to be ready for that. Also, if that code has not been
> fully initialized yet, the driver probing should be deferred.
>
> Moreover, after the probing is complete, it may be necessary to
> put the PM domain in question into the state reflecting the current
> needs of the devices in it, for example, so that power is not drawn
> in vain. The same should be done after removing a driver from
> a device, as the PM domain state may need to be changed to reflect
> the new situation.
>
> For these reasons, introduce new PM domain callbacks, ->activate,
> ->sync and ->dismiss called, respectively, before probing for a
> device driver, after the probing has completed successfully and
> if the probing has failed or the driver has been removed.
>
> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel•com>
> ---
>
> One more update taking Dmitry's comments into account and fixing an
> overoptimization bug spotted by Ulf.
>
> And this actually contains the patch this time (clicked on "Send" too
> early last time, sorry about that).
Reviewed-by: Kevin Hilman <khilman@linaro•org>
Unless I'm missing somthing, this along with the just the genpd part of
Russell's patch should then fix his problem as well.
Kevin
next prev parent reply other threads:[~2015-03-21 0:09 UTC|newest]
Thread overview: 58+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-03-13 15:43 [PATCH 0/9] PM / Domains: Don't leave unused PM domains powered after ->probe() Ulf Hansson
2015-03-13 15:43 ` [PATCH 1/9] PM / Domains: Add dev_pm_domain_get|put() APIs Ulf Hansson
2015-03-14 1:31 ` Rafael J. Wysocki
2015-03-16 9:26 ` Ulf Hansson
2015-03-17 3:01 ` Rafael J. Wysocki
2015-03-17 9:27 ` Ulf Hansson
2015-03-17 14:45 ` Rafael J. Wysocki
2015-03-17 14:25 ` Russell King - ARM Linux
2015-03-18 1:16 ` Rafael J. Wysocki
2015-03-17 14:40 ` Ulf Hansson
2015-03-18 1:09 ` Rafael J. Wysocki
2015-03-18 13:41 ` Ulf Hansson
2015-03-18 15:02 ` [PATCH] driver core / PM: Add callbacks for PM domain initialization/cleanup Rafael J. Wysocki
2015-03-19 8:49 ` Ulf Hansson
2015-03-19 11:45 ` Rafael J. Wysocki
2015-03-19 13:16 ` Ulf Hansson
2015-03-19 13:29 ` Greg Kroah-Hartman
2015-03-19 14:21 ` Rafael J. Wysocki
2015-03-19 14:12 ` Greg Kroah-Hartman
2015-03-19 15:24 ` Rafael J. Wysocki
2015-03-19 14:20 ` Alan Stern
2015-03-19 14:45 ` Ulf Hansson
2015-03-19 15:44 ` Rafael J. Wysocki
2015-03-19 15:37 ` Ulf Hansson
2015-03-19 16:04 ` Rafael J. Wysocki
2015-03-19 15:48 ` Ulf Hansson
2015-03-19 16:18 ` Rafael J. Wysocki
2015-03-19 16:58 ` [PATCH] driver core / PM: Add PM domain callbacks for device setup/cleanup Rafael J. Wysocki
2015-03-19 21:51 ` [PATCH v2] " Rafael J. Wysocki
2015-03-19 22:42 ` Dmitry Torokhov
2015-03-20 0:43 ` Rafael J. Wysocki
2015-03-20 0:43 ` Dmitry Torokhov
2015-03-20 7:45 ` Ulf Hansson
2015-03-20 11:37 ` Ulf Hansson
2015-03-20 12:31 ` Rafael J. Wysocki
2015-03-20 12:57 ` [PATCH v3] " Rafael J. Wysocki
2015-03-20 12:59 ` Rafael J. Wysocki
2015-03-20 13:44 ` Ulf Hansson
2015-03-21 0:09 ` Kevin Hilman [this message]
2015-03-21 1:00 ` Rafael J. Wysocki
2015-03-22 11:46 ` Greg Kroah-Hartman
2015-03-19 14:46 ` [PATCH] driver core / PM: Add callbacks for PM domain initialization/cleanup Geert Uytterhoeven
2015-03-18 15:09 ` [PATCH 1/9] PM / Domains: Add dev_pm_domain_get|put() APIs Rafael J. Wysocki
2015-03-13 15:43 ` [PATCH 2/9] PM / Domains: Enable genpd to support ->get|put() callbacks Ulf Hansson
2015-03-16 2:11 ` Chao
2015-03-13 15:43 ` [PATCH 3/9] amba: Keep PM domain powered during ->probe() Ulf Hansson
2015-03-13 16:03 ` Russell King - ARM Linux
2015-03-16 8:37 ` Ulf Hansson
2015-03-13 15:43 ` [PATCH 4/9] drivercore / platform: " Ulf Hansson
2015-03-13 15:43 ` [PATCH 5/9] i2c: core: " Ulf Hansson
2015-03-13 15:43 ` [PATCH 6/9] spi: " Ulf Hansson
2015-03-13 15:43 ` [PATCH 7/9] mmc: core: Attach PM domain prior probing of SDIO func driver Ulf Hansson
2015-03-17 5:04 ` Aaron Lu
2015-03-13 15:43 ` [PATCH 8/9] mmmc: core: Keep PM domain powered during ->probe() " Ulf Hansson
2015-03-13 16:10 ` Russell King - ARM Linux
2015-03-16 8:24 ` Ulf Hansson
2015-03-13 15:43 ` [PATCH 9/9] Revert "PM / Domains: Power on the PM domain right after attach completes" Ulf Hansson
2015-03-16 9:07 ` Geert Uytterhoeven
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=7hzj77i539.fsf@deeprootsystems.com \
--to=khilman@kernel$(echo .)org \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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