[PATCH libinput] touchpad: exclude Logitech touchpads from disable-while-typing

[Hans de Goede]

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 <peter.hutterer at who-t.net>

Patch looks good to me:

Reviewed-by: Hans de Goede <hdegoede at redhat.com>

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
>