| From 828f84ee8f84710ea1818b3565add268bcb824c8 Mon Sep 17 00:00:00 2001 |
| From: Matt Ranostay <matt@ranostay.consulting> |
| Date: Mon, 16 Jan 2017 18:04:18 -0800 |
| Subject: iio: health: max30100: fixed parenthesis around FIFO count check |
| |
| From: Matt Ranostay <matt@ranostay.consulting> |
| |
| commit 828f84ee8f84710ea1818b3565add268bcb824c8 upstream. |
| |
| 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. |
| |
| Note - the fixes tag is not actually correct, but the fix in this patch |
| would also be needed for it to function correctly so we'll go with that |
| one. Backports should pick up both. |
| |
| Signed-off-by: Matt Ranostay <matt@ranostay.consulting> |
| Fixes: b74fccad7 ("iio: health: max30100: correct FIFO check condition") |
| Signed-off-by: Jonathan Cameron <jic23@kernel.org> |
| Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> |
| |
| --- |
| drivers/iio/health/max30100.c | 2 +- |
| 1 file changed, 1 insertion(+), 1 deletion(-) |
| |
| --- a/drivers/iio/health/max30100.c |
| +++ b/drivers/iio/health/max30100.c |
| @@ -238,7 +238,7 @@ static irqreturn_t max30100_interrupt_ha |
| |
| 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; |