Staging: ipack/devices/ipoctal: read more than one character from RX FIFO.
authorSamuel Iglesias Gonsalvez <siglesias@igalia.com>
Wed, 12 Sep 2012 12:55:43 +0000 (14:55 +0200)
committerGreg Kroah-Hartman <gregkh@linuxfoundation.org>
Wed, 12 Sep 2012 16:56:02 +0000 (09:56 -0700)
The RX FIFO has a size of 3 characters. Check if when we are picking the
oldest one, we have more to read.

Signed-off-by: Samuel Iglesias Gonsalvez <siglesias@igalia.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
drivers/staging/ipack/devices/ipoctal.c

index 8b2be618fc9e210c819dc213c300d7a0718ed9c3..4f79fad7598d2e9688217ed33ce706c34f204ab6 100644 (file)
@@ -134,33 +134,45 @@ static int ipoctal_get_icount(struct tty_struct *tty,
 static void ipoctal_irq_rx(struct ipoctal_channel *channel,
                           struct tty_struct *tty, u8 sr)
 {
-       unsigned char value = ioread8(&channel->regs->r.rhr);
+       unsigned char value;
        unsigned char flag = TTY_NORMAL;
-
-       /* Error: count statistics */
-       if (sr & SR_ERROR) {
-               iowrite8(CR_CMD_RESET_ERR_STATUS, &channel->regs->w.cr);
-
-               if (sr & SR_OVERRUN_ERROR) {
-                       channel->stats.overrun_err++;
-                       /* Overrun doesn't affect the current character*/
-                       tty_insert_flip_char(tty, 0, TTY_OVERRUN);
-               }
-               if (sr & SR_PARITY_ERROR) {
-                       channel->stats.parity_err++;
-                       flag = TTY_PARITY;
-               }
-               if (sr & SR_FRAMING_ERROR) {
-                       channel->stats.framing_err++;
-                       flag = TTY_FRAME;
+       u8 isr;
+
+       do {
+               value = ioread8(&channel->regs->r.rhr);
+               /* Error: count statistics */
+               if (sr & SR_ERROR) {
+                       iowrite8(CR_CMD_RESET_ERR_STATUS, &channel->regs->w.cr);
+
+                       if (sr & SR_OVERRUN_ERROR) {
+                               channel->stats.overrun_err++;
+                               /* Overrun doesn't affect the current character*/
+                               tty_insert_flip_char(tty, 0, TTY_OVERRUN);
+                       }
+                       if (sr & SR_PARITY_ERROR) {
+                               channel->stats.parity_err++;
+                               flag = TTY_PARITY;
+                       }
+                       if (sr & SR_FRAMING_ERROR) {
+                               channel->stats.framing_err++;
+                               flag = TTY_FRAME;
+                       }
+                       if (sr & SR_RECEIVED_BREAK) {
+                               channel->stats.rcv_break++;
+                               flag = TTY_BREAK;
+                       }
                }
-               if (sr & SR_RECEIVED_BREAK) {
-                       channel->stats.rcv_break++;
-                       flag = TTY_BREAK;
-               }
-       }
+               tty_insert_flip_char(tty, value, flag);
 
-       tty_insert_flip_char(tty, value, flag);
+               /* Check if there are more characters in RX FIFO
+                * If there are more, the isr register for this channel
+                * has enabled the RxRDY|FFULL bit.
+                */
+               isr = ioread8(&channel->block_regs->r.isr);
+               sr = ioread8(&channel->regs->r.sr);
+       } while (isr & channel->isr_rx_rdy_mask);
+
+       tty_flip_buffer_push(tty);
 }
 
 static void ipoctal_irq_tx(struct ipoctal_channel *channel)