On 12/1/21 12:14 PM, Al Cooper wrote: > There is a small window in time during resume where the hardware > flow control signal RTS can be asserted (which allows a sender to > resume sending data to the UART) but the baud rate has not yet > been restored. This will cause corrupted data and FRAMING, OVERRUN > and BREAK errors. This is happening because the MCTRL register is > shadowed in uart_port struct and is later used during resume to set > the MCTRL register during both serial8250_do_startup() and > uart_resume_port(). Unfortunately, serial8250_do_startup() > happens before the UART baud rate is restored. The fix is to clear > the shadowed mctrl value at the end of suspend and restore it at the > end of resume. > > Fixes: 41a469482de2 ("serial: 8250: Add new 8250-core based Broadcom STB driver") > Signed-off-by: Al Cooper <alcooperx@xxxxxxxxx> Acked-by: Florian Fainelli <f.fainelli@xxxxxxxxx> -- Florian