From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.free-electrons.com ([62.4.15.54]) by bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux)) id 1dFPle-00042X-6M for linux-mtd@lists.infradead.org; Mon, 29 May 2017 18:51:52 +0000 Date: Mon, 29 May 2017 20:51:27 +0200 From: Boris Brezillon To: Wenyou Yang , Marc Gonzalez , Boris Brezillon , Richard Weinberger , linux-mtd@lists.infradead.org Cc: Marek Vasut , Brian Norris , David Woodhouse , Cyrille Pitchen Subject: Re: [PATCH] mtd: nand: Wait for PAGEPROG to finish in drivers setting NAND_ECC_CUSTOM_PAGE_ACCESS Message-ID: <20170529205127.21590d89@bbrezillon> In-Reply-To: <1494952069-3914-1-git-send-email-boris.brezillon@free-electrons.com> References: <1494952069-3914-1-git-send-email-boris.brezillon@free-electrons.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Tue, 16 May 2017 18:27:49 +0200 Boris Brezillon wrote: > Drivers setting NAND_ECC_CUSTOM_PAGE_ACCESS are supposed to handle the > full read/write page sequence, and waiting for a page to actually be > programmed is part of this write-page sequence. > This is also what is done in ->write_oob_xxx() hooks, so let's do that in > ->write_page_xxx() as well to make it consistent. > Applied to nand/next. > Signed-off-by: Boris Brezillon > --- > drivers/mtd/nand/atmel/nand-controller.c | 6 +++++- > drivers/mtd/nand/nand_base.c | 10 ++++++---- > drivers/mtd/nand/nand_micron.c | 10 ++++++++-- > drivers/mtd/nand/tango_nand.c | 13 ++++++++++++- > 4 files changed, 31 insertions(+), 8 deletions(-) > > diff --git a/drivers/mtd/nand/atmel/nand-controller.c b/drivers/mtd/nand/atmel/nand-controller.c > index 3b2446896147..e5122539d4f1 100644 > --- a/drivers/mtd/nand/atmel/nand-controller.c > +++ b/drivers/mtd/nand/atmel/nand-controller.c > @@ -912,7 +912,7 @@ static int atmel_hsmc_nand_pmecc_write_pg(struct nand_chip *chip, > struct mtd_info *mtd = nand_to_mtd(chip); > struct atmel_nand *nand = to_atmel_nand(chip); > struct atmel_hsmc_nand_controller *nc; > - int ret; > + int ret, status; > > nc = to_hsmc_nand_controller(chip->controller); > > @@ -954,6 +954,10 @@ static int atmel_hsmc_nand_pmecc_write_pg(struct nand_chip *chip, > dev_err(nc->base.dev, "Failed to program NAND page (err = %d)\n", > ret); > > + status = chip->waitfunc(mtd, chip); > + if (status & NAND_STATUS_FAIL) > + return -EIO; > + > return ret; > } > > diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c > index 8dafd2a54e11..08ff98c47e1f 100644 > --- a/drivers/mtd/nand/nand_base.c > +++ b/drivers/mtd/nand/nand_base.c > @@ -2574,11 +2574,13 @@ static int nand_write_page(struct mtd_info *mtd, struct nand_chip *chip, > if (status < 0) > return status; > > - if (nand_standard_page_accessors(&chip->ecc)) > + if (nand_standard_page_accessors(&chip->ecc)) { > chip->cmdfunc(mtd, NAND_CMD_PAGEPROG, -1, -1); > - status = chip->waitfunc(mtd, chip); > - if (status & NAND_STATUS_FAIL) > - return -EIO; > + > + status = chip->waitfunc(mtd, chip); > + if (status & NAND_STATUS_FAIL) > + return -EIO; > + } > > return 0; > } > diff --git a/drivers/mtd/nand/nand_micron.c b/drivers/mtd/nand/nand_micron.c > index 0987f32dc6f7..f3c35fafd51a 100644 > --- a/drivers/mtd/nand/nand_micron.c > +++ b/drivers/mtd/nand/nand_micron.c > @@ -151,15 +151,18 @@ micron_nand_write_page_on_die_ecc(struct mtd_info *mtd, struct nand_chip *chip, > const uint8_t *buf, int oob_required, > int page) > { > + int status; > + > micron_nand_on_die_ecc_setup(chip, true); > > chip->cmdfunc(mtd, NAND_CMD_SEQIN, 0x00, page); > nand_write_page_raw(mtd, chip, buf, oob_required, page); > chip->cmdfunc(mtd, NAND_CMD_PAGEPROG, -1, -1); > + status = chip->waitfunc(mtd, chip); > > micron_nand_on_die_ecc_setup(chip, false); > > - return 0; > + return status & NAND_STATUS_FAIL ? -EIO : 0; > } > > static int > @@ -180,11 +183,14 @@ micron_nand_write_page_raw_on_die_ecc(struct mtd_info *mtd, > const uint8_t *buf, int oob_required, > int page) > { > + int status; > + > chip->cmdfunc(mtd, NAND_CMD_SEQIN, 0x00, page); > nand_write_page_raw(mtd, chip, buf, oob_required, page); > chip->cmdfunc(mtd, NAND_CMD_PAGEPROG, -1, -1); > + status = chip->waitfunc(mtd, chip); > > - return 0; > + return status & NAND_STATUS_FAIL ? -EIO : 0; > } > > enum { > diff --git a/drivers/mtd/nand/tango_nand.c b/drivers/mtd/nand/tango_nand.c > index a2150b15d4c1..8498fa36e533 100644 > --- a/drivers/mtd/nand/tango_nand.c > +++ b/drivers/mtd/nand/tango_nand.c > @@ -295,7 +295,7 @@ static int tango_write_page(struct mtd_info *mtd, struct nand_chip *chip, > const u8 *buf, int oob_required, int page) > { > struct tango_nfc *nfc = to_tango_nfc(chip->controller); > - int err, len = mtd->writesize; > + int err, status, len = mtd->writesize; > > /* Calling tango_write_oob() would send PAGEPROG twice */ > if (oob_required) > @@ -306,6 +306,10 @@ static int tango_write_page(struct mtd_info *mtd, struct nand_chip *chip, > if (err) > return err; > > + status = chip->waitfunc(mtd, chip); > + if (status & NAND_STATUS_FAIL) > + return -EIO; > + > return 0; > } > > @@ -423,9 +427,16 @@ static int tango_read_page_raw(struct mtd_info *mtd, struct nand_chip *chip, > static int tango_write_page_raw(struct mtd_info *mtd, struct nand_chip *chip, > const u8 *buf, int oob_required, int page) > { > + int status; > + > chip->cmdfunc(mtd, NAND_CMD_SEQIN, 0, page); > raw_write(chip, buf, chip->oob_poi); > chip->cmdfunc(mtd, NAND_CMD_PAGEPROG, -1, -1); > + > + status = chip->waitfunc(mtd, chip); > + if (status & NAND_STATUS_FAIL) > + return -EIO; > + > return 0; > } >