ACPI / PM: Change the way power transitions to D3cold are carried out
authorRafael J. Wysocki <rafael.j.wysocki@intel.com>
Thu, 17 Jan 2013 13:11:08 +0000 (14:11 +0100)
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>
Thu, 17 Jan 2013 13:11:08 +0000 (14:11 +0100)
During power transitions into D3cold from any shallower power states
we are supposed to transition the device into D3hot and remove power
from it afterward, but the current code in acpi_device_set_power()
doesn't work this way.

At the same time, though, we need to be careful enough to preserve
backwards compatibility for systems that don't distinguish between
D3hot and D3cold (e.g. designed before ACPI 4).

Modify acpi_device_set_power() so that it works in accordance with
the expectations in both cases.

Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
drivers/acpi/bus.c

index 1f0d457ecbcf851a1805916f01ea931672146420..8e57fc49726ebafcd0bf6ba5c6043c649742c6e9 100644 (file)
@@ -270,6 +270,7 @@ int acpi_device_set_power(struct acpi_device *device, int state)
        int result = 0;
        acpi_status status = AE_OK;
        char object_name[5] = { '_', 'P', 'S', '0' + state, '\0' };
+       bool cut_power = false;
 
        if (!device || (state < ACPI_STATE_D0) || (state > ACPI_STATE_D3_COLD))
                return -EINVAL;
@@ -294,9 +295,13 @@ int acpi_device_set_power(struct acpi_device *device, int state)
                return -ENODEV;
        }
 
-       /* For D3cold we should execute _PS3, not _PS4. */
-       if (state == ACPI_STATE_D3_COLD)
+       /* For D3cold we should first transition into D3hot. */
+       if (state == ACPI_STATE_D3_COLD
+           && device->power.states[ACPI_STATE_D3_COLD].flags.os_accessible) {
+               state = ACPI_STATE_D3_HOT;
                object_name[3] = '3';
+               cut_power = true;
+       }
 
        /*
         * Transition Power
@@ -341,6 +346,9 @@ int acpi_device_set_power(struct acpi_device *device, int state)
                }
        }
 
+       if (cut_power)
+               result = acpi_power_transition(device, ACPI_STATE_D3_COLD);
+
       end:
        if (result)
                printk(KERN_WARNING PREFIX