iio: dac: ad5421: Remove redundant code
authorSachin Kamat <sachin.kamat@linaro.org>
Thu, 24 Oct 2013 11:53:00 +0000 (12:53 +0100)
committerJonathan Cameron <jic23@kernel.org>
Thu, 24 Oct 2013 13:46:00 +0000 (14:46 +0100)
The if check is redundant as the value obtained from
iio_device_register() is already in the required format.
Hence return the function directly. Error messages are already
printed by iio_device_register(); hence not needed.

Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
drivers/iio/dac/ad5421.c

index c44afeb06f56ac890dd5a4ea38f9bde76dbeeec9..3eeaa82075f7bca0649b5e15afb0c02f21a32458 100644 (file)
@@ -514,13 +514,7 @@ static int ad5421_probe(struct spi_device *spi)
                        return ret;
        }
 
-       ret = iio_device_register(indio_dev);
-       if (ret) {
-               dev_err(&spi->dev, "Failed to register iio device: %d\n", ret);
-               return ret;
-       }
-
-       return 0;
+       return iio_device_register(indio_dev);
 }
 
 static int ad5421_remove(struct spi_device *spi)