Comment 0 for bug 1897553

Revision history for this message
Kai-Chuan Hsieh (kchsieh) wrote : Palm detect pressure is too easy to trigger so that the cursor stalled

[Impact]

 * ALPS new i2c touchpads has capabilities for reporting ABS_PRESSURE and ABS_MT_PRESSURE event,
   However, it is very easy to over the default palm detect by pressure threshold 130. When
   the threshold is achieved, the cursor will be stalled, and not be able to keep following the
   finger's movement.

[Test Case]

 * For touchpad with vid is 0488, and have pressure capabilities.

   The pressure capability can be easily obtain by evtest tool, check the ABS event type. There
   should be ABS_PRESSURE and ABS_MT_PRESSURE event.

   Test step:
   Install libinput-tools
   1. $ sudo apt install libinput-tools
   2. $ sudo libinput measure touchpad-pressure
   3. Generally move the finger, the pressure max should not exceed the palm pressure detect
      threshold

[Regression Potential]

 * It only impacts ALPS i2c models
 * It won't impact the touchpad without capability to report pressure event.
 * For those touchpad with pressure capability will make the palm detect pressure be less
   sensitive. However, most firmware can support firmware labeling palm detection, so it is
   okay that the palm detect pressure become less sensitive.

[Other Info]

 * The upstream commit has merged
   https://gitlab.freedesktop.org/libinput/libinput/-/merge_requests/505