acer-wmi: support integer return type from WMI methods
authorLee, Chun-Yi <joeyli.kernel@gmail.com>
Fri, 27 May 2011 06:52:14 +0000 (14:52 +0800)
committerMatthew Garrett <mjg@redhat.com>
Fri, 27 May 2011 16:40:10 +0000 (12:40 -0400)
Acer WMID_GUID1/2 method's return value was declared to integer
type on Gateway notebook.
So, add this patch for support integer return type.

Reference: bko#33032
https://bugzilla.kernel.org/show_bug.cgi?id=33032

Tested on Gateway NV5909H laptop

Tested-by: Filipus Klutiero <chealer@gmail.com>
Cc: Carlos Corbacho <carlos@strangeworlds.co.uk>
Cc: Matthew Garrett <mjg@redhat.com>
Cc: Dmitry Torokhov <dtor@mail.ru>
Cc: Corentin Chary <corentincj@iksaif.net>
Cc: Thomas Renninger <trenn@suse.de>
Signed-off-by: Lee, Chun-Yi <jlee@novell.com>
Signed-off-by: Matthew Garrett <mjg@redhat.com>
drivers/platform/x86/acer-wmi.c

index 5dc843bfc6c22c23337c0e5eaa411142aa25cbcb..005417bd429e815f24cce88c23b4f53ef46afc74 100644 (file)
@@ -754,6 +754,8 @@ WMI_execute_u32(u32 method_id, u32 in, u32 *out)
                (obj->buffer.length == sizeof(u32) ||
                obj->buffer.length == sizeof(u64))) {
                tmp = *((u32 *) obj->buffer.pointer);
+       } else if (obj->type == ACPI_TYPE_INTEGER) {
+               tmp = (u32) obj->integer.value;
        } else {
                tmp = 0;
        }
@@ -884,6 +886,8 @@ static acpi_status WMID_set_capabilities(void)
                (obj->buffer.length == sizeof(u32) ||
                obj->buffer.length == sizeof(u64))) {
                devices = *((u32 *) obj->buffer.pointer);
+       } else if (obj->type == ACPI_TYPE_INTEGER) {
+               devices = (u32) obj->integer.value;
        } else {
                kfree(out.pointer);
                return AE_ERROR;
@@ -1636,6 +1640,8 @@ static u32 get_wmid_devices(void)
                (obj->buffer.length == sizeof(u32) ||
                obj->buffer.length == sizeof(u64))) {
                devices = *((u32 *) obj->buffer.pointer);
+       } else if (obj->type == ACPI_TYPE_INTEGER) {
+               devices = (u32) obj->integer.value;
        }
 
        kfree(out.pointer);