Re: [PATCH 3/3] PM / Domains: Take WAKEUP_PATH driver flag into account in genpd
From: Ulf Hansson <hidden>
Date: 2017-12-15 16:02:45
Also in:
linux-renesas-soc
On 15 December 2017 at 16:56, Ulf Hansson [off-list ref] wrote:
In case the WAKEUP_PATH flag has been set in a later phase than from the ->suspend() callback, the PM core want set the ->power.wakeup_path status
/s/want/don't If another version is needed I fix, else perhaps you can just amend the patch!? Kind regards Uffe
quoted hunk ↗ jump to hunk
flag for the device. Therefore, let's be safe and check it explicitly. Signed-off-by: Ulf Hansson <redacted> --- drivers/base/power/domain.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-)diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c index f9dcc98..32b4ba7 100644 --- a/drivers/base/power/domain.c +++ b/drivers/base/power/domain.c@@ -1038,7 +1038,9 @@ static int genpd_finish_suspend(struct device *dev, bool poweroff) if (IS_ERR(genpd)) return -EINVAL; - if (dev->power.wakeup_path && genpd_is_active_wakeup(genpd)) + if ((dev->power.wakeup_path || + dev_pm_test_driver_flags(dev, DPM_FLAG_WAKEUP_PATH)) && + genpd_is_active_wakeup(genpd)) return 0; if (poweroff)@@ -1093,7 +1095,9 @@ static int genpd_resume_noirq(struct device *dev) if (IS_ERR(genpd)) return -EINVAL; - if (dev->power.wakeup_path && genpd_is_active_wakeup(genpd)) + if ((dev->power.wakeup_path || + dev_pm_test_driver_flags(dev, DPM_FLAG_WAKEUP_PATH)) && + genpd_is_active_wakeup(genpd)) return 0; genpd_lock(genpd); --2.7.4