On Tue, 19 Nov 2019, Hans de Goede wrote: > The Acer SW5-012 2-in-1 keyboard dock uses a Synaptics S91028 touchpad > which is connected to an ITE 8595 USB keyboard controller chip. > > This keyboard has the same quirk for its rfkill / airplane mode hotkey as > other keyboards with the ITE 8595 chip, it only sends a single release > event when pressed and released, it never sends a press event. > > This commit adds this keyboards USB id to the hid-ite id-table, fixing > the rfkill key not working on this keyboard. > > Signed-off-by: Hans de Goede <hdegoede@xxxxxxxxxx> > --- > drivers/hid/hid-ids.h | 1 + > drivers/hid/hid-ite.c | 3 +++ > 2 files changed, 4 insertions(+) > > diff --git a/drivers/hid/hid-ids.h b/drivers/hid/hid-ids.h > index c2e60c727697..50985e321b4f 100644 > --- a/drivers/hid/hid-ids.h > +++ b/drivers/hid/hid-ids.h > @@ -1100,6 +1100,7 @@ > #define USB_DEVICE_ID_SYNAPTICS_LTS2 0x1d10 > #define USB_DEVICE_ID_SYNAPTICS_HD 0x0ac3 > #define USB_DEVICE_ID_SYNAPTICS_QUAD_HD 0x1ac3 > +#define USB_DEVICE_ID_SYNAPTICS_ACER_SWITCH5_012 0x2968 > #define USB_DEVICE_ID_SYNAPTICS_TP_V103 0x5710 > #define USB_DEVICE_ID_SYNAPTICS_ACER_SWITCH5 0x81a7 > > diff --git a/drivers/hid/hid-ite.c b/drivers/hid/hid-ite.c > index a45f2352618d..c436e12feb23 100644 > --- a/drivers/hid/hid-ite.c > +++ b/drivers/hid/hid-ite.c > @@ -40,6 +40,9 @@ static int ite_event(struct hid_device *hdev, struct hid_field *field, > static const struct hid_device_id ite_devices[] = { > { HID_USB_DEVICE(USB_VENDOR_ID_ITE, USB_DEVICE_ID_ITE8595) }, > { HID_USB_DEVICE(USB_VENDOR_ID_258A, USB_DEVICE_ID_258A_6A88) }, > + /* ITE8595 USB kbd ctlr, with Synaptics touchpad connected to it. */ > + { HID_USB_DEVICE(USB_VENDOR_ID_SYNAPTICS, > + USB_DEVICE_ID_SYNAPTICS_ACER_SWITCH5_012) }, > { } Applied, thanks Hans. -- Jiri Kosina SUSE Labs