> Add missing break statement to prevent the code for case > IIO_CHAN_INFO_CALIBBIAS falling through to the default case. > > Also, add a break to the default case for the switch within > case IIO_CHAN_INFO_CALIBBIAS.
fix seems to be cosmetic only... > Addresses-Coverity-ID: 1357377 > Signed-off-by: Gustavo A. R. Silva <gust...@embeddedor.com> > --- > This issue was reported by Coverity and it was tested by compilation only. > Please, verify if this is an actual bug. > > drivers/iio/imu/inv_mpu6050/inv_mpu_core.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/iio/imu/inv_mpu6050/inv_mpu_core.c > b/drivers/iio/imu/inv_mpu6050/inv_mpu_core.c > index 44830bc..6d2268a 100644 > --- a/drivers/iio/imu/inv_mpu6050/inv_mpu_core.c > +++ b/drivers/iio/imu/inv_mpu6050/inv_mpu_core.c > @@ -542,7 +542,9 @@ static int inv_mpu6050_write_raw(struct iio_dev > *indio_dev, > break; > default: > result = -EINVAL; > + break; > } > + break; > default: > result = -EINVAL; > break; > -- Peter Meerwald-Stadler Mobile: +43 664 24 44 418
