On Sat, Jan 21, 2017 at 6:18 AM, Jonathan Cameron <jic23@xxxxxxxxxx> wrote: > On 17/01/17 02:04, Matt Ranostay wrote: >> FIFO was being read every sample after the "almost full" state was >> reached. This was due to an incorrect placement of the parenthesis >> in the while condition check. >> >> Signed-off-by: Matt Ranostay <matt@ranostay.consulting> > Fixes tags please in future. I've applied this to the fixes-togreg > branch and marked for stable with a fixes tag of the last patch to > touch that line (which isn't really correct, but a backport would require > that patch anyway in addition to this one). Oops. Note taken. > > Jonathan >> --- >> drivers/iio/health/max30100.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/iio/health/max30100.c b/drivers/iio/health/max30100.c >> index 90ab8a2d2846..183c14329d6e 100644 >> --- a/drivers/iio/health/max30100.c >> +++ b/drivers/iio/health/max30100.c >> @@ -238,7 +238,7 @@ 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; >> > -- 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