customhidsony - patch for third-party ps4 controllers

[A Former User]

Hello to all,

After the problems that the ps4 controller gave me:
https://retropie.org.uk/forum/topic/27359/ps4-controller-not-detected-in-retropie-4-6-on-pi-4b/37

I have created the diff file with the patch for third party ps4 controllers to be include in Retropie into customhidsony.

The patch has to be applied after the file: 0001-hidsony-gasiafix.diff

I name this patch as 0002-hidsony-thirdpartyps4controllerfix.diff

https://www.mail-archive.com/linux-kernel@vger.kernel.org/msg2440145.html
[PATCH] HID: sony: Support for DS4 clones that do not implement feature report 0x81

There are clones of DualShock 4 that are very similar to the originals,
except of 1) they do not support HID feature report 0x81 and 2) they do
not have any USB Audio interfaces despite they physically have audio
jack.


Such controllers are working fine with Linux when connected via
Bluetooth, but not when connected via USB. Here is how failed USB
connection attempt looks in log:

        usb 1-5: New USB device found, idVendor=054c, idProduct=05c4, 
bcdDevice= 1.00
        usb 1-5: New USB device strings: Mfr=1, Product=2, SerialNumber=0
        usb 1-5: Product: Wireless Controller
        usb 1-5: Manufacturer: Sony Computer Entertainment
        sony 0003:054C:05C4.0007: failed to retrieve feature report 0x81 with 
the DualShock 4 MAC address
        sony 0003:054C:05C4.0007: hidraw6: USB HID v81.11 Gamepad [Sony 
Computer Entertainment Wireless Controller] on usb-0000:00:14.0-5/input0
        sony 0003:054C:05C4.0007: failed to claim input

This patch adds support of using feature report 0x12 as a fallback for
Bluetooth MAC address retrieval. Feature report 0x12 also seems to be
used by DS4Windows[1] for all DS4 controllers.

[1] 
https://github.com/Ryochan7/DS4Windows/blob/1b74a4440089f38a24ee2c2483c1d733a0692b8f/DS4Windows/HidLibrary/HidDevice.cs#L479

Signed-off-by: Ivan Mironov <mironov.i...@gmail.com>

--- a/drivers/hid/hid-sony.c
+++ b/drivers/hid/hid-sony.c
@@ -477,6 +477,7 @@
 
 #define DS4_FEATURE_REPORT_0x02_SIZE 37
 #define DS4_FEATURE_REPORT_0x05_SIZE 41
+#define DS4_FEATURE_REPORT_0x12_SIZE 16
 #define DS4_FEATURE_REPORT_0x81_SIZE 7
 #define DS4_FEATURE_REPORT_0xA3_SIZE 49
 #define DS4_INPUT_REPORT_0x11_SIZE 78
@@ -2465,6 +2466,54 @@
 	return 0;
 }
 
+static int sony_get_usb_ds4_devaddr(struct sony_sc *sc)
+{
+       u8 *buf = NULL;
+       int ret;
+
+       buf = kmalloc(max(DS4_FEATURE_REPORT_0x12_SIZE, DS4_FEATURE_REPORT_0x81_SIZE), GFP_KERNEL);
+       if (!buf)
+               return -ENOMEM;
+
+       /*
+        * The MAC address of a DS4 controller connected via USB can be
+        * retrieved with feature report 0x81. The address begins at
+        * offset 1.
+        */
+       ret = hid_hw_raw_request(sc->hdev, 0x81, buf,
+                       DS4_FEATURE_REPORT_0x81_SIZE, HID_FEATURE_REPORT,
+                       HID_REQ_GET_REPORT);
+       if (ret == DS4_FEATURE_REPORT_0x81_SIZE) {
+               memcpy(sc->mac_address, &buf[1], sizeof(sc->mac_address));
+               goto out_free;
+       }
+       dbg_hid("%s: hid_hw_raw_request(..., 0x81, ...) returned %d\n", __func__, ret);
+
+       /*
+        * Some variants do not implement feature report 0x81 at all.
+        * Fortunately, feature report 0x12 also contains the MAC address of
+        * a controller.
+        */
+       ret = hid_hw_raw_request(sc->hdev, 0x12, buf,
+                       DS4_FEATURE_REPORT_0x12_SIZE, HID_FEATURE_REPORT,
+                       HID_REQ_GET_REPORT);
+       if (ret == DS4_FEATURE_REPORT_0x12_SIZE) {
+               memcpy(sc->mac_address, &buf[1], sizeof(sc->mac_address));
+               goto out_free;
+       }
+       dbg_hid("%s: hid_hw_raw_request(..., 0x12, ...) returned %d\n", __func__, ret);
+
+       hid_err(sc->hdev, "failed to retrieve feature reports 0x81 and 0x12 with the DualShock 4 MAC address\n");
+       ret = ret < 0 ? ret : -EINVAL;
+
+out_free:
+
+       kfree(buf);
+
+       return ret;
+}
+
+
 static int sony_check_add(struct sony_sc *sc)
 {
 	u8 *buf = NULL;
@@ -2485,26 +2534,10 @@
 			return 0;
 		}
 	} else if (sc->quirks & (DUALSHOCK4_CONTROLLER_USB | DUALSHOCK4_DONGLE)) {
-		buf = kmalloc(DS4_FEATURE_REPORT_0x81_SIZE, GFP_KERNEL);
-		if (!buf)
-			return -ENOMEM;
-
-		/*
-		 * The MAC address of a DS4 controller connected via USB can be
-		 * retrieved with feature report 0x81. The address begins at
-		 * offset 1.
-		 */
-		ret = hid_hw_raw_request(sc->hdev, 0x81, buf,
-				DS4_FEATURE_REPORT_0x81_SIZE, HID_FEATURE_REPORT,
-				HID_REQ_GET_REPORT);
-
-		if (ret != DS4_FEATURE_REPORT_0x81_SIZE) {
-			hid_err(sc->hdev, "failed to retrieve feature report 0x81 with the DualShock 4 MAC address\n");
-			ret = ret < 0 ? ret : -EINVAL;
-			goto out_free;
-		}
+		ret = sony_get_usb_ds4_devaddr(sc);
+		if (ret < 0)
+			return ret;
 
-		memcpy(sc->mac_address, &buf[1], sizeof(sc->mac_address));
 
 		snprintf(sc->hdev->uniq, sizeof(sc->hdev->uniq),
 			 "%pMR", sc->mac_address);
@@ -2542,6 +2575,7 @@
 		return 0;
 	}
 
+	dbg_hid("%s: retrieved MAC address: %s\n", __func__, sc->hdev->uniq);
 	ret = sony_check_add_dev_list(sc);
 
 out_free:

I would like someone can include this file into
RetroPie-Setup/scriptmodules/supplementary/customhidsony to be part of Retropie.

Regards

[A Former User]

Could someone publish this patch on
RetroPie-Setup/scriptmodules/supplementary/customhidsony ?