Thread (3 messages) 3 messages, 3 authors, 2021-02-01

Re: [PATCH] PM: domains: Simplify the calculation of variables

From: "Rafael J. Wysocki" <rafael@kernel.org>
Date: 2021-02-01 14:07:07
Also in: lkml

On Mon, Feb 1, 2021 at 11:11 AM Ulf Hansson [off-list ref] wrote:
On Wed, 27 Jan 2021 at 09:42, Abaci Team [off-list ref] wrote:
quoted
Fix the following coccicheck warnings:

./drivers/base/power/domain.c:938:31-33: WARNING !A || A && B is
equivalent to !A || B.

Reported-by: Abaci Robot <redacted>
Suggested-by: Jiapeng Zhong <redacted>
Signed-off-by: Abaci Team <redacted>
Acked-by: Ulf Hansson <redacted>
Applied as 5.12 material, thanks!
quoted
---
 drivers/base/power/domain.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/base/power/domain.c b/drivers/base/power/domain.c
index 9a14eed..e689710 100644
--- a/drivers/base/power/domain.c
+++ b/drivers/base/power/domain.c
@@ -934,8 +934,7 @@ static int genpd_runtime_resume(struct device *dev)
 err_stop:
        genpd_stop_dev(genpd, dev);
 err_poweroff:
-       if (!pm_runtime_is_irq_safe(dev) ||
-               (pm_runtime_is_irq_safe(dev) && genpd_is_irq_safe(genpd))) {
+       if (!pm_runtime_is_irq_safe(dev) || genpd_is_irq_safe(genpd)) {
                genpd_lock(genpd);
                genpd_power_off(genpd, true, 0);
                genpd_unlock(genpd);
--
1.8.3.1
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help