USB: serial: cp210x: disable break signalling on CP2105 SCI
authorJohan Hovold <johan@kernel.org>
Sun, 4 Jun 2023 12:35:04 +0000 (14:35 +0200)
committerJohan Hovold <johan@kernel.org>
Wed, 7 Jun 2023 15:00:23 +0000 (17:00 +0200)
Only the first UART interface (ECI) on CP2105 supports break signalling.

Return an error on requests for break state changes for the second
interface (SCI) to avoid transmitting a garbage character and waiting
when break is not supported.

Tested-by: Corey Minyard <cminyard@mvista.com>
Signed-off-by: Johan Hovold <johan@kernel.org>
drivers/usb/serial/cp210x.c

index 81e49ed..1e61fe0 100644 (file)
@@ -1437,8 +1437,14 @@ static int cp210x_tiocmget(struct tty_struct *tty)
 static int cp210x_break_ctl(struct tty_struct *tty, int break_state)
 {
        struct usb_serial_port *port = tty->driver_data;
+       struct cp210x_serial_private *priv = usb_get_serial_data(port->serial);
        u16 state;
 
+       if (priv->partnum == CP210X_PARTNUM_CP2105) {
+               if (cp210x_interface_num(port->serial) == 1)
+                       return -ENOTTY;
+       }
+
        if (break_state == 0)
                state = BREAK_OFF;
        else