From: Sachin Kamat <sachin.kamat@linaro.org>
Date: Thu, 24 Oct 2013 11:53:00 +0000 (+0100)
Subject: iio: dac: ad5421: Remove redundant code
X-Git-Url: http://git.lede-project.org./?a=commitdiff_plain;h=35b9c0b18eaa81d96841c50452f74ed1a7aa4ae7;p=openwrt%2Fstaging%2Fblogic.git

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>
---

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)