On Sun, 1 Sep 2024 21:08:39 +0900 Gyeyoung Baek <gye976@xxxxxxxxx> wrote: > 'flush_fifo' label performs same task as 'endsession' label > immediately after calling 'env_reset_fifo' function. > So i remove that duplication. > > Signed-off-by: Gyeyoung Baek <gye976@xxxxxxxxx> Ok. This doesn't greatly affect readability and the code ends up a bit shorter. Applied Jonathan > --- > I worked on the wrong branch in v1, and reworked on the latest branch. > > drivers/iio/imu/inv_mpu6050/inv_mpu_ring.c | 7 ++----- > 1 file changed, 2 insertions(+), 5 deletions(-) > > diff --git a/drivers/iio/imu/inv_mpu6050/inv_mpu_ring.c b/drivers/iio/imu/inv_mpu6050/inv_mpu_ring.c > index 3d3b27f28c9d..4753016d6ee0 100644 > --- a/drivers/iio/imu/inv_mpu6050/inv_mpu_ring.c > +++ b/drivers/iio/imu/inv_mpu6050/inv_mpu_ring.c > @@ -121,15 +121,12 @@ irqreturn_t inv_mpu6050_read_fifo(int irq, void *p) > iio_push_to_buffers_with_timestamp(indio_dev, data, timestamp); > } > > -end_session: > - mutex_unlock(&st->lock); > - iio_trigger_notify_done(indio_dev->trig); > - > - return IRQ_HANDLED; > > flush_fifo: > /* Flush HW and SW FIFOs. */ > inv_reset_fifo(indio_dev); > + > +end_session: > mutex_unlock(&st->lock); > iio_trigger_notify_done(indio_dev->trig); >
