Hi Dmitry,
Am 19.11.2016 um 19:12 schrieb Dmitry Torokhov dmitry.torokhov@gmail.com:
On Thu, Oct 27, 2016 at 10:44:15AM +0200, H. Nikolaus Schaller wrote:
this should reduce unnecessary input events.
The duplicates will be filtered out by the input core anyway. I like to keep the drivers simple.
Well, the idea was copied from the ads7846 driver. But there it is used to provide a /sys status.
I have tried to find the code location that really makes sure that the BTN_TOUCH is reported only once, but didn't find it.
My thought is that having it here makes me more sure that it is really filtered because it does not to rely on functions deeply hidden in the input core.
On the other hand we apparently also rely on ABS_X etc. to be filtered by core.
In any case I have tested with and without and it does not make a difference. Maybe it was needed when we started to work on this driver some years ago before posting the patches here.
So I drop it from v8 which will come in some minutes.
Thanks.
BR and thanks, Nikolaus