Input: pixcir_i2c_ts - use devres managed resource allocations
authorRoger Quadros <rogerq@ti.com>
Mon, 19 May 2014 05:43:42 +0000 (22:43 -0700)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Mon, 19 May 2014 06:25:56 +0000 (23:25 -0700)
Use devm_() and friends for allocating memory, input device
and IRQ.

Signed-off-by: Roger Quadros <rogerq@ti.com>
Acked-by: Mugunthan V N <mugunthanvnm@ti.com>
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
drivers/input/touchscreen/pixcir_i2c_ts.c

index 02392d2061d6a01f304216ecfed1dc84b9de60a3..5d36243bdc79455dcb8a41dcced8d287dcd98e4e 100644 (file)
@@ -130,6 +130,7 @@ static int pixcir_i2c_ts_probe(struct i2c_client *client,
 {
        const struct pixcir_ts_platform_data *pdata =
                        dev_get_platdata(&client->dev);
+       struct device *dev = &client->dev;
        struct pixcir_i2c_ts_data *tsdata;
        struct input_dev *input;
        int error;
@@ -139,12 +140,14 @@ static int pixcir_i2c_ts_probe(struct i2c_client *client,
                return -EINVAL;
        }
 
-       tsdata = kzalloc(sizeof(*tsdata), GFP_KERNEL);
-       input = input_allocate_device();
-       if (!tsdata || !input) {
-               dev_err(&client->dev, "Failed to allocate driver data!\n");
-               error = -ENOMEM;
-               goto err_free_mem;
+       tsdata = devm_kzalloc(dev, sizeof(*tsdata), GFP_KERNEL);
+       if (!tsdata)
+               return -ENOMEM;
+
+       input = devm_input_allocate_device(dev);
+       if (!input) {
+               dev_err(dev, "Failed to allocate input device\n");
+               return -ENOMEM;
        }
 
        tsdata->client = client;
@@ -165,29 +168,22 @@ static int pixcir_i2c_ts_probe(struct i2c_client *client,
 
        input_set_drvdata(input, tsdata);
 
-       error = request_threaded_irq(client->irq, NULL, pixcir_ts_isr,
-                                    IRQF_TRIGGER_FALLING | IRQF_ONESHOT,
-                                    client->name, tsdata);
+       error = devm_request_threaded_irq(dev, client->irq, NULL, pixcir_ts_isr,
+                                         IRQF_TRIGGER_FALLING | IRQF_ONESHOT,
+                                         client->name, tsdata);
        if (error) {
-               dev_err(&client->dev, "Unable to request touchscreen IRQ.\n");
-               goto err_free_mem;
+               dev_err(dev, "failed to request irq %d\n", client->irq);
+               return error;
        }
 
        error = input_register_device(input);
        if (error)
-               goto err_free_irq;
+               return error;
 
        i2c_set_clientdata(client, tsdata);
        device_init_wakeup(&client->dev, 1);
 
        return 0;
-
-err_free_irq:
-       free_irq(client->irq, tsdata);
-err_free_mem:
-       input_free_device(input);
-       kfree(tsdata);
-       return error;
 }
 
 static int pixcir_i2c_ts_remove(struct i2c_client *client)
@@ -198,10 +194,6 @@ static int pixcir_i2c_ts_remove(struct i2c_client *client)
 
        tsdata->exiting = true;
        mb();
-       free_irq(client->irq, tsdata);
-
-       input_unregister_device(tsdata->input);
-       kfree(tsdata);
 
        return 0;
 }