diff mbox series

Input: ili210x - Reduce sample period to 15ms

Message ID 20211108114145.84118-1-marex@denx.de (mailing list archive)
State Accepted
Commit 27931d38ce057f36e68bc68cd4bf4ba24bbb9c57
Headers show
Series Input: ili210x - Reduce sample period to 15ms | expand

Commit Message

Marek Vasut Nov. 8, 2021, 11:41 a.m. UTC
Modern devices may redraw display at 60 Hz, make sure we have one input
sample per one frame. Reduce sample period to 15ms, so we would get up
to 66.6 samples per second, although realistically with all the jitter
and extra scheduling wiggle room, we would end up just above 60 samples
per second. This should be a good compromise between sampling too often
and sampling too seldom.

Signed-off-by: Marek Vasut <marex@denx.de>
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Cc: Joe Hung <joe_hung@ilitek.com>
Cc: Luca Hsu <luca_hsu@ilitek.com>
---
 drivers/input/touchscreen/ili210x.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Dmitry Torokhov Nov. 10, 2021, 6:28 a.m. UTC | #1
On Mon, Nov 08, 2021 at 12:41:45PM +0100, Marek Vasut wrote:
> Modern devices may redraw display at 60 Hz, make sure we have one input
> sample per one frame. Reduce sample period to 15ms, so we would get up
> to 66.6 samples per second, although realistically with all the jitter
> and extra scheduling wiggle room, we would end up just above 60 samples
> per second. This should be a good compromise between sampling too often
> and sampling too seldom.
> 
> Signed-off-by: Marek Vasut <marex@denx.de>
> Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Cc: Joe Hung <joe_hung@ilitek.com>
> Cc: Luca Hsu <luca_hsu@ilitek.com>

Applied, thank you.
diff mbox series

Patch

diff --git a/drivers/input/touchscreen/ili210x.c b/drivers/input/touchscreen/ili210x.c
index 6dd5095be567f..ba0e2ae3928a7 100644
--- a/drivers/input/touchscreen/ili210x.c
+++ b/drivers/input/touchscreen/ili210x.c
@@ -14,7 +14,7 @@ 
 #include <linux/slab.h>
 #include <asm/unaligned.h>
 
-#define ILI2XXX_POLL_PERIOD	20
+#define ILI2XXX_POLL_PERIOD	15
 
 #define ILI210X_DATA_SIZE	64
 #define ILI211X_DATA_SIZE	43