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

2015-02-09 Thread Elias Vanderstuyft
On Fri, Feb 6, 2015 at 5:34 PM, Michal Malý madcatxs...@devoid-pointer.net wrote: 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

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

2015-02-09 Thread Sonic Zhang
Hi Dmitry, Since the line I mentioned was removed in v3.19-rc1, I am fine with this patch. Acked-by: Sonic Zhang sonic.zh...@analog.com 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

Re: [PATCH v3 04/20] power_supply: sysfs: Use power_supply_*() API for accessing function attrs

2015-02-09 Thread Stefan Wahren
Hi Krzysztof, Krzysztof Kozlowski k.kozlow...@samsung.com hat am 30. Januar 2015 um 15:47 geschrieben: Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of unregistering the power supply (e.g. by removing the driver). Replace: -

[RFC PATCH] Input: atmel_mxt_ts: Make config CRC calculation match that done by the firmware for mxt224S.

2015-02-09 Thread Martin Fuzzey
Although the protocol guide says the CRC is calculated from T7 onwards on the 224s it appears necessary to calculate from T71 (the previous object). The mismatch is not critical from a functional perspective but causes an incorrect warning Config CRC error, calculated=... file=... to be emitted

[PATCH] Input: atmel_mxt_ts: Configuration data unnecessarily uploaded at each probe.

2015-02-09 Thread Martin Fuzzey
mxt_update_cfg() first checks the info block CRC (data-info_crc) and always does an upload irrespective of configuration CRC if it does not match. However the info_crc is never initialised from the information provided by the firmware which causes an upload to be done at each probe if

[PATCH] Input: atmel_mxt_ts: Fix random configuration load failure

2015-02-09 Thread Martin Fuzzey
Sometimes configuration fails to load with: Bad format: failed to parse object This can occur because the firmware loader does not ensure that the configuration data is null terminated. The scanf() therefore reads arbitrary garbage at the end of file. If one item of garbage happens to

Re: [PATCH v3 04/20] power_supply: sysfs: Use power_supply_*() API for accessing function attrs

2015-02-09 Thread Krzysztof Kozlowski
On pon, 2015-02-09 at 20:02 +0100, Stefan Wahren wrote: Hi Krzysztof, Krzysztof Kozlowski k.kozlow...@samsung.com hat am 30. Januar 2015 um 15:47 geschrieben: Replace direct calls to power supply function attributes with wrappers. Wrappers provide safe access in case of

Re: [PATCH] HID: sony: Enable Gasia third-party PS3 controllers

2015-02-09 Thread Lauri Kasanen
On Mon, 09 Feb 2015 11:08:01 +0100 Oliver Neukum oneu...@suse.de wrote: + ret = usb_interrupt_msg(dev, usb_sndintpipe(dev, 0x02), + buf2, sizeof(buf2), + transfered, USB_CTRL_SET_TIMEOUT); You cannot do this. Even for a single byte

Re: [PATCH 0/6] Fixes for ALPS trackstick

2015-02-09 Thread Dmitry Torokhov
On Sun, Feb 08, 2015 at 01:26:32PM +0100, Pali Rohár wrote: On Monday 02 February 2015 15:27:55 Pali Rohár wrote: On Monday 02 February 2015 11:49:58 Pali Rohár wrote: On Monday 02 February 2015 06:49:31 Dmitry Torokhov wrote: On Sun, Jan 18, 2015 at 10:47:06AM +0100, Pali Rohár wrote:

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

2015-02-09 Thread Krzysztof Kozlowski
On pią, 2015-02-06 at 15:59 +0100, Pavel Machek wrote: 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

Re: [PATCH] HID: sony: Enable Gasia third-party PS3 controllers

2015-02-09 Thread Oliver Neukum
On Sat, 2015-02-07 at 15:48 +0200, Lauri Kasanen wrote: Without this, my Gasia Co.,Ltd PS(R) Gamepad would not send any events. Now everything works including the leds. Based on work by Andrew Haines and Antonio Ospite. cc: Antonio Ospite a...@ao2.it cc: Andrew Haines andrewd...@aol.com

RE: [PATCH 0/2] GPIO joystick driver

2015-02-09 Thread Holmberg, Hans
As far as I can tell, there is no way to specify values for ABS-keys in the device tree binding. It may not be present in device tree binding, but the driver does support it, so I would rather extend the binding than have a brand new driver. All right, I find it a bit like whacking a

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

2015-02-09 Thread Mika Westerberg
On Fri, Feb 06, 2015 at 05:00:13PM +0100, Antonio Ospite wrote: Last doubt, now that I am fixing the DSDT I am going to add an Interrupt resource, but what is the correct way to specify its value? Pin 3 of GPO2 is mapped to IRQ 220 by the mainline kernel, so I added this: Interrupt