platform/x86: msi-laptop: Fix resource cleanup
authorHans de Goede <hdegoede@redhat.com>
Thu, 25 Aug 2022 14:13:36 +0000 (16:13 +0200)
committerHans de Goede <hdegoede@redhat.com>
Fri, 26 Aug 2022 10:59:47 +0000 (12:59 +0200)
Fix the input-device not getting free-ed on probe-errors and
fix the msi_touchpad_dwork not getting cancelled on neither
probe-errors nor on remove.

Fixes: 143a4c0284dc ("msi-laptop: send out touchpad on/off key")
Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Link: https://lore.kernel.org/r/20220825141336.208597-3-hdegoede@redhat.com
drivers/platform/x86/msi-laptop.c

index 65db18c6e3e8fcad83276534a4081086157f2def..5d4b10b8d7719fea4acc04dd00b0634c451d4770 100644 (file)
@@ -1106,6 +1106,8 @@ fail_create_attr:
 fail_create_group:
        if (quirks->load_scm_model) {
                i8042_remove_filter(msi_laptop_i8042_filter);
+               cancel_delayed_work_sync(&msi_touchpad_dwork);
+               input_unregister_device(msi_laptop_input_dev);
                cancel_delayed_work_sync(&msi_rfkill_dwork);
                rfkill_cleanup();
        }
@@ -1125,6 +1127,7 @@ static void __exit msi_cleanup(void)
 {
        if (quirks->load_scm_model) {
                i8042_remove_filter(msi_laptop_i8042_filter);
+               cancel_delayed_work_sync(&msi_touchpad_dwork);
                input_unregister_device(msi_laptop_input_dev);
                cancel_delayed_work_sync(&msi_rfkill_dwork);
                rfkill_cleanup();