[PATCH 4/5] iio: dac: ad5758: remove redundant else after return

Antoniu Miclaus posted 5 patches 1 month, 2 weeks ago
[PATCH 4/5] iio: dac: ad5758: remove redundant else after return
Posted by Antoniu Miclaus 1 month, 2 weeks ago
The else falling back to software reset in ad5758_reset() is
unnecessary since the if block for the GPIO hardware reset already
returns.

Signed-off-by: Antoniu Miclaus <antoniu.miclaus@analog.com>
---
 drivers/iio/dac/ad5758.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/iio/dac/ad5758.c b/drivers/iio/dac/ad5758.c
index 4ed4fda76ea9..c4424a80a35a 100644
--- a/drivers/iio/dac/ad5758.c
+++ b/drivers/iio/dac/ad5758.c
@@ -483,10 +483,10 @@ static int ad5758_reset(struct ad5758_state *st)
 		usleep_range(100, 1000);
 
 		return 0;
-	} else {
-		/* Perform a software reset */
-		return ad5758_soft_reset(st);
 	}
+
+	/* Perform a software reset */
+	return ad5758_soft_reset(st);
 }
 
 static int ad5758_reg_access(struct iio_dev *indio_dev,
-- 
2.43.0