diff options
author | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2013-01-22 06:56:16 -0500 |
---|---|---|
committer | Rafael J. Wysocki <rafael.j.wysocki@intel.com> | 2013-01-22 06:56:16 -0500 |
commit | 87e753b0065f94314ebdacf6593a172cdd7839c8 (patch) | |
tree | c2c44ee42101182bcf811019c92ee1a89025654c | |
parent | e78adb7595a9d585c60a7497345cb6eaeaaacefb (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.c | 5 |
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 | ||
623 | int acpi_power_on_resources(struct acpi_device *device, int state) | 623 | int 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 | ||