Messages in this thread Patch in this message | | | From | Himanshu Jha <> | Subject | [PATCH 08/11] Staging: iio: accel: Use switch statement than if-else | Date | Mon, 5 Mar 2018 13:19:27 +0530 |
| |
Use switch statement instead of if-else pair to explicitly match the only two channels present.
Signed-off-by: Himanshu Jha <himanshujha199640@gmail.com> --- drivers/staging/iio/accel/adis16201.c | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/staging/iio/accel/adis16201.c b/drivers/staging/iio/accel/adis16201.c index 1737708..307d4ab 100644 --- a/drivers/staging/iio/accel/adis16201.c +++ b/drivers/staging/iio/accel/adis16201.c @@ -114,12 +114,15 @@ static int adis16201_read_raw(struct iio_dev *indio_dev, case IIO_CHAN_INFO_SCALE: switch (chan->type) { case IIO_VOLTAGE: - if (chan->channel == 0) { + switch (chan->channel) { + case 0: *val = 1; *val2 = 220000; - } else { + break; + case 1: *val = 0; *val2 = 610000; + break; } return IIO_VAL_INT_PLUS_MICRO; case IIO_TEMP: -- 2.7.4
| |