ACPI: Separate disabling of GPEs from _PTS
authorRafael J. Wysocki <rjw@sisk.pl>
Mon, 7 Jan 2008 23:06:16 +0000 (00:06 +0100)
committerLen Brown <len.brown@intel.com>
Fri, 1 Feb 2008 23:30:56 +0000 (18:30 -0500)
The preparation to enter an ACPI system sleep state is now tied to
the disabling of GPEs, but the GPEs should not be disabled before
suspending devices.  Since on ACPI 1.0x systems the _PTS global
control method should be executed before suspending devices, we
need to disable GPEs separately.

Signed-off-by: Rafael J. Wysocki <rjw@sisk.pl>
Acked-by: Pavel Machek <pavel@suse.cz>
Signed-off-by: Len Brown <len.brown@intel.com>
drivers/acpi/hardware/hwsleep.c
drivers/acpi/sleep/main.c

index 13c93a13785e3e56d415d1f363fa678ba8956adc..fd1c4ba63367469f9826460c6b545407491f9f86 100644 (file)
@@ -229,10 +229,6 @@ acpi_status acpi_enter_sleep_state_prep(u8 sleep_state)
                                "While executing method _SST"));
        }
 
-       /* Disable/Clear all GPEs */
-
-       status = acpi_hw_disable_all_gpes();
-
        return_ACPI_STATUS(status);
 }
 
index fdd8139b3f98926ce8f37061ae9e515e585dbbc7..198ff8a1529a1b7e0a5689c2d73f23570f241cb1 100644 (file)
@@ -91,10 +91,13 @@ static int acpi_pm_begin(suspend_state_t pm_state)
 
 static int acpi_pm_prepare(void)
 {
-       int error = acpi_sleep_prepare(acpi_target_sleep_state);
+       int error;
 
+       error = acpi_sleep_prepare(acpi_target_sleep_state);
        if (error)
                acpi_target_sleep_state = ACPI_STATE_S0;
+       else if (!ACPI_SUCCESS(acpi_hw_disable_all_gpes()))
+               error = -EFAULT;
 
        return error;
 }
@@ -261,7 +264,16 @@ static int acpi_hibernation_start(void)
 
 static int acpi_hibernation_prepare(void)
 {
-       return acpi_sleep_prepare(ACPI_STATE_S4);
+       int error;
+
+       error = acpi_sleep_prepare(ACPI_STATE_S4);
+       if (error)
+               return error;
+
+       if (!ACPI_SUCCESS(acpi_hw_disable_all_gpes()))
+               error = -EFAULT;
+
+       return error;
 }
 
 static int acpi_hibernation_enter(void)
@@ -426,6 +438,7 @@ static void acpi_power_off_prepare(void)
 {
        /* Prepare to power off the system */
        acpi_sleep_prepare(ACPI_STATE_S5);
+       acpi_hw_disable_all_gpes();
 }
 
 static void acpi_power_off(void)