linux-toradex.git/drivers/hid/hid-alps.c, branch v4.9.123

Merge branches 'for-4.8/upstream-fixes', 'for-4.9/alps', 'for-4.9/hid-input', 'for-4.9/intel-ish', 'for-4.9/kye-uclogic-waltop-fixes', 'for-4.9/logitech', 'for-4.9/sony', 'for-4.9/upstream' and 'for-4.9/wacom' into for-linus

2016-10-07T07:59:48+00:00

HID: alps: fix multitouch cursor issue

2016-09-27T11:50:09+00:00

Issue reproduction procedure:

1. three or more fingers put on Touchpad.
2. release fingers from Touchpad.
3. move the cursor by one finger.
4. the cursor does not move.

Cause:

We do not notify multi fingers state correctly to input subsystem.  For
example, when three fingers release from Touchpad, fingers state is 3 -> 0. It
needs to notify first, second and third finger's releasing state.

Fix this by not breaking out on z axis and move x,y,z input handling
code to the correct place so that it's in fact per-finger.

[jkosina@suse.cz: reword changelog]
Signed-off-by: Masaki Ota 
Signed-off-by: Jiri Kosina

HID: alps: fix error return code in alps_input_configured()

2016-09-22T09:10:06+00:00

Fix to return a negative error code from the error handling
case instead of 0, as done elsewhere in this function.

While at it, remove redundant input_free_device(NULL) call.

[jkosina@suse.cz: ammend changelog]
Signed-off-by: Wei Yongjun 
Reviewed-by: Benjamin Tissoires 
Signed-off-by: Jiri Kosina

HID: alps: fix stick device not working after resume

2016-09-22T09:07:16+00:00

The stick device does not work after resume, add U1_SP_ABS_MODE flag can
make the device work after resume.

This has been discovered by pure guesswork, based on how the existing code uses
U1_TP_ABS_MODE flag on both initialization and resume.

I also tested the the patch on an ALPS touchpad without stick device, did not
notice any side effect on suspend/resume, so I made the U1_SP_ABS_MODE flag
mandatory.

[jkosina@suse.cz: made changelog more verbose]
Signed-off-by: Kai-Heng Feng 
Signed-off-by: Jiri Kosina

HID: alps: Fix memory leak

2016-09-19T12:42:22+00:00

A kfree(readbuf) call was missed if hid_hw_raw_request() fails, fix it.
Also avoid mixing direct return with goto in the error paths.

Signed-off-by: Axel Lin 
Reviewed-by: Benjamin Tissoires 
Signed-off-by: Jiri Kosina

HID: alps: a few cleanups

2016-06-23T06:56:36+00:00

Remove an unnecessary codes.
Change input_ivent() function to appropriate function.
Add the device ID of "HID_DEVICE_ID_ALPS_U1_DUAL".

[jkosina@suse.cz: removed unnecessary bitshifts of values passed
 input_report_key() as spotted by Dmitry]
Signed-off-by: Masaki Ota 
Reviewed-by: Dmitry Torokhov 
Signed-off-by: Jiri Kosina

HID: alps: pass correct sizes to hid_hw_raw_request()

2016-06-20T09:16:18+00:00

Calculating size of the report as sizeof(pointer) always passess the length
corresponding to the pointer type, not the underlying data report.

Reported-by: kbuild test robot 
Signed-off-by: Jiri Kosina

HID: alps: struct u1_dev *priv is internal to the driver

2016-06-17T21:15:04+00:00

... and therefore should be marked static.

Signed-off-by: Jiri Kosina

HID: add Alps I2C HID Touchpad-Stick support

2016-06-17T21:12:04+00:00

Add support Alps I2C HID Touchpad and Stick device.

Signed-off-by: Masaki Ota 
Signed-off-by: Jiri Kosina