Commit 68baab14 authored by Andrey Smirnov's avatar Andrey Smirnov Committed by Greg Kroah-Hartman

tty: serial: fsl_lpuart: Use appropriate lpuart32_* I/O funcs

[ Upstream commit 1da17d7c ]

When dealing with 32-bit variant of LPUART IP block appropriate I/O
helpers have to be used to properly deal with endianness
differences. Change all of the offending code to do that.

Fixes: a5fa2660 ("tty/serial/fsl_lpuart: Add CONSOLE_POLL support
for lpuart32.")
Signed-off-by: default avatarAndrey Smirnov <andrew.smirnov@gmail.com>
Cc: Stefan Agner <stefan@agner.ch>
Cc: Bhuvanchandra DV <bhuvanchandra.dv@toradex.com>
Cc: Chris Healy <cphealy@gmail.com>
Cc: Cory Tusar <cory.tusar@zii.aero>
Cc: Lucas Stach <l.stach@pengutronix.de>
Cc: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Jiri Slaby <jslaby@suse.com>
Cc: linux-imx@nxp.com
Cc: linux-serial@vger.kernel.org
Cc: linux-kernel@vger.kernel.org
Link: https://lore.kernel.org/r/20190729195226.8862-14-andrew.smirnov@gmail.comSigned-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Signed-off-by: default avatarSasha Levin <sashal@kernel.org>
parent afe31dc0
...@@ -528,26 +528,26 @@ static int lpuart32_poll_init(struct uart_port *port) ...@@ -528,26 +528,26 @@ static int lpuart32_poll_init(struct uart_port *port)
spin_lock_irqsave(&sport->port.lock, flags); spin_lock_irqsave(&sport->port.lock, flags);
/* Disable Rx & Tx */ /* Disable Rx & Tx */
writel(0, sport->port.membase + UARTCTRL); lpuart32_write(&sport->port, UARTCTRL, 0);
temp = readl(sport->port.membase + UARTFIFO); temp = lpuart32_read(&sport->port, UARTFIFO);
/* Enable Rx and Tx FIFO */ /* Enable Rx and Tx FIFO */
writel(temp | UARTFIFO_RXFE | UARTFIFO_TXFE, lpuart32_write(&sport->port, UARTFIFO,
sport->port.membase + UARTFIFO); temp | UARTFIFO_RXFE | UARTFIFO_TXFE);
/* flush Tx and Rx FIFO */ /* flush Tx and Rx FIFO */
writel(UARTFIFO_TXFLUSH | UARTFIFO_RXFLUSH, lpuart32_write(&sport->port, UARTFIFO,
sport->port.membase + UARTFIFO); UARTFIFO_TXFLUSH | UARTFIFO_RXFLUSH);
/* explicitly clear RDRF */ /* explicitly clear RDRF */
if (readl(sport->port.membase + UARTSTAT) & UARTSTAT_RDRF) { if (lpuart32_read(&sport->port, UARTSTAT) & UARTSTAT_RDRF) {
readl(sport->port.membase + UARTDATA); lpuart32_read(&sport->port, UARTDATA);
writel(UARTFIFO_RXUF, sport->port.membase + UARTFIFO); lpuart32_write(&sport->port, UARTFIFO, UARTFIFO_RXUF);
} }
/* Enable Rx and Tx */ /* Enable Rx and Tx */
writel(UARTCTRL_RE | UARTCTRL_TE, sport->port.membase + UARTCTRL); lpuart32_write(&sport->port, UARTCTRL, UARTCTRL_RE | UARTCTRL_TE);
spin_unlock_irqrestore(&sport->port.lock, flags); spin_unlock_irqrestore(&sport->port.lock, flags);
return 0; return 0;
...@@ -555,18 +555,18 @@ static int lpuart32_poll_init(struct uart_port *port) ...@@ -555,18 +555,18 @@ static int lpuart32_poll_init(struct uart_port *port)
static void lpuart32_poll_put_char(struct uart_port *port, unsigned char c) static void lpuart32_poll_put_char(struct uart_port *port, unsigned char c)
{ {
while (!(readl(port->membase + UARTSTAT) & UARTSTAT_TDRE)) while (!(lpuart32_read(port, UARTSTAT) & UARTSTAT_TDRE))
barrier(); barrier();
writel(c, port->membase + UARTDATA); lpuart32_write(port, UARTDATA, c);
} }
static int lpuart32_poll_get_char(struct uart_port *port) static int lpuart32_poll_get_char(struct uart_port *port)
{ {
if (!(readl(port->membase + UARTSTAT) & UARTSTAT_RDRF)) if (!(lpuart32_read(port, UARTSTAT) & UARTSTAT_RDRF))
return NO_POLL_CHAR; return NO_POLL_CHAR;
return readl(port->membase + UARTDATA); return lpuart32_read(port, UARTDATA);
} }
#endif #endif
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment