mtd: nand: tango: Fix incorrect use of SEQIN command

Linux-MTD Mailing List linux-mtd at lists.infradead.org
Thu Jul 13 10:59:15 PDT 2017


Gitweb:     http://git.infradead.org/?p=mtd-2.6.git;a=commit;h=a186493237a9d8559997c2f97c33c4716d602fd2
Commit:     a186493237a9d8559997c2f97c33c4716d602fd2
Parent:     df5586d7bf6cce421ed48e9b2a88a8bdaa4fd9d0
Author:     Boris Brezillon <boris.brezillon at free-electrons.com>
AuthorDate: Wed May 17 10:47:50 2017 +0200
Committer:  Boris Brezillon <boris.brezillon at free-electrons.com>
CommitDate: Thu Jun 1 10:09:33 2017 +0200

    mtd: nand: tango: Fix incorrect use of SEQIN command
    
    SEQIN is supposed to be used when one wants to start programming a page.
    What we want here is just to change the column within the page, which is
    done with the RNDIN command.
    
    Fixes: 6956e2385a16 ("mtd: nand: add tango NAND flash controller support")
    Cc: stable at vger.kernel.org
    Signed-off-by: Boris Brezillon <boris.brezillon at free-electrons.com>
    Acked-by: Marc Gonzalez <marc_gonzalez at sigmadesigns.com>
---
 drivers/mtd/nand/tango_nand.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/mtd/nand/tango_nand.c b/drivers/mtd/nand/tango_nand.c
index 85e0d97..201979f 100644
--- a/drivers/mtd/nand/tango_nand.c
+++ b/drivers/mtd/nand/tango_nand.c
@@ -332,7 +332,7 @@ static void aux_write(struct nand_chip *chip, const u8 **buf, int len, int *pos)
 
 	if (!*buf) {
 		/* skip over "len" bytes */
-		chip->cmdfunc(mtd, NAND_CMD_SEQIN, *pos, -1);
+		chip->cmdfunc(mtd, NAND_CMD_RNDIN, *pos, -1);
 	} else {
 		tango_write_buf(mtd, *buf, len);
 		*buf += len;



More information about the linux-mtd-cvs mailing list