[PATCH 2/4] iio: light: Remove unnecessary else after return

Roberta Dobrescu roberta.dobrescu at gmail.com
Fri Dec 12 16:31:37 EET 2014


This patch fixes the following checkpatch.pl warning
WARNING: else is not generally useful after a break or return

Signed-off-by: Roberta Dobrescu <roberta.dobrescu at gmail.com>
---
 drivers/iio/light/ltr501.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/drivers/iio/light/ltr501.c b/drivers/iio/light/ltr501.c
index 143277f..eee7d90 100644
--- a/drivers/iio/light/ltr501.c
+++ b/drivers/iio/light/ltr501.c
@@ -171,11 +171,10 @@ static int ltr501_read_raw(struct iio_dev *indio_dev,
 				*val = 0;
 				*val2 = 5000;
 				return IIO_VAL_INT_PLUS_MICRO;
-			} else {
-				*val = 1;
-				*val2 = 0;
-				return IIO_VAL_INT;
 			}
+			*val = 1;
+			*val2 = 0;
+			return IIO_VAL_INT;
 		case IIO_PROXIMITY:
 			i = (data->ps_contr & LTR501_CONTR_PS_GAIN_MASK) >>
 				LTR501_CONTR_PS_GAIN_SHIFT;
-- 
1.9.1



More information about the firefly mailing list