Input: atmel_mxt_ts - use more managed resources
authorSebastian Reichel <sebastian.reichel@collabora.co.uk>
Tue, 25 Jul 2017 21:08:48 +0000 (14:08 -0700)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Mon, 21 Aug 2017 20:10:18 +0000 (13:10 -0700)
Switch mxt_data and interrupt to resource managed allocation methods,
which cleans up the driver slightly and prepares for adding
reset GPIO support.

Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.co.uk>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
drivers/input/touchscreen/atmel_mxt_ts.c

index dd042a9b0aaacc208056376b27a5fd7bf5755f6c..aeb46522073a626114d834d67c74a522f288877d 100644 (file)
@@ -3117,11 +3117,9 @@ static int mxt_probe(struct i2c_client *client, const struct i2c_device_id *id)
        if (IS_ERR(pdata))
                return PTR_ERR(pdata);
 
-       data = kzalloc(sizeof(struct mxt_data), GFP_KERNEL);
-       if (!data) {
-               dev_err(&client->dev, "Failed to allocate memory\n");
+       data = devm_kzalloc(&client->dev, sizeof(struct mxt_data), GFP_KERNEL);
+       if (!data)
                return -ENOMEM;
-       }
 
        snprintf(data->phys, sizeof(data->phys), "i2c-%u-%04x/input0",
                 client->adapter->nr, client->addr);
@@ -3135,19 +3133,20 @@ static int mxt_probe(struct i2c_client *client, const struct i2c_device_id *id)
        init_completion(&data->reset_completion);
        init_completion(&data->crc_completion);
 
-       error = request_threaded_irq(client->irq, NULL, mxt_interrupt,
-                                    pdata->irqflags | IRQF_ONESHOT,
-                                    client->name, data);
+       error = devm_request_threaded_irq(&client->dev, client->irq,
+                                         NULL, mxt_interrupt,
+                                         pdata->irqflags | IRQF_ONESHOT,
+                                         client->name, data);
        if (error) {
                dev_err(&client->dev, "Failed to register interrupt\n");
-               goto err_free_mem;
+               return error;
        }
 
        disable_irq(client->irq);
 
        error = mxt_initialize(data);
        if (error)
-               goto err_free_irq;
+               return error;
 
        error = sysfs_create_group(&client->dev.kobj, &mxt_attr_group);
        if (error) {
@@ -3161,10 +3160,6 @@ static int mxt_probe(struct i2c_client *client, const struct i2c_device_id *id)
 err_free_object:
        mxt_free_input_device(data);
        mxt_free_object_table(data);
-err_free_irq:
-       free_irq(client->irq, data);
-err_free_mem:
-       kfree(data);
        return error;
 }
 
@@ -3172,11 +3167,10 @@ static int mxt_remove(struct i2c_client *client)
 {
        struct mxt_data *data = i2c_get_clientdata(client);
 
+       disable_irq(data->irq);
        sysfs_remove_group(&client->dev.kobj, &mxt_attr_group);
-       free_irq(data->irq, data);
        mxt_free_input_device(data);
        mxt_free_object_table(data);
-       kfree(data);
 
        return 0;
 }