Commit a8cf27be authored by Rafael J. Wysocki's avatar Rafael J. Wysocki

PM / shmobile: Allow the A4R domain to be turned off at run time

After adding PM QoS constraints for the I2C controller in the A4R
domain, that domain can be allowed to be turned off and on by runtime
PM, so remove the "always on" governor from it.

However, the A4R domain has to be "on" when suspend_device_irqs() and
resume_device_irqs() are executed during system suspend and resume,
respectively, so that those functions don't crash while accessing the
INTCS.  For this reason, add a PM notifier to the SH7372 PM code and
make it restore power to A4R before system suspend and remove power
from all unused PM domains after system resume.
Signed-off-by: default avatarRafael J. Wysocki <rjw@sisk.pl>
Acked-by: default avatarMagnus Damm <damm@opensource.se>
parent 9ee27ffb
...@@ -239,7 +239,6 @@ struct sh7372_pm_domain sh7372_d4 = { ...@@ -239,7 +239,6 @@ struct sh7372_pm_domain sh7372_d4 = {
struct sh7372_pm_domain sh7372_a4r = { struct sh7372_pm_domain sh7372_a4r = {
.genpd.name = "A4R", .genpd.name = "A4R",
.bit_shift = 5, .bit_shift = 5,
.gov = &pm_domain_always_on_gov,
.suspend = sh7372_a4r_suspend, .suspend = sh7372_a4r_suspend,
.resume = sh7372_intcs_resume, .resume = sh7372_intcs_resume,
}; };
...@@ -535,9 +534,37 @@ static int sh7372_enter_suspend(suspend_state_t suspend_state) ...@@ -535,9 +534,37 @@ static int sh7372_enter_suspend(suspend_state_t suspend_state)
return 0; return 0;
} }
/**
* sh7372_pm_notifier_fn - SH7372 PM notifier routine.
* @notifier: Unused.
* @pm_event: Event being handled.
* @unused: Unused.
*/
static int sh7372_pm_notifier_fn(struct notifier_block *notifier,
unsigned long pm_event, void *unused)
{
switch (pm_event) {
case PM_SUSPEND_PREPARE:
/*
* This is necessary, because the A4R domain has to be "on"
* when suspend_device_irqs() and resume_device_irqs() are
* executed during system suspend and resume, respectively, so
* that those functions don't crash while accessing the INTCS.
*/
pm_genpd_poweron(&sh7372_a4r.genpd);
break;
case PM_POST_SUSPEND:
pm_genpd_poweroff_unused();
break;
}
return NOTIFY_DONE;
}
static void sh7372_suspend_init(void) static void sh7372_suspend_init(void)
{ {
shmobile_suspend_ops.enter = sh7372_enter_suspend; shmobile_suspend_ops.enter = sh7372_enter_suspend;
pm_notifier(sh7372_pm_notifier_fn, 0);
} }
#else #else
static void sh7372_suspend_init(void) {} static void sh7372_suspend_init(void) {}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment