iio: adc: cc10001: Switch remaining IIO calls in probe to devm_ forms.
authorJonathan Cameron <Jonathan.Cameron@huawei.com>
Sun, 16 Oct 2022 17:09:50 +0000 (18:09 +0100)
committerJonathan Cameron <Jonathan.Cameron@huawei.com>
Wed, 23 Nov 2022 19:44:04 +0000 (19:44 +0000)
As everything else is now handled by devm managed releases the
triggered buffer setup and IIO device registration can also be
moved over to their devm forms allowing dropping of remove().

Only user of drvdata associated with the struct device was the
remove function, so also drop the platform_set_drvdata() call.

Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Reviewed-by: Nuno Sá <nuno.sa@analog.com>
Link: https://lore.kernel.org/r/20221016170950.387751-6-jic23@kernel.org
drivers/iio/adc/cc10001_adc.c

index aecad89c00ff9a57c43967fdbe30822f32099d28..2cde4b44fc6e482afcf8700a1422b3d194c070ea 100644 (file)
@@ -390,33 +390,12 @@ static int cc10001_adc_probe(struct platform_device *pdev)
 
        mutex_init(&adc_dev->lock);
 
-       ret = iio_triggered_buffer_setup(indio_dev, NULL,
-                                        &cc10001_adc_trigger_h, NULL);
+       ret = devm_iio_triggered_buffer_setup(dev, indio_dev, NULL,
+                                             &cc10001_adc_trigger_h, NULL);
        if (ret < 0)
                return ret;
 
-       ret = iio_device_register(indio_dev);
-       if (ret < 0)
-               goto err_cleanup_buffer;
-
-       platform_set_drvdata(pdev, indio_dev);
-
-       return 0;
-
-err_cleanup_buffer:
-       iio_triggered_buffer_cleanup(indio_dev);
-       return ret;
-}
-
-static int cc10001_adc_remove(struct platform_device *pdev)
-{
-       struct iio_dev *indio_dev = platform_get_drvdata(pdev);
-       struct cc10001_adc_device *adc_dev = iio_priv(indio_dev);
-
-       iio_device_unregister(indio_dev);
-       iio_triggered_buffer_cleanup(indio_dev);
-
-       return 0;
+       return devm_iio_device_register(dev, indio_dev);
 }
 
 static const struct of_device_id cc10001_adc_dt_ids[] = {
@@ -431,7 +410,6 @@ static struct platform_driver cc10001_adc_driver = {
                .of_match_table = cc10001_adc_dt_ids,
        },
        .probe  = cc10001_adc_probe,
-       .remove = cc10001_adc_remove,
 };
 module_platform_driver(cc10001_adc_driver);