platform/x86: hp-wmi: remove sparse_keymap_free() calls
authorMichał Kępień <kernel@kempniu.pl>
Thu, 9 Mar 2017 12:11:43 +0000 (13:11 +0100)
committerDarren Hart <dvhart@infradead.org>
Wed, 15 Mar 2017 05:58:06 +0000 (22:58 -0700)
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: Michał Kępień <kernel@kempniu.pl>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
drivers/platform/x86/hp-wmi.c

index 96ffda493266aa68ef57958eb482b9b0c28900ad..7abf92d0ba81ce9f33a2b2b48f30a17fd809da0f 100644 (file)
@@ -673,7 +673,7 @@ static int __init hp_wmi_input_setup(void)
        status = wmi_install_notify_handler(HPWMI_EVENT_GUID, hp_wmi_notify, NULL);
        if (ACPI_FAILURE(status)) {
                err = -EIO;
-               goto err_free_keymap;
+               goto err_free_dev;
        }
 
        err = input_register_device(hp_wmi_input_dev);
@@ -684,8 +684,6 @@ static int __init hp_wmi_input_setup(void)
 
  err_uninstall_notifier:
        wmi_remove_notify_handler(HPWMI_EVENT_GUID);
- err_free_keymap:
-       sparse_keymap_free(hp_wmi_input_dev);
  err_free_dev:
        input_free_device(hp_wmi_input_dev);
        return err;
@@ -694,7 +692,6 @@ static int __init hp_wmi_input_setup(void)
 static void hp_wmi_input_destroy(void)
 {
        wmi_remove_notify_handler(HPWMI_EVENT_GUID);
-       sparse_keymap_free(hp_wmi_input_dev);
        input_unregister_device(hp_wmi_input_dev);
 }