]> Pileus Git - ~andy/linux/commitdiff
iio: adc: viperboard_adc: Use devm_iio_device_alloc
authorSachin Kamat <sachin.kamat@linaro.org>
Tue, 23 Jul 2013 08:58:00 +0000 (09:58 +0100)
committerJonathan Cameron <jic23@kernel.org>
Sat, 3 Aug 2013 17:40:42 +0000 (18:40 +0100)
Using devm_iio_device_alloc makes code simpler.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Cc: Lars Poeschel <poeschel@lemonage.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
drivers/iio/adc/viperboard_adc.c

index 56ac481c73c0bde09575536f6a92066a7fc946c7..09727a71e9fa1578744c4334ad83679274ff24d2 100644 (file)
@@ -124,7 +124,7 @@ static int vprbrd_adc_probe(struct platform_device *pdev)
        int ret;
 
        /* registering iio */
-       indio_dev = iio_device_alloc(sizeof(*adc));
+       indio_dev = devm_iio_device_alloc(&pdev->dev, sizeof(*adc));
        if (!indio_dev) {
                dev_err(&pdev->dev, "failed allocating iio device\n");
                return -ENOMEM;
@@ -142,16 +142,12 @@ static int vprbrd_adc_probe(struct platform_device *pdev)
        ret = iio_device_register(indio_dev);
        if (ret) {
                dev_err(&pdev->dev, "could not register iio (adc)");
-               goto error;
+               return ret;
        }
 
        platform_set_drvdata(pdev, indio_dev);
 
        return 0;
-
-error:
-       iio_device_free(indio_dev);
-       return ret;
 }
 
 static int vprbrd_adc_remove(struct platform_device *pdev)
@@ -159,7 +155,6 @@ static int vprbrd_adc_remove(struct platform_device *pdev)
        struct iio_dev *indio_dev = platform_get_drvdata(pdev);
 
        iio_device_unregister(indio_dev);
-       iio_device_free(indio_dev);
 
        return 0;
 }