mmc: mtk-sd: fix possible incomplete read ops
authorFabien Parent <fparent@baylibre.com>
Thu, 17 Jan 2019 17:06:00 +0000 (18:06 +0100)
committerTom Rini <trini@konsulko.com>
Sat, 26 Jan 2019 13:13:54 +0000 (08:13 -0500)
The code is checking for incomplete read when it see the INT_XFER_COMPL
flag, but it forget to first check whether there is anything left in the
FIFO to copy to the RX buffer. This means that sometimes we will get
errors because of erroneous incomplete read operation.

This commit fixes the driver re-ordering the code so that we first
check for data inside the RX fifo and only after check the status
of the INT_XFER_COMPL flag.

Signed-off-by: Fabien Parent <fparent@baylibre.com>
drivers/mmc/mtk-sd.c

index 0741a525c01cf068ae934ec2276e4732ba2fc41e..e668df70170d04faf76abcd86ecd548492d7c560 100644 (file)
@@ -554,6 +554,14 @@ static int msdc_pio_read(struct msdc_host *host, u8 *ptr, u32 size)
                        break;
                }
 
+               chksz = min(size, (u32)MSDC_FIFO_SIZE);
+
+               if (msdc_fifo_rx_bytes(host) >= chksz) {
+                       msdc_fifo_read(host, ptr, chksz);
+                       ptr += chksz;
+                       size -= chksz;
+               }
+
                if (status & MSDC_INT_XFER_COMPL) {
                        if (size) {
                                pr_err("data not fully read\n");
@@ -562,15 +570,7 @@ static int msdc_pio_read(struct msdc_host *host, u8 *ptr, u32 size)
 
                        break;
                }
-
-               chksz = min(size, (u32)MSDC_FIFO_SIZE);
-
-               if (msdc_fifo_rx_bytes(host) >= chksz) {
-                       msdc_fifo_read(host, ptr, chksz);
-                       ptr += chksz;
-                       size -= chksz;
-               }
-       }
+}
 
        return ret;
 }