staging: dgnc: UART_IIR_XOFF code is buggy
authorDan Carpenter <dan.carpenter@oracle.com>
Wed, 14 May 2014 16:13:07 +0000 (19:13 +0300)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Thu, 15 May 2014 20:46:00 +0000 (13:46 -0700)
The UART_IIR_XOFF was supposed to be a no-op but, because there was a
missing semi-colon, the if statement is not "Empty".  I have just deleted
this code because it was supposed to be a no-op anyway.  UART_IIR_XOFF
is a standard define and not something specific to this driver.

Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/dgnc/dgnc_cls.c
drivers/staging/dgnc/dgnc_cls.h

index 60d9b62c6f1f3b766746cb43aa972e19c54c7e3c..8e265c20db59fefe9be6ee04bbd57a7d93792cab 100644 (file)
@@ -453,10 +453,6 @@ static inline void cls_parse_isr(struct dgnc_board *brd, uint port)
                        cls_copy_data_from_queue_to_uart(ch);
                }
 
-               /* Received Xoff signal/Special character */
-               if (isr & UART_IIR_XOFF)
-                       /* Empty */
-
                /* CTS/RTS change of state */
                if (isr & UART_IIR_CTSRTS) {
                        brd->intr_modem++;
index ffe8535a84a9d93a322464c991a114072f38f4a2..465d79a6f75f6c138f6b5d12909f8962b01226c1 100644 (file)
@@ -61,7 +61,6 @@ struct cls_uart_struct {
 #define UART_16654_FCR_RXTRIGGER_56    0x80
 #define UART_16654_FCR_RXTRIGGER_60     0xC0
 
-#define UART_IIR_XOFF                  0x10    /* Received Xoff signal/Special character */
 #define UART_IIR_CTSRTS                        0x20    /* Received CTS/RTS change of state */
 #define UART_IIR_RDI_TIMEOUT           0x0C    /* Receiver data TIMEOUT */