| Message ID | 20221220092207.428640-1-hadess@hadess.net |
|---|---|
| State | Accepted |
| Commit | cae253d6033da885e71c29c1591b22838a52de76 |
| Headers | show |
| Series | [1/3] Revert "HID: logitech-hidpp: add a module parameter to keep firmware gestures" | expand |
On Tue, 2022-12-20 at 10:22 +0100, Bastien Nocera wrote: > Now that we're in 2022, and the majority of desktop environments can > and > should support touchpad gestures through libinput, remove the legacy > module parameter that made it possible to use gestures implemented in > firmware. > > This will eventually allow simplifying the driver's initialisation > code. > > This reverts commit 9188dbaed68a4b23dc96eba165265c08caa7dc2a. Forgot the signed-off-by, resent as v2. > --- > drivers/hid/hid-logitech-hidpp.c | 10 ---------- > 1 file changed, 10 deletions(-) > > diff --git a/drivers/hid/hid-logitech-hidpp.c b/drivers/hid/hid- > logitech-hidpp.c > index 08ad19097e9e..7f9187201913 100644 > --- a/drivers/hid/hid-logitech-hidpp.c > +++ b/drivers/hid/hid-logitech-hidpp.c > @@ -32,11 +32,6 @@ MODULE_LICENSE("GPL"); > MODULE_AUTHOR("Benjamin Tissoires <benjamin.tissoires@gmail.com>"); > MODULE_AUTHOR("Nestor Lopez Casado <nlopezcasad@logitech.com>"); > > -static bool disable_raw_mode; > -module_param(disable_raw_mode, bool, 0644); > -MODULE_PARM_DESC(disable_raw_mode, > - "Disable Raw mode reporting for touchpads and keep firmware > gestures."); > - > static bool disable_tap_to_click; > module_param(disable_tap_to_click, bool, 0644); > MODULE_PARM_DESC(disable_tap_to_click, > @@ -4355,11 +4350,6 @@ static int hidpp_probe(struct hid_device > *hdev, const struct hid_device_id *id) > hidpp_application_equals(hdev, HID_GD_KEYBOARD)) > hidpp->quirks |= > HIDPP_QUIRK_HIDPP_CONSUMER_VENDOR_KEYS; > > - if (disable_raw_mode) { > - hidpp->quirks &= ~HIDPP_QUIRK_CLASS_WTP; > - hidpp->quirks &= ~HIDPP_QUIRK_NO_HIDINPUT; > - } > - > if (hidpp->quirks & HIDPP_QUIRK_CLASS_WTP) { > ret = wtp_allocate(hdev, id); > if (ret)
diff --git a/drivers/hid/hid-logitech-hidpp.c b/drivers/hid/hid-logitech-hidpp.c index 08ad19097e9e..7f9187201913 100644 --- a/drivers/hid/hid-logitech-hidpp.c +++ b/drivers/hid/hid-logitech-hidpp.c @@ -32,11 +32,6 @@ MODULE_LICENSE("GPL"); MODULE_AUTHOR("Benjamin Tissoires <benjamin.tissoires@gmail.com>"); MODULE_AUTHOR("Nestor Lopez Casado <nlopezcasad@logitech.com>"); -static bool disable_raw_mode; -module_param(disable_raw_mode, bool, 0644); -MODULE_PARM_DESC(disable_raw_mode, - "Disable Raw mode reporting for touchpads and keep firmware gestures."); - static bool disable_tap_to_click; module_param(disable_tap_to_click, bool, 0644); MODULE_PARM_DESC(disable_tap_to_click, @@ -4355,11 +4350,6 @@ static int hidpp_probe(struct hid_device *hdev, const struct hid_device_id *id) hidpp_application_equals(hdev, HID_GD_KEYBOARD)) hidpp->quirks |= HIDPP_QUIRK_HIDPP_CONSUMER_VENDOR_KEYS; - if (disable_raw_mode) { - hidpp->quirks &= ~HIDPP_QUIRK_CLASS_WTP; - hidpp->quirks &= ~HIDPP_QUIRK_NO_HIDINPUT; - } - if (hidpp->quirks & HIDPP_QUIRK_CLASS_WTP) { ret = wtp_allocate(hdev, id); if (ret)