On Wed, May 27, 2020 at 11:22 AM Benjamin Tissoires <benjamin.tissoires@xxxxxxxxxx> wrote: > > On Wed, May 27, 2020 at 8:18 AM Kai-Heng Feng > <kai.heng.feng@xxxxxxxxxxxxx> wrote: > > > > > > > > > On May 26, 2020, at 23:07, Benjamin Tissoires <benjamin.tissoires@xxxxxxxxxx> wrote: > > > > > > Two touchpad/trackstick combos are currently not behaving properly. > > > They define a mouse emulation collection, as per Win8 requirements, > > > but also define a separate mouse collection for the trackstick. > > > > > > The way the kernel currently treat the collections is that it > > > merges both in one device. However, given that the first mouse > > > collection already defines X,Y and left, right buttons, when > > > mapping the events from the second mouse collection, hid-multitouch > > > sees that these events are already mapped, and simply ignores them. > > > > > > To be able to report events from the tracktick, add a new quirked > > > class for it, and manually add the 2 devices we know about. > > > > > > Link: https://bugzilla.kernel.org/show_bug.cgi?id=207235 > > > Cc: stable@xxxxxxxxxxxxxxx > > > Signed-off-by: Benjamin Tissoires <benjamin.tissoires@xxxxxxxxxx> > > > > Tested-by: Kai-Heng Feng <kai.heng.feng@xxxxxxxxxxxxx> > > Thanks for the very fast testing :) > > Pushed to for-5.8/multitouch given that we already are at 5.7-rc7, we > might as well postpone it for one week. > Apologies for the inconvenience, I hadn't noticed my master branch was not up to date with origin. I forced push the branch to have a better history. Cheers, Benjamin > Cheers, > Benjamin > > > > > > --- > > > drivers/hid/hid-multitouch.c | 26 ++++++++++++++++++++++++++ > > > 1 file changed, 26 insertions(+) > > > > > > diff --git a/drivers/hid/hid-multitouch.c b/drivers/hid/hid-multitouch.c > > > index 03c720b47306..39e4da7468e1 100644 > > > --- a/drivers/hid/hid-multitouch.c > > > +++ b/drivers/hid/hid-multitouch.c > > > @@ -69,6 +69,7 @@ MODULE_LICENSE("GPL"); > > > #define MT_QUIRK_ASUS_CUSTOM_UP BIT(17) > > > #define MT_QUIRK_WIN8_PTP_BUTTONS BIT(18) > > > #define MT_QUIRK_SEPARATE_APP_REPORT BIT(19) > > > +#define MT_QUIRK_FORCE_MULTI_INPUT BIT(20) > > > > > > #define MT_INPUTMODE_TOUCHSCREEN 0x02 > > > #define MT_INPUTMODE_TOUCHPAD 0x03 > > > @@ -189,6 +190,7 @@ static void mt_post_parse(struct mt_device *td, struct mt_application *app); > > > #define MT_CLS_WIN_8 0x0012 > > > #define MT_CLS_EXPORT_ALL_INPUTS 0x0013 > > > #define MT_CLS_WIN_8_DUAL 0x0014 > > > +#define MT_CLS_WIN_8_FORCE_MULTI_INPUT 0x0015 > > > > > > /* vendor specific classes */ > > > #define MT_CLS_3M 0x0101 > > > @@ -279,6 +281,15 @@ static const struct mt_class mt_classes[] = { > > > MT_QUIRK_CONTACT_CNT_ACCURATE | > > > MT_QUIRK_WIN8_PTP_BUTTONS, > > > .export_all_inputs = true }, > > > + { .name = MT_CLS_WIN_8_FORCE_MULTI_INPUT, > > > + .quirks = MT_QUIRK_ALWAYS_VALID | > > > + MT_QUIRK_IGNORE_DUPLICATES | > > > + MT_QUIRK_HOVERING | > > > + MT_QUIRK_CONTACT_CNT_ACCURATE | > > > + MT_QUIRK_STICKY_FINGERS | > > > + MT_QUIRK_WIN8_PTP_BUTTONS | > > > + MT_QUIRK_FORCE_MULTI_INPUT, > > > + .export_all_inputs = true }, > > > > > > /* > > > * vendor specific classes > > > @@ -1714,6 +1725,11 @@ static int mt_probe(struct hid_device *hdev, const struct hid_device_id *id) > > > if (id->group != HID_GROUP_MULTITOUCH_WIN_8) > > > hdev->quirks |= HID_QUIRK_MULTI_INPUT; > > > > > > + if (mtclass->quirks & MT_QUIRK_FORCE_MULTI_INPUT) { > > > + hdev->quirks &= ~HID_QUIRK_INPUT_PER_APP; > > > + hdev->quirks |= HID_QUIRK_MULTI_INPUT; > > > + } > > > + > > > timer_setup(&td->release_timer, mt_expired_timeout, 0); > > > > > > ret = hid_parse(hdev); > > > @@ -1926,6 +1942,11 @@ static const struct hid_device_id mt_devices[] = { > > > MT_USB_DEVICE(USB_VENDOR_ID_DWAV, > > > USB_DEVICE_ID_DWAV_EGALAX_MULTITOUCH_C002) }, > > > > > > + /* Elan devices */ > > > + { .driver_data = MT_CLS_WIN_8_FORCE_MULTI_INPUT, > > > + HID_DEVICE(BUS_I2C, HID_GROUP_MULTITOUCH_WIN_8, > > > + USB_VENDOR_ID_ELAN, 0x313a) }, > > > + > > > /* Elitegroup panel */ > > > { .driver_data = MT_CLS_SERIAL, > > > MT_USB_DEVICE(USB_VENDOR_ID_ELITEGROUP, > > > @@ -2056,6 +2077,11 @@ static const struct hid_device_id mt_devices[] = { > > > MT_USB_DEVICE(USB_VENDOR_ID_STANTUM_STM, > > > USB_DEVICE_ID_MTP_STM)}, > > > > > > + /* Synaptics devices */ > > > + { .driver_data = MT_CLS_WIN_8_FORCE_MULTI_INPUT, > > > + HID_DEVICE(BUS_I2C, HID_GROUP_MULTITOUCH_WIN_8, > > > + USB_VENDOR_ID_SYNAPTICS, 0xce08) }, > > > + > > > /* TopSeed panels */ > > > { .driver_data = MT_CLS_TOPSEED, > > > MT_USB_DEVICE(USB_VENDOR_ID_TOPSEED2, > > > -- > > > 2.25.1 > > > > >