[prev in list] [next in list] [prev in thread] [next in thread] 

List:       linux-acpi
Subject:    ACPI LID wakeup device prevents shutdown on Toshiba laptop and
From:       Anisse Astier <anisse () astier ! eu>
Date:       2011-07-28 15:47:59
Message-ID: 20110728174759.292311cc () destiny ! ordissimo
[Download RAW message or body]

Hi,

(See reference: https://bugzilla.kernel.org/show_bug.cgi?id=35262 )

On Toshiba C670D-10C, and as reported on a desktop asus mainboard
computer, we cannot shutdown without the computer rebooting.

The fix for this is simple, just run:
echo " LID" > /proc/acpi/wakeup
before shutdown. Please see below for quick and dirty in-kernel
implementation.

Now I don't know why is that. Perhaps Windows disables (some) wakeup
devices at shutdown, therefore the OEM/BIOS vendor expected any OS to
behave this way in order to halt properly.
Or maybe there's another reason.

I'd like comments on the idea implemented on the patch below, and maybe
suggestions to do this in a better way. 

Thanks,

Anisse

--

From: Anisse Astier <anisse@astier.eu>
Subject: [PATCH] ACPI / PM : Disable ACPI LID wakeup device at shutdown

This device might cause shutdown to be a reboot on some computers:
Toshiba C670D-10C laptop, Asus desktop mainboard.
Device will shutdown, then wakeup immediately afterwards, regardless of
the status of the LID.

Reference: https://bugzilla.kernel.org/show_bug.cgi?id=35262
---
 drivers/acpi/sleep.c |   25 +++++++++++++++++++++++++
 1 files changed, 25 insertions(+), 0 deletions(-)

diff --git a/drivers/acpi/sleep.c b/drivers/acpi/sleep.c
index 6c94960..3859628 100644
--- a/drivers/acpi/sleep.c
+++ b/drivers/acpi/sleep.c
@@ -58,6 +58,30 @@ static struct notifier_block tts_notifier = {
 	.priority	= 0,
 };
 
+static void acpi_disable_obnoxious_wakeup_device(u32 sleep_state)
+{
+	struct list_head *node, *next;
+
+	list_for_each_safe(node, next, &acpi_wakeup_device_list) {
+		struct acpi_device *dev =
+			container_of(node, struct acpi_device, wakeup_list);
+
+		if (!dev->wakeup.flags.valid
+		    || sleep_state > (u32) dev->wakeup.sleep_state
+		    || !(device_may_wakeup(&dev->dev)
+		        || dev->wakeup.prepare_count))
+			continue;
+		if (!strcmp(dev->pnp.bus_id, "LID")) {
+			acpi_set_gpe_wake_mask(dev->wakeup.gpe_device,
+					       dev->wakeup.gpe_number,
+					       ACPI_GPE_DISABLE);
+
+			if (device_may_wakeup(&dev->dev))
+				acpi_disable_wakeup_device_power(dev);
+			}
+	}
+}
+
 static int acpi_sleep_prepare(u32 acpi_state)
 {
 #ifdef CONFIG_ACPI_SLEEP
@@ -75,6 +99,7 @@ static int acpi_sleep_prepare(u32 acpi_state)
 	printk(KERN_INFO PREFIX "Preparing to enter system sleep state S%d\n",
 		acpi_state);
 	acpi_enable_wakeup_devices(acpi_state);
+	acpi_disable_obnoxious_wakeup_device(acpi_state);
 	acpi_enter_sleep_state_prep(acpi_state);
 	return 0;
 }
-- 
1.7.5.3

--
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
[prev in list] [next in list] [prev in thread] [next in thread]