diff options
author | Gulsah Kose <gulsah.1004@gmail.com> | 2014-09-30 17:09:10 -0400 |
---|---|---|
committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2014-10-02 13:04:18 -0400 |
commit | 042e0e463c48765692fa4813a9a26748ca1b1917 (patch) | |
tree | 869d6ffcfd7d54324a52d2e3014acca78c2fa098 | |
parent | e09b142e2b43f151f795579e6dbf1a3db0e74403 (diff) |
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 <gulsah.1004@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
-rw-r--r-- | drivers/staging/iio/accel/adis16220_core.c | 15 |
1 files changed, 7 insertions, 8 deletions
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, | |||
323 | sval = (s16)(sval << (16 - bits)) >> (16 - bits); | 323 | sval = (s16)(sval << (16 - bits)) >> (16 - bits); |
324 | *val = sval; | 324 | *val = sval; |
325 | return IIO_VAL_INT; | 325 | return IIO_VAL_INT; |
326 | } else { | ||
327 | ret = adis_read_reg_16(&st->adis, addr->addr, &uval); | ||
328 | if (ret) | ||
329 | return ret; | ||
330 | bits = addr->bits; | ||
331 | uval &= (1 << bits) - 1; | ||
332 | *val = uval; | ||
333 | return IIO_VAL_INT; | ||
334 | } | 326 | } |
327 | ret = adis_read_reg_16(&st->adis, addr->addr, &uval); | ||
328 | if (ret) | ||
329 | return ret; | ||
330 | bits = addr->bits; | ||
331 | uval &= (1 << bits) - 1; | ||
332 | *val = uval; | ||
333 | return IIO_VAL_INT; | ||
335 | } | 334 | } |
336 | 335 | ||
337 | static const struct iio_chan_spec adis16220_channels[] = { | 336 | static const struct iio_chan_spec adis16220_channels[] = { |