USB: sierra: error handling in runtime PM
authorOliver Neukum <oliver@neukum.org>
Fri, 11 Feb 2011 11:56:40 +0000 (12:56 +0100)
committerGreg Kroah-Hartman <gregkh@suse.de>
Thu, 17 Feb 2011 18:51:13 +0000 (10:51 -0800)
resumption of devices can fail. Errors must be handled.

Signed-off-by: Oliver Neukum <oneukum@suse.de>
Signed-off-by: Greg Kroah-Hartman <gregkh@suse.de>
drivers/usb/serial/sierra.c

index 7481ff8a49e4aaf3f025728e40201c2c4a332ff2..2436796e117b78ac49f23a0eac84ba6303b6192d 100644 (file)
@@ -373,7 +373,10 @@ static int sierra_send_setup(struct usb_serial_port *port)
        if (!do_send)
                return 0;
 
-       usb_autopm_get_interface(serial->interface);
+       retval = usb_autopm_get_interface(serial->interface);
+       if (retval < 0)
+               return retval;
+
        retval = usb_control_msg(serial->dev, usb_rcvctrlpipe(serial->dev, 0),
                0x22, 0x21, val, interface, NULL, 0, USB_CTRL_SET_TIMEOUT);
        usb_autopm_put_interface(serial->interface);
@@ -808,8 +811,12 @@ static void sierra_close(struct usb_serial_port *port)
                mutex_lock(&serial->disc_mutex);
                if (!serial->disconnected) {
                        serial->interface->needs_remote_wakeup = 0;
-                       usb_autopm_get_interface(serial->interface);
-                       sierra_send_setup(port);
+                       /* odd error handling due to pm counters */
+                       if (!usb_autopm_get_interface(serial->interface))
+                               sierra_send_setup(port);
+                       else
+                               usb_autopm_get_interface_no_resume(serial->interface);
+                               
                }
                mutex_unlock(&serial->disc_mutex);
                spin_lock_irq(&intfdata->susp_lock);
@@ -862,7 +869,8 @@ static int sierra_open(struct tty_struct *tty, struct usb_serial_port *port)
                /* get rid of everything as in close */
                sierra_close(port);
                /* restore balance for autopm */
-               usb_autopm_put_interface(serial->interface);
+               if (!serial->disconnected)
+                       usb_autopm_put_interface(serial->interface);
                return err;
        }
        sierra_send_setup(port);