summaryrefslogtreecommitdiffstats
path: root/drivers/platform/x86/wmi.c
diff options
context:
space:
mode:
authorBarnabás Pőcze <pobrn@protonmail.com>2021-09-04 19:56:29 +0200
committerHans de Goede <hdegoede@redhat.com>2021-09-14 12:26:02 +0200
commit736b48aae5e83b5fab16fc9f31354d2cf863aa79 (patch)
treef11a4e7c1543a07586442f991d1b0c4bc121e9ce /drivers/platform/x86/wmi.c
parentplatform/x86: wmi: do not fail if disabling fails (diff)
downloadlinux-736b48aae5e83b5fab16fc9f31354d2cf863aa79.tar.xz
linux-736b48aae5e83b5fab16fc9f31354d2cf863aa79.zip
platform/x86: wmi: simplify error handling logic
The current code carries out the following ACPI status mapping: AE_NOT_FOUND -> AE_OK AE_OK -> AE_OK AE_$X -> AE_$X That is, everything is mapped to itself, except AE_NOT_FOUND. The current code does not do it in the most straighforward way. Simplify the logic. Signed-off-by: Barnabás Pőcze <pobrn@protonmail.com> Link: https://lore.kernel.org/r/20210904175450.156801-26-pobrn@protonmail.com Reviewed-by: Hans de Goede <hdegoede@redhat.com> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Diffstat (limited to 'drivers/platform/x86/wmi.c')
-rw-r--r--drivers/platform/x86/wmi.c7
1 files changed, 3 insertions, 4 deletions
diff --git a/drivers/platform/x86/wmi.c b/drivers/platform/x86/wmi.c
index f4f68b31eb6a..7753836571fe 100644
--- a/drivers/platform/x86/wmi.c
+++ b/drivers/platform/x86/wmi.c
@@ -189,11 +189,10 @@ static acpi_status wmi_method_enable(struct wmi_block *wblock, bool enable)
snprintf(method, 5, "WE%02X", block->notify_id);
status = acpi_execute_simple_method(handle, method, enable);
-
- if (status != AE_OK && status != AE_NOT_FOUND)
- return status;
- else
+ if (status == AE_NOT_FOUND)
return AE_OK;
+
+ return status;
}
/*