Re: [PATCH v3 11/13] HID: hid-multitouch: support for hovering devices

2012-11-13 Thread Henrik Rydberg
On Wed, Nov 07, 2012 at 05:37:34PM +0100, Benjamin Tissoires wrote: Win8 devices supporting hovering must provides InRange HID field. provide the The information that the finger is here but is not touching the surface is sent to the user space through ABS_MT_DISTANCE as required by the

Re: [PATCH v3 11/13] HID: hid-multitouch: support for hovering devices

2012-11-13 Thread Benjamin Tissoires
Hi Henrik, thanks for the review of the patchset. I'll do my best for the next version :) On Tue, Nov 13, 2012 at 5:42 PM, Henrik Rydberg rydb...@euromail.se wrote: On Wed, Nov 07, 2012 at 05:37:34PM +0100, Benjamin Tissoires wrote: Win8 devices supporting hovering must provides InRange HID

Re: [PATCH v3 11/13] HID: hid-multitouch: support for hovering devices

2012-11-13 Thread Benjamin Tissoires
On Tue, Nov 13, 2012 at 7:04 PM, Henrik Rydberg rydb...@euromail.se wrote: Hi Benjamin, Why [-1,1] here? At first, I only used [0,1]. However, it's still the same problem with the information being kept between the touches: if you start an application after having touched your device, you

Re: [PATCH v3 11/13] HID: hid-multitouch: support for hovering devices

2012-11-13 Thread Henrik Rydberg
I just intend to force the update of the distance value at the beginning of the touch. This is just to send a coherent state when the finger goes in range, and to make sure that user-space application do not rely on the undefined value (0) whereas the kernel thought it was set to 1. Right,