diff options
author | Sachin Kamat <sachin.kamat@linaro.org> | 2013-10-24 11:53:00 (GMT) |
---|---|---|
committer | Jonathan Cameron <jic23@kernel.org> | 2013-10-24 13:46:00 (GMT) |
commit | 35b9c0b18eaa81d96841c50452f74ed1a7aa4ae7 (patch) | |
tree | 9b4ec897cfe424f238e2ad79b1645ee4d496f58a /drivers/iio | |
parent | ae0f29d1593fabb8eade5f93352e05f88f6245bd (diff) | |
download | linux-35b9c0b18eaa81d96841c50452f74ed1a7aa4ae7.tar.xz |
iio: dac: ad5421: Remove redundant code
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>
Diffstat (limited to 'drivers/iio')
-rw-r--r-- | drivers/iio/dac/ad5421.c | 8 |
1 files changed, 1 insertions, 7 deletions
diff --git a/drivers/iio/dac/ad5421.c b/drivers/iio/dac/ad5421.c index c44afeb..3eeaa82 100644 --- a/drivers/iio/dac/ad5421.c +++ b/drivers/iio/dac/ad5421.c @@ -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) |