diff options
| author | Shreeya Patel <shreeya.patel23498@gmail.com> | 2018-03-04 18:13:12 +0530 |
|---|---|---|
| committer | Jonathan Cameron <Jonathan.Cameron@huawei.com> | 2018-03-07 21:06:52 +0000 |
| commit | 0df8baeab1ee2a5ccd3c725a44a431fcc63fe416 (patch) | |
| tree | 41a53d13edf09aa3cf802ac428ff238fbcc45d00 | |
| parent | 1026bb35371a499f977174f9f297ce8e2dfd1efb (diff) | |
Staging: iio: adis16209: Adjust a switch statement
Adjust a switch block to explicitly match channels and
return -EINVAL as default case which makes the code
semantically more clear.
Signed-off-by: Shreeya Patel <shreeya.patel23498@gmail.com>
Signed-off-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
| -rw-r--r-- | drivers/staging/iio/accel/adis16209.c | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/drivers/staging/iio/accel/adis16209.c b/drivers/staging/iio/accel/adis16209.c index 68a2f2f80f88..8ffde7ea94c4 100644 --- a/drivers/staging/iio/accel/adis16209.c +++ b/drivers/staging/iio/accel/adis16209.c @@ -155,10 +155,16 @@ static int adis16209_read_raw(struct iio_dev *indio_dev, switch (chan->type) { case IIO_VOLTAGE: *val = 0; - if (chan->channel == 0) + switch (chan->channel) { + case 0: *val2 = 305180; /* 0.30518 mV */ - else + break; + case 1: *val2 = 610500; /* 0.6105 mV */ + break; + default: + return -EINVAL; + } return IIO_VAL_INT_PLUS_MICRO; case IIO_TEMP: *val = -470; |