On 09/11/13 11:20, Sachin Kamat wrote: > 'break' after return is redundant. Remove it. > > Signed-off-by: Sachin Kamat <sachin.kamat@xxxxxxxxxx> Applied to the togreg branch of iio.git > --- > drivers/iio/gyro/adis16130.c | 2 -- > 1 file changed, 2 deletions(-) > > diff --git a/drivers/iio/gyro/adis16130.c b/drivers/iio/gyro/adis16130.c > index ac66fc1..9155cf6 100644 > --- a/drivers/iio/gyro/adis16130.c > +++ b/drivers/iio/gyro/adis16130.c > @@ -103,7 +103,6 @@ static int adis16130_read_raw(struct iio_dev *indio_dev, > default: > return -EINVAL; > } > - break; > case IIO_CHAN_INFO_OFFSET: > switch (chan->type) { > case IIO_ANGL_VEL: > @@ -115,7 +114,6 @@ static int adis16130_read_raw(struct iio_dev *indio_dev, > default: > return -EINVAL; > } > - break; > } > > return -EINVAL; > -- To unsubscribe from this list: send the line "unsubscribe linux-iio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html