diff options
| author | Konstantin Karasyov <konstantin.a.karasyov@intel.com> | 2007-02-21 02:05:58 -0500 |
|---|---|---|
| committer | Len Brown <len.brown@intel.com> | 2007-02-21 02:05:58 -0500 |
| commit | 7292576043666ff39946dee14641fe719ba8c7e8 (patch) | |
| tree | 65e21f949797169d7c2bee2b0794627ffb0407a7 /drivers/acpi/power.c | |
| parent | 255f0385c8e0d6b9005c0e09fffb5bd852f3b506 (diff) | |
| download | lwn-7292576043666ff39946dee14641fe719ba8c7e8.tar.gz lwn-7292576043666ff39946dee14641fe719ba8c7e8.zip | |
ACPI: fix S3 fan resume issue
http://bugzilla.kernel.org/show_bug.cgi?id=7570#c14
Signed-off-by: Konstantin Karasyov <konstantin.a.karasyov@intel.com>
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers/acpi/power.c')
| -rw-r--r-- | drivers/acpi/power.c | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/drivers/acpi/power.c b/drivers/acpi/power.c index 1ef338545dfe..547eee7537bc 100644 --- a/drivers/acpi/power.c +++ b/drivers/acpi/power.c @@ -436,8 +436,6 @@ int acpi_power_transition(struct acpi_device *device, int state) cl = &device->power.states[device->power.state].resources; tl = &device->power.states[state].resources; - device->power.state = ACPI_STATE_UNKNOWN; - if (!cl->count && !tl->count) { result = -ENODEV; goto end; @@ -468,12 +466,15 @@ int acpi_power_transition(struct acpi_device *device, int state) goto end; } - /* We shouldn't change the state till all above operations succeed */ - device->power.state = state; - end: - if (result) + end: + if (result) { + device->power.state = ACPI_STATE_UNKNOWN; printk(KERN_WARNING PREFIX "Transitioning device [%s] to D%d\n", device->pnp.bus_id, state); + } else { + /* We shouldn't change the state till all above operations succeed */ + device->power.state = state; + } return result; } |