Re: [PATCH 2/2] input: misc: New USB eBeam input driver

2013-08-21 Thread Oliver Neukum
On Fri, 2013-08-16 at 22:25 +0200, Yann Cantin wrote: +/* IRQ */ +static int ebeam_read_data(struct ebeam_device *ebeam, unsigned char *pkt) +{ + +/* + * Packet description : 8 bytes + * + * nop packet : FF FF FF FF FF FF FF FF + * + * pkt[0] : Sensors + * bit 1 : ultrasound signal

Re: [PATCH] typo fixes (coordiante - coordinate) in am335x

2013-08-21 Thread Tony Lindgren
* Zubair Lutfullah zubair.lutful...@gmail.com [130715 08:33]: Did a grep for coordiante and replaced them all with coordinate. This applies to the mfd-next tree. This should be safe to apply via the MFD tree as a non-critical fix assuming the bootloaders are not yet using this: Acked-by:

[PATCH v2 1/3] HID: Use hid_parser for pre-scanning the report descriptors

2013-08-21 Thread Benjamin Tissoires
hid_scan_report() implements its own HID report descriptor parsing. It is going to be really bad with the detection of Win 8 certified touchscreen, as this detection relies on a special feature and on the report_size and report_count fields. We can use the existing HID parser in hid-core for

[PATCH v2 3/3] HID: do not init input reports for Win 8 multitouch devices

2013-08-21 Thread Benjamin Tissoires
Some multitouch screens do not like to be polled for input reports. However, the Win8 spec says that all touches should be sent during each report, making the initialization of reports unnecessary. The Win7 spec is less precise, so do not use this for those devices. Add the quirk

[PATCH v2 2/3] HID: detect Win 8 multitouch devices in core

2013-08-21 Thread Benjamin Tissoires
Detecting Win 8 multitouch devices in core allows us to set quirks before the device is parsed through hid_hw_start(). It also simplifies the detection of those devices in hid-multitouch and makes the handling of those devices cleaner. As Win 8 multitouch panels are in the group multitouch and

[PATCH v2 0/3] HID: Win 8 multitouch panels detection in core

2013-08-21 Thread Benjamin Tissoires
Hi guys, this is the v2 of the rework of the pre-scanning of the hid report descriptors. This allows us to be able to detect Win 8 multitouch panels. I tried to take into account all of the previous reviews, and I think the patch series is in a better shape now. Alexander, Srinivas, could you

Re: [PATCH V3 1/2] tps6507x-ts: Add DT support

2013-08-21 Thread Dmitry Torokhov
Hi Manish, On Tue, Aug 20, 2013 at 12:25:32PM +0530, Manish Badarkhe wrote: Hi Prabhakar, On Tue, Aug 20, 2013 at 11:44 AM, Prabhakar Lad All the three above properties as per the documentation above are optional, but the code here makes them required property. May be what you can do

Re: [PATCH v2 1/3] HID: Use hid_parser for pre-scanning the report descriptors

2013-08-21 Thread Henrik Rydberg
Hi Benjamin, this looks pretty good to me, just a few nitpicks below. hid_scan_report() implements its own HID report descriptor parsing. It is going to be really bad with the detection of Win 8 certified touchscreen, as this detection relies on a special feature and on the report_size and

Re: [PATCH v2 2/3] HID: detect Win 8 multitouch devices in core

2013-08-21 Thread Henrik Rydberg
Hi Benjamin, Detecting Win 8 multitouch devices in core allows us to set quirks before the device is parsed through hid_hw_start(). It also simplifies the detection of those devices in hid-multitouch and makes the handling of those devices cleaner. As Win 8 multitouch panels are in the

Rotary encoder platform device driver code examples

2013-08-21 Thread GRENON Loïc
Hi guys, I want to interface a magnetic scale position sensor with a BeagleBone Black (an ARM development platform). While I was reading this following article : http://bwgz57.wordpress.com/2012/04/11/beaglebone-with-a-rotary-encoder/ I discovered that linux has built-in support for rotary

Re: [PATCH v2 0/3] HID: Win 8 multitouch panels detection in core

2013-08-21 Thread Srinivas Pandruvada
On 08/21/2013 08:40 AM, Benjamin Tissoires wrote: Hi guys, this is the v2 of the rework of the pre-scanning of the hid report descriptors. This allows us to be able to detect Win 8 multitouch panels. I tried to take into account all of the previous reviews, and I think the patch series is in a

Re: [PATCH 2/2] input: misc: New USB eBeam input driver

2013-08-21 Thread Yann Cantin
Hi, All issues addressed. Just a point : + +/* input final setup */ +ebeam_setup_input(ebeam, input_dev); + +err = input_register_device(ebeam-input); +if (err) { +dev_dbg(intf-dev, +%s - input_register_device failed, err: %d\n, +