From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from saturn.retrosnub.co.uk ([46.235.226.198]:55568 "EHLO saturn.retrosnub.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750735AbeELJER (ORCPT ); Sat, 12 May 2018 05:04:17 -0400 Date: Sat, 12 May 2018 10:04:12 +0100 From: Jonathan Cameron To: Jean-Baptiste Maneyrol Cc: Martin Kelly , linux-iio@vger.kernel.org Subject: Re: [PATCH 2/2] iio: imu: inv_mpu6050: make loop a do-while Message-ID: <20180512100412.7cdd58e6@archlinux> In-Reply-To: References: <20180501175642.8551-1-mkelly@xevo.com> <20180501175642.8551-3-mkelly@xevo.com> <20180506180816.4c15b826@archlinux> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Sender: linux-iio-owner@vger.kernel.org List-Id: linux-iio@vger.kernel.org On Tue, 8 May 2018 16:34:30 +0200 Jean-Baptiste Maneyrol wrote: > On 06/05/2018 19:08, Jonathan Cameron wrote: > > On Tue, 1 May 2018 10:56:42 -0700 > > Martin Kelly wrote: > > > >> Prior to this loop, we check if fifo_count < bytes_per_datum and bail if > >> so. This means that when we hit the loop, we know that fifo_count >= > >> bytes_per_datum, so the check is unneeded and we can turn the loop into > >> a do-while for a slight performance improvement. > >> > >> Signed-off-by: Martin Kelly > > Seems logical to me, but I'd like to give Jean-Baptiste a chance > > to comment on this one. > > > > If it looks like we have forgotten about it in a week or so do > > give me a poke! > > > > Thanks, > > > > Jonathan > Hello, > > no problem for me, looks good. Anyway, I'm planning additionnal changes > that are going to change completely this loop. Ah well, Applied anyway in the meantime. Applied to the togreg branch of iio.git and pushed out as testing for the autobuilders to play with it. Thanks, Jonathan > > JB > > > >> --- > >> drivers/iio/imu/inv_mpu6050/inv_mpu_ring.c | 4 ++-- > >> 1 file changed, 2 insertions(+), 2 deletions(-) > >> > >> diff --git a/drivers/iio/imu/inv_mpu6050/inv_mpu_ring.c b/drivers/iio/imu/inv_mpu6050/inv_mpu_ring.c > >> index 0cb7c20100ca..11593deaaebd 100644 > >> --- a/drivers/iio/imu/inv_mpu6050/inv_mpu_ring.c > >> +++ b/drivers/iio/imu/inv_mpu6050/inv_mpu_ring.c > >> @@ -174,7 +174,7 @@ irqreturn_t inv_mpu6050_read_fifo(int irq, void *p) > >> if (kfifo_len(&st->timestamps) > > >> fifo_count / bytes_per_datum + INV_MPU6050_TIME_STAMP_TOR) > >> goto flush_fifo; > >> - while (fifo_count >= bytes_per_datum) { > >> + do { > >> result = regmap_bulk_read(st->map, st->reg->fifo_r_w, > >> data, bytes_per_datum); > >> if (result) > >> @@ -188,7 +188,7 @@ irqreturn_t inv_mpu6050_read_fifo(int irq, void *p) > >> iio_push_to_buffers_with_timestamp(indio_dev, data, timestamp); > >> > >> fifo_count -= bytes_per_datum; > >> - } > >> + } while (fifo_count >= bytes_per_datum); > >> > >> end_session: > >> mutex_unlock(&st->lock); > > > > -- > > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > > the body of a message to majordomo@vger.kernel.org > > More majordomo info at http://vger.kernel.org/majordomo-info.html > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-iio" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html