HID: usbhid: replace HID_REPORTED_IDLE with HID_SUSPENDED
authorAlan Stern <stern@rowland.harvard.edu>
Thu, 19 Jul 2012 20:08:45 +0000 (16:08 -0400)
committerJiri Kosina <jkosina@suse.cz>
Fri, 20 Jul 2012 09:24:24 +0000 (11:24 +0200)
This patch (as1595) improves the usbhid driver by using the
HID_SUSPENDED bitflag to indicate that the device is suspended rather
than using HID_REPORTED_IDLE, which the patch removes.

Since HID_SUSPENDED was not being used for anything, and since the
name "HID_REPORTED_IDLE" doesn't convey much meaning, the end result
is easier to read and understand.

Signed-off-by: Alan Stern <stern@rowland.harvard.edu>
CC: Oliver Neukum <oliver@neukum.org>
Signed-off-by: Jiri Kosina <jkosina@suse.cz>
drivers/hid/usbhid/hid-core.c
drivers/hid/usbhid/usbhid.h

index 0fa07d95202d18e8c58a5b68630359c42e91653c..271578b85d914423d0725c2ab05b3243fc1eb04c 100644 (file)
@@ -84,7 +84,7 @@ static int hid_start_in(struct hid_device *hid)
        spin_lock_irqsave(&usbhid->lock, flags);
        if (hid->open > 0 &&
                        !test_bit(HID_DISCONNECTED, &usbhid->iofl) &&
-                       !test_bit(HID_REPORTED_IDLE, &usbhid->iofl) &&
+                       !test_bit(HID_SUSPENDED, &usbhid->iofl) &&
                        !test_and_set_bit(HID_IN_RUNNING, &usbhid->iofl)) {
                rc = usb_submit_urb(usbhid->urbin, GFP_ATOMIC);
                if (rc != 0) {
@@ -222,7 +222,7 @@ static int usbhid_restart_out_queue(struct usbhid_device *usbhid)
                 * If still suspended, don't submit.  Submission will
                 * occur if/when resume drains the queue.
                 */
-               if (test_bit(HID_REPORTED_IDLE, &usbhid->iofl)) {
+               if (test_bit(HID_SUSPENDED, &usbhid->iofl)) {
                        usb_autopm_put_interface_no_suspend(usbhid->intf);
                        return r;
                }
@@ -260,7 +260,7 @@ static int usbhid_restart_ctrl_queue(struct usbhid_device *usbhid)
                 * If still suspended, don't submit.  Submission will
                 * occur if/when resume drains the queue.
                 */
-               if (test_bit(HID_REPORTED_IDLE, &usbhid->iofl)) {
+               if (test_bit(HID_SUSPENDED, &usbhid->iofl)) {
                        usb_autopm_put_interface_no_suspend(usbhid->intf);
                        return r;
                }
@@ -1475,7 +1475,7 @@ static int hid_suspend(struct usb_interface *intf, pm_message_t message)
                    && !test_bit(HID_KEYS_PRESSED, &usbhid->iofl)
                    && (!usbhid->ledcount || ignoreled))
                {
-                       set_bit(HID_REPORTED_IDLE, &usbhid->iofl);
+                       set_bit(HID_SUSPENDED, &usbhid->iofl);
                        spin_unlock_irq(&usbhid->lock);
                        if (hid->driver && hid->driver->suspend) {
                                status = hid->driver->suspend(hid, message);
@@ -1495,7 +1495,7 @@ static int hid_suspend(struct usb_interface *intf, pm_message_t message)
                                return status;
                }
                spin_lock_irq(&usbhid->lock);
-               set_bit(HID_REPORTED_IDLE, &usbhid->iofl);
+               set_bit(HID_SUSPENDED, &usbhid->iofl);
                spin_unlock_irq(&usbhid->lock);
                if (usbhid_wait_io(hid) < 0)
                        return -EIO;
@@ -1525,7 +1525,7 @@ static int hid_resume(struct usb_interface *intf)
        if (!test_bit(HID_STARTED, &usbhid->iofl))
                return 0;
 
-       clear_bit(HID_REPORTED_IDLE, &usbhid->iofl);
+       clear_bit(HID_SUSPENDED, &usbhid->iofl);
        usbhid_mark_busy(usbhid);
 
        if (test_bit(HID_CLEAR_HALT, &usbhid->iofl) ||
@@ -1552,7 +1552,7 @@ static int hid_reset_resume(struct usb_interface *intf)
        struct usbhid_device *usbhid = hid->driver_data;
        int status;
 
-       clear_bit(HID_REPORTED_IDLE, &usbhid->iofl);
+       clear_bit(HID_SUSPENDED, &usbhid->iofl);
        status = hid_post_reset(intf);
        if (status >= 0 && hid->driver && hid->driver->reset_resume) {
                int ret = hid->driver->reset_resume(hid);
index 1883d7b94870d7542827aa443dc3cc32d0de24da..bd87a61e53037dd63c63aff48a277ff172cd964c 100644 (file)
@@ -53,7 +53,6 @@ struct usb_interface *usbhid_find_interface(int minor);
 #define HID_CLEAR_HALT         6
 #define HID_DISCONNECTED       7
 #define HID_STARTED            8
-#define HID_REPORTED_IDLE      9
 #define HID_KEYS_PRESSED       10
 #define HID_NO_BANDWIDTH       11