projects
/
linux.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
| inline |
side by side
(parent:
43aacf8
)
platform/x86: wmi: remove unnecessary variable
author
Barnabás Pőcze
<pobrn@protonmail.com>
Sat, 4 Sep 2021 17:55:13 +0000
(17:55 +0000)
committer
Hans 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
patch
|
blob
|
history
diff --git
a/drivers/platform/x86/wmi.c
b/drivers/platform/x86/wmi.c
index 943f30eb45c3ebcd3eddb0d0a329340381eb6bda..97d46da47ff7eb0c0de093cd7c1448381145829b 100644
(file)
--- a/
drivers/platform/x86/wmi.c
+++ b/
drivers/platform/x86/wmi.c
@@
-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);