* [PATCH] HID: sony: Support for DS4 clones that do not implement feature report 0x81 @ 2021-01-13 17:34 Ivan Mironov 2021-02-13 4:02 ` Ivan Mironov 2022-09-08 8:31 ` Bastien Nocera 0 siblings, 2 replies; 5+ messages in thread From: Ivan Mironov @ 2021-01-13 17:34 UTC (permalink / raw) To: Jiri Kosina; +Cc: Benjamin Tissoires, linux-input, linux-kernel, Ivan Mironov 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.ivan@gmail.com> --- drivers/hid/hid-sony.c | 72 ++++++++++++++++++++++++++++++------------ 1 file changed, 52 insertions(+), 20 deletions(-) diff --git a/drivers/hid/hid-sony.c b/drivers/hid/hid-sony.c index e3a557dc9ffd..97df12180e45 100644 --- a/drivers/hid/hid-sony.c +++ b/drivers/hid/hid-sony.c @@ -491,6 +491,7 @@ struct motion_output_report_02 { #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 @@ -2593,6 +2594,53 @@ static int sony_get_bt_devaddr(struct sony_sc *sc) 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; @@ -2613,26 +2661,9 @@ static int sony_check_add(struct sony_sc *sc) 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; - } - - memcpy(sc->mac_address, &buf[1], sizeof(sc->mac_address)); + ret = sony_get_usb_ds4_devaddr(sc); + if (ret < 0) + return ret; snprintf(sc->hdev->uniq, sizeof(sc->hdev->uniq), "%pMR", sc->mac_address); @@ -2670,6 +2701,7 @@ static int sony_check_add(struct sony_sc *sc) return 0; } + dbg_hid("%s: retrieved MAC address: %s\n", __func__, sc->hdev->uniq); ret = sony_check_add_dev_list(sc); out_free: -- 2.29.2 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] HID: sony: Support for DS4 clones that do not implement feature report 0x81 2021-01-13 17:34 [PATCH] HID: sony: Support for DS4 clones that do not implement feature report 0x81 Ivan Mironov @ 2021-02-13 4:02 ` Ivan Mironov 2022-12-22 22:31 ` Laurent Pinchart 2022-09-08 8:31 ` Bastien Nocera 1 sibling, 1 reply; 5+ messages in thread From: Ivan Mironov @ 2021-02-13 4:02 UTC (permalink / raw) To: Jiri Kosina; +Cc: Benjamin Tissoires, linux-input, linux-kernel Ignore this patch, I am working on a better one. On Wed, 2021-01-13 at 22:34 +0500, Ivan Mironov wrote: > 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.ivan@gmail.com> > --- > drivers/hid/hid-sony.c | 72 ++++++++++++++++++++++++++++++------------ > 1 file changed, 52 insertions(+), 20 deletions(-) > > diff --git a/drivers/hid/hid-sony.c b/drivers/hid/hid-sony.c > index e3a557dc9ffd..97df12180e45 100644 > --- a/drivers/hid/hid-sony.c > +++ b/drivers/hid/hid-sony.c > @@ -491,6 +491,7 @@ struct motion_output_report_02 { > > > #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 > @@ -2593,6 +2594,53 @@ static int sony_get_bt_devaddr(struct sony_sc *sc) > 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; > @@ -2613,26 +2661,9 @@ static int sony_check_add(struct sony_sc *sc) > 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; > - } > - > - memcpy(sc->mac_address, &buf[1], sizeof(sc->mac_address)); > + ret = sony_get_usb_ds4_devaddr(sc); > + if (ret < 0) > + return ret; > > > snprintf(sc->hdev->uniq, sizeof(sc->hdev->uniq), > "%pMR", sc->mac_address); > @@ -2670,6 +2701,7 @@ static int sony_check_add(struct sony_sc *sc) > return 0; > } > > > + dbg_hid("%s: retrieved MAC address: %s\n", __func__, sc->hdev->uniq); > ret = sony_check_add_dev_list(sc); > > > out_free: ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] HID: sony: Support for DS4 clones that do not implement feature report 0x81 2021-02-13 4:02 ` Ivan Mironov @ 2022-12-22 22:31 ` Laurent Pinchart [not found] ` <CAEc3jaCTu+WYP6zikKXX2igTvq5GUKX2Y1fpF0q5ZBaNMytgSA@mail.gmail.com> 0 siblings, 1 reply; 5+ messages in thread From: Laurent Pinchart @ 2022-12-22 22:31 UTC (permalink / raw) To: Ivan Mironov; +Cc: Jiri Kosina, Benjamin Tissoires, linux-input, linux-kernel Hi Ivan, On Sat, Feb 13, 2021 at 09:02:45AM +0500, Ivan Mironov wrote: > Ignore this patch, I am working on a better one. Have you managed to write a better patch ? I've successfuly tested this one with a PS4 controller clone. > On Wed, 2021-01-13 at 22:34 +0500, Ivan Mironov wrote: > > 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.ivan@gmail.com> > > --- > > drivers/hid/hid-sony.c | 72 ++++++++++++++++++++++++++++++------------ > > 1 file changed, 52 insertions(+), 20 deletions(-) > > > > diff --git a/drivers/hid/hid-sony.c b/drivers/hid/hid-sony.c > > index e3a557dc9ffd..97df12180e45 100644 > > --- a/drivers/hid/hid-sony.c > > +++ b/drivers/hid/hid-sony.c > > @@ -491,6 +491,7 @@ struct motion_output_report_02 { > > > > > > #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 > > @@ -2593,6 +2594,53 @@ static int sony_get_bt_devaddr(struct sony_sc *sc) > > 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; > > @@ -2613,26 +2661,9 @@ static int sony_check_add(struct sony_sc *sc) > > 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; > > - } > > - > > - memcpy(sc->mac_address, &buf[1], sizeof(sc->mac_address)); > > + ret = sony_get_usb_ds4_devaddr(sc); > > + if (ret < 0) > > + return ret; > > > > > > snprintf(sc->hdev->uniq, sizeof(sc->hdev->uniq), > > "%pMR", sc->mac_address); > > @@ -2670,6 +2701,7 @@ static int sony_check_add(struct sony_sc *sc) > > return 0; > > } > > > > > > + dbg_hid("%s: retrieved MAC address: %s\n", __func__, sc->hdev->uniq); > > ret = sony_check_add_dev_list(sc); > > > > > > out_free: -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 5+ messages in thread
[parent not found: <CAEc3jaCTu+WYP6zikKXX2igTvq5GUKX2Y1fpF0q5ZBaNMytgSA@mail.gmail.com>]
* Re: [PATCH] HID: sony: Support for DS4 clones that do not implement feature report 0x81 [not found] ` <CAEc3jaCTu+WYP6zikKXX2igTvq5GUKX2Y1fpF0q5ZBaNMytgSA@mail.gmail.com> @ 2022-12-23 23:50 ` Laurent Pinchart 0 siblings, 0 replies; 5+ messages in thread From: Laurent Pinchart @ 2022-12-23 23:50 UTC (permalink / raw) To: Roderick Colenbrander Cc: Ivan Mironov, Jiri Kosina, Benjamin Tissoires, linux-input, lkml Hi Roderick, On Fri, Dec 23, 2022 at 03:41:09PM -0800, Roderick Colenbrander wrote: > Try the new hid-playstation driver for Linux 6.2 (download from git or wait > until rc1). We added ds4 support to it and the Mac address handling code is > different. It may work for your device. I intend to remove ds4 from > hid-sony later. I'll give that a try, but it will take a while. I don't have access to the hardware at the moment, I was testing this at a friend's place, after their son was disappointed that the game controller he had just bought didn't work with RetroPie. When the boy will be a bit older I'll have to teach him to compile his own kernel ;-) > On Thu, Dec 22, 2022, 2:43 PM Laurent Pinchart wrote: > > On Sat, Feb 13, 2021 at 09:02:45AM +0500, Ivan Mironov wrote: > > > Ignore this patch, I am working on a better one. > > > > Have you managed to write a better patch ? I've successfuly tested this > > one with a PS4 controller clone. > > > > > On Wed, 2021-01-13 at 22:34 +0500, Ivan Mironov wrote: > > > > 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.ivan@gmail.com> > > > > --- > > > > drivers/hid/hid-sony.c | 72 ++++++++++++++++++++++++++++++------------ > > > > 1 file changed, 52 insertions(+), 20 deletions(-) > > > > > > > > diff --git a/drivers/hid/hid-sony.c b/drivers/hid/hid-sony.c > > > > index e3a557dc9ffd..97df12180e45 100644 > > > > --- a/drivers/hid/hid-sony.c > > > > +++ b/drivers/hid/hid-sony.c > > > > @@ -491,6 +491,7 @@ struct motion_output_report_02 { > > > > > > > > > > > > #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 > > > > @@ -2593,6 +2594,53 @@ static int sony_get_bt_devaddr(struct sony_sc *sc) > > > > 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; > > > > @@ -2613,26 +2661,9 @@ static int sony_check_add(struct sony_sc *sc) > > > > 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; > > > > - } > > > > - > > > > - memcpy(sc->mac_address, &buf[1], sizeof(sc->mac_address)); > > > > + ret = sony_get_usb_ds4_devaddr(sc); > > > > + if (ret < 0) > > > > + return ret; > > > > > > > > > > > > snprintf(sc->hdev->uniq, sizeof(sc->hdev->uniq), > > > > "%pMR", sc->mac_address); > > > > @@ -2670,6 +2701,7 @@ static int sony_check_add(struct sony_sc *sc) > > > > return 0; > > > > } > > > > > > > > > > > > + dbg_hid("%s: retrieved MAC address: %s\n", __func__, sc->hdev->uniq); > > > > ret = sony_check_add_dev_list(sc); > > > > > > > > > > > > out_free: -- Regards, Laurent Pinchart ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] HID: sony: Support for DS4 clones that do not implement feature report 0x81 2021-01-13 17:34 [PATCH] HID: sony: Support for DS4 clones that do not implement feature report 0x81 Ivan Mironov 2021-02-13 4:02 ` Ivan Mironov @ 2022-09-08 8:31 ` Bastien Nocera 1 sibling, 0 replies; 5+ messages in thread From: Bastien Nocera @ 2022-09-08 8:31 UTC (permalink / raw) To: Ivan Mironov, Jiri Kosina; +Cc: Benjamin Tissoires, linux-input, linux-kernel On Wed, 2021-01-13 at 22:34 +0500, Ivan Mironov wrote: > 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. Do you have the brand/catalogue name for the device on which you tested this? I found a "YCCTEAM" joypad, but it seems that the audio interface is supposed to work (maybe that's only on wireless?). Do you think you could also look into a patch for BlueZ's cable pairing plugin? It uses 0x81 too, even though it does have some fallback code. https://git.kernel.org/pub/scm/bluetooth/bluez.git/tree/plugins/sixaxis.c#n111 ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2022-12-23 23:50 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-01-13 17:34 [PATCH] HID: sony: Support for DS4 clones that do not implement feature report 0x81 Ivan Mironov
2021-02-13 4:02 ` Ivan Mironov
2022-12-22 22:31 ` Laurent Pinchart
[not found] ` <CAEc3jaCTu+WYP6zikKXX2igTvq5GUKX2Y1fpF0q5ZBaNMytgSA@mail.gmail.com>
2022-12-23 23:50 ` Laurent Pinchart
2022-09-08 8:31 ` Bastien Nocera
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).