serial: fsl_lpuart: Remove unnecessary clearing for CRTSCTS

if (termios->c_cflag & CRTSCTS) guarantees that CRTSCTS is not ever set
in the else block so clearing it is unnecessary.

While at it, remove also one pair of extra parenthesis.

Reviewed-by: Johan Hovold <johan@kernel.org>
Signed-off-by: Ilpo Järvinen <ilpo.jarvinen@linux.intel.com>
Link: https://lore.kernel.org/r/20220513082906.11096-5-ilpo.jarvinen@linux.intel.com
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>

diff --git a/drivers/tty/serial/fsl_lpuart.c b/drivers/tty/serial/fsl_lpuart.c
index 7b46b97a6ddd80e1cd42c18386693f884f3088b6..0d6e62f6bb075e913a699924c8e89a51e7b25df1 100644 (file)
--- a/drivers/tty/serial/fsl_lpuart.c
+++ b/drivers/tty/serial/fsl_lpuart.c
@@ -2110,12 +2110,10 @@ lpuart32_set_termios(struct uart_port *port, struct ktermios *termios,
        if (sport->port.rs485.flags & SER_RS485_ENABLED)
                termios->c_cflag &= ~CRTSCTS;
 
-       if (termios->c_cflag & CRTSCTS) {
-               modem |= (UARTMODIR_RXRTSE | UARTMODIR_TXCTSE);
-       } else {
-               termios->c_cflag &= ~CRTSCTS;
+       if (termios->c_cflag & CRTSCTS)
+               modem |= UARTMODIR_RXRTSE | UARTMODIR_TXCTSE;
+       else
                modem &= ~(UARTMODIR_RXRTSE | UARTMODIR_TXCTSE);
-       }
 
        if (termios->c_cflag & CSTOPB)
                bd |= UARTBAUD_SBNS;