cpufreq: suspend governors on system suspend/hibernate
authorMark Brown <broonie@linaro.org>
Thu, 17 Apr 2014 23:48:32 +0000 (00:48 +0100)
committerMark Brown <broonie@linaro.org>
Wed, 23 Apr 2014 22:54:52 +0000 (23:54 +0100)
commitadb665056afe23db0687c1d1a208806c108add08
tree3a36a782bf18e04f5be85e4eded6b14a96b29c93
parent682a3d0201261152f489138d5f034742bccbfe9a
cpufreq: suspend governors on system suspend/hibernate

This patch adds cpufreq suspend/resume calls to dpm_{suspend|resume}()
for handling suspend/resume of cpufreq governors.

Lan Tianyu (Intel) & Jinhyuk Choi (Broadcom) found an issue where the
tunables configuration for clusters/sockets with non-boot CPUs was
lost after system suspend/resume, as we were notifying governors with
CPUFREQ_GOV_POLICY_EXIT on removal of the last CPU for that policy
which caused the tunables memory to be freed.

This is fixed by preventing any governor operations from being
carried out between the device suspend and device resume stages of
system suspend and resume, respectively.

We could have added these callbacks at dpm_{suspend|resume}_noirq()
level, but there is an additional problem that the majority of I/O
devices is already suspended at that point and if cpufreq drivers
want to change the frequency before suspending, then that not be
possible on some platforms (which depend on peripherals like i2c,
regulators, etc).

Reported-and-tested-by: Lan Tianyu <tianyu.lan@intel.com>
Reported-by: Jinhyuk Choi <jinchoi@broadcom.com>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
[rjw: Changelog]
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
(cherry-picked from 2f0aea9363609433b38ab13f0f86f38372668c68)
Signed-off-by: Mark Brown <broonie@linaro.org>
drivers/base/power/main.c
drivers/cpufreq/cpufreq.c
include/linux/cpufreq.h