From c0fd857def5547f612d2c14ae57dda1cef47a92c Mon Sep 17 00:00:00 2001 From: Peter Hutterer Date: Fri, 27 Apr 2018 14:40:57 +1000 Subject: [PATCH] touchpad: fix the trackpoint event counter for the T460s Introduced in 416fa44d80b0f2c53b652ddfa35dd4a156a65c65 but there was a logic error: we claimed to require 3 events from a trackpoint before stopping the touchpad but the timer was only set when we actually stopped the touchpad. So if a trackpoint sends a single event every second, we'd disable the touchpad after 3 seconds for the duration of the timeout, then again 3 seconds later, etc. Fix this by always setting the timeout and resetting the event counter if no activity happened. Signed-off-by: Peter Hutterer --- src/evdev-mt-touchpad.c | 13 ++++++++++--- test/test-trackpoint.c | 30 ++++++++++++++++++++++++++++++ 2 files changed, 40 insertions(+), 3 deletions(-) diff --git a/src/evdev-mt-touchpad.c b/src/evdev-mt-touchpad.c index 7b6825c0..c56838df 100644 --- a/src/evdev-mt-touchpad.c +++ b/src/evdev-mt-touchpad.c @@ -31,6 +31,7 @@ #include "evdev-mt-touchpad.h" #define DEFAULT_TRACKPOINT_ACTIVITY_TIMEOUT ms2us(300) +#define DEFAULT_TRACKPOINT_EVENT_TIMEOUT ms2us(40) #define DEFAULT_KEYBOARD_ACTIVITY_TIMEOUT_1 ms2us(200) #define DEFAULT_KEYBOARD_ACTIVITY_TIMEOUT_2 ms2us(500) #define THUMB_MOVE_TIMEOUT ms2us(300) @@ -1945,8 +1946,10 @@ tp_trackpoint_timeout(uint64_t now, void *data) { struct tp_dispatch *tp = data; - tp_tap_resume(tp, now); - tp->palm.trackpoint_active = false; + if (tp->palm.trackpoint_active) { + tp_tap_resume(tp, now); + tp->palm.trackpoint_active = false; + } tp->palm.trackpoint_event_count = 0; } @@ -1963,9 +1966,13 @@ tp_trackpoint_event(uint64_t time, struct libinput_event *event, void *data) tp->palm.trackpoint_last_event_time = time; tp->palm.trackpoint_event_count++; + /* Require at least three events before enabling palm detection */ - if (tp->palm.trackpoint_event_count < 3) + if (tp->palm.trackpoint_event_count < 3) { + libinput_timer_set(&tp->palm.trackpoint_timer, + time + DEFAULT_TRACKPOINT_EVENT_TIMEOUT); return; + } if (!tp->palm.trackpoint_active) { tp_stop_actions(tp, time); diff --git a/test/test-trackpoint.c b/test/test-trackpoint.c index a924f5e2..1f823331 100644 --- a/test/test-trackpoint.c +++ b/test/test-trackpoint.c @@ -379,6 +379,35 @@ START_TEST(trackpoint_palmdetect_require_min_events) } END_TEST +START_TEST(trackpoint_palmdetect_require_min_events_timeout) +{ + struct litest_device *trackpoint = litest_current_device(); + struct litest_device *touchpad; + struct libinput *li = trackpoint->libinput; + + touchpad = litest_add_device(li, LITEST_SYNAPTICS_I2C); + litest_drain_events(li); + + for (int i = 0; i < 10; i++) { + /* A single event does not trigger palm detection */ + litest_event(trackpoint, EV_REL, REL_X, 1); + litest_event(trackpoint, EV_REL, REL_Y, 1); + litest_event(trackpoint, EV_SYN, SYN_REPORT, 0); + libinput_dispatch(li); + litest_drain_events(li); + + litest_touch_down(touchpad, 0, 30, 30); + litest_touch_move_to(touchpad, 0, 30, 30, 80, 80, 10, 1); + litest_touch_up(touchpad, 0); + litest_assert_only_typed_events(li, LIBINPUT_EVENT_POINTER_MOTION); + + litest_timeout_trackpoint(); + } + + litest_delete_device(touchpad); +} +END_TEST + TEST_COLLECTION(trackpoint) { litest_add("trackpoint:middlebutton", trackpoint_middlebutton, LITEST_POINTINGSTICK, LITEST_ANY); @@ -392,4 +421,5 @@ TEST_COLLECTION(trackpoint) litest_add("trackpoint:palmdetect", trackpoint_palmdetect, LITEST_POINTINGSTICK, LITEST_ANY); litest_add("trackpoint:palmdetect", trackpoint_palmdetect_resume_touch, LITEST_POINTINGSTICK, LITEST_ANY); litest_add("trackpoint:palmdetect", trackpoint_palmdetect_require_min_events, LITEST_POINTINGSTICK, LITEST_ANY); + litest_add("trackpoint:palmdetect", trackpoint_palmdetect_require_min_events_timeout, LITEST_POINTINGSTICK, LITEST_ANY); }