circle.yml: add ninja dist to tests

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: if we don't have a uinput device node, skip the test suite

When running ninja dist in a container, we cannot create devices.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: mkdir -p the udev rules and hwdb directories

Especially /run/udev/rules.d may not exist, causing a test suite failure.

https://bugs.freedesktop.org/show_bug.cgi?id=103527

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

touchpad: allow for multiple paired keyboards

needed for the razer blade keybard which provides multiple event nodes for
one physical device but it's hard/impossible to identify which one is the real
event node we care about.

https://bugs.freedesktop.org/show_bug.cgi?id=103156

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

libinput 1.9.1

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

fallback: fix lid switch event listener being initialized twice

Once the lid is closed, the keyboard event listener is set up to open the lid
for us on keyboard events. With the right sequence, we can trigger the
listener to be added to the list multiple times, triggering an assert in the
list test code (or an infinite loop in the 1.8 branch).

Conditions:
* SW_LID value 1 - sets up the keyboard listener
* keyboard event - sets lid_is_closed to false
* SW_LID value 0 - is ignored because we're already open
* SW_LID value 1 - sets up the keyboard listener again

https://bugs.freedesktop.org/show_bug.cgi?id=103298

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

udev: drop the version field in device groups

The version field is a per device information. We have
no guarantees a touchscreen and a tablet device will share
the same version of the firmware (especially if both
firmwares are from different vendors).

Fixes the touch arbitration for the Dell Canvas 27

Signed-off-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

meson: require GTK 3.20

For GDK_SEAT_CAPABILITY_ALL_POINTING

https://bugs.freedesktop.org/show_bug.cgi?id=103462

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tablet: print what capability is missing when rejecting a device

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

touchpad: cap the edge palm detection zones at 8mm

The main purpose of the edge zone is to detect palms in the area where we
cannot assume a full finger size and thus cannot use any other palm detection
mechanism. 8mm should be large enough that a finger should be detected based
on other properties (size, pressure, ...).

https://bugs.freedesktop.org/show_bug.cgi?id=103330

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Revert "touchpad: cap the edge palm detection zones at 8mm"

This patch only adjusted the left edge, not the right edge which was still on
8%

This reverts commit 3e9e0e2eb18f0eae5ffe2be756903d92b8b88f75.

touchpad: cap the edge palm detection zones at 8mm

The main purpose of the edge zone is to detect palms in the area where we
cannot assume a full finger size and thus cannot use any other palm detection
mechanism. 8mm should be large enough that a finger should be detected based
on other properties (size, pressure, ...).

https://bugs.freedesktop.org/show_bug.cgi?id=103330

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Daniel Martin <consume.noise@gmail.com>

test: add a wacom bamboo test device

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: fix missing BTN_9 for the MobileStudio device

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: fix pid for the 13HDT pad

libinput 1.9.0

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

touchpad: cut down on excessive logging for palm size

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

util: add an extra assert for list_insert()

If we're adding an element that's not null or not a freshly initialized list,
chances are we haven't removed it from a previous list.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Jonas Ådahl <jadahl@gmail.com>

test: unref the litest device *after* removing it from the path interface

Our own reference may be the last one that's still alive if the context is
currently suspended (litest_suspend()). If we unref before removing it from
the path interface, we access already freed memory.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Jonas Ådahl <jadahl@gmail.com>

test: fix a comment in the log_priority test

We don't rely that the lid switch doesn't work in this test, but we always
print a few things when a device gets successfully added.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

libinput 1.8.902

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Fix meson options default values

The keyword is 'value', not 'default'. With meson 0.43 this now stops
building.

https://bugs.freedesktop.org/show_bug.cgi?id=103162

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: restore default permissions for a source file

This was accidentally made 755 as part of e9fc59efc853

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

evdev: fix a coverity warning

'rc' is possibly unused

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: always init the log_handler count and reset it last

When running with -j 1 and CK_FORK=no, the log_handler_count is shared between
the tests. The log_priority tests can invoke the log handler during
libinput_unref(), so on the next day the log handler starts with a nonzero log
handler.

Fix this by always initializing it to 0 in the tests we expect it to be zero
and resetting it last.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

doc: add a section for tablet mode switches

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

doc: add another item to the FAQ

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

doc: update the GNOME stack graph

These days it's mutter that controls the input devices, not g-s-d.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

libinput 1.8.901

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: if we're not root, return with status 77

exit code 77 signals "skip this test"

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

fallback: suspend internal keyboards and trackpoints on tablet-mode switch

Because on some devices the keyboard is where the fingers are holding the
device when in tablet mode.

https://bugs.freedesktop.org/show_bug.cgi?id=102729

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

fallback: add 'interface' to the fallback interface's method names

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

touchpad: invert an if condition to allow for early return

if (foo) {
    everything
}

changed to :

if (!foo)
    return
everything

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

fallback: move the fallback code into a separate file

Split out the fallback-specific device handling from the more generic
evdev-specific handling (which is supposed to be available for all devices).

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

evdev: rename evdev_key_type to just key_type

This is a fallback-specific private enum, don't pretend it's evdev-related

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

evdev: abstract the get_switch_state method

Shove it into the generic dispatch interface so we don't entangle evdev and
fallback.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Use named initializers for the various dispatch interfaces

Better for self-documentation than comments and makes it more obvious if we
initialize something wrongly.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

evdev: change prefix on some fallback-only methods

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Merge branch 'wip/litest-use-sections-for-tests-v2'

fallback: allow for multiple keyboards to toggle the lid switch

Previously we only listened for events on the first one to come up, based on
the assumption that there can only be one internal keyboard. The Razer Blade
laptop keyboards come with with multiple event nodes, all looking like a
normal keyboard. The one that comes up first is one for special keys, so
typing on the internal keyboard after a lid switch does not toggle the write
state.

Fix this by allowing for up to 3 keyboard listeners for a lid switch.

https://bugs.freedesktop.org/show_bug.cgi?id=102039

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: fix test listing

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: fix no-device test name handling

All these tests were stored with "no device" as test name.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: fix test listing

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: fix no-device test name handling

All these tests were stored with "no device" as test name.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: switch to a TEST_DEVICE macro for all the litest test devices

The test device initialization code was a bit of duplicated boilerplate and
required adding a reference to the devices to the 'devices' list in litest.c.
Automate this with a new TEST_DEVICE macro that adds the devices to a custom
section in the binary, then loops throught that section to get the device out.

This reduces the boilerplate for each test device to just the TEST_MACRO and
the LITEST_foo device enum entry. It also now automates the shortname of the
device.

The device's shortname was standardised in this approach as well, lowercase
and dashes only.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: add missing newline to an error message

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: drop obsolete semi-mt allocation for the atmel hover device

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: fix the 'all codes' keyboard device

The ...create() method returned the wrong device, so this one was never
actually used. Once we start using, we get test case failures related to the
device having BTN_foo events as well. For now, just disable those codes so we
have a keyboard with all keys and pass the tests. The rest needs better
fixing.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

timer: flush the timer funcs if our events come in late

Avoid processing an event with a time later than the earliest timer expiry. If
libinput_dispatch() isn't called frequently enough, we may have e.g. a tap
timeout happening but read a subsequent input event first. In that case we can
erroneously trigger or miss out on taps, see wrong palm detection, etc.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>

tablet: always enable the no-proximity-out quirk on HUION tablets

And instead disable it when we do get a proximity out.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>

evdev: update key type check for new key defines

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

include: sync linux kernel header files with 4.12

To get the new key defines

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

evdev: add a comment to the toggle_touch interface

And remove an unnecessary one

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

evdev: drop unused #define DEFAULT_TRACKPOINT_ACCEL

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

evdev: drop unused enum value DISPATCH_LID_SWITCH

Obsolete since f0f632649066c4fc92ec3178b9da747758cb7bb8

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tablet: support tablet devices without BTN_TOOL_PEN

Some devices like the UC Logic WP5540U has BTN_STYLUS but not BTN_TOOL_PEN.
While a kernel bug, let's just handle these correctly anyway.

https://bugs.freedesktop.org/show_bug.cgi?id=102570

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Yay-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>

tablet: add a quirk for the HUION PenTablet that doesn't send proximity out events

Could be fixed in the kernel, but these tablets are effectively abandoned and
fixing them is a one-by-one issue. Let's put the infrastructure in place to
have this fixed once for this type of device and move on.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Yay-by: Benjamin Tissoires <benjamin.tissoires@gmail.com>

test: remove leftover sleep() for debugging in lid switch test

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

evdev: remove duplicate device name in log message

evdev_log_* already writes the device name

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

touchpad: don't resume a disabled touchpad

A touchpad that was disabled by toggling the sendevents option would come back
normally after a lid resume, despite still being nominally disabled.

https://bugzilla.redhat.com/show_bug.cgi?id=1448962

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: add --disable-sendevents option to the debug-events/debug-gui tools

Makes it possible to debug issues with sendevents.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Add helper function for time to timeval conversion

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Merge branch 'wip/ignore-moved-touches'

Merge branch 'wip/sw-tablet-mode'

filter: don't allow an accel factor of 0 on the flat profile

Leave a narrow gap so the mouse moves excruciatingly slow instead of not
moving at all. This allows to recover from overexcited mouse speed slider
movements.

https://bugs.freedesktop.org/show_bug.cgi?id=102501

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

udev: move the EKR into the parent's device group

If we find an EKR, search for the usb hub of the Cintiq, then find the Cintiq
Pen (or Touch) device and assume that device's product id. This way we end up
in the same device group as the Cintiq.

Co-authored-by: Jason Gerecke <jason.gerecke@wacom.com>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

util: add a safe_atoi_base() function

For parsing hex numbers

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

touchpad: if a device has a tablet mode switch, disable the touchpad

On some devices with a tablet mode switch, the touchpad is inacessible when
in tablet mode and we don't really need this except to avoid possible ghost
touches (none have been mentioned so far). On other devices like the Lenovo
Yoga, the touchpad points to the back of the device and it's hard to use the
device without accidentally using the touchpad. For those, disabling the
touchpad is the best solution.

https://bugs.freedesktop.org/show_bug.cgi?id=102408

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Add support for SW_TABLET_MODE

https://bugs.freedesktop.org/show_bug.cgi?id=101008

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

evdev: move lid code to the fallback interface

This was originally designed to deal with devices that only have SW_LID. But
it can be moved into the evdev interface to avoid duplication once we have
SW_TABLET_MODE. The original assumption of the lid switch device being a
standalone device with no other switches is not true, having a separate
dispatch hurts us here.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: remove a leftover debug printf statement

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Add libinput_device_switch_has_switch()

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: make some of the switch tests more generic

Still hardcoded for lid switches, but easier to change for other switches now.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: add test for switch capability

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: rename 'sw' to 'dev' for the lid tests

More in line with other tests and allows us to use 'sw' as name for the actual
switch to be toggled later. The variable name 'sw' stays in those tests where
we have touchpad/keyboard/etc. devices as well.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

evdev: remove declarations for two nonexistent functions

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

tools: restore option handling for the debug gui

window_init zeroes out the struct and loses any option parsing done before.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

evdev: remove unused argument from evdev_tag_lid_switch

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

touchpad: rename the lid_switch_listener to just listener

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: make the lid action helper function a generic switch action helper

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: rename test-lid.c to test-switch.c

So we can add other switch tests to it without confusing too many people.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

touchpad: use motion speed to ignore accidental 2fg touches

Calculate the speed of the touch and compare it against a fixed speed limit.
If a touch exceeds the speed when a second touch is set down, that second
touch is marked as a thumb and ignored (unless it's right next to the other
finger, then it's likely a 2fg scroll).

The speed calculation is simple but has to lag behind by one sample - we reset
the motion history whenever a new finger is set down (to avoid pointer jumps)
so we need to know if the finger was moving fast *before* this happens. Plus,
with the pointer jumps we're more likely to get false positives if we
calculate the speed on actual finger down.

This is the simplest version for now, the speed varies greatly between
movements and should probably be averaged across the last 3-or-so samples.

https://bugs.freedesktop.org/show_bug.cgi?id=99703

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

touchpad: store the time in the motion history

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

touchpad: ignore thumbs during edge scrolling

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: fix the litest_touch_move_to() step counter

We were undercounting by one, causing a slight jump by (step-distance * 2) on
the last event.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: fix indentation for palm touch size test

Somehow this ended up being spaces instead of tabs

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: add a test for removing a touch device with a finger still down

https://bugs.freedesktop.org/show_bug.cgi?id=102385

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

evdev: accept accelerometers if they are tagged as other devices too

Devices tagged as accelerometers may also be other devices like tablet pads.
Only ignore pure accelerometer devices but disable the accelerometer axes for
devices that have multiple types.

https://bugs.freedesktop.org/show_bug.cgi?id=102100

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>
Reviewed-by: Jason Gerecke <jason.gerecke@wacom.com>

test: fix pad ring test for small pad ranges

The current tests worked because all rings had the same range, so our error
margin covered for that. With the upcoming MobileStudio Pro 16 pad device, the
range is half and our error margins don't work anymore. Switch to a more
reliable approach that tests every integer value the wheel can send, even
though it relies on kernel filtering.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

Add support for LIBINPUT_IGNORE_DEVICE

The recommended way to have libinput ignore specific devices so far was to
remove the ID_INPUT* properties from the device. That may also affect other
pieces of the stack that need access to this device.

For the niche case of a device that should only be ignored by libinput but
otherwise be treated normally by the system, we now support the
LIBINPUT_IGNORE_DEVICE property.

If the property is set to "0", it's equivalent to being unset. This gets
around some technical limitations in udev where unsetting a property is
impossible via a hwdb entry.

https://bugs.freedesktop.org/show_bug.cgi?id=102229

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: expose litest_create() to test cases

And make it init the full litest device minus the libinput device. This
enables us to add litest devices that aren't handled by libinput.

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

udev: update pressure range for Dell Latitude E6620

"This gives some more sensitivity to the fingers without introducing spurious
touches and movements."

https://bugs.freedesktop.org/show_bug.cgi?id=101670

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

build: Add -Iinclude to unit tests

It seems the unit tests rely on another part of <linux/input.h> which I
missed in the previous commit (5cf4b35b).

Signed-off-by: Philip Withnall <withnall@endlessm.com>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

build: Add -Iinclude to libinput and its tools

Various files use #include <linux/input.h> and, if the system input.h is
too old, will fail to compile. Use the internal copy by adding -Iinclude
to the build command lines. This was the case in the old autotools build
system.

Signed-off-by: Philip Withnall <withnall@endlessm.com>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: fix typo in test case name

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

test: when restoring the log handler to defaults, also restore log priority

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

touchpad: Allow larger palm sizes.

On MBP13,3 the touch areas are quite large, and a thumb size easily gets
to 1000+. Hence need to be able to set palm sizes > 1024 (using 1200
currently).

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

evdev: disable evdev event debugging again

This snuck in with 432fbc33cd6a2181ac213eeb66239df8312836a6

Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

touchpad: Enable timestamp smoothing support for bluetooth touchpads

Bluetooth wreaks havoc with the timestamp of the input events coming
from the touchpad, enable timestamp smoothing support to counter this.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>

filter: Add timestamp smoothing support

Some devices, specifically some bluetooth touchpads generate quite
unreliable timestamps for their events. The problem seems to be that
(some of) these touchpads sample at aprox 90 Hz, but the bluetooth stack
only communicates about every 30 ms (*) and then sends mutiple HID input
reports in one batch.

This results in 2-4 packets / SYNs every 30 ms. With timestamps really
close together. The finger coordinate deltas in these packets change by
aprox. the same amount between each packet when moving a finger at
constant speed. But the time deltas are e.g. 28 ms, 1 ms, 1 ms resulting
in calculate_tracker_velocity returning vastly different speeds for the
1st and 2nd packet, which in turn results in very "jerky" mouse pointer
movement.

*) Maybe it is waiting for a transmit time slot or some such.

This commit adds support for a real simple timestamp smoothing algorithm,
intended *only* for use with touchpads. Since touchpads will send a
contineous stream of events at their sample rate when a finger is down,
this filter simply assumes that any events which are under
event_delta_smooth_threshold us apart are part of a smooth continuous
stream of events with each event being event_delta_smooth_value us apart.

Theoritically a very still finger may send the exact same coordinates
and pressure twice, but even if this happens that is not a problem because
a still finger generates coordinates changes below the hyst treshold so
we ignore it anyways.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
Signed-off-by: Peter Hutterer <peter.hutterer@who-t.net>