From: Benjamin Tissoires <benjamin.tissoires@redhat.com>
To: Kai-Heng Feng <kai.heng.feng@canonical.com>
Cc: Jiri Kosina <jikos@kernel.org>,
"open list:INTEL INTEGRATED SENSOR HUB DRIVER"
<linux-input@vger.kernel.org>,
open list <linux-kernel@vger.kernel.org>,
"3.8+" <stable@vger.kernel.org>
Subject: Re: [PATCH] HID: multitouch: enable multi-input as a quirk for some devices
Date: Wed, 27 May 2020 11:50:02 +0200 [thread overview]
Message-ID: <CAO-hwJJE6_8j-XVjVskJwmHW=DM9i5aSZZ=35jLDfjf4E1spZQ@mail.gmail.com> (raw)
In-Reply-To: <CAO-hwJLPF4pSHQqFp-ogZAxKu15nbuKULTRbudhD8L4RFv4w4g@mail.gmail.com>
On Wed, May 27, 2020 at 11:22 AM Benjamin Tissoires
<benjamin.tissoires@redhat.com> wrote:
>
> On Wed, May 27, 2020 at 8:18 AM Kai-Heng Feng
> <kai.heng.feng@canonical.com> wrote:
> >
> >
> >
> > > On May 26, 2020, at 23:07, Benjamin Tissoires <benjamin.tissoires@redhat.com> 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@vger.kernel.org
> > > Signed-off-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
> >
> > Tested-by: Kai-Heng Feng <kai.heng.feng@canonical.com>
>
> 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
> > >
> >
next prev parent reply other threads:[~2020-05-27 9:50 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-05-26 15:07 [PATCH] HID: multitouch: enable multi-input as a quirk for some devices Benjamin Tissoires
2020-05-27 6:18 ` Kai-Heng Feng
2020-05-27 9:22 ` Benjamin Tissoires
2020-05-27 9:50 ` Benjamin Tissoires [this message]
2020-05-27 16:58 ` Sasha Levin
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to='CAO-hwJJE6_8j-XVjVskJwmHW=DM9i5aSZZ=35jLDfjf4E1spZQ@mail.gmail.com' \
--to=benjamin.tissoires@redhat.com \
--cc=jikos@kernel.org \
--cc=kai.heng.feng@canonical.com \
--cc=linux-input@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=stable@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).