Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1400883

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

From Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Newsgroups linux.kernel
Subject Re: [PATCH 2/2] HID: multitouch: enable the Surface 3 Type Cover to report multitouch data
Date 2016-05-13 18:10 +0200
Message-ID <rynP6-70e-33@gated-at.bofh.it> (permalink)
References <rxZD4-7Er-5@gated-at.bofh.it> <rxZD3-7Er-3@gated-at.bofh.it> <rymgj-55V-27@gated-at.bofh.it> <rymJk-5r8-3@gated-at.bofh.it>
Organization Intel Finland Oy

Show all headers | View raw


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@redhat.com>
> > > ---
> > > 
> > > Andy, would you mind checking if this series is sufficient to
> > > enable
> > > the
> > > TypeCover of the Surface Book?
> > Tested-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> > 
> > 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.

-- 
Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Intel Finland Oy

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

[PATCH 2/2] HID: multitouch: enable the Surface 3 Type Cover to report multitouch data Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-05-12 16:20 +0200
  Re: [PATCH 2/2] HID: multitouch: enable the Surface 3 Type Cover to  report multitouch data Andy Shevchenko <andriy.shevchenko@linux.intel.com> - 2016-05-13 16:30 +0200
    Re: [PATCH 2/2] HID: multitouch: enable the Surface 3 Type Cover to  report multitouch data Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-05-13 17:00 +0200
      Re: [PATCH 2/2] HID: multitouch: enable the Surface 3 Type Cover to  report multitouch data Andy Shevchenko <andriy.shevchenko@linux.intel.com> - 2016-05-13 18:10 +0200
        Re: [PATCH 2/2] HID: multitouch: enable the Surface 3 Type Cover to  report multitouch data Andy Shevchenko <andriy.shevchenko@linux.intel.com> - 2016-05-13 18:30 +0200
          Re: [PATCH 2/2] HID: multitouch: enable the Surface 3 Type Cover to  report multitouch data Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-05-20 10:00 +0200
        Re: [PATCH 2/2] HID: multitouch: enable the Surface 3 Type Cover to  report multitouch data Benjamin Tissoires <benjamin.tissoires@redhat.com> - 2016-05-13 18:30 +0200
  Re: [PATCH 2/2] HID: multitouch: enable the Surface 3 Type Cover to  report multitouch data Bastien Nocera <hadess@hadess.net> - 2016-05-17 20:00 +0200

csiph-web