HID: multitouch: do not set HID_QUIRK_NO_INIT_REPORTS
authorBenjamin Tissoires <benjamin.tissoires@redhat.com>
Tue, 20 Mar 2018 11:04:50 +0000 (12:04 +0100)
committerJiri Kosina <jkosina@suse.cz>
Fri, 23 Mar 2018 14:46:09 +0000 (15:46 +0100)
It is set by default now, so there is no point setting it in the driver

Signed-off-by: Benjamin Tissoires <benjamin.tissoires@redhat.com>
Acked-by: Peter Hutterer <peter.hutterer@who-t.net>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
drivers/hid/hid-multitouch.c

index c4d89830cd1f27c6f3da2934be6f1f9707c5e2e6..f05d372ba71e62e5be8de29e59017bfbb338e28c 100644 (file)
@@ -380,7 +380,6 @@ static const struct attribute_group mt_attribute_group = {
 
 static void mt_get_feature(struct hid_device *hdev, struct hid_report *report)
 {
-       struct mt_device *td = hid_get_drvdata(hdev);
        int ret, size = hid_report_len(report);
        u8 *buf;
 
@@ -388,7 +387,7 @@ static void mt_get_feature(struct hid_device *hdev, struct hid_report *report)
         * Do not fetch the feature report if the device has been explicitly
         * marked as non-capable.
         */
-       if (td->initial_quirks & HID_QUIRK_NO_INIT_REPORTS)
+       if (hdev->quirks & HID_QUIRK_NO_INIT_REPORTS)
                return;
 
        buf = hid_alloc_report_buf(report, GFP_KERNEL);
@@ -1470,21 +1469,6 @@ static int mt_probe(struct hid_device *hdev, const struct hid_device_id *id)
         */
        hdev->quirks |= HID_QUIRK_MULTI_INPUT;
 
-       /*
-        * Some multitouch screens do not like to be polled for input
-        * reports. Fortunately, the Win8 spec says that all touches
-        * should be sent during each report, making the initialization
-        * of input reports unnecessary. For Win7 devices, well, let's hope
-        * they will still be happy (this is only be a problem if a touch
-        * was already there while probing the device).
-        *
-        * In addition some touchpads do not behave well if we read
-        * all feature reports from them. Instead we prevent
-        * initial report fetching and then selectively fetch each
-        * report we are interested in.
-        */
-       hdev->quirks |= HID_QUIRK_NO_INIT_REPORTS;
-
        timer_setup(&td->release_timer, mt_expired_timeout, 0);
 
        ret = hid_parse(hdev);