Commit 576057bf authored by Dmitry Torokhov's avatar Dmitry Torokhov

Input: ili210x - switch to using devm_device_add_group()

By switching to devm_device_add_group() we can complete driver conversion
to using managed resources and get rid of ili210x_i2c_remove().
Signed-off-by: default avatarDmitry Torokhov <dmitry.torokhov@gmail.com>
parent 49588917
...@@ -376,7 +376,7 @@ static int ili210x_i2c_probe(struct i2c_client *client, ...@@ -376,7 +376,7 @@ static int ili210x_i2c_probe(struct i2c_client *client,
return error; return error;
} }
error = sysfs_create_group(&dev->kobj, &ili210x_attr_group); error = devm_device_add_group(dev, &ili210x_attr_group);
if (error) { if (error) {
dev_err(dev, "Unable to create sysfs attributes, err: %d\n", dev_err(dev, "Unable to create sysfs attributes, err: %d\n",
error); error);
...@@ -386,7 +386,7 @@ static int ili210x_i2c_probe(struct i2c_client *client, ...@@ -386,7 +386,7 @@ static int ili210x_i2c_probe(struct i2c_client *client,
error = input_register_device(priv->input); error = input_register_device(priv->input);
if (error) { if (error) {
dev_err(dev, "Cannot register input device, err: %d\n", error); dev_err(dev, "Cannot register input device, err: %d\n", error);
goto err_remove_sysfs; return error;
} }
device_init_wakeup(dev, 1); device_init_wakeup(dev, 1);
...@@ -396,17 +396,6 @@ static int ili210x_i2c_probe(struct i2c_client *client, ...@@ -396,17 +396,6 @@ static int ili210x_i2c_probe(struct i2c_client *client,
client->irq, firmware.id, firmware.major, firmware.minor); client->irq, firmware.id, firmware.major, firmware.minor);
return 0; return 0;
err_remove_sysfs:
sysfs_remove_group(&dev->kobj, &ili210x_attr_group);
return error;
}
static int ili210x_i2c_remove(struct i2c_client *client)
{
sysfs_remove_group(&client->dev.kobj, &ili210x_attr_group);
return 0;
} }
static int __maybe_unused ili210x_i2c_suspend(struct device *dev) static int __maybe_unused ili210x_i2c_suspend(struct device *dev)
...@@ -454,7 +443,6 @@ static struct i2c_driver ili210x_ts_driver = { ...@@ -454,7 +443,6 @@ static struct i2c_driver ili210x_ts_driver = {
}, },
.id_table = ili210x_i2c_id, .id_table = ili210x_i2c_id,
.probe = ili210x_i2c_probe, .probe = ili210x_i2c_probe,
.remove = ili210x_i2c_remove,
}; };
module_i2c_driver(ili210x_ts_driver); module_i2c_driver(ili210x_ts_driver);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment