USB: serial: ti_usb_3410_5052: reduce drain delay to one char
authorJohan Hovold <johan@kernel.org>
Mon, 12 Apr 2021 09:38:14 +0000 (11:38 +0200)
committerJohan Hovold <johan@kernel.org>
Tue, 13 Apr 2021 16:19:33 +0000 (18:19 +0200)
The three-character drain delay was added by commit f1175daa5312 ("USB:
ti_usb_3410_5052: kill custom closing_wait") when removing the custom
closing-wait implementation, which used a fixed 20 ms poll period and
drain delay.

This was likely a bit too conservative as a one-character timeout (e.g.
33 ms at 300 bps) should be enough to compensate for the lack of a
transmitter empty bit in the TUSB5052 line-status register.

Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Johan Hovold <johan@kernel.org>
drivers/usb/serial/ti_usb_3410_5052.c

index 03839289d6c0932f46ab14fe510655601b3b4906..8ed64115987f7143110db949f5907300fedaf502 100644 (file)
@@ -610,7 +610,11 @@ static int ti_port_probe(struct usb_serial_port *port)
 
        usb_set_serial_port_data(port, tport);
 
-       port->port.drain_delay = 3;
+       /*
+        * The TUSB5052 LSR does not tell when the transmitter shift register
+        * has emptied so add a one-character drain delay.
+        */
+       port->port.drain_delay = 1;
 
        return 0;
 }