Skip to content

Commit fabb1c2

Browse files
Ulf Hanssonrafaeljw
authored andcommitted
PM: domains: Drop redundant code for genpd always-on governor
Due to recent changes, the always-on governor is always used with a genpd that has the GENPD_FLAG_RPM_ALWAYS_ON flag being set. This means genpd, doesn't invoke the governor's ->power_down_ok() callback, which makes the code in the governor redundant, so let's drop it. Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
1 parent 27656dc commit fabb1c2

1 file changed

Lines changed: 0 additions & 6 deletions

File tree

drivers/base/power/domain_governor.c

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -327,11 +327,6 @@ static bool default_power_down_ok(struct dev_pm_domain *pd)
327327
return _default_power_down_ok(pd, ktime_get());
328328
}
329329

330-
static bool always_on_power_down_ok(struct dev_pm_domain *domain)
331-
{
332-
return false;
333-
}
334-
335330
#ifdef CONFIG_CPU_IDLE
336331
static bool cpu_power_down_ok(struct dev_pm_domain *pd)
337332
{
@@ -401,6 +396,5 @@ struct dev_power_governor simple_qos_governor = {
401396
* pm_genpd_gov_always_on - A governor implementing an always-on policy
402397
*/
403398
struct dev_power_governor pm_domain_always_on_gov = {
404-
.power_down_ok = always_on_power_down_ok,
405399
.suspend_ok = default_suspend_ok,
406400
};

0 commit comments

Comments
 (0)