Re: [PATCH v16 3/3] Input: Add TouchNetix axiom i2c touchscreen driver

From: Kamel BOUHARA
Date: Fri Jul 12 2024 - 05:12:01 EST


[...]

+/*
+ * axiom devices are typically configured to report touches at a rate
+ * of 100Hz (10ms) for systems that require polling for reports.
+ * When reports are polled, it will be expected to occasionally
+ * observe the overflow bit being set in the reports.
+ * This indicates that reports are not being read fast enough.

This comment is strange; if reports are not read quickly enough at
the default rate, why not set the default rate at the slowest for
which the overflow bit is not set?


The rate has been set to the give a good user experience regardless of
this overflow bit that isn't even processed here.


and therefore I might remove the part refering to this overflow bit in the comment.

Regards,
Kamel