Re: [PATCH] HID: logitech-dj: add support for the static device in the Powerplay mat/receiver

From
Filipe Laíns <hidden>
Date
2020-02-08 19:08:07
Also in
lkml
Reply to
Filipe Laíns, 2020-01-15
On Wed, 2020-01-15 at 20:18 +0000, Filipe Laíns wrote:
quoted hunk ↗ jump to hunk
The Logitech G Powerplay has a lightspeed receiver with a static
HID++
device with ID 7 attached to it to. It is used to configure the led
on
the mat. For this reason I increased the max number of devices.

Signed-off-by: Filipe Laíns <redacted>
---
 drivers/hid/hid-logitech-dj.c | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)
diff --git a/drivers/hid/hid-logitech-dj.c b/drivers/hid/hid-
logitech-dj.c
index bb50d6e7745b..79294b873057 100644
--- a/drivers/hid/hid-logitech-dj.c
+++ b/drivers/hid/hid-logitech-dj.c
@@ -16,11 +16,11 @@
 #include <asm/unaligned.h>
 #include "hid-ids.h"
 
-#define DJ_MAX_PAIRED_DEVICES			6
+#define DJ_MAX_PAIRED_DEVICES			7
 #define DJ_MAX_NUMBER_NOTIFS			8
 #define DJ_RECEIVER_INDEX			0
 #define DJ_DEVICE_INDEX_MIN			1
-#define DJ_DEVICE_INDEX_MAX			6
+#define DJ_DEVICE_INDEX_MAX			7
 
 #define DJREPORT_SHORT_LENGTH			15
 #define DJREPORT_LONG_LENGTH			32
@@ -980,6 +980,11 @@ static void logi_hidpp_recv_queue_notif(struct
hid_device *hdev,
 		break;
 	}
 
+	/* custom receiver device (eg. powerplay) */
+	if (hidpp_report->device_index == 7) {
+		workitem.reports_supported |= HIDPP;
+	}
+
 	if (workitem.type == WORKITEM_TYPE_EMPTY) {
 		hid_warn(hdev,
 			 "unusable device of type %s (0x%02x) connected
on slot %d",
Benjamin, is there any updates on this? Were you able to test it?

Thank you,
Filipe Laíns

Attachments

Thread (5 messages) 5 messages, 2 authors, last reply 2020-03-10
Keyboard shortcuts
hback out one level
jnext message in thread / next series-diff position
kprevious message in thread / previous series-diff position
ldrill in
Escclose help / fold thread tree
?toggle this help