Commit cd1aaef0 authored by Michał Kępień's avatar Michał Kępień Committed by Darren Hart

platform/x86: acer-wmi: remove sparse_keymap_free() calls

As sparse_keymap_setup() now uses a managed memory allocation for the
keymap copy it creates, the latter is freed automatically.  Remove all
calls to sparse_keymap_free().
Signed-off-by: default avatarMichał Kępień <kernel@kempniu.pl>
Signed-off-by: default avatarAndy Shevchenko <andriy.shevchenko@linux.intel.com>
parent 98f64777
...@@ -1987,7 +1987,7 @@ static int __init acer_wmi_input_setup(void) ...@@ -1987,7 +1987,7 @@ static int __init acer_wmi_input_setup(void)
acer_wmi_notify, NULL); acer_wmi_notify, NULL);
if (ACPI_FAILURE(status)) { if (ACPI_FAILURE(status)) {
err = -EIO; err = -EIO;
goto err_free_keymap; goto err_free_dev;
} }
err = input_register_device(acer_wmi_input_dev); err = input_register_device(acer_wmi_input_dev);
...@@ -1998,8 +1998,6 @@ static int __init acer_wmi_input_setup(void) ...@@ -1998,8 +1998,6 @@ static int __init acer_wmi_input_setup(void)
err_uninstall_notifier: err_uninstall_notifier:
wmi_remove_notify_handler(ACERWMID_EVENT_GUID); wmi_remove_notify_handler(ACERWMID_EVENT_GUID);
err_free_keymap:
sparse_keymap_free(acer_wmi_input_dev);
err_free_dev: err_free_dev:
input_free_device(acer_wmi_input_dev); input_free_device(acer_wmi_input_dev);
return err; return err;
...@@ -2008,7 +2006,6 @@ static int __init acer_wmi_input_setup(void) ...@@ -2008,7 +2006,6 @@ static int __init acer_wmi_input_setup(void)
static void acer_wmi_input_destroy(void) static void acer_wmi_input_destroy(void)
{ {
wmi_remove_notify_handler(ACERWMID_EVENT_GUID); wmi_remove_notify_handler(ACERWMID_EVENT_GUID);
sparse_keymap_free(acer_wmi_input_dev);
input_unregister_device(acer_wmi_input_dev); input_unregister_device(acer_wmi_input_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