Re: [PATCH v2] HID: multitouch: Fetch feature reports on demand for Win8 devices

2015-10-11 Thread Jiri Kosina
On Wed, 7 Oct 2015, Mika Westerberg wrote: > Some newer Intel Skylake based Dell laptops with Win8 precision touchpad > fail when initial feature reports are fetched from it. Below is an example > output with some additional debug included: > > i2c_hid i2c-DLL0704:01: Fetching the HID descriptor

Re: [PATCH v2] HID: multitouch: Fetch feature reports on demand for Win8 devices

2015-10-09 Thread Benjamin Tissoires
On Oct 09 2015 or thereabouts, Mika Westerberg wrote: > On Thu, Oct 08, 2015 at 06:10:23PM -0700, Andrew Duggan wrote: > > No, this touchpad doesn't have HID_QUIRK_NO_INIT_REPORTS set and > > usbhid/hid-core.c is calling usbhid_init_reports() which is retrieving > > feature 8. But, I did some addit

Re: [PATCH v2] HID: multitouch: Fetch feature reports on demand for Win8 devices

2015-10-09 Thread Mika Westerberg
On Thu, Oct 08, 2015 at 06:10:23PM -0700, Andrew Duggan wrote: > No, this touchpad doesn't have HID_QUIRK_NO_INIT_REPORTS set and > usbhid/hid-core.c is calling usbhid_init_reports() which is retrieving > feature 8. But, I did some additional testing and I wasn't able to reproduce > this issue on o

Re: [PATCH v2] HID: multitouch: Fetch feature reports on demand for Win8 devices

2015-10-08 Thread Andrew Duggan
On 10/08/2015 02:40 AM, Mika Westerberg wrote: On Wed, Oct 07, 2015 at 06:12:42PM -0700, Andrew Duggan wrote: I can confirm that it is working on the Dell XPS 13 and the buttonpad property is set correctly. However, I also tried it on an older non production precision touchpad and it didn't work

Re: [PATCH v2] HID: multitouch: Fetch feature reports on demand for Win8 devices

2015-10-08 Thread Mika Westerberg
On Wed, Oct 07, 2015 at 06:12:42PM -0700, Andrew Duggan wrote: > I can confirm that it is working on the Dell XPS 13 and the buttonpad > property is set correctly. However, I also tried it on an older non > production precision touchpad and it didn't work. It reports that it fails > to fetch featur

Re: [PATCH v2] HID: multitouch: Fetch feature reports on demand for Win8 devices

2015-10-07 Thread Andrew Duggan
On 10/07/2015 08:56 AM, Seth Forshee wrote: On Wed, Oct 07, 2015 at 09:34:10AM -0400, Benjamin Tissoires wrote: [sorry for the late answer Jiri, I was on vacations last week] On Oct 07 2015 or thereabouts, Mika Westerberg wrote: Some newer Intel Skylake based Dell laptops with Win8 precision

Re: [PATCH v2] HID: multitouch: Fetch feature reports on demand for Win8 devices

2015-10-07 Thread Seth Forshee
On Wed, Oct 07, 2015 at 09:34:10AM -0400, Benjamin Tissoires wrote: > [sorry for the late answer Jiri, I was on vacations last week] > > On Oct 07 2015 or thereabouts, Mika Westerberg wrote: > > Some newer Intel Skylake based Dell laptops with Win8 precision touchpad > > fail when initial feature

Re: [PATCH v2] HID: multitouch: Fetch feature reports on demand for Win8 devices

2015-10-07 Thread Benjamin Tissoires
[sorry for the late answer Jiri, I was on vacations last week] On Oct 07 2015 or thereabouts, Mika Westerberg wrote: > Some newer Intel Skylake based Dell laptops with Win8 precision touchpad > fail when initial feature reports are fetched from it. Below is an example > output with some additional

[PATCH v2] HID: multitouch: Fetch feature reports on demand for Win8 devices

2015-10-07 Thread Mika Westerberg
Some newer Intel Skylake based Dell laptops with Win8 precision touchpad fail when initial feature reports are fetched from it. Below is an example output with some additional debug included: i2c_hid i2c-DLL0704:01: Fetching the HID descriptor i2c_hid i2c-DLL0704:01: __i2c_hid_command: cmd=20 00