diff options
author | Tai-hwa Liang <[email protected]> | 2012-09-19 11:10:47 -0700 |
---|---|---|
committer | Dmitry Torokhov <[email protected]> | 2012-09-19 14:58:07 -0700 |
commit | 30ebb7fa0e3e92145b859ad6e44aa6dc636b4103 (patch) | |
tree | ecfaf412c27c5096cd7a368c240f27fd29563467 | |
parent | 59d1f5c41a21f0d87973f2dedd40efd927a201d6 (diff) |
Input: sentelic - filter out erratic movement when lifting finger
When lifing finger off the surface some versions of touchpad send movement
packets with very low coordinates, which cause cursor to jump to the upper
left corner of the screen. Let's ignore least significant bits of X and Y
coordinates if higher bits are all zeroes and consider finger not touching
the pad.
Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=43197
Reported-and-tested-by: Aleksey Spiridonov <[email protected]>
Tested-by: Eddie Dunn <[email protected]>
Tested-by: Jakub Luzny <[email protected]>
Tested-by: Olivier Goffart <[email protected]>
Signed-off-by: Tai-hwa Liang <[email protected]>
Signed-off-by: Dmitry Torokhov <[email protected]>
-rw-r--r-- | drivers/input/mouse/sentelic.c | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/drivers/input/mouse/sentelic.c b/drivers/input/mouse/sentelic.c index 3f5649f19082..a261d8576919 100644 --- a/drivers/input/mouse/sentelic.c +++ b/drivers/input/mouse/sentelic.c @@ -721,6 +721,17 @@ static psmouse_ret_t fsp_process_byte(struct psmouse *psmouse) switch (psmouse->packet[0] >> FSP_PKT_TYPE_SHIFT) { case FSP_PKT_TYPE_ABS: + + if ((packet[0] == 0x48 || packet[0] == 0x49) && + packet[1] == 0 && packet[2] == 0) { + /* + * Ignore coordinate noise when finger leaving the + * surface, otherwise cursor may jump to upper-left + * corner. + */ + packet[3] &= 0xf0; + } + abs_x = GET_ABS_X(packet); abs_y = GET_ABS_Y(packet); |