This is a note to let you know that I've just added the patch titled

    ACPI / PM: Do not execute _PS0 for devices without _PSC during 
initialization

to the 3.9-stable tree which can be found at:
    
http://www.kernel.org/git/?p=linux/kernel/git/stable/stable-queue.git;a=summary

The filename of the patch is:
     
acpi-pm-do-not-execute-_ps0-for-devices-without-_psc-during-initialization.patch
and it can be found in the queue-3.9 subdirectory.

If you, or anyone else, feels it should not be added to the stable tree,
please let <[email protected]> know about it.


>From 7cd8407d53ef5fb0280fcbe34f42311472f90feb Mon Sep 17 00:00:00 2001
From: "Rafael J. Wysocki" <[email protected]>
Date: Wed, 5 Jun 2013 14:01:19 +0200
Subject: ACPI / PM: Do not execute _PS0 for devices without _PSC during 
initialization

From: "Rafael J. Wysocki" <[email protected]>

commit 7cd8407d53ef5fb0280fcbe34f42311472f90feb upstream.

Commit b378549 (ACPI / PM: Do not power manage devices in unknown
initial states) added code to force devices without _PSC, but having
_PS0 defined in the ACPI namespace, into ACPI power state D0 by
executing _PS0 for them.  That turned out to break Toshiba P870-303,
however, so revert that code.

References: https://bugzilla.kernel.org/show_bug.cgi?id=58201
Reported-and-tested-by: Jerome Cantenot <[email protected]>
Tracked-down-by: Lan Tianyu <[email protected]>
Signed-off-by: Rafael J. Wysocki <[email protected]>
Signed-off-by: Greg Kroah-Hartman <[email protected]>

---
 drivers/acpi/device_pm.c |   10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

--- a/drivers/acpi/device_pm.c
+++ b/drivers/acpi/device_pm.c
@@ -269,11 +269,13 @@ int acpi_bus_init_power(struct acpi_devi
                if (result)
                        return result;
        } else if (state == ACPI_STATE_UNKNOWN) {
-               /* No power resources and missing _PSC? Try to force D0. */
+               /*
+                * No power resources and missing _PSC?  Cross fingers and make
+                * it D0 in hope that this is what the BIOS put the device into.
+                * [We tried to force D0 here by executing _PS0, but that broke
+                * Toshiba P870-303 in a nasty way.]
+                */
                state = ACPI_STATE_D0;
-               result = acpi_dev_pm_explicit_set(device, state);
-               if (result)
-                       return result;
        }
        device->power.state = state;
        return 0;


Patches currently in stable-queue which might be from 
[email protected] are

queue-3.9/acpi-video-ignore-bios-initial-backlight-value-for-hp-m4.patch
queue-3.9/acpi-video-ignore-bios-initial-backlight-value-for-hp-pavilion-g6.patch
queue-3.9/acpi-cpufreq-set-current-frequency-based-on-target-p-state.patch
queue-3.9/acpi-scan-do-not-match-drivers-against-objects-having-scan-handlers.patch
queue-3.9/x86-platform-hp_wmi-fix-bluetooth_rfkill-misuse-in-hp_wmi_rfkill_setup.patch
queue-3.9/acpi-pm-do-not-execute-_ps0-for-devices-without-_psc-during-initialization.patch
--
To unsubscribe from this list: send the line "unsubscribe stable" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to