serial: 8250: Eliminate needless local in irq handlers
authorPeter Hurley <peter@hurleysoftware.com>
Sun, 10 Apr 2016 03:49:41 +0000 (20:49 -0700)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Sat, 30 Apr 2016 16:26:55 +0000 (09:26 -0700)
dma_err is not used other than for boolean evaluation;
substitute the actual operation directly.

Signed-off-by: Peter Hurley <peter@hurleysoftware.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/tty/serial/8250/8250_omap.c
drivers/tty/serial/8250/8250_port.c

index 6f760510e46da08a4beaa41d47e47e8ccdece352..7c8c88653374c813e048fc381db45c28735217c6 100644 (file)
@@ -1033,7 +1033,6 @@ static int omap_8250_dma_handle_irq(struct uart_port *port)
        unsigned char status;
        unsigned long flags;
        u8 iir;
-       int dma_err = 0;
 
        serial8250_rpm_get(up);
 
@@ -1048,9 +1047,7 @@ static int omap_8250_dma_handle_irq(struct uart_port *port)
        status = serial_port_in(port, UART_LSR);
 
        if (status & (UART_LSR_DR | UART_LSR_BI)) {
-
-               dma_err = omap_8250_rx_dma(up, iir);
-               if (dma_err) {
+               if (omap_8250_rx_dma(up, iir)) {
                        status = serial8250_rx_chars(up, status);
                        omap_8250_rx_dma(up, 0);
                }
@@ -1066,8 +1063,7 @@ static int omap_8250_dma_handle_irq(struct uart_port *port)
                         * try again due to an earlier failer which
                         * might have been resolved by now.
                         */
-                       dma_err = omap_8250_tx_dma(up);
-                       if (dma_err)
+                       if (omap_8250_tx_dma(up))
                                serial8250_tx_chars(up);
                }
        }
index a803ddfd5a599fb18b38cdcaeb0ed92964697d77..40d2d303dc7f3a0e29688a8cf5a9a65f12b1642a 100644 (file)
@@ -1796,7 +1796,6 @@ int serial8250_handle_irq(struct uart_port *port, unsigned int iir)
        unsigned char status;
        unsigned long flags;
        struct uart_8250_port *up = up_to_u8250p(port);
-       int dma_err = 0;
 
        if (iir & UART_IIR_NO_INT)
                return 0;
@@ -1808,10 +1807,7 @@ int serial8250_handle_irq(struct uart_port *port, unsigned int iir)
        DEBUG_INTR("status = %x...", status);
 
        if (status & (UART_LSR_DR | UART_LSR_BI)) {
-               if (up->dma)
-                       dma_err = up->dma->rx_dma(up, iir);
-
-               if (!up->dma || dma_err)
+               if (!up->dma || up->dma->rx_dma(up, iir))
                        status = serial8250_rx_chars(up, status);
        }
        serial8250_modem_status(up);