Bluetooth: btusb: also be quiet when suspending
authorPaul Bolle <pebolle@tiscali.nl>
Sun, 9 Oct 2011 10:12:16 +0000 (12:12 +0200)
committerGustavo F. Padovan <padovan@profusion.mobi>
Mon, 10 Oct 2011 21:02:31 +0000 (18:02 -0300)
usb_submit_urb() returns -ENODEV when a usb device is disconnected. In
commit 4935f1c164ac528dff3538f97953b385ba500710 ("Bluetooth: btusb: be
quiet on device disconnect") I stopped treating that return as an error
in the three btusb_*_complete() functions.

It turns out btusb_send_frame() generates a similar error if the system
is suspended while the bluetooth usb device is enabled. The sensible
thing to do here seems to be to treat -ENODEV (and -EPERM) just like the
btusb_*_complete() functions now do.

Signed-off-by: Paul Bolle <pebolle@tiscali.nl>
Signed-off-by: Gustavo F. Padovan <padovan@profusion.mobi>
drivers/bluetooth/btusb.c

index 675246a6f7ef2c38ff3fbbf863c9bb62a908dd64..18fde3be530fb64b3865784485faeb91a7aab7bb 100644 (file)
@@ -767,7 +767,9 @@ skip_waking:
 
        err = usb_submit_urb(urb, GFP_ATOMIC);
        if (err < 0) {
-               BT_ERR("%s urb %p submission failed", hdev->name, urb);
+               if (err != -EPERM && err != -ENODEV)
+                       BT_ERR("%s urb %p submission failed (%d)",
+                                               hdev->name, urb, -err);
                kfree(urb->setup_packet);
                usb_unanchor_urb(urb);
        } else {