[PATCH libinput 2/2] tablet: support tablet devices without BTN_TOOL_PEN
Benjamin Tissoires
benjamin.tissoires at gmail.com
Tue Sep 19 07:00:11 UTC 2017
On Thu, Sep 14, 2017 at 3:04 AM, Peter Hutterer <peter.hutterer at who-t.net>
wrote:
> 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 at who-t.net>
> ---
>
Hi,
I had a look at the kernel and here is what I think:
long story short: hid-uclogic uses "pure" HID, i.e. through hid-input.c, to
assign and process usages. It could be said that there is no logic, the
driver just forwards the incoming events.
So, knowing that BTN_TOOL_PEN is mapped when we see an InRange HID usage,
this means that either the tablet are not fixed because they appeared after
DIGIMend died, or they simply don't provide the information.
I think the latter is more probable and will be more frequent.
So basically, all we *could* do in the kernel is to emulate BTN_TOOL_PEN
based on heuristics, which is basically what you are doing here. And I
think having heuristics in user space is much better than having them in
the kernel where any fix will have a 3 months delay before going into the
users hands.
So I think the libinput only fix is the correct one (IMHO).
Yay-by: Benjamin Tissoires <benjamin.tissoires at gmail.com>
Cheers,
Benjamin
> meson.build | 1 +
> src/evdev-tablet.c | 16 +++++-
> test/litest-device-uclogic-tablet.c | 106 ++++++++++++++++++++++++++++++
> ++++++
> test/litest.c | 2 +
> test/litest.h | 1 +
> test/test-tablet.c | 2 +
> 6 files changed, 126 insertions(+), 2 deletions(-)
> create mode 100644 test/litest-device-uclogic-tablet.c
>
> diff --git a/meson.build b/meson.build
> index 41381553..dc3ab1b6 100644
> --- a/meson.build
> +++ b/meson.build
> @@ -569,6 +569,7 @@ if get_option('tests')
> 'test/litest-device-trackpoint.c',
> 'test/litest-device-touch-screen.c',
> 'test/litest-device-touchscreen-fuzz.c',
> + 'test/litest-device-uclogic-tablet.c',
> 'test/litest-device-wacom-bamboo-16fg-pen.c',
> 'test/litest-device-wacom-cintiq-12wx-pen.c',
> 'test/litest-device-wacom-cintiq-13hdt-finger.c',
> diff --git a/src/evdev-tablet.c b/src/evdev-tablet.c
> index 057b498c..7e9281d1 100644
> --- a/src/evdev-tablet.c
> +++ b/src/evdev-tablet.c
> @@ -1975,7 +1975,8 @@ tablet_reject_device(struct evdev_device *device)
> !libevdev_has_event_code(evdev, EV_ABS, ABS_Y))
> goto out;
>
> - if (!libevdev_has_event_code(evdev, EV_KEY, BTN_TOOL_PEN))
> + if (!libevdev_has_event_code(evdev, EV_KEY, BTN_TOOL_PEN) &&
> + !libevdev_has_event_code(evdev, EV_KEY, BTN_STYLUS))
> goto out;
>
> if (evdev_device_get_size(device, &w, &h) != 0)
> @@ -1996,7 +1997,9 @@ static int
> tablet_init(struct tablet_dispatch *tablet,
> struct evdev_device *device)
> {
> + struct libevdev *evdev = device->evdev;
> enum libinput_tablet_tool_axis axis;
> + bool want_proximity_quirk = false;
> int rc;
>
> tablet->base.dispatch_type = DISPATCH_TABLET;
> @@ -2026,7 +2029,16 @@ tablet_init(struct tablet_dispatch *tablet,
>
> tablet_set_status(tablet, TABLET_TOOL_OUT_OF_PROXIMITY);
>
> - if (device->model_flags & EVDEV_MODEL_TABLET_NO_PROXIMITY_OUT) {
> + if (!libevdev_has_event_code(evdev, EV_KEY, BTN_TOOL_PEN)) {
> + libevdev_enable_event_code(evdev, EV_KEY, BTN_TOOL_PEN,
> NULL);
> + want_proximity_quirk = true;
> + tablet->quirks.proximity_out_forced = true;
> + }
> +
> + if (device->model_flags & EVDEV_MODEL_TABLET_NO_PROXIMITY_OUT)
> + want_proximity_quirk = true;
> +
> + if (want_proximity_quirk) {
> tablet->quirks.need_to_force_prox_out = true;
> libinput_timer_init(&tablet->quirks.prox_out_timer,
> tablet_libinput_context(tablet),
> diff --git a/test/litest-device-uclogic-tablet.c
> b/test/litest-device-uclogic-tablet.c
> new file mode 100644
> index 00000000..b0393d65
> --- /dev/null
> +++ b/test/litest-device-uclogic-tablet.c
> @@ -0,0 +1,106 @@
> +/*
> + * Copyright © 2017 Red Hat, Inc.
> + *
> + * Permission is hereby granted, free of charge, to any person obtaining a
> + * copy of this software and associated documentation files (the
> "Software"),
> + * to deal in the Software without restriction, including without
> limitation
> + * the rights to use, copy, modify, merge, publish, distribute,
> sublicense,
> + * and/or sell copies of the Software, and to permit persons to whom the
> + * Software is furnished to do so, subject to the following conditions:
> + *
> + * The above copyright notice and this permission notice (including the
> next
> + * paragraph) shall be included in all copies or substantial portions of
> the
> + * Software.
> + *
> + * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
> EXPRESS OR
> + * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
> MERCHANTABILITY,
> + * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT
> SHALL
> + * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR
> OTHER
> + * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
> + * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
> + * DEALINGS IN THE SOFTWARE.
> + */
> +
> +#include "config.h"
> +
> +#include "litest.h"
> +#include "litest-int.h"
> +
> +static void litest_uclogic_tablet_setup(void)
> +{
> + struct litest_device *d = litest_create_device(LITEST_
> UCLOGIC_TABLET);
> + litest_set_current_device(d);
> +}
> +
> +static struct input_event proximity_in[] = {
> + { .type = EV_ABS, .code = ABS_X, .value = LITEST_AUTO_ASSIGN },
> + { .type = EV_ABS, .code = ABS_Y, .value = LITEST_AUTO_ASSIGN },
> + { .type = EV_ABS, .code = ABS_PRESSURE, .value =
> LITEST_AUTO_ASSIGN },
> + { .type = EV_SYN, .code = SYN_REPORT, .value = 0 },
> + { .type = -1, .code = -1 },
> +};
> +
> +static struct input_event proximity_out[] = {
> + { .type = -1, .code = -1 },
> +};
> +
> +static struct input_event motion[] = {
> + { .type = EV_ABS, .code = ABS_X, .value = LITEST_AUTO_ASSIGN },
> + { .type = EV_ABS, .code = ABS_Y, .value = LITEST_AUTO_ASSIGN },
> + { .type = EV_ABS, .code = ABS_PRESSURE, .value =
> LITEST_AUTO_ASSIGN },
> + { .type = EV_SYN, .code = SYN_REPORT, .value = 0 },
> + { .type = -1, .code = -1 },
> +};
> +
> +static int
> +get_axis_default(struct litest_device *d, unsigned int evcode, int32_t
> *value)
> +{
> + switch (evcode) {
> + case ABS_PRESSURE:
> + *value = 100;
> + return 0;
> + }
> + return 1;
> +}
> +
> +static struct litest_device_interface interface = {
> + .tablet_proximity_in_events = proximity_in,
> + .tablet_proximity_out_events = proximity_out,
> + .tablet_motion_events = motion,
> +
> + .get_axis_default = get_axis_default,
> +};
> +
> +static struct input_absinfo absinfo[] = {
> + { ABS_X, 0, 32767, 0, 0, 235 },
> + { ABS_Y, 0, 32767, 0, 0, 323 },
> + { ABS_PRESSURE, 0, 1023, 0, 0, 0 },
> + { .value = -1 },
> +};
> +
> +static struct input_id input_id = {
> + .bustype = 0x3,
> + .vendor = 0x256c,
> + .product = 0x6e,
> +};
> +
> +static int events[] = {
> + EV_KEY, BTN_TOUCH,
> + EV_KEY, BTN_STYLUS,
> + EV_KEY, BTN_STYLUS2,
> + EV_MSC, MSC_SCAN,
> + -1, -1,
> +};
> +
> +struct litest_test_device litest_uclogic_tablet_device = {
> + .type = LITEST_UCLOGIC_TABLET,
> + .features = LITEST_TABLET,
> + .shortname = "uclogic-tablet",
> + .setup = litest_uclogic_tablet_setup,
> + .interface = &interface,
> +
> + .name = "uclogic PenTablet Pen",
> + .id = &input_id,
> + .events = events,
> + .absinfo = absinfo,
> +};
> diff --git a/test/litest.c b/test/litest.c
> index 868dfb69..cb95a670 100644
> --- a/test/litest.c
> +++ b/test/litest.c
> @@ -418,6 +418,7 @@ extern struct litest_test_device
> litest_gpio_keys_device;
> extern struct litest_test_device litest_ignored_mouse_device;
> extern struct litest_test_device litest_wacom_mobilestudio_
> 13hdt_pad_device;
> extern struct litest_test_device litest_thinkpad_extrabuttons_device;
> +extern struct litest_test_device litest_uclogic_tablet_device;
>
> struct litest_test_device* devices[] = {
> &litest_synaptics_clickpad_device,
> @@ -487,6 +488,7 @@ struct litest_test_device* devices[] = {
> &litest_ignored_mouse_device,
> &litest_wacom_mobilestudio_13hdt_pad_device,
> &litest_thinkpad_extrabuttons_device,
> + &litest_uclogic_tablet_device,
> NULL,
> };
>
> diff --git a/test/litest.h b/test/litest.h
> index 8f3c3bc9..5868045c 100644
> --- a/test/litest.h
> +++ b/test/litest.h
> @@ -238,6 +238,7 @@ enum litest_device_type {
> LITEST_IGNORED_MOUSE,
> LITEST_WACOM_MOBILESTUDIO_PRO_16_PAD,
> LITEST_THINKPAD_EXTRABUTTONS,
> + LITEST_UCLOGIC_TABLET,
> };
>
> enum litest_device_feature {
> diff --git a/test/test-tablet.c b/test/test-tablet.c
> index 4a36c79a..66b4f79c 100644
> --- a/test/test-tablet.c
> +++ b/test/test-tablet.c
> @@ -260,6 +260,8 @@ tablet_has_proxout_quirk(struct litest_device *dev)
>
> has_quirk = !!udev_device_get_property_value(udev_device,
> "LIBINPUT_MODEL_TABLET_NO_PROXIMITY_OUT");
> + if (!has_quirk)
> + has_quirk = !libevdev_has_event_code(dev->evdev, EV_KEY,
> BTN_TOOL_PEN);
>
> udev_device_unref(udev_device);
>
> --
> 2.13.5
>
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/wayland-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/wayland-devel/attachments/20170919/dd0c13af/attachment-0001.html>
More information about the wayland-devel
mailing list