USB: serial: set drain delay at port probe
authorJohan Hovold <jhovold@gmail.com>
Wed, 26 Jun 2013 14:47:23 +0000 (16:47 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Tue, 23 Jul 2013 23:21:50 +0000 (16:21 -0700)
The port drain delay is constant and should be set at port probe rather
than open.

Signed-off-by: Johan Hovold <jhovold@gmail.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/usb/serial/cypress_m8.c
drivers/usb/serial/f81232.c
drivers/usb/serial/io_ti.c
drivers/usb/serial/oti6858.c
drivers/usb/serial/pl2303.c
drivers/usb/serial/spcp8x5.c
drivers/usb/serial/ti_usb_3410_5052.c

index e948dc0..558605d 100644 (file)
@@ -495,6 +495,8 @@ static int cypress_generic_port_probe(struct usb_serial_port *port)
        }
        usb_set_serial_port_data(port, priv);
 
+       port->port.drain_delay = 256;
+
        return 0;
 }
 
@@ -625,7 +627,7 @@ static int cypress_open(struct tty_struct *tty, struct usb_serial_port *port)
                                                        __func__, result);
                cypress_set_dead(port);
        }
-       port->port.drain_delay = 256;
+
        return result;
 } /* cypress_open */
 
index 75e85cb..639a18f 100644 (file)
@@ -207,7 +207,6 @@ static int f81232_open(struct tty_struct *tty, struct usb_serial_port *port)
                return result;
        }
 
-       port->port.drain_delay = 256;
        return 0;
 }
 
@@ -322,6 +321,8 @@ static int f81232_port_probe(struct usb_serial_port *port)
 
        usb_set_serial_port_data(port, priv);
 
+       port->port.drain_delay = 256;
+
        return 0;
 }
 
index 60054e7..d43adc0 100644 (file)
@@ -1872,8 +1872,6 @@ static int edge_open(struct tty_struct *tty, struct usb_serial_port *port)
 
        ++edge_serial->num_ports_open;
 
-       port->port.drain_delay = 1;
-
        goto release_es_lock;
 
 unlink_int_urb:
@@ -2473,6 +2471,7 @@ static int edge_port_probe(struct usb_serial_port *port)
        }
 
        port->port.closing_wait = msecs_to_jiffies(closing_wait * 10);
+       port->port.drain_delay = 1;
 
        return 0;
 }
index cf509c8..baa2ea5 100644 (file)
@@ -343,6 +343,8 @@ static int oti6858_port_probe(struct usb_serial_port *port)
 
        usb_set_serial_port_data(port, priv);
 
+       port->port.drain_delay = 256;   /* FIXME: check the FIFO length */
+
        return 0;
 }
 
@@ -558,7 +560,7 @@ static int oti6858_open(struct tty_struct *tty, struct usb_serial_port *port)
        /* setup termios */
        if (tty)
                oti6858_set_termios(tty, port, &tmp_termios);
-       port->port.drain_delay = 256;   /* FIXME: check the FIFO length */
+
        return 0;
 }
 
index cb6bbed..ed9edb2 100644 (file)
@@ -235,6 +235,8 @@ static int pl2303_port_probe(struct usb_serial_port *port)
 
        usb_set_serial_port_data(port, priv);
 
+       port->port.drain_delay = 256;
+
        return 0;
 }
 
@@ -521,7 +523,6 @@ static int pl2303_open(struct tty_struct *tty, struct usb_serial_port *port)
                return result;
        }
 
-       port->port.drain_delay = 256;
        return 0;
 }
 
index ddf6c47..4abac28 100644 (file)
@@ -169,6 +169,8 @@ static int spcp8x5_port_probe(struct usb_serial_port *port)
 
        usb_set_serial_port_data(port, priv);
 
+       port->port.drain_delay = 256;
+
        return 0;
 }
 
@@ -411,8 +413,6 @@ static int spcp8x5_open(struct tty_struct *tty, struct usb_serial_port *port)
        if (tty)
                spcp8x5_set_termios(tty, port, NULL);
 
-       port->port.drain_delay = 256;
-
        return usb_serial_generic_open(tty, port);
 }
 
index 7182bb7..18db7b3 100644 (file)
@@ -441,6 +441,8 @@ static int ti_port_probe(struct usb_serial_port *port)
 
        usb_set_serial_port_data(port, tport);
 
+       port->port.drain_delay = 3;
+
        return 0;
 }
 
@@ -582,8 +584,6 @@ static int ti_open(struct tty_struct *tty, struct usb_serial_port *port)
        tport->tp_is_open = 1;
        ++tdev->td_open_port_count;
 
-       port->port.drain_delay = 3;
-
        goto release_lock;
 
 unlink_int_urb: