diff options
author | Sachin Kamat <sachin.kamat@linaro.org> | 2013-10-24 07:53:00 -0400 |
---|---|---|
committer | Jonathan Cameron <jic23@kernel.org> | 2013-10-24 09:46:00 -0400 |
commit | 35b9c0b18eaa81d96841c50452f74ed1a7aa4ae7 (patch) | |
tree | 9b4ec897cfe424f238e2ad79b1645ee4d496f58a /drivers/iio | |
parent | ae0f29d1593fabb8eade5f93352e05f88f6245bd (diff) |
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 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) | |||
514 | return ret; | 514 | return ret; |
515 | } | 515 | } |
516 | 516 | ||
517 | ret = iio_device_register(indio_dev); | 517 | return iio_device_register(indio_dev); |
518 | if (ret) { | ||
519 | dev_err(&spi->dev, "Failed to register iio device: %d\n", ret); | ||
520 | return ret; | ||
521 | } | ||
522 | |||
523 | return 0; | ||
524 | } | 518 | } |
525 | 519 | ||
526 | static int ad5421_remove(struct spi_device *spi) | 520 | static int ad5421_remove(struct spi_device *spi) |