linux-toradex.git/drivers/hid/hid-sony.c, branch v4.11

HID: sony: Fix input device leak when connecting a DS4 twice using USB/BT

2017-03-06T13:43:16+00:00

When a user connects a DS4 twice using USB and BT, we reject the
second device connection after the setup work. We then perform
a cleanup, but during cleanup we are not removing the touchpad
device. This leads to leakage of an input device, which we would
never remove. It can likely result into a kernel oops as well
when the touchpad evdev node is accessed and the underlaying HID
device has been removed from the system.

[jkosina@suse.cz: added stable annotation]
Fixes: ac797b95f532 ("HID: sony: Make the DS4 touchpad a separate device")
Cc: stable@vger.kernel.org
Signed-off-by: Roderick Colenbrander 
Reviewed-by: Benjamin Tissoires 
Signed-off-by: Jiri Kosina

HID: sony: Ignore DS4 dongle reports when no device is connected

2016-12-19T14:07:37+00:00

When the DS4 dongle is connected, it always generates HID reports
even when no DS4 is paired to it. This patch adds logic to ignore
HID reports from the dongle if there is no DS4 currently attached.

Signed-off-by: Roderick Colenbrander 
Signed-off-by: Jiri Kosina

HID: sony: Use DS4 MAC address as unique identifier on USB

2016-12-19T14:07:37+00:00

The DS4 MAC address is reported as a unique identified when
using Bluetooth. For USB there is no unique identifier reported
yet, so use the MAC address.

Signed-off-by: Roderick Colenbrander 
Signed-off-by: Jiri Kosina

HID: sony: Fix error handling bug when touchpad registration fails

2016-12-19T14:07:37+00:00

The error handling code in sony_input_configured in general uses goto
based cleanup. Recently we migrated code from sony_probe to here, but
we didn't update the existing touchpad registration code, which was
already here to use the goto.

Signed-off-by: Roderick Colenbrander 
Signed-off-by: Jiri Kosina

HID: sony: Support DS4 dongle

2016-11-29T13:31:20+00:00

Add support for USB based DS4 dongle device, which allows connecting
a DS4 through Bluetooth, but hides Bluetooth from the host system.

Signed-off-by: Roderick Colenbrander 
Signed-off-by: Jiri Kosina

HID: sony: Comply to Linux gamepad spec for DS4

2016-11-28T13:54:57+00:00

The DS4 side of hid-sony used the hid-core layer to assign buttons
and axes based on the HID report descriptors. The default mapping
was strange e.g. right stick using ABS_Z/ABS_RZ or the physical
'south button' being reported as BTN_EAST etcetera.

This patch makes the DS4 side ofi the hid-sony driver comply to
the Linux game controller spec as suggested in a discussion with
Dmitry on the linux-input list.

Currently the main user of the DS4 is the SDL2 library, which has
a mapping table using vendor/device/version as a key. In order to
not break SDL2 we discussed adjusting the version number, so it
can have both mappings. This was discust on linux-input and we
discussed privately with SDL2 developers.

Signed-off-by: Roderick Colenbrander 
Signed-off-by: Jiri Kosina

HID: sony: Make the DS4 touchpad a separate device

2016-11-28T13:54:57+00:00

The dualshock 4 supports both analog sticks of which one uses
ABS_X/_Y and a touchpad. In a recent discussion with Dmitry about
some input-mt changes we proposed for disabling pointer emulation from
input_mt_sync_frame, Dmitry mentioned ABS_X/_Y should report the
same data as ABS_MT_POSITION_X/_Y. The current driver is mixing axes
for different subdevices. It was suggested to make the touchpad
its own sub-device.

This patch turns the touchpad into its own device. In addition
this patch also moves the button underneath the touchpad into
the new device. It felt like this button should be part of the
device. No known user space application (not even SDL2) seems to
be using it.

Signed-off-by: Roderick Colenbrander 
Signed-off-by: Jiri Kosina

HID: sony: Fix memory issue when connecting device using both Bluetooth and USB

2016-11-28T13:54:56+00:00

A previous patch moved most input initialization from sony_probe to
sony_input_configured to avoid some race conditions. The driver has some
special logic to prevent the device to get registered twice in case the
user connects it both over Bluetooth and USB. When this condition
happens sony_input_configured returns a failure, but sony_probe continues
as hid_hw_start doesn't fail. As was discussed on linux-input, it is
acceptable for this function to fail.

This patch adds a check for the HID_CLAIMED_INPUT flag within sony_probe
to determine whether initialization succeeded correctly. The flag is
not set by the HID layer when sony_input_configured fails.

Signed-off-by: Roderick Colenbrander 
Signed-off-by: Jiri Kosina

HID: sony: Update device ids

2016-10-10T08:43:26+00:00

Support additional DS4 model.

Signed-off-by: Roderick Colenbrander 
Reviewed-by: Benjamin Tissoires 
Signed-off-by: Jiri Kosina

HID: sony: Adjust value range for motion sensors

2016-10-10T08:43:26+00:00

The motion sensor values are 16-bit, so make the value range match.
It is hard to reach the upper values, but they can be reached. At
least the current accelerometer value of 8192 is very easy to pass.

It is still not nice that the motion sensors live in no man's land
in between ABS_MISC and ABS_MT_SLOT, but that's something for another
time, which the proposed ABS_ACCEL_*/ABS_GYRO_* were meant for.

Signed-off-by: Roderick Colenbrander 
Reviewed-by: Benjamin Tissoires 
Signed-off-by: Jiri Kosina