Re: [PATCH 2/2] HID: multitouch: enable the Surface 3 Type Cover to report multitouch data

From: Benjamin Tissoires
Date: Fri May 20 2016 - 03:58:45 EST


On May 13 2016 or thereabouts, Andy Shevchenko wrote:
> On Fri, 2016-05-13 at 19:09 +0300, Andy Shevchenko wrote:
> > On Fri, 2016-05-13 at 16:49 +0200, Benjamin Tissoires wrote:
> > >
> > > On May 13 2016 or thereabouts, Andy Shevchenko wrote:
> > > >
> > > >
> > > > On Thu, 2016-05-12 at 16:12 +0200, Benjamin Tissoires wrote:
> > > > >
> > > > >
> > > > > There is no reasons to filter out keyboard and consumer control
> > > > > collections
> > > > > in hid-multitouch.
> > > > > With the previous hid-input fix, there is now a full support of
> > > > > the
> > > > > Type
> > > > > Cover and we can remove all specific bits from hid-core and hid-
> > > > > microsoft.
> > > > >
> > > > > hid-multitouch will automatically set HID_QUIRK_NO_INIT_REPORTS
> > > > > so
> > > > > we
> > > > > can
> > > > > also remove it from the list of ushbid quirks.
> > > > >
> > > > > Signed-off-by: Benjamin Tissoires <benjamin.tissoires@xxxxxxxxxx
> > > > > >
> > > > > ---
> > > > >
> > > > > Andy, would you mind checking if this series is sufficient to
> > > > > enable
> > > > > the
> > > > > TypeCover of the Surface Book?
> > > > Tested-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx>
> > > >
> > > > Noticed little difference that there is no event for the device
> > > > anymore,
> > > > just /dev/usb/hiddev0. Would it work properly under X?
> > > Not sure what you mean.
> > > I get 3 input nodes:
> > > Microsoft Surface Type Cover Keyboard
> > > Microsoft Surface Type Cover Consumer Control
> > > Microsoft Surface Type Cover Touchpad
> > >
> > > Each on this input device is properly assigned an event node and X
> > > works
> > > with them.
> > >
> > > If there is no /dev/input/eventX created for your cover, then this
> > > is
> > > not normal and needs to be investigated.
> > There is no /dev/input/eventX for the touchpad.Â
> > It prints that input device is under /usb.../input8, but the actual
> > node
> > I got events from is /dev/usb/hiddev0.
>
> Correction, it seems the event node is there (event6), but I got nothing
> from it

Would you mind sending me a hid-recorder[1] trace of the device while
typing few keys and using the touchpad?

Not receiving events from the actual touchpad node means that the
device did not switch to the mutlitouch mode, or that something else
is going on. I should be able to see that in the hid-recorder log.


[1] http://bentiss.github.io/hid-replay-docs/

Cheers,
Benjamin