Input: melfas_mip4 - use managed devm_device_add_group
authorAndi Shyti <andi@etezian.org>
Fri, 29 Sep 2017 23:38:23 +0000 (16:38 -0700)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Fri, 29 Sep 2017 23:51:03 +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/melfas_mip4.c

index 05108c2..6892f0e 100644 (file)
@@ -1433,13 +1433,6 @@ static const struct attribute_group mip4_attr_group = {
        .attrs = mip4_attrs,
 };
 
-static void mip4_sysfs_remove(void *_data)
-{
-       struct mip4_ts *ts = _data;
-
-       sysfs_remove_group(&ts->client->dev.kobj, &mip4_attr_group);
-}
-
 static int mip4_probe(struct i2c_client *client, const struct i2c_device_id *id)
 {
        struct mip4_ts *ts;
@@ -1535,21 +1528,13 @@ static int mip4_probe(struct i2c_client *client, const struct i2c_device_id *id)
                return error;
        }
 
-       error = sysfs_create_group(&client->dev.kobj, &mip4_attr_group);
+       error = devm_device_add_group(&client->dev, &mip4_attr_group);
        if (error) {
                dev_err(&client->dev,
                        "Failed to create sysfs attribute group: %d\n", error);
                return error;
        }
 
-       error = devm_add_action(&client->dev, mip4_sysfs_remove, ts);
-       if (error) {
-               mip4_sysfs_remove(ts);
-               dev_err(&client->dev,
-                       "Failed to install sysfs remoce action: %d\n", error);
-               return error;
-       }
-
        return 0;
 }