iio:mcp4725: Use devm_iio_device_alloc
authorPeter Meerwald <pmeerw@pmeerw.net>
Mon, 29 Jul 2013 22:18:00 +0000 (23:18 +0100)
committerJonathan Cameron <jic23@kernel.org>
Sat, 3 Aug 2013 21:18:54 +0000 (22:18 +0100)
Signed-off-by: Peter Meerwald <pmeerw@pmeerw.net>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
drivers/iio/dac/mcp4725.c

index 18a78cd21e8833b8de9cf2fbab7265098ee17d4a..1f4a48e6a82c33f29b8985ad556698f0630e21af 100644 (file)
@@ -292,15 +292,12 @@ static int mcp4725_probe(struct i2c_client *client,
 
        if (!platform_data || !platform_data->vref_mv) {
                dev_err(&client->dev, "invalid platform data");
-               err = -EINVAL;
-               goto exit;
+               return -EINVAL;
        }
 
-       indio_dev = iio_device_alloc(sizeof(*data));
-       if (indio_dev == NULL) {
-               err = -ENOMEM;
-               goto exit;
-       }
+       indio_dev = devm_iio_device_alloc(&client->dev, sizeof(*data));
+       if (indio_dev == NULL)
+               return -ENOMEM;
        data = iio_priv(indio_dev);
        i2c_set_clientdata(client, indio_dev);
        data->client = client;
@@ -317,7 +314,7 @@ static int mcp4725_probe(struct i2c_client *client,
        err = i2c_master_recv(client, inbuf, 3);
        if (err < 0) {
                dev_err(&client->dev, "failed to read DAC value");
-               goto exit_free_device;
+               return err;
        }
        pd = (inbuf[0] >> 1) & 0x3;
        data->powerdown = pd > 0 ? true : false;
@@ -326,25 +323,16 @@ static int mcp4725_probe(struct i2c_client *client,
 
        err = iio_device_register(indio_dev);
        if (err)
-               goto exit_free_device;
+               return err;
 
        dev_info(&client->dev, "MCP4725 DAC registered\n");
 
        return 0;
-
-exit_free_device:
-       iio_device_free(indio_dev);
-exit:
-       return err;
 }
 
 static int mcp4725_remove(struct i2c_client *client)
 {
-       struct iio_dev *indio_dev = i2c_get_clientdata(client);
-
-       iio_device_unregister(indio_dev);
-       iio_device_free(indio_dev);
-
+       iio_device_unregister(i2c_get_clientdata(client));
        return 0;
 }