[PATCH] Input: ili210x - Reduce sample period to 15ms

[PATCH] Input: ili210x - Reduce sample period to 15ms

[Marek Vasut]

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(-)

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
-- 
2.33.0

Re: [PATCH] Input: ili210x - Reduce sample period to 15ms

[Dmitry Torokhov]

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.

-- 
Dmitry