aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/platform/x86/wmi.c
diff options
context:
space:
mode:
authorZhang Rui <rui.zhang@intel.com>2013-09-02 20:32:07 -0400
committerRafael J. Wysocki <rafael.j.wysocki@intel.com>2013-09-23 19:37:56 -0400
commit54f14c277d78cd4fee860f5daf391a5759f8596f (patch)
tree376d17754d25ca9a711ae766d07d74371bd45b5f /drivers/platform/x86/wmi.c
parente2e196069fd68fcf8c20fd478f8b683e5403e575 (diff)
wmi: convert acpi_get_handle() to acpi_has_method()
acpi_has_method() is a new ACPI API introduced to check the existence of an ACPI control method. It can be used to replace acpi_get_handle() in the case that 1. the calling function doesn't need the ACPI handle of the control method. and 2. the calling function doesn't care the reason why the method is unavailable. Convert acpi_get_handle() to acpi_has_method() in drivers/platform/x86/wmi.c in this patch. Signed-off-by: Zhang Rui <rui.zhang@intel.com> CC: Matthew Garrett <matthew.garrett@nebula.com> Signed-off-by: Rafael J. Wysocki <rafael.j.wysocki@intel.com>
Diffstat (limited to 'drivers/platform/x86/wmi.c')
-rw-r--r--drivers/platform/x86/wmi.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/drivers/platform/x86/wmi.c b/drivers/platform/x86/wmi.c
index 3bb55060bd05..62e8c221d01e 100644
--- a/drivers/platform/x86/wmi.c
+++ b/drivers/platform/x86/wmi.c
@@ -347,7 +347,7 @@ struct acpi_buffer *out)
347{ 347{
348 struct guid_block *block = NULL; 348 struct guid_block *block = NULL;
349 struct wmi_block *wblock = NULL; 349 struct wmi_block *wblock = NULL;
350 acpi_handle handle, wc_handle; 350 acpi_handle handle;
351 acpi_status status, wc_status = AE_ERROR; 351 acpi_status status, wc_status = AE_ERROR;
352 struct acpi_object_list input; 352 struct acpi_object_list input;
353 union acpi_object wq_params[1]; 353 union acpi_object wq_params[1];
@@ -387,8 +387,7 @@ struct acpi_buffer *out)
387 * expensive, but have no corresponding WCxx method. So we 387 * expensive, but have no corresponding WCxx method. So we
388 * should not fail if this happens. 388 * should not fail if this happens.
389 */ 389 */
390 wc_status = acpi_get_handle(handle, wc_method, &wc_handle); 390 if (acpi_has_method(handle, wc_method))
391 if (ACPI_SUCCESS(wc_status))
392 wc_status = acpi_execute_simple_method(handle, 391 wc_status = acpi_execute_simple_method(handle,
393 wc_method, 1); 392 wc_method, 1);
394 } 393 }