diff options
author | Zhao Yakui <yakui.zhao@intel.com> | 2009-12-15 09:01:57 -0500 |
---|---|---|
committer | Len Brown <len.brown@intel.com> | 2009-12-16 01:12:35 -0500 |
commit | 13c199c0d0cf78b27592991129fb8cbcfc5164de (patch) | |
tree | 0f205e067541102085580224b22b73c2056fceca /drivers/acpi | |
parent | 22763c5cf3690a681551162c15d34d935308c8d7 (diff) |
ACPI: Use the return result of ACPI lid notifier chain correctly
On some laptops it will return NOTIFY_OK(non-zero) when calling the ACPI LID
notifier. Then it is used as the result of ACPI LID resume function, which
will complain the following warning message in course of suspend/resume:
>PM: Device PNP0C0D:00 failed to resume: error 1
This patch is to eliminate the above warning message.
http://bugzilla.kernel.org/show_bug.cgi?id=14782
Signed-off-by: Zhao Yakui <yakui.zhao@intel.com>
Signed-off-by: Len Brown <len.brown@intel.com>
Diffstat (limited to 'drivers/acpi')
-rw-r--r-- | drivers/acpi/button.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/drivers/acpi/button.c b/drivers/acpi/button.c index 0c9c6a9a002c..8a95e8329df7 100644 --- a/drivers/acpi/button.c +++ b/drivers/acpi/button.c | |||
@@ -282,6 +282,13 @@ static int acpi_lid_send_state(struct acpi_device *device) | |||
282 | if (ret == NOTIFY_DONE) | 282 | if (ret == NOTIFY_DONE) |
283 | ret = blocking_notifier_call_chain(&acpi_lid_notifier, state, | 283 | ret = blocking_notifier_call_chain(&acpi_lid_notifier, state, |
284 | device); | 284 | device); |
285 | if (ret == NOTIFY_DONE || ret == NOTIFY_OK) { | ||
286 | /* | ||
287 | * It is also regarded as success if the notifier_chain | ||
288 | * returns NOTIFY_OK or NOTIFY_DONE. | ||
289 | */ | ||
290 | ret = 0; | ||
291 | } | ||
285 | return ret; | 292 | return ret; |
286 | } | 293 | } |
287 | 294 | ||