linux-kernel - [PATCH] serial: max310x: turn off transmitter before activating AutoCTS or auto transmitter flow control

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [thread-next>] [day] [month] [year] [list]

Message-ID: <bd9fb8e75d0e45218eb110c25bb539ef@SSSDEEX.i.sigma-surface-science.com>
Date:   Fri, 30 Aug 2019 09:58:41 +0000
From:   Christoph Vogtländer 
        <Christoph.Vogtlaender@...ma-surface-science.com>
To:     unlisted-recipients:; (no To-header on input)
CC:     Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        Jiri Slaby <jslaby@...e.com>,
        "linux-serial@...r.kernel.org" <linux-serial@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        Christoph Vogtländer 
        <Christoph.Vogtlaender@...ma-surface-science.com>
Subject: [PATCH] serial: max310x: turn off transmitter before activating
 AutoCTS or auto transmitter flow control

As documented in the data-sheet, the transmitter must be disabled before
activating AutoCTS or auto transmitter flow control. Accordingly, the
transmitter must be enabled after AutoCTS or auto transmitter flow
control gets deactivated.

Signed-off-by: Christoph Vogtländer <c.vogtlaender@...ma-surface-science.com>
---
 drivers/tty/serial/max310x.c | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/drivers/tty/serial/max310x.c b/drivers/tty/serial/max310x.c
index fb5a7e0e58e9..adfef6dae4a7 100644
--- a/drivers/tty/serial/max310x.c
+++ b/drivers/tty/serial/max310x.c
@@ -860,6 +860,15 @@ static void max310x_set_termios(struct uart_port *port,
 max310x_port_write(port, MAX310X_XON1_REG, termios->c_cc[VSTART]);
 max310x_port_write(port, MAX310X_XOFF1_REG, termios->c_cc[VSTOP]);

+/* Disable transmitter before enabling AutoCTS or auto transmitter
+ * flow control
+ */
+if (termios->c_cflag & CRTSCTS || termios->c_iflag & IXOFF) {
+max310x_port_update(port, MAX310X_MODE1_REG,
+    MAX310X_MODE1_TXDIS_BIT,
+    MAX310X_MODE1_TXDIS_BIT);
+}
+
 port->status &= ~(UPSTAT_AUTOCTS | UPSTAT_AUTORTS | UPSTAT_AUTOXOFF);

 if (termios->c_cflag & CRTSCTS) {
@@ -878,6 +887,15 @@ static void max310x_set_termios(struct uart_port *port,
 }
 max310x_port_write(port, MAX310X_FLOWCTRL_REG, flow);

+/* Enable transmitter after disabling AutoCTS and auto transmitter
+ * flow control
+ */
+if (!(termios->c_cflag & CRTSCTS) && !(termios->c_iflag & IXOFF)) {
+max310x_port_update(port, MAX310X_MODE1_REG,
+    MAX310X_MODE1_TXDIS_BIT,
+    0);
+}
+
 /* Get baud rate generator configuration */
 baud = uart_get_baud_rate(port, termios, old,
   port->uartclk / 16 / 0xffff,
--
2.22.1


Sigma Surface Science GmbH, Idsteiner Str. 78, 65232 Taunusstein. Amtsgericht Wiesbaden, HRB 27422. Geschäftsführer: Norbert Nold