serial: altera_jtaguart: switch status to u32
authorJiri Slaby <jirislaby@kernel.org>
Mon, 31 Jul 2023 08:02:40 +0000 (10:02 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Mon, 31 Jul 2023 15:16:05 +0000 (17:16 +0200)
'status' is assigned a result from readl(). There is no need for the
variable to be 'unsigned long'. readl() returns 32bit values.

Provided, this is a Nios II driver (32-bit), there is no change in
semantics. This only makes the type explicit.

Signed-off-by: Jiri Slaby (SUSE) <jirislaby@kernel.org>
Cc: Tobias Klauser <tklauser@distanz.ch>
Link: https://lore.kernel.org/r/20230731080244.2698-7-jirislaby@kernel.org
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/serial/altera_jtaguart.c

index 6203ca1de7696f8e4ac1434dda4f885a650c48d4..5fab4c97889132bc65a2341fc1a2834fbec1d956 100644 (file)
@@ -110,7 +110,7 @@ static void altera_jtaguart_set_termios(struct uart_port *port,
 
 static void altera_jtaguart_rx_chars(struct uart_port *port)
 {
-       unsigned long status;
+       u32 status;
        u8 ch;
 
        while ((status = readl(port->membase + ALTERA_JTAGUART_DATA_REG)) &