| Message ID | l7xYjnA9EGfZe03FsrFhnH2aMq8qS8plWhHVvOtY_l4ShZ1NV6HA6hn9aI-jAzbLYUGFCIQCIKkx9z42Uoj4-AZDwBfRcAecYIn-0ZA5upE=@protonmail.com (mailing list archive) |
|---|---|
| State | Superseded |
| Delegated to: | Jiri Kosina |
| Headers | show |
| Series | [v3,1/4] HID: logitech: Add MX Mice over Bluetooth | expand |
On Sun, 2019-10-06 at 01:04 +0000, Mazin Rezk wrote: > This patch adds support for several MX mice over Bluetooth. The device IDs > have been copied from the libratbag device database and their features > have been based on their DJ device counterparts. > > Signed-off-by: Mazin Rezk <mnrzk@protonmail.com> > --- > drivers/hid/hid-logitech-hidpp.c | 18 ++++++++++++++++++ > 1 file changed, 18 insertions(+) > > diff --git a/drivers/hid/hid-logitech-hidpp.c b/drivers/hid/hid-logitech-hidpp.c > index 0179f7ed77e5..85fd0c17cc2f 100644 > --- a/drivers/hid/hid-logitech-hidpp.c > +++ b/drivers/hid/hid-logitech-hidpp.c > @@ -3773,6 +3773,24 @@ static const struct hid_device_id hidpp_devices[] = { > { /* MX5500 keyboard over Bluetooth */ > HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb30b), > .driver_data = HIDPP_QUIRK_HIDPP_CONSUMER_VENDOR_KEYS }, > + { /* MX Anywhere 2 mouse over Bluetooth */ > + HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb013), > + .driver_data = HIDPP_QUIRK_HI_RES_SCROLL_X2121 }, > + { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb018), > + .driver_data = HIDPP_QUIRK_HI_RES_SCROLL_X2121 }, > + { /* MX Anywhere 2S mouse over Bluetooth */ > + HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb01a), > + .driver_data = HIDPP_QUIRK_HI_RES_SCROLL_X2121 }, > + { /* MX Master mouse over Bluetooth */ > + HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb012), > + .driver_data = HIDPP_QUIRK_HI_RES_SCROLL_X2121 }, > + { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb017), > + .driver_data = HIDPP_QUIRK_HI_RES_SCROLL_X2121 }, > + { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb01e), > + .driver_data = HIDPP_QUIRK_HI_RES_SCROLL_X2121 }, > + { /* MX Master 2S mouse over Bluetooth */ > + HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb019), > + .driver_data = HIDPP_QUIRK_HI_RES_SCROLL_X2121 }, > {} > }; > > -- > 2.23.0 I think you should only add the mice you tested. We are not sure if this devices actually do work properly with the current stack. I will try to test some devices after Tuesday. Filipe Laíns
On Sunday, October 6, 2019 11:07 AM, Filipe Laíns <lains@archlinux.org> wrote: > On Sun, 2019-10-06 at 01:04 +0000, Mazin Rezk wrote: > > > This patch adds support for several MX mice over Bluetooth. The device IDs > > have been copied from the libratbag device database and their features > > have been based on their DJ device counterparts. > > > > Signed-off-by: Mazin Rezk mnrzk@protonmail.com > > > > ----------------------------------------------- > > > > drivers/hid/hid-logitech-hidpp.c | 18 ++++++++++++++++++ > > 1 file changed, 18 insertions(+) > > diff --git a/drivers/hid/hid-logitech-hidpp.c b/drivers/hid/hid-logitech-hidpp.c > > index 0179f7ed77e5..85fd0c17cc2f 100644 > > --- a/drivers/hid/hid-logitech-hidpp.c > > +++ b/drivers/hid/hid-logitech-hidpp.c > > @@ -3773,6 +3773,24 @@ static const struct hid_device_id hidpp_devices[] = { > > { /* MX5500 keyboard over Bluetooth */ > > HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb30b), > > .driver_data = HIDPP_QUIRK_HIDPP_CONSUMER_VENDOR_KEYS }, > > > > - { /* MX Anywhere 2 mouse over Bluetooth */ > > - HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb013), > > - .driver_data = HIDPP_QUIRK_HI_RES_SCROLL_X2121 }, > > - { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb018), > > - .driver_data = HIDPP_QUIRK_HI_RES_SCROLL_X2121 }, > > - { /* MX Anywhere 2S mouse over Bluetooth */ > > - HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb01a), > > - .driver_data = HIDPP_QUIRK_HI_RES_SCROLL_X2121 }, > > - { /* MX Master mouse over Bluetooth */ > > - HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb012), > > - .driver_data = HIDPP_QUIRK_HI_RES_SCROLL_X2121 }, > > - { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb017), > > - .driver_data = HIDPP_QUIRK_HI_RES_SCROLL_X2121 }, > > - { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb01e), > > - .driver_data = HIDPP_QUIRK_HI_RES_SCROLL_X2121 }, > > - { /* MX Master 2S mouse over Bluetooth */ > > - HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb019), > > - .driver_data = HIDPP_QUIRK_HI_RES_SCROLL_X2121 }, > > {} > > }; > > > > > > -- > > 2.23.0 > > I think you should only add the mice you tested. We are not sure if > this devices actually do work properly with the current stack. I will > try to test some devices after Tuesday. > > Filipe Laíns I have only really been able to test this patch on the MX Master (0xb01e). However, I suspect that many of the added devices work in a similar way. I could completely remove the devices I have not tested but I feel like it would be better if we somehow kept track of what devices have been tested.
diff --git a/drivers/hid/hid-logitech-hidpp.c b/drivers/hid/hid-logitech-hidpp.c index 0179f7ed77e5..85fd0c17cc2f 100644 --- a/drivers/hid/hid-logitech-hidpp.c +++ b/drivers/hid/hid-logitech-hidpp.c @@ -3773,6 +3773,24 @@ static const struct hid_device_id hidpp_devices[] = { { /* MX5500 keyboard over Bluetooth */ HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb30b), .driver_data = HIDPP_QUIRK_HIDPP_CONSUMER_VENDOR_KEYS }, + { /* MX Anywhere 2 mouse over Bluetooth */ + HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb013), + .driver_data = HIDPP_QUIRK_HI_RES_SCROLL_X2121 }, + { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb018), + .driver_data = HIDPP_QUIRK_HI_RES_SCROLL_X2121 }, + { /* MX Anywhere 2S mouse over Bluetooth */ + HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb01a), + .driver_data = HIDPP_QUIRK_HI_RES_SCROLL_X2121 }, + { /* MX Master mouse over Bluetooth */ + HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb012), + .driver_data = HIDPP_QUIRK_HI_RES_SCROLL_X2121 }, + { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb017), + .driver_data = HIDPP_QUIRK_HI_RES_SCROLL_X2121 }, + { HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb01e), + .driver_data = HIDPP_QUIRK_HI_RES_SCROLL_X2121 }, + { /* MX Master 2S mouse over Bluetooth */ + HID_BLUETOOTH_DEVICE(USB_VENDOR_ID_LOGITECH, 0xb019), + .driver_data = HIDPP_QUIRK_HI_RES_SCROLL_X2121 }, {} };
This patch adds support for several MX mice over Bluetooth. The device IDs have been copied from the libratbag device database and their features have been based on their DJ device counterparts. Signed-off-by: Mazin Rezk <mnrzk@protonmail.com> --- drivers/hid/hid-logitech-hidpp.c | 18 ++++++++++++++++++ 1 file changed, 18 insertions(+) -- 2.23.0