On 10/29/13 11:39, Sachin Kamat wrote:
> devm_iio_device_register simplifies the code.
>
> Signed-off-by: Sachin Kamat <sachin.kamat@xxxxxxxxxx>
A simple device so this one is fine.
Applied to the togreg branch of iio.git
> ---
> drivers/staging/iio/gyro/adis16060_core.c | 10 +---------
> 1 file changed, 1 insertion(+), 9 deletions(-)
>
> diff --git a/drivers/staging/iio/gyro/adis16060_core.c b/drivers/staging/iio/gyro/adis16060_core.c
> index 6d3d771..d5d395c 100644
> --- a/drivers/staging/iio/gyro/adis16060_core.c
> +++ b/drivers/staging/iio/gyro/adis16060_core.c
> @@ -167,7 +167,7 @@ static int adis16060_r_probe(struct spi_device *spi)
> indio_dev->channels = adis16060_channels;
> indio_dev->num_channels = ARRAY_SIZE(adis16060_channels);
>
> - ret = iio_device_register(indio_dev);
> + ret = devm_iio_device_register(&spi->dev, indio_dev);
> if (ret)
> return ret;
>
> @@ -175,13 +175,6 @@ static int adis16060_r_probe(struct spi_device *spi)
> return 0;
> }
>
> -/* fixme, confirm ordering in this function */
> -static int adis16060_r_remove(struct spi_device *spi)
> -{
> - iio_device_unregister(spi_get_drvdata(spi));
> - return 0;
> -}
> -
> static int adis16060_w_probe(struct spi_device *spi)
> {
> int ret;
> @@ -211,7 +204,6 @@ static struct spi_driver adis16060_r_driver = {
> .owner = THIS_MODULE,
> },
> .probe = adis16060_r_probe,
> - .remove = adis16060_r_remove,
> };
>
> static struct spi_driver adis16060_w_driver = {
>
--
To unsubscribe from this list: send the line "unsubscribe linux-iio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
