[PATCH 0/2 libinput] [RFC] Add touchpad wobbliness detection

Konstantin Kharlamov Hi-Angel at yandex.ru
Sun Feb 18 10:09:22 UTC 2018


For the purposes it seems to work fine — it's marked RFC because I see a
small oddness, and I think it's better to ask someone more
acknowledgable in libinput codebase.

For some reason every time I run 

	libinput debug-events --verbose | grep bled

I see a message about hysteresis being enabled, which then never appears
until I stop the command, and re-run it again. It looks like every time
the command ran, touchpad being reinitialized, making the function
tp_init_hysteresis() to ran again.

If this is an intentional behavior, then I have no other question, and
the patchset should be fine. Though it would be nice if someone with a
"non-wobbly" touchpad could test that it does not enable hysteresis for
them (it all should be good, but testing never hurts :)

P.S.: I don't have commit rights.

Konstantin Kharlamov (2):
  touchpad: remove the code for disabling hysteresis
  touchpad: add wobbling detection

 src/evdev-mt-touchpad.c | 42 ++++++++++++++++++++++++++++--------------
 src/evdev-mt-touchpad.h |  1 +
 2 files changed, 29 insertions(+), 14 deletions(-)

-- 
2.15.1



More information about the wayland-devel mailing list