[PATCH v2 5/5] iio: light: tcs3472: move standalone return to default case
Aldo Conte <[email protected]>
| Newsgroups | dev.linux.lists.linux-kernel-mentees,org.kernel.vger.linux-iio,org.kernel.vger.linux-kernel |
|---|---|
| Message-ID | <[email protected]> |
Move the trailing 'return -EINVAL' statements at the end of tcs3472_read_raw() and tcs3472_write_raw() into explicit default: cases inside the respective switch statements. This removes the need for a separate return statement after the switch. No functional change. Suggested-by: Andy Shevchenko <[email protected]> Signed-off-by: Aldo Conte <[email protected]> --- drivers/iio/light/tcs3472.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/drivers/iio/light/tcs3472.c b/drivers/iio/light/tcs3472.c index f8f70399a4dc..4ba2def244b7 100644 --- a/drivers/iio/light/tcs3472.c +++ b/drivers/iio/light/tcs3472.c @@ -291,8 +291,9 @@ static int tcs3472_read_raw(struct iio_dev *indio_dev, cycle_us); return IIO_VAL_INT_PLUS_MICRO; } + default: + return -EINVAL; } - return -EINVAL; } static int tcs3472_write_raw(struct iio_dev *indio_dev, @@ -344,8 +345,9 @@ static int tcs3472_write_raw(struct iio_dev *indio_dev, return -EINVAL; case IIO_CHAN_INFO_SAMP_FREQ: return tcs3472_set_sampling_freq(data, val, val2); + default: + return -EINVAL; } - return -EINVAL; } /* -- 2.54.0