staging: iio: accel: remove unneeded braces around single statements
authorMark Stenglein <mark@stengle.in>
Thu, 23 Mar 2017 06:18:12 +0000 (02:18 -0400)
committerJonathan Cameron <jic23@kernel.org>
Sat, 25 Mar 2017 17:02:26 +0000 (17:02 +0000)
Fixes three checkpatch warnings due to braces used when single
statements are sufficient.

Signed-off-by: Mark Stenglein <mark@stengle.in>
Signed-off-by: Jonathan Cameron <jic23@kernel.org>
drivers/staging/iio/accel/adis16209.c
drivers/staging/iio/accel/adis16240.c

index 52fa2e0..8485c02 100644 (file)
@@ -255,9 +255,8 @@ static int adis16209_read_raw(struct iio_dev *indio_dev,
                }
                addr = adis16209_addresses[chan->scan_index][0];
                ret = adis_read_reg_16(st, addr, &val16);
-               if (ret) {
+               if (ret)
                        return ret;
-               }
                val16 &= (1 << bits) - 1;
                val16 = (s16)(val16 << (16 - bits)) >> (16 - bits);
                *val = val16;
index 6e3c95c..109cd94 100644 (file)
@@ -290,9 +290,8 @@ static int adis16240_read_raw(struct iio_dev *indio_dev,
                bits = 10;
                addr = adis16240_addresses[chan->scan_index][0];
                ret = adis_read_reg_16(st, addr, &val16);
-               if (ret) {
+               if (ret)
                        return ret;
-               }
                val16 &= (1 << bits) - 1;
                val16 = (s16)(val16 << (16 - bits)) >> (16 - bits);
                *val = val16;
@@ -301,9 +300,8 @@ static int adis16240_read_raw(struct iio_dev *indio_dev,
                bits = 10;
                addr = adis16240_addresses[chan->scan_index][1];
                ret = adis_read_reg_16(st, addr, &val16);
-               if (ret) {
+               if (ret)
                        return ret;
-               }
                val16 &= (1 << bits) - 1;
                val16 = (s16)(val16 << (16 - bits)) >> (16 - bits);
                *val = val16;