From: Gulsah Kose Date: Tue, 30 Sep 2014 21:09:10 +0000 (+0300) Subject: staging: iio: accel: Removed unnecessary else expression. X-Git-Url: https://git.karo-electronics.de/?a=commitdiff_plain;h=042e0e463c48765692fa4813a9a26748ca1b1917;p=linux-beck.git staging: iio: accel: Removed unnecessary else expression. This patch fixes "else is not generally useful after a break or return" checkpatch.pl warning in adis16220_core.c Signed-off-by: Gulsah Kose Signed-off-by: Greg Kroah-Hartman --- diff --git a/drivers/staging/iio/accel/adis16220_core.c b/drivers/staging/iio/accel/adis16220_core.c index d9a1a141099e..d478f5130a0f 100644 --- a/drivers/staging/iio/accel/adis16220_core.c +++ b/drivers/staging/iio/accel/adis16220_core.c @@ -323,15 +323,14 @@ static int adis16220_read_raw(struct iio_dev *indio_dev, sval = (s16)(sval << (16 - bits)) >> (16 - bits); *val = sval; return IIO_VAL_INT; - } else { - ret = adis_read_reg_16(&st->adis, addr->addr, &uval); - if (ret) - return ret; - bits = addr->bits; - uval &= (1 << bits) - 1; - *val = uval; - return IIO_VAL_INT; } + ret = adis_read_reg_16(&st->adis, addr->addr, &uval); + if (ret) + return ret; + bits = addr->bits; + uval &= (1 << bits) - 1; + *val = uval; + return IIO_VAL_INT; } static const struct iio_chan_spec adis16220_channels[] = {