USB: serial: ch341: fix receiver regression
authorJohan Hovold <johan@kernel.org>
Thu, 6 Feb 2020 11:18:19 +0000 (12:18 +0100)
committerJohan Hovold <johan@kernel.org>
Mon, 10 Feb 2020 09:10:35 +0000 (10:10 +0100)
commit7c3d02285ad558691f27fde760bcd841baa27eab
tree59e031390629c18c68810914ca48e9b3b991828b
parentc664a4fa8f69308b8f624cff4fa1294e9aef880d
USB: serial: ch341: fix receiver regression

While assumed not to make a difference, not using the factor-2 prescaler
makes the receiver more susceptible to errors.

Specifically, there have been reports of problems with devices that
cannot generate a 115200 rate with a smaller error than 2.1% (e.g.
117647 bps). But this can also be reproduced with a low-speed RS232
tranceiver at 115200 when the input rate matches the nominal rate.

So whenever possible, enable the factor-2 prescaler and halve the
divisor in order to use settings closer to that of the previous
algorithm.

Fixes: 35714565089e ("USB: serial: ch341: reimplement line-speed handling")
Cc: stable <stable@vger.kernel.org> # 5.5
Reported-by: Jakub Nantl <jn@forever.cz>
Tested-by: Jakub Nantl <jn@forever.cz>
Reviewed-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: Johan Hovold <johan@kernel.org>
drivers/usb/serial/ch341.c