platform/x86: asus-wmi: Fix hwmon device cleanup
authorYurii Pavlovskyi <yurii.pavlovskyi@gmail.com>
Tue, 14 May 2019 18:50:30 +0000 (20:50 +0200)
committerAndy Shevchenko <andriy.shevchenko@linux.intel.com>
Mon, 17 Jun 2019 12:22:47 +0000 (15:22 +0300)
The driver does not clean up the hwmon device on exit or error. To
reproduce the bug, repeat rmmod, insmod to verify that device number
/sys/devices/platform/asus-nb-wmi/hwmon/hwmon?? grows every time. Replace
call for registering device with devm_* version that unregisters it
automatically.

Signed-off-by: Yurii Pavlovskyi <yurii.pavlovskyi@gmail.com>
Reviewed-by: Daniel Drake <drake@endlessm.com>
Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
drivers/platform/x86/asus-wmi.c

index 7e946ce7f66f62217ae237976953dc3caa6cc8db..6a590ec4c4ef9a1ebb08b3e1ebcf3b52fea02417 100644 (file)
@@ -1428,11 +1428,12 @@ __ATTRIBUTE_GROUPS(hwmon_attribute);
 
 static int asus_wmi_hwmon_init(struct asus_wmi *asus)
 {
+       struct device *dev = &asus->platform_device->dev;
        struct device *hwmon;
 
-       hwmon = hwmon_device_register_with_groups(&asus->platform_device->dev,
-                                                 "asus", asus,
-                                                 hwmon_attribute_groups);
+       hwmon = devm_hwmon_device_register_with_groups(dev, "asus", asus,
+                       hwmon_attribute_groups);
+
        if (IS_ERR(hwmon)) {
                pr_err("Could not register asus hwmon device\n");
                return PTR_ERR(hwmon);