diff options
| author | Sachin Kamat <sachin.kamat@linaro.org> | 2013-10-24 15:53:00 +0400 |
|---|---|---|
| committer | Jonathan Cameron <jic23@kernel.org> | 2013-10-24 17:46:00 +0400 |
| commit | 35b9c0b18eaa81d96841c50452f74ed1a7aa4ae7 (patch) | |
| tree | 9b4ec897cfe424f238e2ad79b1645ee4d496f58a /drivers/iio/dac | |
| 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/dac')
| -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 c44afeb06f56..3eeaa82075f7 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) |