aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorRafael J. Wysocki <rafael.j.wysocki@intel.com>2013-01-22 06:56:16 -0500
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2013-01-22 06:56:16 -0500
commit87e753b0065f94314ebdacf6593a172cdd7839c8 (patch)
treec2c44ee42101182bcf811019c92ee1a89025654c
parente78adb7595a9d585c60a7497345cb6eaeaaacefb (diff)
ACPI / PM: Sanitize checks in acpi_power_on_resources()
After the only user of acpi_power_on_resources(), acpi_bus_init_power(), has been changed to avoid calling it for state equal to ACPI_STATE_D3_COLD, it doesn't have to special case that state any more. For this reason, modify the checks in acpi_power_on_resources() so that it returns -EINVAL for ACPI_STATE_D3_COLD as it should. Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
-rw-r--r--drivers/acpi/power.c5
1 files changed, 1 insertions, 4 deletions
diff --git a/drivers/acpi/power.c b/drivers/acpi/power.c
index 089a7c39348f..6db261c237d4 100644
--- a/drivers/acpi/power.c
+++ b/drivers/acpi/power.c
@@ -622,12 +622,9 @@ int acpi_power_get_inferred_state(struct acpi_device *device, int *state)
622 622
623int acpi_power_on_resources(struct acpi_device *device, int state) 623int acpi_power_on_resources(struct acpi_device *device, int state)
624{ 624{
625 if (!device || state < ACPI_STATE_D0 || state > ACPI_STATE_D3_COLD) 625 if (!device || state < ACPI_STATE_D0 || state > ACPI_STATE_D3_HOT)
626 return -EINVAL; 626 return -EINVAL;
627 627
628 if (state == ACPI_STATE_D3_COLD)
629 return 0;
630
631 return acpi_power_on_list(&device->power.states[state].resources); 628 return acpi_power_on_list(&device->power.states[state].resources);
632} 629}
633 630