diff options
| author | Denis 'GNUtoo' Carikli <GNUtoo@cyberdimension.org> | 2020-04-19 22:19:30 +0200 |
|---|---|---|
| committer | Joonas Kylmälä <joonas.kylmala@iki.fi> | 2020-06-25 14:10:03 -0400 |
| commit | 32d24591b9b4af3ad8edec4e246fa45cbac2276f (patch) | |
| tree | 1c838ab7e5d02de88651388ae93b14778004fbdb | |
| parent | 70f91bc2f325e8a34e412cd01453f3f4000f177e (diff) | |
| download | kernel_replicant_linux-history/replicant-10/5.8rc2.tar.gz kernel_replicant_linux-history/replicant-10/5.8rc2.tar.bz2 kernel_replicant_linux-history/replicant-10/5.8rc2.zip | |
Input: mms114: don't report 0 pressure while slidinghistory/replicant-10/5.8rc2
When sliding we manage to have events that look like that:
Event: time 1571859641.595517, -------------- SYN_REPORT ------------
Event: time 1571859641.606593, type 3 (EV_ABS), code 54 (ABS_MT_POSITION_Y), value 1193
Event: time 1571859641.606593, type 3 (EV_ABS), code 48 (ABS_MT_TOUCH_MAJOR), value 21
Event: time 1571859641.606593, type 3 (EV_ABS), code 58 (ABS_MT_PRESSURE), value 0
Event: time 1571859641.606593, type 3 (EV_ABS), code 1 (ABS_Y), value 1193
Event: time 1571859641.606593, type 3 (EV_ABS), code 24 (ABS_PRESSURE), value 0
Event: time 1571859641.606593, -------------- SYN_REPORT ------------
However in Documentation/input/multi-touch-protocol.rst, we
have:
ABS_MT_PRESSURE
The pressure, in arbitrary units, on the contact
area. May be used instead of TOUCH and WIDTH for
pressure-based devices or any device with a spatial
signal intensity distribution.
Because of that userspace may use a ABS_MT_PRESSURE of 0 as
an indication that the sliding stopped.
However having an ABS_MT_PRESSURE of 0 is not consistent
with having ABS_MT_TOUCH_MAJOR > 0 along with an
ABS_MT_TRACKING_ID that is still valid.
This was tested on the following devices:
- GT-I9300 with a glass screen protection
- GT-I9300 without any screen protection
- GT-N7105 with a glass screen protection
Reported-by: Javi Ferrer <javi.f.o@gmail.com>
Signed-off-by: Denis 'GNUtoo' Carikli <GNUtoo@cyberdimension.org>
| -rw-r--r-- | drivers/input/touchscreen/mms114.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/drivers/input/touchscreen/mms114.c b/drivers/input/touchscreen/mms114.c index 1f96657310b7..1e2c0fe44e5e 100644 --- a/drivers/input/touchscreen/mms114.c +++ b/drivers/input/touchscreen/mms114.c @@ -184,7 +184,10 @@ static void mms114_process_mt(struct mms114_data *data, struct mms114_touch *tou if (touch->pressed) { touchscreen_report_pos(input_dev, &data->props, x, y, true); input_report_abs(input_dev, ABS_MT_TOUCH_MAJOR, touch->width); - input_report_abs(input_dev, ABS_MT_PRESSURE, touch->strength); + if (touch->strength) { + input_report_abs(input_dev, ABS_MT_PRESSURE, + touch->strength); + } } } |