Input: ad7879 - use managed devm_device_add_group
authorAndi Shyti <andi@etezian.org>
Fri, 29 Sep 2017 23:47:53 +0000 (16:47 -0700)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Fri, 29 Sep 2017 23:51:07 +0000 (16:51 -0700)
Commit 57b8ff070f98 ("driver core: add devm_device_add_group() and
friends") has added the managed version for creating sysfs group files.

Use devm_device_add_group instead of sysfs_create_group and remove the
action that cleans the sysfs file when exiting the driver.

Signed-off-by: Andi Shyti <andi@etezian.org>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
drivers/input/touchscreen/ad7879.c

index 196028c452106186a611f5232aeeb2593286f537..7d74a0ae2c9426c49b290cd211e44684baca397a 100644 (file)
@@ -524,13 +524,6 @@ static int ad7879_parse_dt(struct device *dev, struct ad7879 *ts)
        return 0;
 }
 
-static void ad7879_cleanup_sysfs(void *_ts)
-{
-       struct ad7879 *ts = _ts;
-
-       sysfs_remove_group(&ts->dev->kobj, &ad7879_attr_group);
-}
-
 int ad7879_probe(struct device *dev, struct regmap *regmap,
                 int irq, u16 bustype, u8 devid)
 {
@@ -658,11 +651,7 @@ int ad7879_probe(struct device *dev, struct regmap *regmap,
 
        __ad7879_disable(ts);
 
-       err = sysfs_create_group(&dev->kobj, &ad7879_attr_group);
-       if (err)
-               return err;
-
-       err = devm_add_action_or_reset(dev, ad7879_cleanup_sysfs, ts);
+       err = devm_device_add_group(dev, &ad7879_attr_group);
        if (err)
                return err;