This patch enables support for Toshiba BENAND. It was originally posted on
  https://lkml.org/lkml/2015/7/24/571

This patch is rewrote to adapt the recent mtd "separate vendor specific code
from core" cleanup process.

Signed-off-by: KOBAYASHI Yoshitake <yoshitake.kobaya...@toshiba.co.jp>
---
 drivers/mtd/nand/Kconfig        |  17 ++++++
 drivers/mtd/nand/nand_base.c    |  15 ++++++
 drivers/mtd/nand/nand_toshiba.c | 112 +++++++++++++++++++++++++++++++++++++++-
 include/linux/mtd/nand.h        |   1 +
 4 files changed, 143 insertions(+), 2 deletions(-)

diff --git a/drivers/mtd/nand/Kconfig b/drivers/mtd/nand/Kconfig
index 0bd2319..6634d4b 100644
--- a/drivers/mtd/nand/Kconfig
+++ b/drivers/mtd/nand/Kconfig
@@ -36,6 +36,23 @@ config MTD_NAND_ECC_BCH
          ECC codes. They are used with NAND devices requiring more than 1 bit
          of error correction.
 
+config MTD_NAND_BENAND
+       bool "Support for Toshiba BENAND (Built-in ECC NAND)"
+       default n
+       help
+         This enables support for Toshiba BENAND.
+         Toshiba BENAND is a SLC NAND solution that automatically
+         generates ECC inside NAND chip.
+
+config MTD_NAND_BENAND_ECC_STATUS
+       bool "Enable ECC Status Read Command(0x7A)"
+       depends on MTD_NAND_BENAND
+       default n
+       help
+         This enables support for ECC Status Read Command(0x7A) of BENAND.
+         When this enables, report the real number of bitflips.
+         In other cases, report the assumud number.
+
 config MTD_SM_COMMON
        tristate
        default n
diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c
index 43722a8..ab8652e 100644
--- a/drivers/mtd/nand/nand_base.c
+++ b/drivers/mtd/nand/nand_base.c
@@ -4186,6 +4186,7 @@ static const char * const nand_ecc_modes[] = {
        [NAND_ECC_HW_SYNDROME]  = "hw_syndrome",
        [NAND_ECC_HW_OOB_FIRST] = "hw_oob_first",
        [NAND_ECC_ON_DIE]       = "on-die",
+       [NAND_ECC_BENAND]       = "benand",
 };
 
 static int of_get_nand_ecc_mode(struct device_node *np)
@@ -4751,6 +4752,19 @@ int nand_scan_tail(struct mtd_info *mtd)
                        ecc->write_oob = nand_write_oob_std;
                break;
 
+       case NAND_ECC_BENAND:
+               if (!ecc->read_page || !ecc->read_subpage) {
+                       WARN(1, "No ECC functions supplied; benand ECC not 
possible\n");
+                       ret = -EINVAL;
+                       goto err_free;
+               }
+               ecc->write_page = nand_write_page_raw;
+               ecc->read_page_raw = nand_read_page_raw;
+               ecc->write_page_raw = nand_write_page_raw;
+               ecc->read_oob = nand_read_oob_std;
+               ecc->write_oob = nand_write_oob_std;
+               break;
+
        case NAND_ECC_NONE:
                pr_warn("NAND_ECC_NONE selected by board driver. This is not 
recommended!\n");
                ecc->read_page = nand_read_page_raw;
@@ -4831,6 +4845,7 @@ int nand_scan_tail(struct mtd_info *mtd)
        /* Large page NAND with SOFT_ECC should support subpage reads */
        switch (ecc->mode) {
        case NAND_ECC_SOFT:
+       case NAND_ECC_BENAND:
                if (chip->page_shift > 9)
                        chip->options |= NAND_SUBPAGE_READ;
                break;
diff --git a/drivers/mtd/nand/nand_toshiba.c b/drivers/mtd/nand/nand_toshiba.c
index fa787ba..bb3c852 100644
--- a/drivers/mtd/nand/nand_toshiba.c
+++ b/drivers/mtd/nand/nand_toshiba.c
@@ -17,6 +17,97 @@
 
 #include <linux/mtd/nand.h>
 
+/* ECC Status Read Command for BENAND */
+#define NAND_CMD_ECC_STATUS    0x7A
+
+/* Recommended to rewrite for BENAND */
+#define NAND_STATUS_RECOM_REWRT        0x08
+
+
+static int toshiba_nand_benand_status_chk(struct mtd_info *mtd,
+                       struct nand_chip *chip)
+{
+       unsigned int max_bitflips = 0;
+       u8 status;
+
+       /* Check Read Status */
+       chip->cmdfunc(mtd, NAND_CMD_STATUS, -1, -1);
+       status = chip->read_byte(mtd);
+
+       /* timeout */
+       if (!(status & NAND_STATUS_READY)) {
+               pr_debug("BENAND : Time Out!\n");
+               return -EIO;
+       }
+
+       /* uncorrectable */
+       else if (status & NAND_STATUS_FAIL)
+               mtd->ecc_stats.failed++;
+
+       /* correctable */
+       else if (status & NAND_STATUS_RECOM_REWRT) {
+               if (chip->cmd_ctrl &&
+                       IS_ENABLED(CONFIG_MTD_NAND_BENAND_ECC_STATUS)) {
+
+                       int i;
+                       u8 ecc_status;
+                       unsigned int bitflips;
+
+                       /* Check Read ECC Status */
+                       chip->cmd_ctrl(mtd, NAND_CMD_ECC_STATUS,
+                               NAND_NCE | NAND_CLE | NAND_CTRL_CHANGE);
+                       /* Get bitflips info per 512Byte */
+                       for (i = 0; i < mtd->writesize >> 9; i++) {
+                               ecc_status = chip->read_byte(mtd);
+                               bitflips = ecc_status & 0x0f;
+                               max_bitflips = max_t(unsigned int,
+                                               max_bitflips, bitflips);
+                       }
+                       mtd->ecc_stats.corrected += max_bitflips;
+               } else {
+                       /*
+                        * If can't use chip->cmd_ctrl,
+                        * we can't get real number of bitflips.
+                        * So, we set max_bitflips mtd->bitflip_threshold.
+                        */
+                       max_bitflips = mtd->bitflip_threshold;
+                       mtd->ecc_stats.corrected += max_bitflips;
+               }
+       }
+
+       return max_bitflips;
+}
+
+static int toshiba_nand_read_page_benand(struct mtd_info *mtd,
+                       struct nand_chip *chip, uint8_t *buf,
+                       int oob_required, int page)
+{
+       unsigned int max_bitflips = 0;
+
+       chip->ecc.read_page_raw(mtd, chip, buf, oob_required, page);
+       max_bitflips = toshiba_nand_benand_status_chk(mtd, chip);
+
+       return max_bitflips;
+}
+
+static int toshiba_nand_read_subpage_benand(struct mtd_info *mtd,
+                       struct nand_chip *chip, uint32_t data_offs,
+                       uint32_t readlen, uint8_t *bufpoi, int page)
+{
+       uint8_t *p;
+       unsigned int max_bitflips = 0;
+
+       if (data_offs != 0)
+               chip->cmdfunc(mtd, NAND_CMD_RNDOUT, data_offs, -1);
+
+       p = bufpoi + data_offs;
+       chip->read_buf(mtd, p, readlen);
+
+       max_bitflips = toshiba_nand_benand_status_chk(mtd, chip);
+
+       return max_bitflips;
+}
+
 static void toshiba_nand_decode_id(struct nand_chip *chip)
 {
        struct mtd_info *mtd = nand_to_mtd(chip);
@@ -33,15 +124,32 @@ static void toshiba_nand_decode_id(struct nand_chip *chip)
         */
        if (chip->id.len >= 6 && nand_is_slc(chip) &&
            (chip->id.data[5] & 0x7) == 0x6 /* 24nm */ &&
-           !(chip->id.data[4] & 0x80) /* !BENAND */)
-               mtd->oobsize = 32 * mtd->writesize >> 9;
+           (chip->id.data[4] & 0x80) /* BENAND */) {
+               if (IS_ENABLED(CONFIG_MTD_NAND_BENAND))
+                       chip->ecc.mode = NAND_ECC_BENAND;
+       } else {
+               mtd->oobsize = 32 * mtd->writesize >> 9;        /* !BENAND */
+       }
 }
 
 static int toshiba_nand_init(struct nand_chip *chip)
 {
+       struct mtd_info *mtd = nand_to_mtd(chip);
+
        if (nand_is_slc(chip))
                chip->bbt_options |= NAND_BBT_SCAN2NDPAGE;
 
+       if (chip->ecc.mode == NAND_ECC_BENAND) {
+               chip->ecc.options = NAND_ECC_CUSTOM_PAGE_ACCESS;
+               chip->ecc.bytes = 0;
+               chip->ecc.strength = 8;
+               chip->ecc.total = 0;
+               chip->ecc.read_page = toshiba_nand_read_page_benand;
+               chip->ecc.read_subpage = toshiba_nand_read_subpage_benand;
+
+               mtd_set_ooblayout(mtd, &nand_ooblayout_lp_ops);
+       }
+
        return 0;
 }
 
diff --git a/include/linux/mtd/nand.h b/include/linux/mtd/nand.h
index 3a6a77f..6821334 100644
--- a/include/linux/mtd/nand.h
+++ b/include/linux/mtd/nand.h
@@ -117,6 +117,7 @@ typedef enum {
        NAND_ECC_HW_SYNDROME,
        NAND_ECC_HW_OOB_FIRST,
        NAND_ECC_ON_DIE,
+       NAND_ECC_BENAND,
 } nand_ecc_modes_t;
 
 enum nand_ecc_algo {
-- 
2.7.4


Reply via email to