From: Geert Uytterhoeven <geert+renesas@glider.be>
To: "Rafael J . Wysocki" <rjw@rjwysocki.net>,
Ulf Hansson <ulf.hansson@linaro.org>,
Kevin Hilman <khilman@kernel.org>
Cc: Simon Horman <horms@verge.net.au>,
Magnus Damm <magnus.damm@gmail.com>,
Matthias Brugger <matthias.bgg@gmail.com>,
Heiko Stuebner <heiko@sntech.de>,
linux-pm@vger.kernel.org, linux-mediatek@lists.infradead.org,
linux-rockchip@lists.infradead.org,
linux-renesas-soc@vger.kernel.org, linux-kernel@vger.kernel.org,
Geert Uytterhoeven <geert+renesas@glider.be>
Subject: [PATCH v2 0/5] PM / Domains: Remove gpd_dev_ops.active_wakeup() callback
Date: Tue, 31 Oct 2017 19:11:29 +0100 [thread overview]
Message-ID: <1509473494-19740-1-git-send-email-geert+renesas@glider.be> (raw)
Hi Rafael, Ulf, Kevin,
It is quite common for PM Domains to require slave devices to be kept
active during system suspend if they are to be used as wakeup sources.
To enable this, currently each PM Domain or driver has to provide its
own gpd_dev_ops.active_wakeup() callback.
All existing callbacks either return always true, or a fixed value
depending on the PM Domain.
Hence this patch series simplifies active wakeup handling by replacing
the callback by a flag:
- Patch 1 adds a new new flag GENPD_FLAG_ACTIVE_WAKEUP, to be set by
PM Domain drivers that want to use the new handling,
- Patches 2-4 convert all existing users of the callback to the new
flag,
- Patch 5 removes the callback.
This series was extracted from "[PATCH 00/10] PM / Domain: renesas: Fix
active wakeup behavior", and retains only PM Domain changes to existing
drivers.
Changes compared to v1 (most suggested by Ulf):
- Use the flag in se instead of setting up an "always true" callback,
- Convert the mediatek and rockchip PM Domain drivers,
- Remove the callback.
If this idea is accepted, I think it would be good to still queue the first
patch for v4.15. That would allow the changes to the PM Domain drivers
to go in through the respective platform trees in v4.16.
Then the final patch has to be postponed until v4.17.
Alternatively, everything can go in through the PM tree in one shot.
Thanks for your comments!
Geert Uytterhoeven (5):
PM / Domains: Allow genpd users to specify default active wakeup
behavior
[RFC] ARM: shmobile: pm-rmobile: Use GENPD_FLAG_ACTIVE_WAKEUP
[RFC] soc: mediatek: Use GENPD_FLAG_ACTIVE_WAKEUP
[RFC] soc: rockchip: power-domain: Use GENPD_FLAG_ACTIVE_WAKEUP
[RFC] PM / Domains: Remove gpd_dev_ops.active_wakeup() callback
arch/arm/mach-shmobile/pm-rmobile.c | 8 +-------
drivers/base/power/domain.c | 13 ++++---------
drivers/soc/mediatek/mtk-scpsys.c | 14 ++------------
drivers/soc/rockchip/pm_domains.c | 14 ++------------
include/linux/pm_domain.h | 8 ++++----
5 files changed, 13 insertions(+), 44 deletions(-)
--
2.7.4
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
next reply other threads:[~2017-10-31 18:11 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-31 18:11 Geert Uytterhoeven [this message]
2017-10-31 18:11 ` [PATCH v2 1/5] PM / Domains: Allow genpd users to specify default active wakeup behavior Geert Uytterhoeven
2017-11-02 13:59 ` Kevin Hilman
2017-10-31 18:11 ` [PATCH v2 2/5] [RFC] ARM: shmobile: pm-rmobile: Use GENPD_FLAG_ACTIVE_WAKEUP Geert Uytterhoeven
2017-11-02 8:18 ` Simon Horman
2017-10-31 18:11 ` [PATCH v2 3/5] [RFC] soc: mediatek: " Geert Uytterhoeven
2017-11-02 11:32 ` Matthias Brugger
2017-10-31 18:11 ` [PATCH v2 4/5] [RFC] soc: rockchip: power-domain: " Geert Uytterhoeven
2017-11-02 9:51 ` Heiko Stübner
2017-10-31 18:11 ` [PATCH v2 5/5] [RFC] PM / Domains: Remove gpd_dev_ops.active_wakeup() callback Geert Uytterhoeven
2017-11-02 14:00 ` Kevin Hilman
2017-11-01 10:10 ` [PATCH v2 0/5] " Ulf Hansson
2017-11-02 9:14 ` 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=1509473494-19740-1-git-send-email-geert+renesas@glider.be \
--to=geert+renesas@glider.be \
--cc=heiko@sntech.de \
--cc=horms@verge.net.au \
--cc=khilman@kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mediatek@lists.infradead.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=linux-rockchip@lists.infradead.org \
--cc=magnus.damm@gmail.com \
--cc=matthias.bgg@gmail.com \
--cc=rjw@rjwysocki.net \
--cc=ulf.hansson@linaro.org \
/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).