staging:iio:ade7758: Do not return a error in remove function
authorLars-Peter Clausen <lars@metafoo.de>
Sat, 22 Sep 2012 08:56:00 +0000 (09:56 +0100)
committerJonathan Cameron <jic23@kernel.org>
Sat, 22 Sep 2012 09:18:42 +0000 (10:18 +0100)
In the Linux device driver model the remove callback is not allowed to fail and
the device will be removed regardless of the return value of the remove
callback. So if we abort in the remove function and do not free all resources we
will create a resource leak. Also all kinds of undefined behaviour are expected
to happen since the IIO device is still there while its parent is already gone.

The error which the driver tries to handle in the remove function is
non-critical, so we can just ignore it and continue to free all resources and
remove the IIO device.

Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
drivers/staging/iio/meter/ade7758_core.c

index 958f8f235b42143d3376aadf638db37ddff06db5..a0fef77d8e5e6389c3ebd6d86e348b222962ea19 100644 (file)
@@ -966,13 +966,9 @@ static int __devexit ade7758_remove(struct spi_device *spi)
 {
        struct iio_dev *indio_dev = spi_get_drvdata(spi);
        struct ade7758_state *st = iio_priv(indio_dev);
-       int ret;
 
        iio_device_unregister(indio_dev);
-       ret = ade7758_stop_device(&indio_dev->dev);
-       if (ret)
-               goto err_ret;
-
+       ade7758_stop_device(&indio_dev->dev);
        ade7758_remove_trigger(indio_dev);
        ade7758_uninitialize_ring(indio_dev);
        ade7758_unconfigure_ring(indio_dev);
@@ -981,8 +977,7 @@ static int __devexit ade7758_remove(struct spi_device *spi)
 
        iio_device_free(indio_dev);
 
-err_ret:
-       return ret;
+       return 0;
 }
 
 static const struct spi_device_id ade7758_id[] = {