[PATCH v2 5/7] Input: synaptics - remove TOPBUTTONPAD property for Lenovos 2015

2015-02-06 Thread Benjamin Tissoires
The 2015 series of the Lenovo thinkpads added back the hardware buttons on top of the touchpad for the trackstick. Unfortunately, Lenovo used the PNPIDs that are supposed to be 5 buttons touchpads, so the new laptops also have the INPUT_PROP_TOPBUTTONPAD. Yay! Instead of manually removing each

[PATCH v2 4/7] Input: synaptics - retrieve the extended capabilities in query $10

2015-02-06 Thread Benjamin Tissoires
Newer Synaptics touchpads need to get information from the query $10. Retrieve it if available. Signed-off-by: Benjamin Tissoires benjamin.tissoi...@redhat.com --- v2: - brand new drivers/input/mouse/synaptics.c | 23 --- drivers/input/mouse/synaptics.h | 23

[PATCH v2 7/7] Input: synaptics - Remove X1 Carbon 3rd gen from the topbuttonpad list

2015-02-06 Thread Benjamin Tissoires
Lenovo decided to switch back to physical buttons for the trackstick on their latest series. The PNPId list was provided before they reverted back to physical buttons, so it contains the new models too. We can know from the touchpad capabilities that the touchpad has physical buttons, so removing

[PATCH v2 6/7] Input: synaptics - re-route tracksticks buttons on the Lenovo 2015 series

2015-02-06 Thread Benjamin Tissoires
The 2015 series of the Lenovo thinkpads added back the hardware buttons on top of the touchpad for the trackstick. Unfortunately, they are wired to the touchpad, and not the trackstick. Thus, they are seen as extra buttons from the kernel point of view. This leads to a problem in user space

[PATCH v2 2/7] Input: synaptics - handle spurious release of trackstick buttons

2015-02-06 Thread Benjamin Tissoires
The Fimware 8.1 has a bug in which the extra buttons are only sent when the ExtBit is 1. This should be fixed in a future FW update which should have a bump of the minor version. Cc: sta...@vger.kernel.org Signed-off-by: Benjamin Tissoires benjamin.tissoi...@redhat.com --- v2: - break out extra

[PATCH v2 3/7] Input: synaptics - do not retrieve the board id on old firmwares

2015-02-06 Thread Benjamin Tissoires
The board id capability has been added in firmware 7.5. Cc: sta...@vger.kernel.org Signed-off-by: Benjamin Tissoires benjamin.tissoi...@redhat.com --- v2: - brand new drivers/input/mouse/synaptics.c | 4 1 file changed, 4 insertions(+) diff --git a/drivers/input/mouse/synaptics.c

[PATCH v2 1/7] Input: synaptics - fix middle button on Lenovo 2015 products

2015-02-06 Thread Benjamin Tissoires
From: Dmitry Torokhov dmitry.torok...@gmail.com On the X1 Carbon 3rd gen (with a 2015 broadwell cpu), the physical middle button of the trackstick (attached to the touchpad serio device, of course) seems to get lost. Actually, the touchpads reports 3 extra buttons, which falls in the switch

Re: [PATCH] blackfin: convert pinmux request API into empty static inline functions for bf609 and bf548

2015-02-06 Thread Dmitry Torokhov
On Fri, Feb 06, 2015 at 05:41:39PM +0800, Sonic Zhang wrote: From: Sonic Zhang sonic.zh...@analog.com Signed-off-by: Sonic Zhang sonic.zh...@analog.com Reviewed-by: Dmitry Torokhov dmitry.torok...@gmail.com I do not think arch changes shoudl go through my tree though... ---

RE: [PATCH 0/2] GPIO joystick driver

2015-02-06 Thread Holmberg, Hans
Hi Dmitry, Why do we need this new driver when we have gpio-keys driver that can be configured to emit ABS_* events? As far as I can tell, there is no way to specify values for ABS-keys in the device tree binding. While both the device tree binding and the gpio-keys driver could possibly

RE: [PATCH 2/2] input: gpio-joy - GPIO joystick driver

2015-02-06 Thread Holmberg, Hans
+static int gpio_joy_probe(struct platform_device *pdev) { + struct device *dev = pdev-dev; + struct input_dev *input; + struct gpio_joy_drvdata *ddata; + int i, err; + unsigned int debounce_ms; This should be a u32 given you pass it to a function expecting a u32 pointer.

RE: [PATCH 1/2] Documentation: Device Tree binding information for gpio-joy driver

2015-02-06 Thread Holmberg, Hans
The binding should be for a class of hardware, not for a driver. Please describe the class of hardware the binding represents, and remove references to the Linux driver. Will do. +Required properties: + - compatible= gpio-joy; That does not strike me as a very informative

Re: [PATCH v2] Input: da9052_tsi: remove unnecessary worker

2015-02-06 Thread Dmitry Torokhov
On Fri, Feb 06, 2015 at 12:05:48PM +0100, Michael Grzeschik wrote: With the datardy irq we get the information if the pen got pulled from the screen. This patch changes the irq by checking this condition every time instead of triggering the worker. Signed-off-by: Michael Grzeschik

Re: [PATCH v2] Input: da9052_tsi: remove unnecessary worker

2015-02-06 Thread Michael Grzeschik
On Fri, Feb 06, 2015 at 03:24:48PM -0800, Dmitry Torokhov wrote: On Fri, Feb 06, 2015 at 12:05:48PM +0100, Michael Grzeschik wrote: With the datardy irq we get the information if the pen got pulled from the screen. This patch changes the irq by checking this condition every time instead

Re: [PATCH v2] Input: da9052_tsi: remove unnecessary worker

2015-02-06 Thread Dmitry Torokhov
On Sat, Feb 07, 2015 at 12:41:24AM +0100, Michael Grzeschik wrote: On Fri, Feb 06, 2015 at 03:24:48PM -0800, Dmitry Torokhov wrote: On Fri, Feb 06, 2015 at 12:05:48PM +0100, Michael Grzeschik wrote: With the datardy irq we get the information if the pen got pulled from the screen. This

Re: [PATCH v3 02/20] power_supply: Move run-time configuration to separate structure

2015-02-06 Thread Darren Hart
On Fri, Jan 30, 2015 at 03:47:40PM +0100, Krzysztof Kozlowski wrote: Add new structure 'power_supply_config' for holding run-time initialization data like of_node, supplies and private driver data. The power_supply_register() function is changed so all power supply drivers need updating.

Re: [PATCH] Input: bfin_rotary - mark suspend and resume code as __maybe_unused

2015-02-06 Thread Sonic Zhang
Hi Dmitry, On Fri, Feb 6, 2015 at 2:09 PM, Dmitry Torokhov dmitry.torok...@gmail.com wrote: Instead of using #ifdef to guard potentially unused suspend and resume code let's mark them as __maybe_unused so they still get discarded if they are not used but we do not get warning. This allows for

[PATCH 1/2] Documentation: Device Tree binding information for gpio-joy driver

2015-02-06 Thread Hans Holmberg
This document describes the Device Tree bindings for the gpio-joy driver. Signed-off-by: Hans Holmberg hans.holmb...@intel.com --- .../devicetree/bindings/input/gpio-joy.txt | 29 ++ 1 file changed, 29 insertions(+) create mode 100644

[PATCH 0/2] GPIO joystick driver

2015-02-06 Thread Hans Holmberg
This series adds a driver for digital joysticks connected via GPIOs. Tested on Rasperry Pi and Minnowboard MAX using evtest. The DSDT AML-code snippet below illustrates how to configure a joystick on the Minnowboard MAX, using the new _DSD support in the kernel. Scope (_SB) { Device

[PATCH 2/2] input: gpio-joy - GPIO joystick driver

2015-02-06 Thread Hans Holmberg
This is an interrupt-driven driver for digital joysticks connected to GPIOs. Supports any digital joystick with signals for up, down, left, right and one signal for trigger button press, i.e. C64/Atari joysticks. Signed-off-by: Hans Holmberg hans.holmb...@intel.com ---

Re: [RFT/PATCH] Input: bfin_rotary - introduce open and close methods

2015-02-06 Thread Sonic Zhang
Hi Dmitry, On Fri, Feb 6, 2015 at 2:56 PM, Dmitry Torokhov dmitry.torok...@gmail.com wrote: Introduce open and close methods for the input device to postpone enabling the device until it is needed. Signed-off-by: Dmitry Torokhov dmitry.torok...@gmail.com --- Hi Sonic, Could you please

Re: [RFT/PATCH] Input: bfin_rotary - introduce open and close methods

2015-02-06 Thread Sonic Zhang
Hi Dmitry, After apply your patch manually to my blackfin kernel tree, I got kernel panic when probe the rotary. NULL pointer access Kernel OOPS in progress Deferred Exception context CURRENT PROCESS: COMM=swapper PID=1 CPU=0 invalid mm return address: [0x00167648]; contents of: 0x00167620:

[PATCH] blackfin: convert pinmux request API into empty static inline functions for bf609 and bf548

2015-02-06 Thread Sonic Zhang
From: Sonic Zhang sonic.zh...@analog.com Signed-off-by: Sonic Zhang sonic.zh...@analog.com --- arch/blackfin/include/asm/portmux.h | 15 +++ 1 file changed, 11 insertions(+), 4 deletions(-) diff --git a/arch/blackfin/include/asm/portmux.h b/arch/blackfin/include/asm/portmux.h

Re: [PATCH 1/2] Documentation: Device Tree binding information for gpio-joy driver

2015-02-06 Thread Mark Rutland
On Fri, Feb 06, 2015 at 09:32:01AM +, Hans Holmberg wrote: This document describes the Device Tree bindings for the gpio-joy driver. Signed-off-by: Hans Holmberg hans.holmb...@intel.com --- .../devicetree/bindings/input/gpio-joy.txt | 29 ++ 1 file

Re: [PATCH 2/2] input: gpio-joy - GPIO joystick driver

2015-02-06 Thread Mark Rutland
On Fri, Feb 06, 2015 at 09:32:02AM +, Hans Holmberg wrote: This is an interrupt-driven driver for digital joysticks connected to GPIOs. Supports any digital joystick with signals for up, down, left, right and one signal for trigger button press, i.e. C64/Atari joysticks. Signed-off-by:

Re: [PATCH 3/5] Input: synaptics - Query min dimensions for fw v8.1

2015-02-06 Thread Benjamin Tissoires
Thanks Daniel for the respin. On Fri, Feb 6, 2015 at 8:25 AM, Daniel Martin daniel.mar...@secunet.com wrote: From: Daniel Martin consume.no...@gmail.com Query the min dimensions even if the check SYN_EXT_CAP_REQUESTS(priv-capabilities) = 7 fails, but we know that the firmware version 8.1

Re: [PATCH v3 20/20] arm: mach-pxa: Decrement the power supply's device reference counter

2015-02-06 Thread Krzysztof Kozlowski
On pią, 2015-02-06 at 14:49 +0100, Pavel Machek wrote: On Fri 2015-01-30 15:47:58, Krzysztof Kozlowski wrote: Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski k.kozlow...@samsung.com Reviewed-by: Bartlomiej

Re: [v3] synaptics: match PNP-Id is not sufficient for min/max quirks

2015-02-06 Thread Benjamin Tissoires
Again, Thanks for the respin. On Fri, Feb 6, 2015 at 8:25 AM, Daniel Martin daniel.mar...@secunet.com wrote: Hi folks, this is the third version of my patches to address the bug: https://bugzilla.kernel.org/show_bug.cgi?id=91541 One task on top of patch 5 remains: Find suitable board id

[PATCH v2] Input: da9052_tsi: remove unnecessary worker

2015-02-06 Thread Michael Grzeschik
With the datardy irq we get the information if the pen got pulled from the screen. This patch changes the irq by checking this condition every time instead of triggering the worker. Signed-off-by: Michael Grzeschik m.grzesc...@pengutronix.de --- v1 - v2: - removed adc_on variable -

[PATCH v4 2/6] Input: synaptics - Log queried and quirked dimension values

2015-02-06 Thread Benjamin Tissoires
From: Daniel Martin consume.no...@gmail.com Logging the dimension values we queried (info) and the values we use from a quirk to overwrite (warn) can be helpful for debugging. This partly relates to bug: https://bugzilla.kernel.org/show_bug.cgi?id=91541 Cc: sta...@vger.kernel.org

[PATCH v4 1/6] Input: synaptics - Split synaptics_resolution(), query first

2015-02-06 Thread Benjamin Tissoires
From: Daniel Martin consume.no...@gmail.com Split the function synaptics_resolution() into synaptics_resolution() and synaptics_quirks(). synaptics_resolution() will be called before synaptics_quirks() to query dimensions and resolutions before overwriting them with quirks. Cc:

Re: About Goodix-TS on Bay Trail, and ACPI and interrupts

2015-02-06 Thread Antonio Ospite
On Tue, 27 Jan 2015 15:45:59 +0100 Antonio Ospite a...@ao2.it wrote: On Wed, 21 Jan 2015 12:16:54 +0200 Mika Westerberg mika.westerb...@intel.com wrote: On Tue, Jan 20, 2015 at 05:56:45PM +0100, Antonio Ospite wrote: [...] [9.056300] gpiochip_get_desc: hwnum: 68 chip-ngpio: 44 [

[PATCH v4 3/6] Input: synaptics - Query min dimensions for fw v8.1

2015-02-06 Thread Benjamin Tissoires
From: Daniel Martin consume.no...@gmail.com Query the min dimensions even if the check SYN_EXT_CAP_REQUESTS(priv-capabilities) = 7 fails, but we know that the firmware version 8.1 is safe. With that we don't need quirks for post-2013 models anymore as they expose correct min and max

[PATCH v4 4/6] Input: synaptics - Remove obsolete min/max quirk for X240

2015-02-06 Thread Benjamin Tissoires
From: Daniel Martin consume.no...@gmail.com The firmware of the X240 (LEN0035, 2013/12) exposes the same values x [1232..5710], y [1156..4696] as the quirk applies. Cc: sta...@vger.kernel.org Signed-off-by: Daniel Martin consume.no...@gmail.com --- drivers/input/mouse/synaptics.c | 2 +- 1

[PATCH v4 5/6] Input: synaptics - Support min/max board id in min_max_pnpid_table

2015-02-06 Thread Benjamin Tissoires
From: Daniel Martin daniel.mar...@secunet.com Add a min/max range for board ids to the min/max coordinates quirk. This makes it possible to restrict quirks to specific models based upon their board id. The define ANY_BOARD_ID (0) serves as a wildcard. Bugzilla:

[PATCH v4 6/6] Input: synaptics - Skip quirks when post-2013 dimensions

2015-02-06 Thread Benjamin Tissoires
post-2013 Lenovo laptops provide correct min/max dimensions, which are different with the ones currently quirked. According to https://bugzilla.kernel.org/show_bug.cgi?id=91541 The following board ids are assigned in the post-2013 touchpads: t440p/t440s: LEN0036 - 2964/2962 t540p: LEN0034 -

[PATCH] Input: pixcir_i2c_ts - Add RESET gpio

2015-02-06 Thread Roger Quadros
The controller has a RESET pin which is usually controlled over a GPIO line. If such a GPIO is provided, perform a RESET during probe. Signed-off-by: Roger Quadros rog...@ti.com --- .../bindings/input/touchscreen/pixcir_i2c_ts.txt | 3 ++ drivers/input/touchscreen/pixcir_i2c_ts.c |

[v3] synaptics: match PNP-Id is not sufficient for min/max quirks

2015-02-06 Thread Daniel Martin
Hi folks, this is the third version of my patches to address the bug: https://bugzilla.kernel.org/show_bug.cgi?id=91541 One task on top of patch 5 remains: Find suitable board id ranges. I.e. in the first comment of the bug, the board id 2962 is the lowest one that I know of for a post-2013

[PATCH 4/5] Input: synaptics - Remove obsolete min/max quirk for X240

2015-02-06 Thread Daniel Martin
From: Daniel Martin consume.no...@gmail.com The firmware of the X240 (LEN0035, 2013/12) exposes the same values x [1232..5710], y [1156..4696] as the quirk applies. Signed-off-by: Daniel Martin consume.no...@gmail.com --- drivers/input/mouse/synaptics.c | 2 +- 1 file changed, 1

[PATCH 2/5] Input: synaptics - Log queried and quirked dimension values

2015-02-06 Thread Daniel Martin
From: Daniel Martin consume.no...@gmail.com Logging the dimension values we queried (info) and the values we use from a quirk to overwrite (warn) can be helpful for debugging. This partly relates to bug: https://bugzilla.kernel.org/show_bug.cgi?id=91541 Signed-off-by: Daniel Martin

[PATCH 3/5] Input: synaptics - Query min dimensions for fw v8.1

2015-02-06 Thread Daniel Martin
From: Daniel Martin consume.no...@gmail.com Query the min dimensions even if the check SYN_EXT_CAP_REQUESTS(priv-capabilities) = 7 fails, but we know that the firmware version 8.1 is safe. With that we don't need quirks for post-2013 models anymore as they expose correct min and max

[PATCH 5/5] Input: synaptics - Support min/max board id in min_max_pnpid_table

2015-02-06 Thread Daniel Martin
Add a min/max range for board ids to the min/max coordinates quirk. This makes it possible to restrict quirks to specific models based upon their board id. The define ANY_BOARD_ID (0) serves as a wildcard. Bugzilla: https://bugzilla.kernel.org/show_bug.cgi?id=91541 Signed-off-by: Daniel Martin

Re: [PATCH] input: elan: Fix wrong %p extension

2015-02-06 Thread Dmitry Torokhov
On Fri, Feb 06, 2015 at 02:51:13PM +0100, Rasmus Villemoes wrote: There's no %px extension. From the context I think the intention was to dump the five bytes which were not as expected, and for that one should use %ph. Signed-off-by: Rasmus Villemoes li...@rasmusvillemoes.dk Applied, thank

[PATCH 3/4] HID: Introduce a module parameter to disable automatic switch of Logitech gaming wheels from compatibility to native mode. This only applies to multimode wheels.

2015-02-06 Thread Michal Malý
Introduce a module parameter to disable automatic switch of Logitech gaming wheels from compatibility to native mode. This only applies to multimode wheels. Signed-off-by: Michal Malý madcatxs...@devoid-pointer.net --- drivers/hid/hid-lg.c| 6 ++ drivers/hid/hid-lg4ff.c | 4 +++- 2

[PATCH 4/4] HID: Allow switching of Logitech gaming wheels between available compatibility modes through sysfs. This only applies to multimode wheels.

2015-02-06 Thread Michal Malý
Allow switching of Logitech gaming wheels between available compatibility modes through sysfs. This only applies to multimode wheels. Signed-off-by: Michal Malý madcatxs...@devoid-pointer.net --- .../ABI/testing/sysfs-driver-hid-logitech-lg4ff| 45 + drivers/hid/hid-lg4ff.c

[PATCH 0/4] HID: Improve handling of multimode Logitech handling wheels

2015-02-06 Thread Michal Malý
This patch series improves handling of various Logitech gaming wheels and allows switching between various compatibility modes which might be useful to improve compatibility with very old games and testing purposes. Signed-off-by: Michal Malý madcatxs...@devoid-pointer.net Michal Malý (4):

[PATCH 1/4] HID: Identify Logitech gaming wheels in compatibility modes accordingly to Logitech specifications

2015-02-06 Thread Michal Malý
Identify Logitech gaming wheels in compatibility modes accordingly to Logitech specifications. Logitech specification contains a general method of identifying various models of their gaming wheels while they are in compatibility mode. This patch implements the method instead of checking against

Re: [PATCH 0/4] HID: Improve handling of multimode Logitech handling wheels

2015-02-06 Thread simon
This patch series improves handling of various Logitech gaming wheels and allows switching between various compatibility modes which might be useful to improve compatibility with very old games and testing purposes. Hi all, We should note that part 1 performs a very important function of

[PATCH] input: elan: Fix wrong %p extension

2015-02-06 Thread Rasmus Villemoes
There's no %px extension. From the context I think the intention was to dump the five bytes which were not as expected, and for that one should use %ph. Signed-off-by: Rasmus Villemoes li...@rasmusvillemoes.dk --- I think the bug is mostly harmless, since the unrecognized extension is simply

Re: [PATCH v3 20/20] arm: mach-pxa: Decrement the power supply's device reference counter

2015-02-06 Thread Pavel Machek
On Fri 2015-01-30 15:47:58, Krzysztof Kozlowski wrote: Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski k.kozlow...@samsung.com Reviewed-by: Bartlomiej Zolnierkiewicz b.zolnier...@samsung.com Reviewed-by: Sebastian Reichel

Re: [PATCH v3 20/20] arm: mach-pxa: Decrement the power supply's device reference counter

2015-02-06 Thread Pavel Machek
On Fri 2015-02-06 15:43:08, Krzysztof Kozlowski wrote: On pią, 2015-02-06 at 14:49 +0100, Pavel Machek wrote: On Fri 2015-01-30 15:47:58, Krzysztof Kozlowski wrote: Use power_supply_put() to decrement the power supply's device reference counter. Signed-off-by: Krzysztof Kozlowski

Re: [PATCH 0/2] GPIO joystick driver

2015-02-06 Thread Dmitry Torokhov
Hi Hans, On February 6, 2015 1:32:00 AM PST, Hans Holmberg hans.holmb...@intel.com wrote: This series adds a driver for digital joysticks connected via GPIOs. Why do we need this new driver when we have gpio-keys driver that can be configured to emit ABS_* events? Thanks. -- Dmitry -- To

Re: [PATCH] Input: bfin_rotary - mark suspend and resume code as __maybe_unused

2015-02-06 Thread Geert Uytterhoeven
On Fri, Feb 6, 2015 at 10:06 AM, Sonic Zhang sonic@gmail.com wrote: static struct platform_driver bfin_rotary_device_driver = { .probe = bfin_rotary_probe, .remove = bfin_rotary_remove, .driver = { .name = bfin-rotary,

[PATCH v4 0/6] synaptics: match PNP-Id is not sufficient for min/max quirks

2015-02-06 Thread Benjamin Tissoires
Hi, Here is the v4 of Daniel's patch series regarding https://bugzilla.kernel.org/show_bug.cgi?id=91541 I am resending the whole series to add the cc: stable@ flags and to fix v3 5/5 which did not applied properly on top of input.git/for-next. Cheers, Benjamin Benjamin Tissoires (1): Input:

Re: [PATCH v4 3/6] Input: synaptics - Query min dimensions for fw v8.1

2015-02-06 Thread Benjamin Tissoires
On 02/06/2015 10:44 AM, Benjamin Tissoires wrote: From: Daniel Martin consume.no...@gmail.com Query the min dimensions even if the check SYN_EXT_CAP_REQUESTS(priv-capabilities) = 7 fails, but we know that the firmware version 8.1 is safe. With that we don't need quirks for post-2013

[PATCH 2/4] HID: Display the real wheel model and supported alternate modes through sysfs. This applies only to multimode wheels.

2015-02-06 Thread Michal Malý
Display the real wheel model and supported alternate modes through sysfs. This applies only to multimode wheels. Signed-off-by: Michal Malý madcatxs...@devoid-pointer.net --- drivers/hid/hid-lg4ff.c | 209 ++-- 1 file changed, 202 insertions(+), 7

Re: [PATCH] Input: pixcir_i2c_ts - Add RESET gpio

2015-02-06 Thread Dmitry Torokhov
Hi Roger, On Fri, Feb 06, 2015 at 02:53:10PM +0200, Roger Quadros wrote: The controller has a RESET pin which is usually controlled over a GPIO line. If such a GPIO is provided, perform a RESET during probe. Signed-off-by: Roger Quadros rog...@ti.com ---