On Mon, Aug 12, 2019 at 6:26 PM Hans de Goede [off-list ref] wrote:

Hi,

On 12-08-19 18:08, Benjamin Tissoires wrote:

quoted

Both the G700 and the G700s are sharing the same receiver.
Include support for this receiver in hid-logitech-dj so that userspace
can differentiate both.

Signed-off-by: Benjamin Tissoires <redacted>

Nice, looks good to me:

Reviewed-by: Hans de Goede <redacted>

thanks

Applied to for-5.4/logitech

Cheers,
Benjamin

Regards,

Hans

quoted

---
  drivers/hid/hid-logitech-dj.c | 5 +++++
  1 file changed, 5 insertions(+)

diff --git a/drivers/hid/hid-logitech-dj.c b/drivers/hid/hid-logitech-dj.c
index c547cba05fbb..d6250b0cb9f8 100644
--- a/drivers/hid/hid-logitech-dj.c
+++ b/drivers/hid/hid-logitech-dj.c

@@ -959,6 +959,7 @@ static void logi_hidpp_recv_queue_notif(struct hid_device *hdev,
              break;
      case 0x07:
              device_type = "eQUAD step 4 Gaming";
+             logi_hidpp_dev_conn_notif_equad(hdev, hidpp_report, &workitem);
              break;
      case 0x08:
              device_type = "eQUAD step 4 for gamepads";

@@ -1832,6 +1833,10 @@ static const struct hid_device_id logi_dj_receivers[] = {
        HID_USB_DEVICE(USB_VENDOR_ID_LOGITECH,
                       USB_DEVICE_ID_LOGITECH_NANO_RECEIVER_2),
       .driver_data = recvr_type_hidpp},
+     { /* Logitech G700(s) receiver (0xc531) */
+       HID_USB_DEVICE(USB_VENDOR_ID_LOGITECH,
+             0xc531),
+      .driver_data = recvr_type_gaming_hidpp},
      { /* Logitech lightspeed receiver (0xc539) */
        HID_USB_DEVICE(USB_VENDOR_ID_LOGITECH,
              USB_DEVICE_ID_LOGITECH_NANO_RECEIVER_LIGHTSPEED),