USB: serial: fix generic disconnect implementation
authorJohan Hovold <jhovold@gmail.com>
Thu, 21 Mar 2013 11:36:41 +0000 (12:36 +0100)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 25 Mar 2013 20:48:26 +0000 (13:48 -0700)
There is no need for the generic disconnect callback to stop the read
and write urbs a second time as this has already been taken care of by
close (which is called from hangup as part of disconnect).

Signed-off-by: Johan Hovold <jhovold@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/serial/generic.c

index 131d6cb..a6d0ac6 100644 (file)
@@ -102,7 +102,7 @@ int usb_serial_generic_open(struct tty_struct *tty, struct usb_serial_port *port
 }
 EXPORT_SYMBOL_GPL(usb_serial_generic_open);
 
-static void generic_cleanup(struct usb_serial_port *port)
+void usb_serial_generic_close(struct usb_serial_port *port)
 {
        unsigned long flags;
        int i;
@@ -120,11 +120,6 @@ static void generic_cleanup(struct usb_serial_port *port)
                        usb_kill_urb(port->read_urbs[i]);
        }
 }
-
-void usb_serial_generic_close(struct usb_serial_port *port)
-{
-       generic_cleanup(port);
-}
 EXPORT_SYMBOL_GPL(usb_serial_generic_close);
 
 int usb_serial_generic_prepare_write_buffer(struct usb_serial_port *port,
@@ -507,11 +502,6 @@ EXPORT_SYMBOL_GPL(usb_serial_generic_resume);
 
 void usb_serial_generic_disconnect(struct usb_serial *serial)
 {
-       int i;
-
-       /* stop reads and writes on all ports */
-       for (i = 0; i < serial->num_ports; ++i)
-               generic_cleanup(serial->port[i]);
 }
 EXPORT_SYMBOL_GPL(usb_serial_generic_disconnect);