Commit 21397cac authored by Barnabás Pőcze's avatar Barnabás Pőcze Committed by Hans de Goede
Browse files

platform/x86: wmi: remove unnecessary variable 



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: default avatarBarnabás Pőcze <pobrn@protonmail.com>
Link: https://lore.kernel.org/r/20210904175450.156801-7-pobrn@protonmail.com


Reviewed-by: default avatarHans de Goede <hdegoede@redhat.com>
Signed-off-by: default avatarHans de Goede <hdegoede@redhat.com>
parent 43aacf83

drivers/platform/x86/wmi.c

+1 −4
Original line number Diff line number Diff line
@@ -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);