iio: light: tsl2563: Use devm_* APIs

devm_* APIs are device managed and make code simpler.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Reviewed-by: Amit Kucheria <amit.kucheria@linaro.org>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>

diff --git a/drivers/iio/light/tsl2563.c b/drivers/iio/light/tsl2563.c
index 1f529f36f138bd2b4490eb50ac610e8bec73d9fd..ebb962c5c323a025f9c9e1ef8cfb27f40f2ba8d5 100644 (file)
--- a/drivers/iio/light/tsl2563.c
+++ b/drivers/iio/light/tsl2563.c
@@ -702,7 +702,7 @@ static int tsl2563_probe(struct i2c_client *client,
        int err = 0;
        u8 id = 0;
 
-       indio_dev = iio_device_alloc(sizeof(*chip));
+       indio_dev = devm_iio_device_alloc(&client->dev, sizeof(*chip));
        if (!indio_dev)
                return -ENOMEM;
 
@@ -714,13 +714,13 @@ static int tsl2563_probe(struct i2c_client *client,
        err = tsl2563_detect(chip);
        if (err) {
                dev_err(&client->dev, "detect error %d\n", -err);
-               goto fail1;
+               return err;
        }
 
        err = tsl2563_read_id(chip, &id);
        if (err) {
                dev_err(&client->dev, "read id error %d\n", -err);
-               goto fail1;
+               return err;
        }
 
        mutex_init(&chip->lock);
@@ -751,7 +751,7 @@ static int tsl2563_probe(struct i2c_client *client,
                indio_dev->info = &tsl2563_info_no_irq;
 
        if (client->irq) {
-               err = request_threaded_irq(client->irq,
+               err = devm_request_threaded_irq(&client->dev, client->irq,
                                           NULL,
                                           &tsl2563_event_handler,
                                           IRQF_TRIGGER_RISING | IRQF_ONESHOT,
@@ -759,14 +759,14 @@ static int tsl2563_probe(struct i2c_client *client,
                                           indio_dev);
                if (err) {
                        dev_err(&client->dev, "irq request error %d\n", -err);
-                       goto fail1;
+                       return err;
                }
        }
 
        err = tsl2563_configure(chip);
        if (err) {
                dev_err(&client->dev, "configure error %d\n", -err);
-               goto fail2;
+               return err;
        }
 
        INIT_DELAYED_WORK(&chip->poweroff_work, tsl2563_poweroff_work);
@@ -777,19 +777,14 @@ static int tsl2563_probe(struct i2c_client *client,
        err = iio_device_register(indio_dev);
        if (err) {
                dev_err(&client->dev, "iio registration error %d\n", -err);
-               goto fail3;
+               goto fail;
        }
 
        return 0;
 
-fail3:
+fail:
        cancel_delayed_work(&chip->poweroff_work);
        flush_scheduled_work();
-fail2:
-       if (client->irq)
-               free_irq(client->irq, indio_dev);
-fail1:
-       iio_device_free(indio_dev);
        return err;
 }
 
@@ -807,10 +802,6 @@ static int tsl2563_remove(struct i2c_client *client)
                                  chip->intr);
        flush_scheduled_work();
        tsl2563_set_power(chip, 0);
-       if (client->irq)
-               free_irq(client->irq, indio_dev);
-
-       iio_device_free(indio_dev);
 
        return 0;
 }