Commit f6c7b803 authored by Hans de Goede's avatar Hans de Goede Committed by Andy Shevchenko

platform/x86: peaq-wmi: Remove unnecessary checks from peaq_wmi_exit

peaq_wmi_exit will only ever get called if peaq_wmi_init succeeds, so
there is no need to repeat the checks from peaq_wmi_init.
Suggested-by: default avatarAndy Shevchenko <andy.shevchenko@gmail.com>
Signed-off-by: default avatarHans de Goede <hdegoede@redhat.com>
Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
parent d6fa71f1
...@@ -66,7 +66,7 @@ static void peaq_wmi_poll(struct input_polled_dev *dev) ...@@ -66,7 +66,7 @@ static void peaq_wmi_poll(struct input_polled_dev *dev)
} }
/* Some other devices (Shuttle XS35) use the same WMI GUID for other purposes */ /* Some other devices (Shuttle XS35) use the same WMI GUID for other purposes */
static const struct dmi_system_id peaq_dmi_table[] = { static const struct dmi_system_id peaq_dmi_table[] __initconst = {
{ {
.matches = { .matches = {
DMI_MATCH(DMI_SYS_VENDOR, "PEAQ"), DMI_MATCH(DMI_SYS_VENDOR, "PEAQ"),
...@@ -102,12 +102,6 @@ static int __init peaq_wmi_init(void) ...@@ -102,12 +102,6 @@ static int __init peaq_wmi_init(void)
static void __exit peaq_wmi_exit(void) static void __exit peaq_wmi_exit(void)
{ {
if (!dmi_check_system(peaq_dmi_table))
return;
if (!wmi_has_guid(PEAQ_DOLBY_BUTTON_GUID))
return;
input_unregister_polled_device(peaq_poll_dev); input_unregister_polled_device(peaq_poll_dev);
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment