Input: rohm_bu21023 - use managed devm_device_add_group
Commit 57b8ff070f
("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>
This commit is contained in:
parent
b887df52f8
commit
a71b8b5c4a
|
@ -1103,13 +1103,6 @@ static void rohm_ts_close(struct input_dev *input_dev)
|
||||||
ts->initialized = false;
|
ts->initialized = false;
|
||||||
}
|
}
|
||||||
|
|
||||||
static void rohm_ts_remove_sysfs_group(void *_dev)
|
|
||||||
{
|
|
||||||
struct device *dev = _dev;
|
|
||||||
|
|
||||||
sysfs_remove_group(&dev->kobj, &rohm_ts_attr_group);
|
|
||||||
}
|
|
||||||
|
|
||||||
static int rohm_bu21023_i2c_probe(struct i2c_client *client,
|
static int rohm_bu21023_i2c_probe(struct i2c_client *client,
|
||||||
const struct i2c_device_id *id)
|
const struct i2c_device_id *id)
|
||||||
{
|
{
|
||||||
|
@ -1180,20 +1173,12 @@ static int rohm_bu21023_i2c_probe(struct i2c_client *client,
|
||||||
return error;
|
return error;
|
||||||
}
|
}
|
||||||
|
|
||||||
error = sysfs_create_group(&dev->kobj, &rohm_ts_attr_group);
|
error = devm_device_add_group(dev, &rohm_ts_attr_group);
|
||||||
if (error) {
|
if (error) {
|
||||||
dev_err(dev, "failed to create sysfs group: %d\n", error);
|
dev_err(dev, "failed to create sysfs group: %d\n", error);
|
||||||
return error;
|
return error;
|
||||||
}
|
}
|
||||||
|
|
||||||
error = devm_add_action(dev, rohm_ts_remove_sysfs_group, dev);
|
|
||||||
if (error) {
|
|
||||||
rohm_ts_remove_sysfs_group(dev);
|
|
||||||
dev_err(dev, "Failed to add sysfs cleanup action: %d\n",
|
|
||||||
error);
|
|
||||||
return error;
|
|
||||||
}
|
|
||||||
|
|
||||||
return error;
|
return error;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue