lkml.org 
[lkml]   [2022]   [Aug]   [17]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH 1/4] iio/adc: ingenic: fix channel offsets in buffer
Date
Consumers expect the buffer to only contain enabled channels. While
preparing the buffer, the driver also (incorrectly) inserts empty data
for disabled channels, causing the enabled channels to appear at wrong
offsets. Fix that.

Fixes: b96952f498db ("IIO: Ingenic JZ47xx: Add touchscreen mode.")
Tested-by: Paul Cercueil <paul@crapouillou.net>
Signed-off-by: Artur Rojek <contact@artur-rojek.eu>
---
drivers/iio/adc/ingenic-adc.c | 7 +++----
1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/drivers/iio/adc/ingenic-adc.c b/drivers/iio/adc/ingenic-adc.c
index a7325dbbb99a..5a932c375a89 100644
--- a/drivers/iio/adc/ingenic-adc.c
+++ b/drivers/iio/adc/ingenic-adc.c
@@ -804,11 +804,10 @@ static irqreturn_t ingenic_adc_irq(int irq, void *data)
unsigned int i;
u32 tdat[3];

- for (i = 0; i < ARRAY_SIZE(tdat); mask >>= 2, i++) {
+ memset(tdat, 0, ARRAY_SIZE(tdat));
+ for (i = 0; mask && i < ARRAY_SIZE(tdat); mask >>= 2) {
if (mask & 0x3)
- tdat[i] = readl(adc->base + JZ_ADC_REG_ADTCH);
- else
- tdat[i] = 0;
+ tdat[i++] = readl(adc->base + JZ_ADC_REG_ADTCH);
}

iio_push_to_buffers(iio_dev, tdat);
--
2.37.2
\
 
 \ /
  Last update: 2022-08-17 12:59    [W:0.112 / U:0.592 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site