HID: sony: Fix SHANWAN PS3 GamePad rumble and led settings support
authorHongye Yuan <outmatch@gmail.com>
Wed, 9 Jan 2019 01:18:09 +0000 (09:18 +0800)
committerBenjamin Tissoires <benjamin.tissoires@redhat.com>
Wed, 9 Jan 2019 13:56:30 +0000 (14:56 +0100)
Rumble support on SHANWAN PS3 gamepad is not working when a user
program has asked it to. If a HID Output Reports is sent via Control
Channel then it will be discard by gamepad, thus rumble motor and led
settings in Output Report are ignored.

This patch therefore sends HID Output Report via Interrupt Channel to
SHANWAN gamepad instead of Control Channel, fixing rumble motor and
led settings.

Signed-off-by: Hongye Yuan <outmatch@gmail.com>
Reviewed-by: Bastien Nocera <hadess@hadess.net>
Signed-off-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
drivers/hid/hid-sony.c

index fc82f02..26fae90 100644 (file)
@@ -2100,9 +2100,14 @@ static void sixaxis_send_output_report(struct sony_sc *sc)
                }
        }
 
-       hid_hw_raw_request(sc->hdev, report->report_id, (u8 *)report,
-                       sizeof(struct sixaxis_output_report),
-                       HID_OUTPUT_REPORT, HID_REQ_SET_REPORT);
+       /* SHANWAN controllers require output reports via intr channel */
+       if (sc->quirks & SHANWAN_GAMEPAD)
+               hid_hw_output_report(sc->hdev, (u8 *)report,
+                               sizeof(struct sixaxis_output_report));
+       else
+               hid_hw_raw_request(sc->hdev, report->report_id, (u8 *)report,
+                               sizeof(struct sixaxis_output_report),
+                               HID_OUTPUT_REPORT, HID_REQ_SET_REPORT);
 }
 
 static void dualshock4_send_output_report(struct sony_sc *sc)