platform/x86: wmi: remove unnecessary variable
authorBarnabás Pőcze <pobrn@protonmail.com>
Sat, 4 Sep 2021 17:55:13 +0000 (17:55 +0000)
committerHans de Goede <hdegoede@redhat.com>
Tue, 14 Sep 2021 10:26:00 +0000 (12:26 +0200)
The `status` variable was assigned at the end, and then
immediately returned. Remove it altogether, and return
the previously assigned value directly.

Signed-off-by: Barnabás Pőcze <pobrn@protonmail.com>
Link: https://lore.kernel.org/r/20210904175450.156801-7-pobrn@protonmail.com
Reviewed-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
drivers/platform/x86/wmi.c

index 943f30eb45c3ebcd3eddb0d0a329340381eb6bda..97d46da47ff7eb0c0de093cd7c1448381145829b 100644 (file)
@@ -254,7 +254,6 @@ acpi_status wmidev_evaluate_method(struct wmi_device *wdev, u8 instance,
        struct guid_block *block;
        struct wmi_block *wblock;
        acpi_handle handle;
-       acpi_status status;
        struct acpi_object_list input;
        union acpi_object params[3];
        char method[5] = "WM";
@@ -290,9 +289,7 @@ acpi_status wmidev_evaluate_method(struct wmi_device *wdev, u8 instance,
 
        strncat(method, block->object_id, 2);
 
-       status = acpi_evaluate_object(handle, method, &input, out);
-
-       return status;
+       return acpi_evaluate_object(handle, method, &input, out);
 }
 EXPORT_SYMBOL_GPL(wmidev_evaluate_method);