It turns out that the Apple Mac Mini comes back from suspend in legacy 
mode.

The simplest and most obvious fix would seem to be the following trivial 
one-liner, which just enables ACPI mode after any suspend event.

Comments? Can anybody see any downsides to just doing something this 
obvious, and doing it unconditionally?  I've seen a much more complicated 
patch that only does this for Apple hardware, but I wouldn't be surprised 
at all if this is more common.

In fact, googling for the symptoms ("nobody cared" and "acpi_irq" and 
"suspend") gives quite a number of hits, so I would not be surprised at 
all if this is not Apple-related at all, but that we should always have 
done this.

Comments? 

                Linus

---
diff --git a/drivers/acpi/sleep/main.c b/drivers/acpi/sleep/main.c
index 56f861e..7b6c146 100644
--- a/drivers/acpi/sleep/main.c
+++ b/drivers/acpi/sleep/main.c
@@ -109,6 +108,11 @@ static int acpi_pm_enter(suspend_state_t
        local_irq_restore(flags);
        printk(KERN_DEBUG "Back to C!\n");
 
+       /*
+        * Make sure we're back in ACPI mode!
+        */
+       acpi_enable();          
+
        /* restore processor state
         * We should only be here if we're coming back from STR or STD.
         * And, in the case of the latter, the memory image should have already
-
To unsubscribe from this list: send the line "unsubscribe linux-acpi" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to