Re: [PATCH v2 0/2] HID: wacom: add support of Bamboo PAD

2015-02-26 Thread Jiri Kosina
On Wed, 25 Feb 2015, Benjamin Tissoires wrote: So here is the v2 of the patch set to support the Bamboo PAD. It has been tested by Josep (thanks!) on the wired version and I developped it on the wireless version. I guess it should be good to go if the reviews goes well. Cheers, Benjamin

Re: [PATCH 0/2] Cleanup and fix touch_down implementation

2015-02-26 Thread Jiri Kosina
On Fri, 20 Feb 2015, Ping Cheng wrote: Hi Jiri, This set is based on your for-3.20/upstream-fixes branch. If pen comes in proximity while touch is down, we force touch up before sending pen events. This idea has been implemented for Tablet PCs. But other tablets that support both pen and

Re: [PATCH 3/4] HID: uclogic: present only the working interfaces on the Huion tablets

2015-02-26 Thread Benjamin Tissoires
On Feb 25 2015 or thereabouts, Benjamin Tissoires wrote: On Feb 25 2015 or thereabouts, Nikolai Kondrashov wrote: On 02/25/2015 02:05 AM, Benjamin Tissoires wrote: The Huion tablets show 3 interfaces. Only the first and the third are currently used. Also remove HID_QUIRK_MULTI_INPUT for

[PATCH v3 2/2] HID: wacom: add full support of the Wacom Bamboo PAD

2015-02-26 Thread Benjamin Tissoires
The stylus of this device works just fine out of the box. The touch is seen by default as a mouse with relative events and some gestures. The wireless and the wired version have slightly different firmwares, but the debug mode 2 on the feature 2 is common to the 2 devices. In this mode, all the

Re: [PATCH v2 0/4] input: ft5x06: Fix userspace reported maximum value

2015-02-26 Thread Markus Pargmann
Hi Maxime, On Tue, Feb 24, 2015 at 05:55:08PM +0100, Maxime Ripard wrote: Hi Markus, On Mon, Feb 23, 2015 at 10:49:36PM +0100, Markus Pargmann wrote: Hi, On Thu, Nov 13, 2014 at 03:06:54PM +0100, Maxime Ripard wrote: Hi, The current ft5x06 reports to the user-space that its

Re: udev timeout for mma8450 unless the accelerometer is shaken

2015-02-26 Thread Stefan Sauer
udev is 208-r4 - most likely not having the patch. Thanks for the info, will see if I can bump the version. On Thu, Feb 26, 2015 at 3:02 PM, Bastien Nocera had...@hadess.net wrote: On Thu, 2015-02-26 at 14:05 +0100, Stefan Sauer wrote: hi, on my beaglebone-black I get these lines during

Re: udev timeout for mma8450 unless the accelerometer is shaken

2015-02-26 Thread Bastien Nocera
On Thu, 2015-02-26 at 14:05 +0100, Stefan Sauer wrote: hi, on my beaglebone-black I get these lines during bootup [ 71.745951] udevd[105]: worker [113] /devices/ocp.2/4819c000.i2c/i2c-2/2-001d/input/input2/event1 timeout; kill it [ 71.771991] udevd[105]: seq 1243

Re: [PATCH v6 6/8] input: touchscreen: imx25 tcq driver

2015-02-26 Thread Markus Pargmann
Hi, On Mon, Feb 02, 2015 at 09:51:34AM -0800, Dmitry Torokhov wrote: On Mon, Feb 02, 2015 at 05:05:39PM +0100, Markus Pargmann wrote: Hi, On Fri, Jan 30, 2015 at 10:57:21AM -0800, Dmitry Torokhov wrote: On Thu, Jan 29, 2015 at 07:56:40PM +0530, Varka Bhadram wrote: Hi, On

Re: [PATCH v2 1/2] DT: i2c: Deprecate adi,adxl34x compatible string

2015-02-26 Thread Laurent Pinchart
Hi Wolfram, On Monday 26 January 2015 13:09:47 Wolfram Sang wrote: If you drop adi,adxl346, checkpatch will start complaining if it encounters it in a .dts. Boah, this is annoying. That means we need an 346 entry even if it is not different from 345 (which is fine by me).

udev timeout for mma8450 unless the accelerometer is shaken

2015-02-26 Thread Stefan Sauer
hi, on my beaglebone-black I get these lines during bootup [ 71.745951] udevd[105]: worker [113] /devices/ocp.2/4819c000.i2c/i2c-2/2-001d/input/input2/event1 timeout; kill it [ 71.771991] udevd[105]: seq 1243 '/devices/ocp.2/4819c000.i2c/i2c-2/2-001d/input/input2/event1' killed [

[PATCH v2 5/6] HID: uclogic: discard the extra Pen input node on Huion tablets

2015-02-26 Thread Benjamin Tissoires
Some Huion tablets present 2 HID Pen interfaces. Only one is used, so we can drop the unused one. Signed-off-by: Benjamin Tissoires benjamin.tissoi...@redhat.com --- New in v2 drivers/hid/hid-uclogic.c | 20 1 file changed, 20 insertions(+) diff --git

[PATCH v2 4/6] HID: uclogic: apply quirk NO_EMPTY_INPUT

2015-02-26 Thread Benjamin Tissoires
NO_EMPTY_INPUT is useful when MULTI_INPUT is set. It prevents to create empty input nodes that user space does not know what to do with. It does not seem to be required at the moment, this is just a preventive patch. This check is only made during the plug of the device, so it does not hurt to

[PATCH] HID: remove 2 unused usbh.h includes

2015-02-26 Thread Benjamin Tissoires
These 2 are left over from the USB dependency cleaning, so there is no need to keep them. Signed-off-by: Benjamin Tissoires benjamin.tissoi...@redhat.com --- drivers/hid/hid-multitouch.c | 1 - drivers/hid/hid-steelseries.c | 1 - 2 files changed, 2 deletions(-) diff --git

[PATCH v2 2/6] HID: uclogic: merge hid-huion driver in hid-uclogic

2015-02-26 Thread Benjamin Tissoires
From: Nikolai Kondrashov nikolai.kondras...@redhat.com Merge the hid-huion driver into hid-uclogic as all the devices supported by hid-huion are in fact UC-Logic devices. Signed-off-by: Nikolai Kondrashov nikolai.kondras...@redhat.com Signed-off-by: Benjamin Tissoires

[PATCH v2 1/6] HID: uclogic: Set quirks from inside the driver

2015-02-26 Thread Benjamin Tissoires
Based on a patch from: Nikolai Kondrashov nikolai.kondras...@redhat.com Most of the tablets handled by hid-uclogic already use MULTI_INPUT. For the ones which are not quirked in usbhid/hidquirks, they have a custom report descriptor which contains only one report per HID interface. For those

[PATCH] HID: hid-sensor-hub: Fix sparse warning

2015-02-26 Thread Srinivas Pandruvada
Address issue reported by kbuild test robot fengguang...@intel.com sparse warnings: (new ones prefixed by ) drivers/hid/hid-sensor-hub.c:223:39: sparse: incorrect type in argument 3 (different base types) drivers/hid/hid-sensor-hub.c:223:39:expected signed int [signed] [usertype]

[PATCH v2 0/6] HID: huion/uclogic merge and few additions

2015-02-26 Thread Benjamin Tissoires
Hi, this is the v2 of the merge of Huion and UcLogic. I removed the previous 3/4 as mentioned during the review and added 3 more. Note that 6/6 could have been squashed in 2/6, but I kept it separate to keep the first 2 reviewed. Cheers, Benjamin Benjamin Tissoires (6): HID: uclogic: Set

Re: [DIGImend-devel] [PATCH 3/4]....

2015-02-26 Thread Benjamin Tissoires
Hi Vince, On Feb 26 2015 or thereabouts, Vince Herried wrote: I'm overwhelmed by these updates My understanding of the code is not that good. Currently i have a version of the code that gives two interfaces from my Huion h610. I have gotten the buttons to work using xsetwacom. I dont

Re: [PATCH v4 11/20] power_supply: Change ownership from driver to core

2015-02-26 Thread Lee Jones
On Mon, 23 Feb 2015, Krzysztof Kozlowski wrote: Change the ownership of power_supply structure from each driver implementing the class to the power supply core. The patch changes power_supply_register() function thus all drivers implementing power supply class are adjusted. Each driver

Re: [PATCH v4 11/20] power_supply: Change ownership from driver to core

2015-02-26 Thread Krzysztof Kozlowski
On czw, 2015-02-26 at 01:45 +0100, Sebastian Reichel wrote: Hi, On Mon, Feb 23, 2015 at 12:47:32PM +0100, Krzysztof Kozlowski wrote: Change the ownership of power_supply structure from each driver implementing the class to the power supply core. (...) I would like to merge this via

Re: [PATCH] drivers: input: mouse: Add Foxconn's CNTouch driver

2015-02-26 Thread Thomas Sanchez
Hi, Any news/thought regarding this patch ? Thanks, Cheers, 2015-02-15 13:44 GMT+01:00 Thomas Sanchez thomas.san...@gmail.com: This commit introduces a new driver for Foxconn CNTouch trackpad that can be found on LDLC's laptop IRIS family. The driver has been inspired from usbmouse.c.

[PATCH] pidff driver should work correctly with more than one concurrent effect

2015-02-26 Thread Jim Keir
The PID driver (usbhid/hid-pidff.c) does not set the effect ID when uploading an effect. The result is that the initial upload works but subsequent uploads to modify effect parameters are all directed at the last-created effect. The targeted effect ID must be passed back to the device when effect

Re: [PATCH 1/7] thinkpad_acpi: Remember adaptive kbd presence

2015-02-26 Thread Bastien Nocera
On Wed, 2015-02-25 at 22:18 -0800, Darren Hart wrote: On Fri, Feb 20, 2015 at 03:44:10PM +0100, Bastien Nocera wrote: Rather than checking on each suspend and resume whether the laptop has an adaptive keyboard, check when the driver is initialised. Bastien, am I awaiting another version of

Re: [PATCH] Input: mma8450 - convert to using managed resources

2015-02-26 Thread Stefan Sauer
Thanks, tested it and confirming it works them same. On Thu, Feb 26, 2015 at 1:11 PM, Stefan Sauer enso...@google.com wrote: Thanks, tested it and confirming it works them same. On Thu, Feb 26, 2015 at 2:48 AM, Dmitry Torokhov dmitry.torok...@gmail.com wrote: This simplifies error handling

Re: Inactive keyboard on Acer Aspire Switch 11

2015-02-26 Thread Benjamin Tissoires
On Thu, Feb 26, 2015 at 6:18 PM, Florian Echtler f...@butterbrot.org wrote: Hello everyone, I've recently bought an Acer Aspire Switch 11 convertible tablet/notebook hybrid device. Ubuntu 14.04.2 with kernel 3.16 runs almost perfectly, with one notable exception: on the keyboard dock, only

Re: [PATCH v3 2/2] HID: wacom: add full support of the Wacom Bamboo PAD

2015-02-26 Thread Ping Cheng
On Thu, Feb 26, 2015 at 8:28 AM, Benjamin Tissoires benjamin.tissoi...@redhat.com wrote: The stylus of this device works just fine out of the box. The touch is seen by default as a mouse with relative events and some gestures. The wireless and the wired version have slightly different

[PATCHv2 0/4] FocalTech touchpad fixes

2015-02-26 Thread Mathias Gottschlag
Hi, Changes since the last version: - I added a commit which clamps the reported coordinates to valid values, even if the values reported by the touchpad are wrong/misinterpreted. - PSMOUSE_CMD_SETSCALE11 is now encapsulated in a member function of struct psmouse, like the other configuration

[PATCHv2 4/4] psmouse: Disable palm detection in the focaltech driver.

2015-02-26 Thread Mathias Gottschlag
Apparently, the threshold for large contact area seems to be rather low on some devices, causing the touchpad to frequently freeze during normal usage. Because we do now know how we are supposed to use the value in question, this commit just drops the related code completely. Signed-off-by:

Inactive keyboard on Acer Aspire Switch 11

2015-02-26 Thread Florian Echtler
Hello everyone, I've recently bought an Acer Aspire Switch 11 convertible tablet/notebook hybrid device. Ubuntu 14.04.2 with kernel 3.16 runs almost perfectly, with one notable exception: on the keyboard dock, only the touchpad works, not the keyboard itself. I've done a bit of digging in the

Re: [PATCH] HID: remove 2 unused usbh.h includes

2015-02-26 Thread Jiri Kosina
On Thu, 26 Feb 2015, Benjamin Tissoires wrote: These 2 are left over from the USB dependency cleaning, so there is no need to keep them. Signed-off-by: Benjamin Tissoires benjamin.tissoi...@redhat.com I have fixed the filename in the subject :) and applied. Thanks. ---

[PATCHv2 1/4] psmouse: Remove hardcoded touchpad size from the focaltech driver.

2015-02-26 Thread Mathias Gottschlag
The size has in most cases already been fetched from the touchpad, the hardcoded values should have been removed. Signed-off-by: Mathias Gottschlag mgottsch...@gmail.com --- drivers/input/mouse/focaltech.c | 5 + 1 file changed, 1 insertion(+), 4 deletions(-) diff --git

[PATCHv2 3/4] psmouse: Disable resolution/rate/scale changes for FocalTech touchpads.

2015-02-26 Thread Mathias Gottschlag
These PS/2 commands make some touchpads stop responding, so this commit adds some dummy functions to replace the generic implementation. Because scale changes were not encapsulated in a method of struct psmouse yet, this commit adds a method set_scale to psmouse. Signed-off-by: Mathias Gottschlag

[PATCHv2 2/4] psmouse: Ensure that the focaltech driver reports consistent coordinates.

2015-02-26 Thread Mathias Gottschlag
We don't know whether x_max or y_max really hold the maximum possible coordinates, and we don't know for sure whether we correctly interpret the coordinates sent by the touchpad, so we clamp the reported values to prevent confusion in userspace code. Signed-off-by: Mathias Gottschlag

Re: [PATCH v2 1/2] Input: touchscreen-iproc: Add Broadcom iProc touchscreen driver

2015-02-26 Thread Jonathan Richardson
Hi Dmitry, Thanks. I'll go through his patch and make the appropriate changes to our driver. Jon On 15-02-24 03:18 PM, Dmitry Torokhov wrote: Jonathan, On Wed, Feb 11, 2015 at 10:45:34AM -0800, Jonathan Richardson wrote: Pinging maintainers... Am I ok to go ahead with the current