Hi,

On 21-04-16 08:06, Peter Hutterer wrote:
Logitech does not sell internal touchpads, the closest ones are the TK820 and
the K400 series devices. Neither of which need DWT, the touchpad is next to
the keyboard.

Signed-off-by: Peter Hutterer <[email protected]>

Patch looks good to me:

Reviewed-by: Hans de Goede <[email protected]>

Regards,

Hans



---
  src/evdev-mt-touchpad.c | 4 ++++
  src/libinput-util.h     | 1 +
  2 files changed, 5 insertions(+)

diff --git a/src/evdev-mt-touchpad.c b/src/evdev-mt-touchpad.c
index 40c51c6..1974e2a 100644
--- a/src/evdev-mt-touchpad.c
+++ b/src/evdev-mt-touchpad.c
@@ -1335,6 +1335,10 @@ tp_want_dwt(struct evdev_device *touchpad,
        if (bus_tp == BUS_I8042 && bus_kbd != bus_tp)
                return false;

+       /* Logitech does not have internal touchpads */
+       if (vendor_tp == VENDOR_ID_LOGITECH)
+               return false;
+
        /* For Apple touchpads, always use its internal keyboard */
        if (vendor_tp == VENDOR_ID_APPLE) {
                return vendor_kbd == vendor_tp &&
diff --git a/src/libinput-util.h b/src/libinput-util.h
index 66748ea..8f814cc 100644
--- a/src/libinput-util.h
+++ b/src/libinput-util.h
@@ -37,6 +37,7 @@
  #include "libinput.h"

  #define VENDOR_ID_APPLE 0x5ac
+#define VENDOR_ID_LOGITECH 0x46d
  #define VENDOR_ID_WACOM 0x56a
  #define VENDOR_ID_SYNAPTICS_SERIAL 0x002
  #define PRODUCT_ID_APPLE_KBD_TOUCHPAD 0x273

_______________________________________________
wayland-devel mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to