[PATCH v3.11-rc2] HID: sony: fix HID mapping for PS3 sixaxis controller

2013-07-24 Thread Benjamin Tissoires
Commit f04d51404f51 (HID: driver for PS2/3 Buzz controllers) introduced an input_mapping() callback, but set the return value to -1 to all devices except the Buzz controllers. The result of this is that the Sixaxis input device is not populated, making it useless. Signed-off-by: Benjamin

Re: [PATCH v3.11-rc2] HID: sony: fix HID mapping for PS3 sixaxis controller

2013-07-24 Thread Jiri Kosina
On Wed, 24 Jul 2013, Benjamin Tissoires wrote: Commit f04d51404f51 (HID: driver for PS2/3 Buzz controllers) introduced an input_mapping() callback, but set the return value to -1 to all devices except the Buzz controllers. The result of this is that the Sixaxis input device is not populated,

Re: [PATCH] HID: multitouch: do not init reports for multitouch devices

2013-07-24 Thread Benjamin Tissoires
Hi Henrik, On Fri, Jul 19, 2013 at 11:04 PM, rydb...@euromail.se wrote: Hi Benjamin, 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

[PATCH 0/2] HID special drivers converted to devres API

2013-07-24 Thread Benjamin Tissoires
Hi Jiri, It's been a long time since I told you that I was about to work on the devres API... So here are the first patches. My deductions are that it is safe to run devm_* functions within the hid subsystem. With this, we can simplify a little the error path of some drivers. The gain is more

[PATCH 2/2] HID: multitouch: devm conversion

2013-07-24 Thread Benjamin Tissoires
HID special drivers can use safely the devres API. Use it to remove 25 lines of code and to clean up a little the error paths. Besides the pur kzalloc - devm_kzalloc conversions, I changed the place of the allocation of the new name. Doing this right in mt_input_configured() removes the kstrdup

[PATCH 1/2] HID: trivial devm conversion for special hid drivers

2013-07-24 Thread Benjamin Tissoires
It is safe to use devres allocation within the hid subsystem: - the devres release is called _after_ the call to .remove(), meaning that no freed pointers will exists while removing the device - if a .probe() fails, devres releases all the allocated ressources before going to the next driver:

Re: [PATCH 1/2] HID: trivial devm conversion for special hid drivers

2013-07-24 Thread Andy Shevchenko
On Wed, Jul 24, 2013 at 6:29 PM, Benjamin Tissoires benjamin.tissoi...@redhat.com wrote: It is safe to use devres allocation within the hid subsystem: - the devres release is called _after_ the call to .remove(), meaning that no freed pointers will exists while removing the device - if a

Re: [PATCH] HID: i2c-hid: add DT bindings

2013-07-24 Thread Benjamin Tissoires
On Tue, Jun 18, 2013 at 11:51 AM, Jiri Kosina jkos...@suse.cz wrote: On Thu, 13 Jun 2013, Benjamin Tissoires wrote: Add device tree based support for HID over I2C devices. Tested on an Odroid-X board with a Synaptics touchpad. Signed-off-by: Benjamin Tissoires benjamin.tissoi...@redhat.com

Re: [PATCH 1/2] HID: trivial devm conversion for special hid drivers

2013-07-24 Thread Benjamin Tissoires
On Wed, Jul 24, 2013 at 5:37 PM, Andy Shevchenko andy.shevche...@gmail.com wrote: On Wed, Jul 24, 2013 at 6:29 PM, Benjamin Tissoires benjamin.tissoi...@redhat.com wrote: It is safe to use devres allocation within the hid subsystem: - the devres release is called _after_ the call to .remove(),

Re: [PATCH 2/2] HID: multitouch: devm conversion

2013-07-24 Thread Andy Shevchenko
On Wed, Jul 24, 2013 at 6:29 PM, Benjamin Tissoires benjamin.tissoi...@redhat.com wrote: HID special drivers can use safely the devres API. Use it to remove 25 lines of code and to clean up a little the error paths. Besides the pur kzalloc - devm_kzalloc conversions, I changed the Typo 'pur'

[PATCH v3 1/3] Input: omap-keypad: Cleanup - use bitfiled instead of hardcoded values

2013-07-24 Thread Illia Smyrnov
Use bitfiled instead of hardcoded values to set KBD_CTRL, use BIT macro, remove unused defines. Signed-off-by: Illia Smyrnov illia.smyr...@ti.com Reviewed-by: Felipe Balbi ba...@ti.com --- drivers/input/keyboard/omap4-keypad.c | 25 +++-- 1 files changed, 11 insertions(+),

[PATCH v3 0/3] Input: omap-keypad: Convert to threaded IRQ and cleanup

2013-07-24 Thread Illia Smyrnov
Replace unclear hardcoded values with bit field, convert to threaded IRQ and clear interrupts when open the keypad. Based on top of v3.11-rc2. Tested on OMAP4 SDP. Illia Smyrnov (3): Input: omap-keypad: Cleanup - use bitfiled instead of hardcoded values Input: omap-keypad: Convert to

[PATCH v3 3/3] Input: omap-keypad: Clear interrupts on open

2013-07-24 Thread Illia Smyrnov
Clear interrupts when open keypad. According to TRM, the recommended way for keyboard controller initialization is clear the interrupt-status register, then set up certain keyboard events for generating an interrupt request and set up expected source of wake-up event that generates a wake-up

[PATCH v3 2/3] Input: omap-keypad: Convert to threaded IRQ

2013-07-24 Thread Illia Smyrnov
Convert to use threaded IRQ. Cc: Felipe Balbi ba...@ti.com Signed-off-by: Illia Smyrnov illia.smyr...@ti.com Reviewed-by: Felipe Balbi ba...@ti.com --- drivers/input/keyboard/omap4-keypad.c | 29 - 1 files changed, 20 insertions(+), 9 deletions(-) diff --git

Re: [PATCH v3 3/3] Input: omap-keypad: Clear interrupts on open

2013-07-24 Thread Felipe Balbi
Hi, On Wed, Jul 24, 2013 at 06:54:53PM +0300, Illia Smyrnov wrote: Clear interrupts when open keypad. According to TRM, the recommended way for keyboard controller initialization is clear the interrupt-status register, then set up certain keyboard events for generating an interrupt request

Re: [PATCH 2/2] HID: multitouch: devm conversion

2013-07-24 Thread Benjamin Tissoires
On Wed, Jul 24, 2013 at 5:44 PM, Andy Shevchenko andy.shevche...@gmail.com wrote: On Wed, Jul 24, 2013 at 6:29 PM, Benjamin Tissoires benjamin.tissoi...@redhat.com wrote: HID special drivers can use safely the devres API. Use it to remove 25 lines of code and to clean up a little the error

[PATCH v2 1/2] HID: trivial devm conversion for special hid drivers

2013-07-24 Thread Benjamin Tissoires
It is safe to use devres allocation within the hid subsystem: - the devres release is called _after_ the call to .remove(), meaning that no freed pointers will exists while removing the device - if a .probe() fails, devres releases all the allocated ressources before going to the next driver:

[PATCH v2 0/2] HID special drivers converted to devres API

2013-07-24 Thread Benjamin Tissoires
Hi guys, this is the v2 of this series. Cheers, Benjamin changes since v1: - make the exit code of probe() in hid-a4tech.c returning 0 - fix a typo in the commit message of 2/2 Benjamin Tissoires (2): HID: trivial devm conversion for special hid drivers HID: multitouch: devm conversion

[PATCH v2 2/2] HID: multitouch: devm conversion

2013-07-24 Thread Benjamin Tissoires
HID special drivers can use safely the devres API. Use it to remove 25 lines of code and to clean up a little the error paths. Besides the basic kzalloc - devm_kzalloc conversions, I changed the place of the allocation of the new name. Doing this right in mt_input_configured() removes the kstrdup

Re: [PATCH v2 1/2] HID: trivial devm conversion for special hid drivers

2013-07-24 Thread Andy Shevchenko
On Wed, Jul 24, 2013 at 8:38 PM, Benjamin Tissoires benjamin.tissoi...@redhat.com wrote: It is safe to use devres allocation within the hid subsystem: - the devres release is called _after_ the call to .remove(), meaning that no freed pointers will exists while removing the device - if a

[PATCH] Input: egalax_ts - Provide a .remove function

2013-07-24 Thread Fabio Estevam
Provide a .remove function so that we can unregister the input device. Signed-off-by: Fabio Estevam fabio.este...@freescale.com --- drivers/input/touchscreen/egalax_ts.c | 11 +++ 1 file changed, 11 insertions(+) diff --git a/drivers/input/touchscreen/egalax_ts.c

Re: [PATCH] Input: egalax_ts - Provide a .remove function

2013-07-24 Thread Dmitry Torokhov
Hi Fabio, On Wednesday, July 24, 2013 05:46:09 PM Fabio Estevam wrote: Provide a .remove function so that we can unregister the input device. Signed-off-by: Fabio Estevam fabio.este...@freescale.com --- drivers/input/touchscreen/egalax_ts.c | 11 +++ 1 file changed, 11

[RESEND PATCH v5] Input: sysrq - DT binding for key sequence

2013-07-24 Thread mathieu . poirier
I haven't heard back from anyone after making the last modifications a week ago. If it's all good with you Rob, please ack it so that Dmitry can pick it up on his side. Thanks, Mathieu. From 37318cb3c6339747f4e96e09f07f732a395a5ae1 Mon Sep 17 00:00:00 2001 From: Mathieu J. Poirier

[RESEND PATCH v5] Input: sysrq - DT binding for key sequence

2013-07-24 Thread mathieu . poirier
Sorry about the noise - device tree mailing has changed. I haven't heard back from anyone after making the last modifications a week ago. If it's all good with you Rob, please ack it so that Dmitry can pick it up on his side. Thanks, Mathieu. From 37318cb3c6339747f4e96e09f07f732a395a5ae1 Mon

[PATCH] Input: max11801_ts - convert to devm

2013-07-24 Thread Fabio Estevam
From: Fabio Estevam fabio.este...@freescale.com Converting to devm functions can make the code smaller and cleaner. Signed-off-by: Fabio Estevam fabio.este...@freescale.com --- drivers/input/touchscreen/max11801_ts.c | 37 - 1 file changed, 9 insertions(+), 28

[RFC ebeam PATCH 0/2] new USB eBeam input driver

2013-07-24 Thread Yann Cantin
Hi, New USB input driver for eBeam devices. Currently supported (tested) : - Luidia eBeam classic projection and edge projection models - Nec interactive solution NP01Wi1 NP01Wi2 accessories. From basic usb point of view, all these devices are indistinguishable : they have the same usb ids and

[RFC ebeam PATCH 1/2] hid: Blacklist eBeam devices

2013-07-24 Thread Yann Cantin
Signed-off-by: Yann Cantin yann.can...@laposte.net --- drivers/hid/hid-core.c | 3 +++ drivers/hid/hid-ids.h | 3 +++ 2 files changed, 6 insertions(+) diff --git a/drivers/hid/hid-core.c b/drivers/hid/hid-core.c index 36668d1..da5dfa0 100644 --- a/drivers/hid/hid-core.c +++

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

2013-07-24 Thread Yann Cantin
Signed-off-by: Yann Cantin yann.can...@laposte.net --- Documentation/ABI/testing/sysfs-driver-ebeam | 53 ++ drivers/input/misc/Kconfig | 22 + drivers/input/misc/Makefile | 1 + drivers/input/misc/ebeam.c | 763 +++