2014-07-21 08:18:39

by Himangi Saraogi

[permalink] [raw]
Subject: [PATCH] staging: iio: ad5930: Use devm_iio_device_register

This patch introduces the use of iio_device_register and does away with
the call to the coressponding unregister function in the remove function
of the driver. The remove function is no longer required and is
completely removed.

Signed-off-by: Himangi Saraogi <[email protected]>
Acked-by: Julia Lawall <[email protected]>
---
drivers/staging/iio/frequency/ad5930.c | 10 +---------
1 file changed, 1 insertion(+), 9 deletions(-)

diff --git a/drivers/staging/iio/frequency/ad5930.c b/drivers/staging/iio/frequency/ad5930.c
index a4aeee6..662cfb9 100644
--- a/drivers/staging/iio/frequency/ad5930.c
+++ b/drivers/staging/iio/frequency/ad5930.c
@@ -106,7 +106,7 @@ static int ad5930_probe(struct spi_device *spi)
idev->info = &ad5930_info;
idev->modes = INDIO_DIRECT_MODE;

- ret = iio_device_register(idev);
+ ret = devm_iio_device_register(&spi->dev, idev);
if (ret)
return ret;
spi->max_speed_hz = 2000000;
@@ -117,20 +117,12 @@ static int ad5930_probe(struct spi_device *spi)
return 0;
}

-static int ad5930_remove(struct spi_device *spi)
-{
- iio_device_unregister(spi_get_drvdata(spi));
-
- return 0;
-}
-
static struct spi_driver ad5930_driver = {
.driver = {
.name = DRV_NAME,
.owner = THIS_MODULE,
},
.probe = ad5930_probe,
- .remove = ad5930_remove,
};
module_spi_driver(ad5930_driver);

--
1.9.1