mirror of https://gitee.com/openkylin/linux.git
PM / Domains: Move genpd_power_off() above genpd_power_on()
Following changes in genpd_power_on() makes it invoke genpd_power_off(). To enable these changes and avoiding to declare genpd_power_off(), let's move its implementation above genpd_power_on(). In this way, following changes should become easier to review. Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
This commit is contained in:
Ulf Hansson
Rafael J. Wysocki
parent
16d0bf1882
commit
1f8728b7ad
|
|
@ -273,6 +273,87 @@ static void genpd_queue_power_off_work(struct generic_pm_domain *genpd)
|
||||||
queue_work(pm_wq, &genpd->power_off_work);
|
queue_work(pm_wq, &genpd->power_off_work);
|
||||||
}
|
}
|
||||||
|
|
||||||
|
/**
|
||||||
|
* genpd_power_off - Remove power from a given PM domain.
|
||||||
|
* @genpd: PM domain to power down.
|
||||||
|
* @is_async: PM domain is powered down from a scheduled work
|
||||||
|
*
|
||||||
|
* If all of the @genpd's devices have been suspended and all of its subdomains
|
||||||
|
* have been powered down, remove power from @genpd.
|
||||||
|
*/
|
||||||
|
static int genpd_power_off(struct generic_pm_domain *genpd, bool is_async)
|
||||||
|
{
|
||||||
|
struct pm_domain_data *pdd;
|
||||||
|
struct gpd_link *link;
|
||||||
|
unsigned int not_suspended = 0;
|
||||||
|
|
||||||
|
/*
|
||||||
|
* Do not try to power off the domain in the following situations:
|
||||||
|
* (1) The domain is already in the "power off" state.
|
||||||
|
* (2) System suspend is in progress.
|
||||||
|
*/
|
||||||
|
if (genpd->status == GPD_STATE_POWER_OFF
|
||||||
|
|| genpd->prepared_count > 0)
|
||||||
|
return 0;
|
||||||
|
|
||||||
|
if (atomic_read(&genpd->sd_count) > 0)
|
||||||
|
return -EBUSY;
|
||||||
|
|
||||||
|
list_for_each_entry(pdd, &genpd->dev_list, list_node) {
|
||||||
|
enum pm_qos_flags_status stat;
|
||||||
|
|
||||||
|
stat = dev_pm_qos_flags(pdd->dev,
|
||||||
|
PM_QOS_FLAG_NO_POWER_OFF
|
||||||
|
| PM_QOS_FLAG_REMOTE_WAKEUP);
|
||||||
|
if (stat > PM_QOS_FLAGS_NONE)
|
||||||
|
return -EBUSY;
|
||||||
|
|
||||||
|
/*
|
||||||
|
* Do not allow PM domain to be powered off, when an IRQ safe
|
||||||
|
* device is part of a non-IRQ safe domain.
|
||||||
|
*/
|
||||||
|
if (!pm_runtime_suspended(pdd->dev) ||
|
||||||
|
irq_safe_dev_in_no_sleep_domain(pdd->dev, genpd))
|
||||||
|
not_suspended++;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (not_suspended > 1 || (not_suspended == 1 && is_async))
|
||||||
|
return -EBUSY;
|
||||||
|
|
||||||
|
if (genpd->gov && genpd->gov->power_down_ok) {
|
||||||
|
if (!genpd->gov->power_down_ok(&genpd->domain))
|
||||||
|
return -EAGAIN;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (genpd->power_off) {
|
||||||
|
int ret;
|
||||||
|
|
||||||
|
if (atomic_read(&genpd->sd_count) > 0)
|
||||||
|
return -EBUSY;
|
||||||
|
|
||||||
|
/*
|
||||||
|
* If sd_count > 0 at this point, one of the subdomains hasn't
|
||||||
|
* managed to call genpd_power_on() for the master yet after
|
||||||
|
* incrementing it. In that case genpd_power_on() will wait
|
||||||
|
* for us to drop the lock, so we can call .power_off() and let
|
||||||
|
* the genpd_power_on() restore power for us (this shouldn't
|
||||||
|
* happen very often).
|
||||||
|
*/
|
||||||
|
ret = _genpd_power_off(genpd, true);
|
||||||
|
if (ret)
|
||||||
|
return ret;
|
||||||
|
}
|
||||||
|
|
||||||
|
genpd->status = GPD_STATE_POWER_OFF;
|
||||||
|
|
||||||
|
list_for_each_entry(link, &genpd->slave_links, slave_node) {
|
||||||
|
genpd_sd_counter_dec(link->master);
|
||||||
|
genpd_queue_power_off_work(link->master);
|
||||||
|
}
|
||||||
|
|
||||||
|
return 0;
|
||||||
|
}
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* genpd_power_on - Restore power to a given PM domain and its masters.
|
* genpd_power_on - Restore power to a given PM domain and its masters.
|
||||||
* @genpd: PM domain to power up.
|
* @genpd: PM domain to power up.
|
||||||
|
|
@ -367,87 +448,6 @@ static int genpd_dev_pm_qos_notifier(struct notifier_block *nb,
|
||||||
return NOTIFY_DONE;
|
return NOTIFY_DONE;
|
||||||
}
|
}
|
||||||
|
|
||||||
/**
|
|
||||||
* genpd_power_off - Remove power from a given PM domain.
|
|
||||||
* @genpd: PM domain to power down.
|
|
||||||
* @is_async: PM domain is powered down from a scheduled work
|
|
||||||
*
|
|
||||||
* If all of the @genpd's devices have been suspended and all of its subdomains
|
|
||||||
* have been powered down, remove power from @genpd.
|
|
||||||
*/
|
|
||||||
static int genpd_power_off(struct generic_pm_domain *genpd, bool is_async)
|
|
||||||
{
|
|
||||||
struct pm_domain_data *pdd;
|
|
||||||
struct gpd_link *link;
|
|
||||||
unsigned int not_suspended = 0;
|
|
||||||
|
|
||||||
/*
|
|
||||||
* Do not try to power off the domain in the following situations:
|
|
||||||
* (1) The domain is already in the "power off" state.
|
|
||||||
* (2) System suspend is in progress.
|
|
||||||
*/
|
|
||||||
if (genpd->status == GPD_STATE_POWER_OFF
|
|
||||||
|| genpd->prepared_count > 0)
|
|
||||||
return 0;
|
|
||||||
|
|
||||||
if (atomic_read(&genpd->sd_count) > 0)
|
|
||||||
return -EBUSY;
|
|
||||||
|
|
||||||
list_for_each_entry(pdd, &genpd->dev_list, list_node) {
|
|
||||||
enum pm_qos_flags_status stat;
|
|
||||||
|
|
||||||
stat = dev_pm_qos_flags(pdd->dev,
|
|
||||||
PM_QOS_FLAG_NO_POWER_OFF
|
|
||||||
| PM_QOS_FLAG_REMOTE_WAKEUP);
|
|
||||||
if (stat > PM_QOS_FLAGS_NONE)
|
|
||||||
return -EBUSY;
|
|
||||||
|
|
||||||
/*
|
|
||||||
* Do not allow PM domain to be powered off, when an IRQ safe
|
|
||||||
* device is part of a non-IRQ safe domain.
|
|
||||||
*/
|
|
||||||
if (!pm_runtime_suspended(pdd->dev) ||
|
|
||||||
irq_safe_dev_in_no_sleep_domain(pdd->dev, genpd))
|
|
||||||
not_suspended++;
|
|
||||||
}
|
|
||||||
|
|
||||||
if (not_suspended > 1 || (not_suspended == 1 && is_async))
|
|
||||||
return -EBUSY;
|
|
||||||
|
|
||||||
if (genpd->gov && genpd->gov->power_down_ok) {
|
|
||||||
if (!genpd->gov->power_down_ok(&genpd->domain))
|
|
||||||
return -EAGAIN;
|
|
||||||
}
|
|
||||||
|
|
||||||
if (genpd->power_off) {
|
|
||||||
int ret;
|
|
||||||
|
|
||||||
if (atomic_read(&genpd->sd_count) > 0)
|
|
||||||
return -EBUSY;
|
|
||||||
|
|
||||||
/*
|
|
||||||
* If sd_count > 0 at this point, one of the subdomains hasn't
|
|
||||||
* managed to call genpd_power_on() for the master yet after
|
|
||||||
* incrementing it. In that case genpd_power_on() will wait
|
|
||||||
* for us to drop the lock, so we can call .power_off() and let
|
|
||||||
* the genpd_power_on() restore power for us (this shouldn't
|
|
||||||
* happen very often).
|
|
||||||
*/
|
|
||||||
ret = _genpd_power_off(genpd, true);
|
|
||||||
if (ret)
|
|
||||||
return ret;
|
|
||||||
}
|
|
||||||
|
|
||||||
genpd->status = GPD_STATE_POWER_OFF;
|
|
||||||
|
|
||||||
list_for_each_entry(link, &genpd->slave_links, slave_node) {
|
|
||||||
genpd_sd_counter_dec(link->master);
|
|
||||||
genpd_queue_power_off_work(link->master);
|
|
||||||
}
|
|
||||||
|
|
||||||
return 0;
|
|
||||||
}
|
|
||||||
|
|
||||||
/**
|
/**
|
||||||
* genpd_power_off_work_fn - Power off PM domain whose subdomain count is 0.
|
* genpd_power_off_work_fn - Power off PM domain whose subdomain count is 0.
|
||||||
* @work: Work structure used for scheduling the execution of this function.
|
* @work: Work structure used for scheduling the execution of this function.
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue