Commit 1d25a056 authored by Johan Hovold's avatar Johan Hovold Committed by Greg Kroah-Hartman

USB: serial: ch341: fix resume after reset

commit ce5e2928 upstream.

Fix reset-resume handling which failed to resubmit the read and
interrupt URBs, thereby leaving a port that was open before suspend in a
broken state until closed and reopened.

Fixes: 1ded7ea4 ("USB: ch341 serial: fix port number changed after
resume")
Fixes: 2bfd1c96 ("USB: serial: ch341: remove reset_resume callback")
Signed-off-by: default avatarJohan Hovold <johan@kernel.org>
Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent a1a0612b
...@@ -554,14 +554,23 @@ static int ch341_tiocmget(struct tty_struct *tty) ...@@ -554,14 +554,23 @@ static int ch341_tiocmget(struct tty_struct *tty)
static int ch341_reset_resume(struct usb_serial *serial) static int ch341_reset_resume(struct usb_serial *serial)
{ {
struct ch341_private *priv; struct usb_serial_port *port = serial->port[0];
struct ch341_private *priv = usb_get_serial_port_data(port);
priv = usb_get_serial_port_data(serial->port[0]); int ret;
/* reconfigure ch341 serial port after bus-reset */ /* reconfigure ch341 serial port after bus-reset */
ch341_configure(serial->dev, priv); ch341_configure(serial->dev, priv);
return 0; if (test_bit(ASYNCB_INITIALIZED, &port->port.flags)) {
ret = usb_submit_urb(port->interrupt_in_urb, GFP_NOIO);
if (ret) {
dev_err(&port->dev, "failed to submit interrupt urb: %d\n",
ret);
return ret;
}
}
return usb_serial_generic_resume(serial);
} }
static struct usb_serial_driver ch341_device = { static struct usb_serial_driver ch341_device = {
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment