platform/x86: dell-wmi-aio: remove sparse_keymap_free() calls
authorMichał Kępień <kernel@kempniu.pl>
Thu, 9 Mar 2017 12:11:40 +0000 (13:11 +0100)
committerDarren Hart <dvhart@infradead.org>
Wed, 15 Mar 2017 05:58:04 +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/dell-wmi-aio.c

index dbc97a3..50c2078 100644 (file)
@@ -152,12 +152,10 @@ static int __init dell_wmi_aio_input_setup(void)
        err = input_register_device(dell_wmi_aio_input_dev);
        if (err) {
                pr_info("Unable to register input device\n");
-               goto err_free_keymap;
+               goto err_free_dev;
        }
        return 0;
 
-err_free_keymap:
-       sparse_keymap_free(dell_wmi_aio_input_dev);
 err_free_dev:
        input_free_device(dell_wmi_aio_input_dev);
        return err;
@@ -192,7 +190,6 @@ static int __init dell_wmi_aio_init(void)
        err = wmi_install_notify_handler(guid, dell_wmi_aio_notify, NULL);
        if (err) {
                pr_err("Unable to register notify handler - %d\n", err);
-               sparse_keymap_free(dell_wmi_aio_input_dev);
                input_unregister_device(dell_wmi_aio_input_dev);
                return err;
        }
@@ -206,7 +203,6 @@ static void __exit dell_wmi_aio_exit(void)
 
        guid = dell_wmi_aio_find();
        wmi_remove_notify_handler(guid);
-       sparse_keymap_free(dell_wmi_aio_input_dev);
        input_unregister_device(dell_wmi_aio_input_dev);
 }