|
Takashi Iwai |
ccf980 |
From 452a3e723f75880757acf87b053935c43aa89f89 Mon Sep 17 00:00:00 2001
|
|
Takashi Iwai |
ccf980 |
From: "Rafael J. Wysocki" <rafael.j.wysocki@intel.com>
|
|
Takashi Iwai |
ccf980 |
Date: Thu, 4 Nov 2021 22:54:17 +0100
|
|
Takashi Iwai |
ccf980 |
Subject: [PATCH] ACPI: PM: Fix device wakeup power reference counting error
|
|
Takashi Iwai |
ccf980 |
Git-commit: 452a3e723f75880757acf87b053935c43aa89f89
|
|
Takashi Iwai |
ccf980 |
Patch-mainline: v5.16-rc1
|
|
Takashi Iwai |
001336 |
References: git-fixes stable-5.14.19
|
|
Takashi Iwai |
ccf980 |
|
|
Takashi Iwai |
ccf980 |
Fix a device wakeup power reference counting error introduced by
|
|
Takashi Iwai |
ccf980 |
commit a2d7b2e004af ("ACPI: PM: Fix sharing of wakeup power
|
|
Takashi Iwai |
ccf980 |
resources") because of a coding mistake.
|
|
Takashi Iwai |
ccf980 |
|
|
Takashi Iwai |
ccf980 |
Fixes: a2d7b2e004af ("ACPI: PM: Fix sharing of wakeup power resources")
|
|
Takashi Iwai |
ccf980 |
Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
|
|
Takashi Iwai |
ccf980 |
Acked-by: Takashi Iwai <tiwai@suse.de>
|
|
Takashi Iwai |
ccf980 |
|
|
Takashi Iwai |
ccf980 |
---
|
|
Takashi Iwai |
ccf980 |
drivers/acpi/power.c | 8 +++-----
|
|
Takashi Iwai |
ccf980 |
1 file changed, 3 insertions(+), 5 deletions(-)
|
|
Takashi Iwai |
ccf980 |
|
|
Takashi Iwai |
ccf980 |
diff --git a/drivers/acpi/power.c b/drivers/acpi/power.c
|
|
Takashi Iwai |
ccf980 |
index 112256154880..5dcb02ededbc 100644
|
|
Takashi Iwai |
ccf980 |
--- a/drivers/acpi/power.c
|
|
Takashi Iwai |
ccf980 |
+++ b/drivers/acpi/power.c
|
|
Takashi Iwai |
ccf980 |
@@ -757,13 +757,11 @@ int acpi_disable_wakeup_device_power(struct acpi_device *dev)
|
|
Takashi Iwai |
ccf980 |
|
|
Takashi Iwai |
ccf980 |
mutex_lock(&acpi_device_lock);
|
|
Takashi Iwai |
ccf980 |
|
|
Takashi Iwai |
ccf980 |
- if (dev->wakeup.prepare_count > 1) {
|
|
Takashi Iwai |
ccf980 |
- dev->wakeup.prepare_count--;
|
|
Takashi Iwai |
ccf980 |
+ /* Do nothing if wakeup power has not been enabled for this device. */
|
|
Takashi Iwai |
ccf980 |
+ if (dev->wakeup.prepare_count <= 0)
|
|
Takashi Iwai |
ccf980 |
goto out;
|
|
Takashi Iwai |
ccf980 |
- }
|
|
Takashi Iwai |
ccf980 |
|
|
Takashi Iwai |
ccf980 |
- /* Do nothing if wakeup power has not been enabled for this device. */
|
|
Takashi Iwai |
ccf980 |
- if (!dev->wakeup.prepare_count)
|
|
Takashi Iwai |
ccf980 |
+ if (--dev->wakeup.prepare_count > 0)
|
|
Takashi Iwai |
ccf980 |
goto out;
|
|
Takashi Iwai |
ccf980 |
|
|
Takashi Iwai |
ccf980 |
err = acpi_device_sleep_wake(dev, 0, 0, 0);
|
|
Takashi Iwai |
ccf980 |
--
|
|
Takashi Iwai |
ccf980 |
2.26.2
|
|
Takashi Iwai |
ccf980 |
|