This get lost in the inbox? :)
Same fix as for APDS9660 issue in the FIFO
On Thu, Mar 17, 2016 at 8:48 PM, Matt Ranostay <mranostay@xxxxxxxxx> wrote:
> Correct issue that the last entry in FIFO was being read twice due
> to an incorrect decrement of entry count variable before condition
> check.
>
> Signed-off-by: Matt Ranostay <mranostay@xxxxxxxxx>
> ---
> drivers/iio/health/max30100.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/iio/health/max30100.c b/drivers/iio/health/max30100.c
> index 09db893..90ab8a2d 100644
> --- a/drivers/iio/health/max30100.c
> +++ b/drivers/iio/health/max30100.c
> @@ -238,12 +238,13 @@ static irqreturn_t max30100_interrupt_handler(int irq, void *private)
>
> mutex_lock(&data->lock);
>
> - while (cnt-- || (cnt = max30100_fifo_count(data) > 0)) {
> + while (cnt || (cnt = max30100_fifo_count(data) > 0)) {
> ret = max30100_read_measurement(data);
> if (ret)
> break;
>
> iio_push_to_buffers(data->indio_dev, data->buffer);
> + cnt--;
> }
>
> mutex_unlock(&data->lock);
> --
> 2.7.0
>
--
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
