Blackfin: nand: drain the write buffer before returning
authorAndrew Caldwell <Andrew.Caldwell@analog.com>
Fri, 7 May 2010 19:10:07 +0000 (15:10 -0400)
committerScott Wood <scottwood@freescale.com>
Wed, 26 May 2010 22:59:13 +0000 (17:59 -0500)
The current Blackfin nand write function fills up the write buffer but
returns before it has had a chance to drain.  On faster systems, this
isn't a problem as the operation finishes before the ECC registers are
read, but on slower systems the ECC may be incomplete when the core tries
to read it.

So wait for the buffer to drain once we're done writing to it.

Signed-off-by: Andrew Caldwell <Andrew.Caldwell@analog.com>
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
drivers/mtd/nand/bfin_nand.c

index f134ef1..6d3d450 100644 (file)
@@ -75,7 +75,7 @@ static void bfin_nfc_cmd_ctrl(struct mtd_info *mtd, int cmd, unsigned int ctrl)
 int bfin_nfc_devready(struct mtd_info *mtd)
 {
        pr_stamp();
-       return (bfin_read_NFC_STAT() & NBUSY ? 1 : 0);
+       return (bfin_read_NFC_STAT() & NBUSY) ? 1 : 0;
 }
 
 /*
@@ -132,6 +132,11 @@ static void bfin_nfc_write_buf(struct mtd_info *mtd, const uint8_t *buf, int len
 
                bfin_write_NFC_DATA_WR(buf[i]);
        }
+
+       /* Wait for the buffer to drain before we return */
+       while (!(bfin_read_NFC_STAT() & WB_EMPTY))
+               if (ctrlc())
+                       return;
 }
 
 /*