[PATCH libinput 3/4] touchpad: work thumb detection into the tap state machine
Peter Hutterer
peter.hutterer at who-t.net
Tue Jul 7 23:56:05 PDT 2015
Most thumbs are detected a few events into the sequence. Work this into parts
of the tapping state machine. Only the most common use-case is handled here -
if the first finger ends up being marked as a thumb, we return to the idle
state and ignore that touch sequence.
At any other state, we handle thumbs like any other finger.
Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
---
doc/touchpad-tap-state-machine.svg | 18 +++++++++++++++
src/evdev-mt-touchpad-tap.c | 47 ++++++++++++++++++++++++++++++++++++++
2 files changed, 65 insertions(+)
diff --git a/doc/touchpad-tap-state-machine.svg b/doc/touchpad-tap-state-machine.svg
index 89c34fa..b1c5995 100644
--- a/doc/touchpad-tap-state-machine.svg
+++ b/doc/touchpad-tap-state-machine.svg
@@ -1179,5 +1179,23 @@
[Not supported by viewer]</text>
</switch>
</g>
+ <path d="M 2095.54 407 C 2098.34 403.86 2102.32 402.05 2106.52 402 L 2140.49 402 C 2144.69 402.05 2148.67 403.86 2151.47 407 L 2171.45 430 C 2172.01 431.28 2172.01 432.72 2171.45 434 L 2151.47 457 C 2148.67 460.14 2144.69 461.95 2140.49 462 L 2106.52 462 C 2102.32 461.95 2098.34 460.14 2095.54 457 L 2075.56 434 C 2075 432.72 2075 431.28 2075.56 430 L 2095.54 407 Z" fill="#ff99cc" stroke="#000000" stroke-width="2" stroke-miterlimit="10" pointer-events="none"/>
+ <g fill="#000000" font-family="Helvetica" text-anchor="middle" font-size="12px">
+ <text x="2123.25" y="435.75">
+ thumb</text>
+ </g>
+ <path d="M 1801.42 332 L 2068.92 415.05" fill="none" stroke="#000000" stroke-miterlimit="10" pointer-events="none"/>
+ <path d="M 2073.93 416.61 L 2066.21 417.88 L 2068.92 415.05 L 2068.28 411.19 Z" fill="#000000" stroke="#000000" stroke-miterlimit="10" pointer-events="none"/>
+ <path d="M 2152.5 488.7 C 2168.62 494.91 2186.38 494.91 2202.5 488.7 C 2218.62 482.5 2236.38 482.5 2252.5 488.7 L 2252.5 541.28 C 2236.38 535.08 2218.62 535.08 2202.5 541.28 C 2186.38 547.49 2168.62 547.49 2152.5 541.28 L 2152.5 488.7 Z" fill="#ffd966" stroke="#000000" stroke-width="2" stroke-miterlimit="10" pointer-events="none"/>
+ <g fill="#000000" font-family="Helvetica" text-anchor="middle" font-size="12px">
+ <text x="2202.25" y="518.75">
+ TOUCH_DEAD</text>
+ </g>
+ <path d="M 2152.05 462 L 2167.18 477.89" fill="none" stroke="#000000" stroke-miterlimit="10" pointer-events="none"/>
+ <path d="M 2170.8 481.69 L 2163.43 479.03 L 2167.18 477.89 L 2168.5 474.21 Z" fill="#000000" stroke="#000000" stroke-miterlimit="10" pointer-events="none"/>
+ <path d="M 2217.81 547.5 L 2324.65 774.24" fill="none" stroke="#000000" stroke-miterlimit="10" pointer-events="none"/>
+ <path d="M 2326.89 778.99 L 2320.74 774.15 L 2324.65 774.24 L 2327.07 771.16 Z" fill="#000000" stroke="#000000" stroke-miterlimit="10" pointer-events="none"/>
+ <path d="M 1959.61 577 L 2084.82 466.22" fill="none" stroke="#000000" stroke-miterlimit="10" pointer-events="none"/>
+ <path d="M 2088.75 462.74 L 2085.83 470 L 2084.82 466.22 L 2081.19 464.76 Z" fill="#000000" stroke="#000000" stroke-miterlimit="10" pointer-events="none"/>
</g>
</svg>
diff --git a/src/evdev-mt-touchpad-tap.c b/src/evdev-mt-touchpad-tap.c
index a43e235..9bbf6b8 100644
--- a/src/evdev-mt-touchpad-tap.c
+++ b/src/evdev-mt-touchpad-tap.c
@@ -47,6 +47,7 @@ enum tap_event {
TAP_EVENT_RELEASE,
TAP_EVENT_BUTTON,
TAP_EVENT_TIMEOUT,
+ TAP_EVENT_THUMB,
};
/*****************************************
@@ -93,6 +94,7 @@ tap_event_to_str(enum tap_event event)
CASE_RETURN_STRING(TAP_EVENT_RELEASE);
CASE_RETURN_STRING(TAP_EVENT_TIMEOUT);
CASE_RETURN_STRING(TAP_EVENT_BUTTON);
+ CASE_RETURN_STRING(TAP_EVENT_THUMB);
}
return NULL;
}
@@ -166,6 +168,10 @@ tp_tap_idle_handle_event(struct tp_dispatch *tp,
case TAP_EVENT_BUTTON:
tp->tap.state = TAP_STATE_DEAD;
break;
+ case TAP_EVENT_THUMB:
+ log_bug_libinput(libinput,
+ "invalid tap event, no fingers down, no thumb\n");
+ break;
}
}
@@ -193,6 +199,12 @@ tp_tap_touch_handle_event(struct tp_dispatch *tp,
case TAP_EVENT_BUTTON:
tp->tap.state = TAP_STATE_DEAD;
break;
+ case TAP_EVENT_THUMB:
+ tp->tap.state = TAP_STATE_IDLE;
+ t->tap.is_thumb = true;
+ t->tap.state = TAP_TOUCH_STATE_DEAD;
+ tp_tap_clear_timer(tp);
+ break;
}
}
@@ -216,6 +228,11 @@ tp_tap_hold_handle_event(struct tp_dispatch *tp,
case TAP_EVENT_BUTTON:
tp->tap.state = TAP_STATE_DEAD;
break;
+ case TAP_EVENT_THUMB:
+ tp->tap.state = TAP_STATE_IDLE;
+ t->tap.is_thumb = true;
+ t->tap.state = TAP_TOUCH_STATE_DEAD;
+ break;
}
}
@@ -244,6 +261,8 @@ tp_tap_tapped_handle_event(struct tp_dispatch *tp,
tp->tap.state = TAP_STATE_DEAD;
tp_tap_notify(tp, time, 1, LIBINPUT_BUTTON_STATE_RELEASED);
break;
+ case TAP_EVENT_THUMB:
+ break;
}
}
@@ -275,6 +294,8 @@ tp_tap_touch2_handle_event(struct tp_dispatch *tp,
case TAP_EVENT_BUTTON:
tp->tap.state = TAP_STATE_DEAD;
break;
+ case TAP_EVENT_THUMB:
+ break;
}
}
@@ -299,6 +320,8 @@ tp_tap_touch2_hold_handle_event(struct tp_dispatch *tp,
case TAP_EVENT_BUTTON:
tp->tap.state = TAP_STATE_DEAD;
break;
+ case TAP_EVENT_THUMB:
+ break;
}
}
@@ -328,6 +351,8 @@ tp_tap_touch3_handle_event(struct tp_dispatch *tp,
case TAP_EVENT_BUTTON:
tp->tap.state = TAP_STATE_DEAD;
break;
+ case TAP_EVENT_THUMB:
+ break;
}
}
@@ -351,6 +376,8 @@ tp_tap_touch3_hold_handle_event(struct tp_dispatch *tp,
case TAP_EVENT_BUTTON:
tp->tap.state = TAP_STATE_DEAD;
break;
+ case TAP_EVENT_THUMB:
+ break;
}
}
@@ -375,6 +402,8 @@ tp_tap_dragging_or_doubletap_handle_event(struct tp_dispatch *tp,
tp->tap.state = TAP_STATE_DEAD;
tp_tap_notify(tp, time, 1, LIBINPUT_BUTTON_STATE_RELEASED);
break;
+ case TAP_EVENT_THUMB:
+ break;
}
}
@@ -408,6 +437,8 @@ tp_tap_dragging_handle_event(struct tp_dispatch *tp,
tp->tap.state = TAP_STATE_DEAD;
tp_tap_notify(tp, time, 1, LIBINPUT_BUTTON_STATE_RELEASED);
break;
+ case TAP_EVENT_THUMB:
+ break;
}
}
@@ -433,6 +464,8 @@ tp_tap_dragging_wait_handle_event(struct tp_dispatch *tp,
tp->tap.state = TAP_STATE_DEAD;
tp_tap_notify(tp, time, 1, LIBINPUT_BUTTON_STATE_RELEASED);
break;
+ case TAP_EVENT_THUMB:
+ break;
}
}
@@ -459,6 +492,8 @@ tp_tap_dragging_tap_handle_event(struct tp_dispatch *tp,
tp->tap.state = TAP_STATE_DEAD;
tp_tap_notify(tp, time, 1, LIBINPUT_BUTTON_STATE_RELEASED);
break;
+ case TAP_EVENT_THUMB:
+ break;
}
}
@@ -484,6 +519,8 @@ tp_tap_dragging2_handle_event(struct tp_dispatch *tp,
tp->tap.state = TAP_STATE_DEAD;
tp_tap_notify(tp, time, 1, LIBINPUT_BUTTON_STATE_RELEASED);
break;
+ case TAP_EVENT_THUMB:
+ break;
}
}
@@ -518,6 +555,8 @@ tp_tap_multitap_handle_event(struct tp_dispatch *tp,
tp->tap.state = TAP_STATE_IDLE;
tp_tap_clear_timer(tp);
break;
+ case TAP_EVENT_THUMB:
+ break;
}
}
@@ -556,6 +595,8 @@ tp_tap_multitap_down_handle_event(struct tp_dispatch *tp,
tp_tap_notify(tp, time, 1, LIBINPUT_BUTTON_STATE_RELEASED);
tp_tap_clear_timer(tp);
break;
+ case TAP_EVENT_THUMB:
+ break;
}
}
@@ -576,6 +617,8 @@ tp_tap_dead_handle_event(struct tp_dispatch *tp,
case TAP_EVENT_TIMEOUT:
case TAP_EVENT_BUTTON:
break;
+ case TAP_EVENT_THUMB:
+ break;
}
}
@@ -731,6 +774,10 @@ tp_tap_handle_state(struct tp_dispatch *tp, uint64_t time)
}
tp_tap_handle_event(tp, t, TAP_EVENT_MOTION, time);
+ } else if (tp->tap.state != TAP_STATE_IDLE &&
+ t->is_thumb &&
+ !t->tap.is_thumb) {
+ tp_tap_handle_event(tp, t, TAP_EVENT_THUMB, time);
}
}
--
2.4.3
More information about the wayland-devel
mailing list