USB: serial: ch341: fix modem-status handling
authorJohan Hovold <johan@kernel.org>
Fri, 6 Jan 2017 18:15:17 +0000 (19:15 +0100)
committerJohan Hovold <johan@kernel.org>
Mon, 16 Jan 2017 11:30:45 +0000 (12:30 +0100)
commita0467a967f347842b30739aae636c44980265265
tree43f5fa7489be7b63961e1fe03280f495caf87fd7
parent56e6d90797b4d8300e617a4f7b6ac216cf248041
USB: serial: ch341: fix modem-status handling

The modem-status register was read as part of device configuration at
port_probe and then again at open (and reset-resume). During open (and
reset-resume) the MSR was read before submitting the interrupt URB,
something which could lead to an MSR-change going unnoticed when it
races with open (reset-resume).

Fix this by dropping the redundant reconfiguration of the port at every
open, and only read the MSR after the interrupt URB has been submitted.

Fixes: 664d5df92e88 ("USB: usb-serial ch341: support for DTR/RTS/CTS")
Signed-off-by: Johan Hovold <johan@kernel.org>
drivers/usb/serial/ch341.c