* [PATCH v4 01/12] mtd: nand: add a helper to check the SLC/MLC nand chip
2013-09-25 6:58 [PATCH v4 00/12] About the SLC/MLC Huang Shijie
@ 2013-09-25 6:58 ` Huang Shijie
2013-09-25 6:58 ` [PATCH v4 02/12] mtd: nand: rename the cellinfo to bits_per_cell Huang Shijie
` (11 subsequent siblings)
12 siblings, 0 replies; 20+ messages in thread
From: Huang Shijie @ 2013-09-25 6:58 UTC (permalink / raw)
To: dwmw2; +Cc: Huang Shijie, computersforpeace, linux-mtd, dedekind1
Add a helper to check if a nand chip is SLC or MLC.
This helper makes the code more readable.
Signed-off-by: Huang Shijie <b32955@freescale.com>
---
drivers/mtd/nand/denali.c | 2 +-
drivers/mtd/nand/nand_base.c | 14 ++++++--------
include/linux/mtd/nand.h | 9 +++++++++
3 files changed, 16 insertions(+), 9 deletions(-)
diff --git a/drivers/mtd/nand/denali.c b/drivers/mtd/nand/denali.c
index 154b033..370b9dd 100644
--- a/drivers/mtd/nand/denali.c
+++ b/drivers/mtd/nand/denali.c
@@ -1520,7 +1520,7 @@ int denali_init(struct denali_nand_info *denali)
* so just let controller do 15bit ECC for MLC and 8bit ECC for
* SLC if possible.
* */
- if (denali->nand.cellinfo & NAND_CI_CELLTYPE_MSK &&
+ if (!nand_is_slc(&denali->nand) &&
(denali->mtd.oobsize > (denali->bbtskipbytes +
ECC_15BITS * (denali->mtd.writesize /
ECC_SECTOR_SIZE)))) {
diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c
index 00022b4..f256385 100644
--- a/drivers/mtd/nand/nand_base.c
+++ b/drivers/mtd/nand/nand_base.c
@@ -3107,8 +3107,7 @@ static void nand_decode_ext_id(struct mtd_info *mtd, struct nand_chip *chip,
* ID to decide what to do.
*/
if (id_len == 6 && id_data[0] == NAND_MFR_SAMSUNG &&
- (chip->cellinfo & NAND_CI_CELLTYPE_MSK) &&
- id_data[5] != 0x00) {
+ !nand_is_slc(chip) && id_data[5] != 0x00) {
/* Calc pagesize */
mtd->writesize = 2048 << (extid & 0x03);
extid >>= 2;
@@ -3140,7 +3139,7 @@ static void nand_decode_ext_id(struct mtd_info *mtd, struct nand_chip *chip,
(((extid >> 1) & 0x04) | (extid & 0x03));
*busw = 0;
} else if (id_len == 6 && id_data[0] == NAND_MFR_HYNIX &&
- (chip->cellinfo & NAND_CI_CELLTYPE_MSK)) {
+ !nand_is_slc(chip)) {
unsigned int tmp;
/* Calc pagesize */
@@ -3203,7 +3202,7 @@ static void nand_decode_ext_id(struct mtd_info *mtd, struct nand_chip *chip,
* - ID byte 5, bit[7]: 1 -> BENAND, 0 -> raw SLC
*/
if (id_len >= 6 && id_data[0] == NAND_MFR_TOSHIBA &&
- !(chip->cellinfo & NAND_CI_CELLTYPE_MSK) &&
+ nand_is_slc(chip) &&
(id_data[5] & 0x7) == 0x6 /* 24nm */ &&
!(id_data[4] & 0x80) /* !BENAND */) {
mtd->oobsize = 32 * mtd->writesize >> 9;
@@ -3264,11 +3263,11 @@ static void nand_decode_bbm_options(struct mtd_info *mtd,
* Micron devices with 2KiB pages and on SLC Samsung, Hynix, Toshiba,
* AMD/Spansion, and Macronix. All others scan only the first page.
*/
- if ((chip->cellinfo & NAND_CI_CELLTYPE_MSK) &&
+ if (!nand_is_slc(chip) &&
(maf_id == NAND_MFR_SAMSUNG ||
maf_id == NAND_MFR_HYNIX))
chip->bbt_options |= NAND_BBT_SCANLASTPAGE;
- else if ((!(chip->cellinfo & NAND_CI_CELLTYPE_MSK) &&
+ else if ((nand_is_slc(chip) &&
(maf_id == NAND_MFR_SAMSUNG ||
maf_id == NAND_MFR_HYNIX ||
maf_id == NAND_MFR_TOSHIBA ||
@@ -3744,8 +3743,7 @@ int nand_scan_tail(struct mtd_info *mtd)
chip->ecc.total = chip->ecc.steps * chip->ecc.bytes;
/* Allow subpage writes up to ecc.steps. Not possible for MLC flash */
- if (!(chip->options & NAND_NO_SUBPAGE_WRITE) &&
- !(chip->cellinfo & NAND_CI_CELLTYPE_MSK)) {
+ if (!(chip->options & NAND_NO_SUBPAGE_WRITE) && nand_is_slc(chip)) {
switch (chip->ecc.steps) {
case 2:
mtd->subpage_sft = 1;
diff --git a/include/linux/mtd/nand.h b/include/linux/mtd/nand.h
index 1295481..5c05bab 100644
--- a/include/linux/mtd/nand.h
+++ b/include/linux/mtd/nand.h
@@ -795,4 +795,13 @@ static inline int onfi_get_sync_timing_mode(struct nand_chip *chip)
return le16_to_cpu(chip->onfi_params.src_sync_timing_mode);
}
+/*
+ * Check if it is a SLC nand.
+ * The !nand_is_slc() can be used to check the MLC/TLC nand chips.
+ * We do not distinguish the MLC and TLC now.
+ */
+static inline bool nand_is_slc(struct nand_chip *chip)
+{
+ return !(chip->cellinfo & NAND_CI_CELLTYPE_MSK);
+}
#endif /* __LINUX_MTD_NAND_H */
--
1.7.1
^ permalink raw reply related [flat|nested] 20+ messages in thread* [PATCH v4 02/12] mtd: nand: rename the cellinfo to bits_per_cell
2013-09-25 6:58 [PATCH v4 00/12] About the SLC/MLC Huang Shijie
2013-09-25 6:58 ` [PATCH v4 01/12] mtd: nand: add a helper to check the SLC/MLC nand chip Huang Shijie
@ 2013-09-25 6:58 ` Huang Shijie
2013-09-30 22:15 ` Brian Norris
2013-09-25 6:58 ` [PATCH v4 03/12] mtd: nand: add the "bits per cell" info for legacy ID NAND Huang Shijie
` (10 subsequent siblings)
12 siblings, 1 reply; 20+ messages in thread
From: Huang Shijie @ 2013-09-25 6:58 UTC (permalink / raw)
To: dwmw2; +Cc: Huang Shijie, computersforpeace, linux-mtd, dedekind1
The @cellinfo fields contains unused information, such as write caching,
internal chip numbering, etc. But we only use it to check the SLC or MLC.
This patch tries to make it more clear and simple, renames the @cellinfo
to @bits_per_cell.
In order to avoiding the bisect issue, this patch also does the following
changes:
(0) add a macro NAND_CI_CELLTYPE_SHIFT to avoid the hardcode.
(1) add a helper to parse out the cell type : nand_get_bits_per_cell()
(2) parse out the cell type for extended-ID chips and the full-id nand chips.
Signed-off-by: Huang Shijie <b32955@freescale.com>
---
drivers/mtd/nand/nand_base.c | 14 ++++++++++++--
include/linux/mtd/nand.h | 7 ++++---
2 files changed, 16 insertions(+), 5 deletions(-)
diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c
index f256385..c84c35f 100644
--- a/drivers/mtd/nand/nand_base.c
+++ b/drivers/mtd/nand/nand_base.c
@@ -3081,6 +3081,16 @@ static int nand_id_len(u8 *id_data, int arrlen)
return arrlen;
}
+/* Extract the bits of per cell from the 3rd byte of the extended ID */
+static int nand_get_bits_per_cell(u8 cellinfo)
+{
+ int bits;
+
+ bits = cellinfo & NAND_CI_CELLTYPE_MSK;
+ bits >>= NAND_CI_CELLTYPE_SHIFT;
+ return bits + 1;
+}
+
/*
* Many new NAND share similar device ID codes, which represent the size of the
* chip. The rest of the parameters must be decoded according to generic or
@@ -3091,7 +3101,7 @@ static void nand_decode_ext_id(struct mtd_info *mtd, struct nand_chip *chip,
{
int extid, id_len;
/* The 3rd id byte holds MLC / multichip data */
- chip->cellinfo = id_data[2];
+ chip->bits_per_cell = nand_get_bits_per_cell(id_data[2]);
/* The 4th id byte is the important one */
extid = id_data[3];
@@ -3291,7 +3301,7 @@ static bool find_full_id_nand(struct mtd_info *mtd, struct nand_chip *chip,
mtd->erasesize = type->erasesize;
mtd->oobsize = type->oobsize;
- chip->cellinfo = id_data[2];
+ chip->bits_per_cell = nand_get_bits_per_cell(id_data[2]);
chip->chipsize = (uint64_t)type->chipsize << 20;
chip->options |= type->options;
chip->ecc_strength_ds = NAND_ECC_STRENGTH(type);
diff --git a/include/linux/mtd/nand.h b/include/linux/mtd/nand.h
index 5c05bab..0eb4a63 100644
--- a/include/linux/mtd/nand.h
+++ b/include/linux/mtd/nand.h
@@ -198,6 +198,7 @@ typedef enum {
/* Cell info constants */
#define NAND_CI_CHIPNR_MSK 0x03
#define NAND_CI_CELLTYPE_MSK 0x0C
+#define NAND_CI_CELLTYPE_SHIFT 2
/* Keep gcc happy */
struct nand_chip;
@@ -477,7 +478,7 @@ struct nand_buffers {
* @badblockbits: [INTERN] minimum number of set bits in a good block's
* bad block marker position; i.e., BBM == 11110111b is
* not bad when badblockbits == 7
- * @cellinfo: [INTERN] MLC/multichip data from chip ident
+ * @bits_per_cell: [INTERN] the bits of per cell. i.e., 1 means SLC.
* @ecc_strength_ds: [INTERN] ECC correctability from the datasheet.
* Minimum amount of bit errors per @ecc_step_ds guaranteed
* to be correctable. If unknown, set to zero.
@@ -558,7 +559,7 @@ struct nand_chip {
int pagebuf;
unsigned int pagebuf_bitflips;
int subpagesize;
- uint8_t cellinfo;
+ uint8_t bits_per_cell;
uint16_t ecc_strength_ds;
uint16_t ecc_step_ds;
int badblockpos;
@@ -802,6 +803,6 @@ static inline int onfi_get_sync_timing_mode(struct nand_chip *chip)
*/
static inline bool nand_is_slc(struct nand_chip *chip)
{
- return !(chip->cellinfo & NAND_CI_CELLTYPE_MSK);
+ return chip->bits_per_cell == 1;
}
#endif /* __LINUX_MTD_NAND_H */
--
1.7.1
^ permalink raw reply related [flat|nested] 20+ messages in thread* Re: [PATCH v4 02/12] mtd: nand: rename the cellinfo to bits_per_cell
2013-09-25 6:58 ` [PATCH v4 02/12] mtd: nand: rename the cellinfo to bits_per_cell Huang Shijie
@ 2013-09-30 22:15 ` Brian Norris
0 siblings, 0 replies; 20+ messages in thread
From: Brian Norris @ 2013-09-30 22:15 UTC (permalink / raw)
To: Huang Shijie; +Cc: linux-mtd, dwmw2, dedekind1
On Wed, Sep 25, 2013 at 02:58:11PM +0800, Huang Shijie wrote:
> The @cellinfo fields contains unused information, such as write caching,
> internal chip numbering, etc. But we only use it to check the SLC or MLC.
>
> This patch tries to make it more clear and simple, renames the @cellinfo
> to @bits_per_cell.
>
> In order to avoiding the bisect issue, this patch also does the following
> changes:
> (0) add a macro NAND_CI_CELLTYPE_SHIFT to avoid the hardcode.
>
> (1) add a helper to parse out the cell type : nand_get_bits_per_cell()
>
> (2) parse out the cell type for extended-ID chips and the full-id nand chips.
>
> Signed-off-by: Huang Shijie <b32955@freescale.com>
> ---
> drivers/mtd/nand/nand_base.c | 14 ++++++++++++--
> include/linux/mtd/nand.h | 7 ++++---
> 2 files changed, 16 insertions(+), 5 deletions(-)
>
> diff --git a/include/linux/mtd/nand.h b/include/linux/mtd/nand.h
> index 5c05bab..0eb4a63 100644
> --- a/include/linux/mtd/nand.h
> +++ b/include/linux/mtd/nand.h
> @@ -477,7 +478,7 @@ struct nand_buffers {
> * @badblockbits: [INTERN] minimum number of set bits in a good block's
> * bad block marker position; i.e., BBM == 11110111b is
> * not bad when badblockbits == 7
> - * @cellinfo: [INTERN] MLC/multichip data from chip ident
> + * @bits_per_cell: [INTERN] the bits of per cell. i.e., 1 means SLC.
s/bits of/number of bits/
> * @ecc_strength_ds: [INTERN] ECC correctability from the datasheet.
> * Minimum amount of bit errors per @ecc_step_ds guaranteed
> * to be correctable. If unknown, set to zero.
I'll squash that change into the patch myself if the rest looks good.
Brian
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH v4 03/12] mtd: nand: add the "bits per cell" info for legacy ID NAND
2013-09-25 6:58 [PATCH v4 00/12] About the SLC/MLC Huang Shijie
2013-09-25 6:58 ` [PATCH v4 01/12] mtd: nand: add a helper to check the SLC/MLC nand chip Huang Shijie
2013-09-25 6:58 ` [PATCH v4 02/12] mtd: nand: rename the cellinfo to bits_per_cell Huang Shijie
@ 2013-09-25 6:58 ` Huang Shijie
2013-09-25 6:58 ` [PATCH v4 04/12] mtd: nand: set the cell information for ONFI nand Huang Shijie
` (9 subsequent siblings)
12 siblings, 0 replies; 20+ messages in thread
From: Huang Shijie @ 2013-09-25 6:58 UTC (permalink / raw)
To: dwmw2; +Cc: Huang Shijie, computersforpeace, linux-mtd, dedekind1
The legacy ID NAND are all SLC.
This patch sets 1 to the @bits_per_cell for the legacy ID NAND,
which means they are all SLC.
Signed-off-by: Huang Shijie <b32955@freescale.com>
---
drivers/mtd/nand/nand_base.c | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c
index c84c35f..0dedf71 100644
--- a/drivers/mtd/nand/nand_base.c
+++ b/drivers/mtd/nand/nand_base.c
@@ -3237,6 +3237,9 @@ static void nand_decode_id(struct mtd_info *mtd, struct nand_chip *chip,
mtd->oobsize = mtd->writesize / 32;
*busw = type->options & NAND_BUSWIDTH_16;
+ /* All legacy ID NAND are small-page, SLC */
+ chip->bits_per_cell = 1;
+
/*
* Check for Spansion/AMD ID + repeating 5th, 6th byte since
* some Spansion chips have erasesize that conflicts with size
--
1.7.1
^ permalink raw reply related [flat|nested] 20+ messages in thread* [PATCH v4 04/12] mtd: nand: set the cell information for ONFI nand
2013-09-25 6:58 [PATCH v4 00/12] About the SLC/MLC Huang Shijie
` (2 preceding siblings ...)
2013-09-25 6:58 ` [PATCH v4 03/12] mtd: nand: add the "bits per cell" info for legacy ID NAND Huang Shijie
@ 2013-09-25 6:58 ` Huang Shijie
2013-09-25 6:58 ` [PATCH v4 05/12] mtd: nand: print out the cell information for nand chip Huang Shijie
` (8 subsequent siblings)
12 siblings, 0 replies; 20+ messages in thread
From: Huang Shijie @ 2013-09-25 6:58 UTC (permalink / raw)
To: dwmw2; +Cc: Huang Shijie, computersforpeace, linux-mtd, dedekind1
The current code does not set the SLC/MLC information for onfi nand.
(This makes that the kernel treats all the onfi nand as SLC nand.)
This patch fills the cell information for ONFI nands.
Signed-off-by: Huang Shijie <b32955@freescale.com>
---
drivers/mtd/nand/nand_base.c | 1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c
index 0dedf71..94d9084 100644
--- a/drivers/mtd/nand/nand_base.c
+++ b/drivers/mtd/nand/nand_base.c
@@ -2991,6 +2991,7 @@ static int nand_flash_detect_onfi(struct mtd_info *mtd, struct nand_chip *chip,
mtd->oobsize = le16_to_cpu(p->spare_bytes_per_page);
chip->chipsize = le32_to_cpu(p->blocks_per_lun);
chip->chipsize *= (uint64_t)mtd->erasesize * p->lun_count;
+ chip->bits_per_cell = p->bits_per_cell;
if (onfi_feature(chip) & ONFI_FEATURE_16_BIT_BUS)
*busw = NAND_BUSWIDTH_16;
--
1.7.1
^ permalink raw reply related [flat|nested] 20+ messages in thread* [PATCH v4 05/12] mtd: nand: print out the cell information for nand chip
2013-09-25 6:58 [PATCH v4 00/12] About the SLC/MLC Huang Shijie
` (3 preceding siblings ...)
2013-09-25 6:58 ` [PATCH v4 04/12] mtd: nand: set the cell information for ONFI nand Huang Shijie
@ 2013-09-25 6:58 ` Huang Shijie
2013-09-30 22:49 ` Brian Norris
2013-09-25 6:58 ` [PATCH v4 06/12] mtd: gpmi: rewrite the gpmi_ecc_write_oob() to support the jffs2 Huang Shijie
` (7 subsequent siblings)
12 siblings, 1 reply; 20+ messages in thread
From: Huang Shijie @ 2013-09-25 6:58 UTC (permalink / raw)
To: dwmw2; +Cc: Huang Shijie, computersforpeace, linux-mtd, dedekind1
Print out the cell information for nand chip.
(Since the message is too long, this patch also splits the log
with two separate pr_info())
Signed-off-by: Huang Shijie <b32955@freescale.com>
---
drivers/mtd/nand/nand_base.c | 11 +++++++----
1 files changed, 7 insertions(+), 4 deletions(-)
diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c
index 94d9084..b94309b 100644
--- a/drivers/mtd/nand/nand_base.c
+++ b/drivers/mtd/nand/nand_base.c
@@ -3460,11 +3460,14 @@ ident_done:
if (mtd->writesize > 512 && chip->cmdfunc == nand_command)
chip->cmdfunc = nand_command_lp;
- pr_info("NAND device: Manufacturer ID: 0x%02x, Chip ID: 0x%02x (%s %s),"
- " %dMiB, page size: %d, OOB size: %d\n",
+ pr_info("NAND device: Manufacturer ID: 0x%02x,"
+ "Chip ID: 0x%02x (%s %s) \n",
*maf_id, *dev_id, nand_manuf_ids[maf_idx].name,
- chip->onfi_version ? chip->onfi_params.model : type->name,
- (int)(chip->chipsize >> 20), mtd->writesize, mtd->oobsize);
+ chip->onfi_version ? chip->onfi_params.model : type->name);
+
+ pr_info("NAND device: %dMiB, %s, page size: %d, OOB size: %d\n",
+ (int)(chip->chipsize >> 20), nand_is_slc(chip) ? "SLC" : "MLC",
+ mtd->writesize, mtd->oobsize);
return type;
}
--
1.7.1
^ permalink raw reply related [flat|nested] 20+ messages in thread* Re: [PATCH v4 05/12] mtd: nand: print out the cell information for nand chip
2013-09-25 6:58 ` [PATCH v4 05/12] mtd: nand: print out the cell information for nand chip Huang Shijie
@ 2013-09-30 22:49 ` Brian Norris
2013-10-03 14:34 ` Artem Bityutskiy
0 siblings, 1 reply; 20+ messages in thread
From: Brian Norris @ 2013-09-30 22:49 UTC (permalink / raw)
To: Huang Shijie; +Cc: linux-mtd, dwmw2, dedekind1
On Wed, Sep 25, 2013 at 02:58:14PM +0800, Huang Shijie wrote:
> Print out the cell information for nand chip.
>
> (Since the message is too long, this patch also splits the log
> with two separate pr_info())
>
> Signed-off-by: Huang Shijie <b32955@freescale.com>
> ---
> drivers/mtd/nand/nand_base.c | 11 +++++++----
> 1 files changed, 7 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c
> index 94d9084..b94309b 100644
> --- a/drivers/mtd/nand/nand_base.c
> +++ b/drivers/mtd/nand/nand_base.c
> @@ -3460,11 +3460,14 @@ ident_done:
> if (mtd->writesize > 512 && chip->cmdfunc == nand_command)
> chip->cmdfunc = nand_command_lp;
>
> - pr_info("NAND device: Manufacturer ID: 0x%02x, Chip ID: 0x%02x (%s %s),"
> - " %dMiB, page size: %d, OOB size: %d\n",
> + pr_info("NAND device: Manufacturer ID: 0x%02x,"
> + "Chip ID: 0x%02x (%s %s) \n",
You're adding excess whitespace before the '\n'. I'll squash this
myself.
> *maf_id, *dev_id, nand_manuf_ids[maf_idx].name,
> - chip->onfi_version ? chip->onfi_params.model : type->name,
> - (int)(chip->chipsize >> 20), mtd->writesize, mtd->oobsize);
> + chip->onfi_version ? chip->onfi_params.model : type->name);
> +
> + pr_info("NAND device: %dMiB, %s, page size: %d, OOB size: %d\n",
> + (int)(chip->chipsize >> 20), nand_is_slc(chip) ? "SLC" : "MLC",
> + mtd->writesize, mtd->oobsize);
>
> return type;
> }
Brian
^ permalink raw reply [flat|nested] 20+ messages in thread* Re: [PATCH v4 05/12] mtd: nand: print out the cell information for nand chip
2013-09-30 22:49 ` Brian Norris
@ 2013-10-03 14:34 ` Artem Bityutskiy
2013-10-03 15:53 ` Brian Norris
0 siblings, 1 reply; 20+ messages in thread
From: Artem Bityutskiy @ 2013-10-03 14:34 UTC (permalink / raw)
To: Brian Norris; +Cc: Huang Shijie, dwmw2, linux-mtd
On Mon, 2013-09-30 at 15:49 -0700, Brian Norris wrote:
> On Wed, Sep 25, 2013 at 02:58:14PM +0800, Huang Shijie wrote:
> > Print out the cell information for nand chip.
> >
> > (Since the message is too long, this patch also splits the log
> > with two separate pr_info())
> >
> > Signed-off-by: Huang Shijie <b32955@freescale.com>
> > ---
> > drivers/mtd/nand/nand_base.c | 11 +++++++----
> > 1 files changed, 7 insertions(+), 4 deletions(-)
> >
> > diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c
> > index 94d9084..b94309b 100644
> > --- a/drivers/mtd/nand/nand_base.c
> > +++ b/drivers/mtd/nand/nand_base.c
> > @@ -3460,11 +3460,14 @@ ident_done:
> > if (mtd->writesize > 512 && chip->cmdfunc == nand_command)
> > chip->cmdfunc = nand_command_lp;
> >
> > - pr_info("NAND device: Manufacturer ID: 0x%02x, Chip ID: 0x%02x (%s %s),"
> > - " %dMiB, page size: %d, OOB size: %d\n",
> > + pr_info("NAND device: Manufacturer ID: 0x%02x,"
> > + "Chip ID: 0x%02x (%s %s) \n",
>
> You're adding excess whitespace before the '\n'. I'll squash this
> myself.
Nowadays' fashion is to never split strings, Huang.
--
Best Regards,
Artem Bityutskiy
^ permalink raw reply [flat|nested] 20+ messages in thread* Re: [PATCH v4 05/12] mtd: nand: print out the cell information for nand chip
2013-10-03 14:34 ` Artem Bityutskiy
@ 2013-10-03 15:53 ` Brian Norris
0 siblings, 0 replies; 20+ messages in thread
From: Brian Norris @ 2013-10-03 15:53 UTC (permalink / raw)
To: dedekind1; +Cc: Huang Shijie, dwmw2, linux-mtd
On 10/03/2013 07:34 AM, Artem Bityutskiy wrote:
> On Mon, 2013-09-30 at 15:49 -0700, Brian Norris wrote:
>> On Wed, Sep 25, 2013 at 02:58:14PM +0800, Huang Shijie wrote:
>>> Print out the cell information for nand chip.
>>>
>>> (Since the message is too long, this patch also splits the log
>>> with two separate pr_info())
>>>
>>> Signed-off-by: Huang Shijie <b32955@freescale.com>
>>> ---
>>> drivers/mtd/nand/nand_base.c | 11 +++++++----
>>> 1 files changed, 7 insertions(+), 4 deletions(-)
>>>
>>> diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c
>>> index 94d9084..b94309b 100644
>>> --- a/drivers/mtd/nand/nand_base.c
>>> +++ b/drivers/mtd/nand/nand_base.c
>>> @@ -3460,11 +3460,14 @@ ident_done:
>>> if (mtd->writesize > 512 && chip->cmdfunc == nand_command)
>>> chip->cmdfunc = nand_command_lp;
>>>
>>> - pr_info("NAND device: Manufacturer ID: 0x%02x, Chip ID: 0x%02x (%s %s),"
>>> - " %dMiB, page size: %d, OOB size: %d\n",
>>> + pr_info("NAND device: Manufacturer ID: 0x%02x,"
>>> + "Chip ID: 0x%02x (%s %s) \n",
>>
>> You're adding excess whitespace before the '\n'. I'll squash this
>> myself.
>
> Nowadays' fashion is to never split strings, Huang.
checkpatch.pl actually warns about this now, so I already fixed it but
didn't remember to mention it. Also, in splitting this string, Huang
removed a space (fixed as well).
Brian
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH v4 06/12] mtd: gpmi: rewrite the gpmi_ecc_write_oob() to support the jffs2
2013-09-25 6:58 [PATCH v4 00/12] About the SLC/MLC Huang Shijie
` (4 preceding siblings ...)
2013-09-25 6:58 ` [PATCH v4 05/12] mtd: nand: print out the cell information for nand chip Huang Shijie
@ 2013-09-25 6:58 ` Huang Shijie
2013-09-30 22:47 ` Brian Norris
2013-09-25 6:58 ` [PATCH v4 07/12] mtd: nand: add more comment for MTD_NANDFLASH/MTD_MLCNANDFLASH Huang Shijie
` (6 subsequent siblings)
12 siblings, 1 reply; 20+ messages in thread
From: Huang Shijie @ 2013-09-25 6:58 UTC (permalink / raw)
To: dwmw2; +Cc: Huang Shijie, computersforpeace, linux-mtd, dedekind1
When we use the ECC info which is get from the nand chip's datasheet,
we may have some freed oob area now.
This patch rewrites the gpmi_ecc_write_oob() to implement the ecc.write_oob().
We also update the comment for gpmi_hw_ecclayout.
Yes! We can support the JFFS2 for the SLC nand now.
Signed-off-by: Huang Shijie <b32955@freescale.com>
---
drivers/mtd/nand/gpmi-nand/gpmi-nand.c | 31 ++++++++++++++++++++++---------
1 files changed, 22 insertions(+), 9 deletions(-)
diff --git a/drivers/mtd/nand/gpmi-nand/gpmi-nand.c b/drivers/mtd/nand/gpmi-nand/gpmi-nand.c
index b4306bc..eba7b79 100644
--- a/drivers/mtd/nand/gpmi-nand/gpmi-nand.c
+++ b/drivers/mtd/nand/gpmi-nand/gpmi-nand.c
@@ -45,7 +45,10 @@ static struct nand_bbt_descr gpmi_bbt_descr = {
.pattern = scan_ff_pattern
};
-/* We will use all the (page + OOB). */
+/*
+ * We may change the layout if we can get the ECC info from the datasheet,
+ * else we will use all the (page + OOB).
+ */
static struct nand_ecclayout gpmi_hw_ecclayout = {
.eccbytes = 0,
.eccpos = { 0, },
@@ -1262,14 +1265,24 @@ static int gpmi_ecc_read_oob(struct mtd_info *mtd, struct nand_chip *chip,
static int
gpmi_ecc_write_oob(struct mtd_info *mtd, struct nand_chip *chip, int page)
{
- /*
- * The BCH will use all the (page + oob).
- * Our gpmi_hw_ecclayout can only prohibit the JFFS2 to write the oob.
- * But it can not stop some ioctls such MEMWRITEOOB which uses
- * MTD_OPS_PLACE_OOB. So We have to implement this function to prohibit
- * these ioctls too.
- */
- return -EPERM;
+ struct nand_oobfree *of = mtd->ecclayout->oobfree;
+ int status = 0;
+
+ /* Do we have available oob area? */
+ if (!of->length)
+ return -EPERM;
+
+ if (!nand_is_slc(chip))
+ return -EPERM;
+
+ pr_debug("page number is %d\n", page);
+
+ chip->cmdfunc(mtd, NAND_CMD_SEQIN, mtd->writesize + of->offset, page);
+ chip->write_buf(mtd, chip->oob_poi + of->offset, of->length);
+ chip->cmdfunc(mtd, NAND_CMD_PAGEPROG, -1, -1);
+
+ status = chip->waitfunc(mtd, chip);
+ return status & NAND_STATUS_FAIL ? -EIO : 0;
}
static int gpmi_block_markbad(struct mtd_info *mtd, loff_t ofs)
--
1.7.1
^ permalink raw reply related [flat|nested] 20+ messages in thread* Re: [PATCH v4 06/12] mtd: gpmi: rewrite the gpmi_ecc_write_oob() to support the jffs2
2013-09-25 6:58 ` [PATCH v4 06/12] mtd: gpmi: rewrite the gpmi_ecc_write_oob() to support the jffs2 Huang Shijie
@ 2013-09-30 22:47 ` Brian Norris
0 siblings, 0 replies; 20+ messages in thread
From: Brian Norris @ 2013-09-30 22:47 UTC (permalink / raw)
To: Huang Shijie; +Cc: linux-mtd, dwmw2, dedekind1
On Wed, Sep 25, 2013 at 02:58:15PM +0800, Huang Shijie wrote:
> When we use the ECC info which is get from the nand chip's datasheet,
> we may have some freed oob area now.
>
> This patch rewrites the gpmi_ecc_write_oob() to implement the ecc.write_oob().
> We also update the comment for gpmi_hw_ecclayout.
>
> Yes! We can support the JFFS2 for the SLC nand now.
>
> Signed-off-by: Huang Shijie <b32955@freescale.com>
> ---
> drivers/mtd/nand/gpmi-nand/gpmi-nand.c | 31 ++++++++++++++++++++++---------
> 1 files changed, 22 insertions(+), 9 deletions(-)
>
> diff --git a/drivers/mtd/nand/gpmi-nand/gpmi-nand.c b/drivers/mtd/nand/gpmi-nand/gpmi-nand.c
> index b4306bc..eba7b79 100644
> --- a/drivers/mtd/nand/gpmi-nand/gpmi-nand.c
> +++ b/drivers/mtd/nand/gpmi-nand/gpmi-nand.c
> @@ -45,7 +45,10 @@ static struct nand_bbt_descr gpmi_bbt_descr = {
> .pattern = scan_ff_pattern
> };
>
> -/* We will use all the (page + OOB). */
> +/*
> + * We may change the layout if we can get the ECC info from the datasheet,
> + * else we will use all the (page + OOB).
> + */
> static struct nand_ecclayout gpmi_hw_ecclayout = {
> .eccbytes = 0,
> .eccpos = { 0, },
> @@ -1262,14 +1265,24 @@ static int gpmi_ecc_read_oob(struct mtd_info *mtd, struct nand_chip *chip,
> static int
> gpmi_ecc_write_oob(struct mtd_info *mtd, struct nand_chip *chip, int page)
> {
> - /*
> - * The BCH will use all the (page + oob).
> - * Our gpmi_hw_ecclayout can only prohibit the JFFS2 to write the oob.
> - * But it can not stop some ioctls such MEMWRITEOOB which uses
> - * MTD_OPS_PLACE_OOB. So We have to implement this function to prohibit
> - * these ioctls too.
> - */
> - return -EPERM;
> + struct nand_oobfree *of = mtd->ecclayout->oobfree;
> + int status = 0;
> +
> + /* Do we have available oob area? */
> + if (!of->length)
> + return -EPERM;
> +
> + if (!nand_is_slc(chip))
> + return -EPERM;
> +
> + pr_debug("page number is %d\n", page);
I feel like I commented on this earlier, but I guess I never actually
sent my comment? Anyway, I don't think we need this pr_debug(). That may
be useful for certain kinds of debugging, but it's really not useful at
all to most people. So I'll drop it when I push, or else if we have v5,
please remove it there.
> +
> + chip->cmdfunc(mtd, NAND_CMD_SEQIN, mtd->writesize + of->offset, page);
> + chip->write_buf(mtd, chip->oob_poi + of->offset, of->length);
> + chip->cmdfunc(mtd, NAND_CMD_PAGEPROG, -1, -1);
> +
> + status = chip->waitfunc(mtd, chip);
> + return status & NAND_STATUS_FAIL ? -EIO : 0;
> }
>
> static int gpmi_block_markbad(struct mtd_info *mtd, loff_t ofs)
Brian
^ permalink raw reply [flat|nested] 20+ messages in thread
* [PATCH v4 07/12] mtd: nand: add more comment for MTD_NANDFLASH/MTD_MLCNANDFLASH
2013-09-25 6:58 [PATCH v4 00/12] About the SLC/MLC Huang Shijie
` (5 preceding siblings ...)
2013-09-25 6:58 ` [PATCH v4 06/12] mtd: gpmi: rewrite the gpmi_ecc_write_oob() to support the jffs2 Huang Shijie
@ 2013-09-25 6:58 ` Huang Shijie
2013-09-25 6:58 ` [PATCH v4 08/12] mtd: nand: add a helper to detect the nand type Huang Shijie
` (5 subsequent siblings)
12 siblings, 0 replies; 20+ messages in thread
From: Huang Shijie @ 2013-09-25 6:58 UTC (permalink / raw)
To: dwmw2; +Cc: Huang Shijie, computersforpeace, linux-mtd, dedekind1
In current code, the MTD_NANDFLASH is used to represent both the SLC and
MLC. It is confusing to us.
By adding an explicit comment about these two macros, this patch makes it
clear that:
MTD_NANDFLASH : stands for SLC nand,
MTD_MLCNANDFLASH : stands for MLC nand(including TLC).
Signed-off-by: Huang Shijie <b32955@freescale.com>
---
include/uapi/mtd/mtd-abi.h | 4 ++--
1 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/include/uapi/mtd/mtd-abi.h b/include/uapi/mtd/mtd-abi.h
index 36eace0..16a9406 100644
--- a/include/uapi/mtd/mtd-abi.h
+++ b/include/uapi/mtd/mtd-abi.h
@@ -94,10 +94,10 @@ struct mtd_write_req {
#define MTD_RAM 1
#define MTD_ROM 2
#define MTD_NORFLASH 3
-#define MTD_NANDFLASH 4
+#define MTD_NANDFLASH 4 /* stand for SLC nand */
#define MTD_DATAFLASH 6
#define MTD_UBIVOLUME 7
-#define MTD_MLCNANDFLASH 8
+#define MTD_MLCNANDFLASH 8 /* stand for MLC nand(including TLC) */
#define MTD_WRITEABLE 0x400 /* Device is writeable */
#define MTD_BIT_WRITEABLE 0x800 /* Single bits can be flipped */
--
1.7.1
^ permalink raw reply related [flat|nested] 20+ messages in thread* [PATCH v4 08/12] mtd: nand: add a helper to detect the nand type
2013-09-25 6:58 [PATCH v4 00/12] About the SLC/MLC Huang Shijie
` (6 preceding siblings ...)
2013-09-25 6:58 ` [PATCH v4 07/12] mtd: nand: add more comment for MTD_NANDFLASH/MTD_MLCNANDFLASH Huang Shijie
@ 2013-09-25 6:58 ` Huang Shijie
2013-09-25 6:58 ` [PATCH v4 09/12] mtd: mtd-abi: " Huang Shijie
` (4 subsequent siblings)
12 siblings, 0 replies; 20+ messages in thread
From: Huang Shijie @ 2013-09-25 6:58 UTC (permalink / raw)
To: dwmw2; +Cc: Huang Shijie, computersforpeace, linux-mtd, dedekind1
This helper detects that whether the mtd's type is nand type.
Now, it's clear that the MTD_NANDFLASH stands for SLC nand only.
So use the mtd_type_is_nand() to replace the old check method
to do the nand type (include the SLC and MLC) check.
Signed-off-by: Huang Shijie <b32955@freescale.com>
---
drivers/mtd/inftlcore.c | 2 +-
drivers/mtd/nftlcore.c | 2 +-
drivers/mtd/ssfdc.c | 2 +-
drivers/mtd/tests/nandbiterrs.c | 2 +-
drivers/mtd/tests/oobtest.c | 2 +-
drivers/mtd/tests/pagetest.c | 2 +-
drivers/mtd/tests/subpagetest.c | 2 +-
include/linux/mtd/mtd.h | 5 +++++
8 files changed, 12 insertions(+), 7 deletions(-)
diff --git a/drivers/mtd/inftlcore.c b/drivers/mtd/inftlcore.c
index 3af3514..b66b541 100644
--- a/drivers/mtd/inftlcore.c
+++ b/drivers/mtd/inftlcore.c
@@ -50,7 +50,7 @@ static void inftl_add_mtd(struct mtd_blktrans_ops *tr, struct mtd_info *mtd)
struct INFTLrecord *inftl;
unsigned long temp;
- if (mtd->type != MTD_NANDFLASH || mtd->size > UINT_MAX)
+ if (!mtd_type_is_nand(mtd) || mtd->size > UINT_MAX)
return;
/* OK, this is moderately ugly. But probably safe. Alternatives? */
if (memcmp(mtd->name, "DiskOnChip", 10))
diff --git a/drivers/mtd/nftlcore.c b/drivers/mtd/nftlcore.c
index c5f4ebf..46f27de 100644
--- a/drivers/mtd/nftlcore.c
+++ b/drivers/mtd/nftlcore.c
@@ -50,7 +50,7 @@ static void nftl_add_mtd(struct mtd_blktrans_ops *tr, struct mtd_info *mtd)
struct NFTLrecord *nftl;
unsigned long temp;
- if (mtd->type != MTD_NANDFLASH || mtd->size > UINT_MAX)
+ if (!mtd_type_is_nand(mtd) || mtd->size > UINT_MAX)
return;
/* OK, this is moderately ugly. But probably safe. Alternatives? */
if (memcmp(mtd->name, "DiskOnChip", 10))
diff --git a/drivers/mtd/ssfdc.c b/drivers/mtd/ssfdc.c
index ab2a52a..daf82ba 100644
--- a/drivers/mtd/ssfdc.c
+++ b/drivers/mtd/ssfdc.c
@@ -290,7 +290,7 @@ static void ssfdcr_add_mtd(struct mtd_blktrans_ops *tr, struct mtd_info *mtd)
int cis_sector;
/* Check for small page NAND flash */
- if (mtd->type != MTD_NANDFLASH || mtd->oobsize != OOB_SIZE ||
+ if (!mtd_type_is_nand(mtd) || mtd->oobsize != OOB_SIZE ||
mtd->size > UINT_MAX)
return;
diff --git a/drivers/mtd/tests/nandbiterrs.c b/drivers/mtd/tests/nandbiterrs.c
index 3cd3aab..6f97615 100644
--- a/drivers/mtd/tests/nandbiterrs.c
+++ b/drivers/mtd/tests/nandbiterrs.c
@@ -349,7 +349,7 @@ static int __init mtd_nandbiterrs_init(void)
goto exit_mtddev;
}
- if (mtd->type != MTD_NANDFLASH) {
+ if (!mtd_type_is_nand(mtd)) {
pr_info("this test requires NAND flash\n");
err = -ENODEV;
goto exit_nand;
diff --git a/drivers/mtd/tests/oobtest.c b/drivers/mtd/tests/oobtest.c
index ff35c46..2e9e2d1 100644
--- a/drivers/mtd/tests/oobtest.c
+++ b/drivers/mtd/tests/oobtest.c
@@ -289,7 +289,7 @@ static int __init mtd_oobtest_init(void)
return err;
}
- if (mtd->type != MTD_NANDFLASH) {
+ if (!mtd_type_is_nand(mtd)) {
pr_info("this test requires NAND flash\n");
goto out;
}
diff --git a/drivers/mtd/tests/pagetest.c b/drivers/mtd/tests/pagetest.c
index 44b96e9..ed2d3f6 100644
--- a/drivers/mtd/tests/pagetest.c
+++ b/drivers/mtd/tests/pagetest.c
@@ -353,7 +353,7 @@ static int __init mtd_pagetest_init(void)
return err;
}
- if (mtd->type != MTD_NANDFLASH) {
+ if (!mtd_type_is_nand(mtd)) {
pr_info("this test requires NAND flash\n");
goto out;
}
diff --git a/drivers/mtd/tests/subpagetest.c b/drivers/mtd/tests/subpagetest.c
index e2c0adf..a876371 100644
--- a/drivers/mtd/tests/subpagetest.c
+++ b/drivers/mtd/tests/subpagetest.c
@@ -299,7 +299,7 @@ static int __init mtd_subpagetest_init(void)
return err;
}
- if (mtd->type != MTD_NANDFLASH) {
+ if (!mtd_type_is_nand(mtd)) {
pr_info("this test requires NAND flash\n");
goto out;
}
diff --git a/include/linux/mtd/mtd.h b/include/linux/mtd/mtd.h
index f9bfe52..88409b8 100644
--- a/include/linux/mtd/mtd.h
+++ b/include/linux/mtd/mtd.h
@@ -354,6 +354,11 @@ static inline int mtd_has_oob(const struct mtd_info *mtd)
return mtd->_read_oob && mtd->_write_oob;
}
+static inline int mtd_type_is_nand(const struct mtd_info *mtd)
+{
+ return mtd->type == MTD_NANDFLASH || mtd->type == MTD_MLCNANDFLASH;
+}
+
static inline int mtd_can_have_bb(const struct mtd_info *mtd)
{
return !!mtd->_block_isbad;
--
1.7.1
^ permalink raw reply related [flat|nested] 20+ messages in thread* [PATCH v4 09/12] mtd: mtd-abi: add a helper to detect the nand type
2013-09-25 6:58 [PATCH v4 00/12] About the SLC/MLC Huang Shijie
` (7 preceding siblings ...)
2013-09-25 6:58 ` [PATCH v4 08/12] mtd: nand: add a helper to detect the nand type Huang Shijie
@ 2013-09-25 6:58 ` Huang Shijie
2013-09-25 6:58 ` [PATCH v4 10/12] mtd: add MTD_MLCNANDFLASH case for mtd_type_show() Huang Shijie
` (3 subsequent siblings)
12 siblings, 0 replies; 20+ messages in thread
From: Huang Shijie @ 2013-09-25 6:58 UTC (permalink / raw)
To: dwmw2; +Cc: Huang Shijie, computersforpeace, linux-mtd, dedekind1
The helper is for user applications, and it is just a copy of
the kernel helper: mtd_type_is_nand();
Signed-off-by: Huang Shijie <b32955@freescale.com>
---
include/uapi/mtd/mtd-abi.h | 5 +++++
1 files changed, 5 insertions(+), 0 deletions(-)
diff --git a/include/uapi/mtd/mtd-abi.h b/include/uapi/mtd/mtd-abi.h
index 16a9406..66c2b0c 100644
--- a/include/uapi/mtd/mtd-abi.h
+++ b/include/uapi/mtd/mtd-abi.h
@@ -275,4 +275,9 @@ enum mtd_file_modes {
MTD_FILE_MODE_RAW,
};
+static inline int mtd_type_is_nand_user(const struct mtd_info_user *mtd)
+{
+ return mtd->type == MTD_NANDFLASH || mtd->type == MTD_MLCNANDFLASH;
+}
+
#endif /* __MTD_ABI_H__ */
--
1.7.1
^ permalink raw reply related [flat|nested] 20+ messages in thread* [PATCH v4 10/12] mtd: add MTD_MLCNANDFLASH case for mtd_type_show()
2013-09-25 6:58 [PATCH v4 00/12] About the SLC/MLC Huang Shijie
` (8 preceding siblings ...)
2013-09-25 6:58 ` [PATCH v4 09/12] mtd: mtd-abi: " Huang Shijie
@ 2013-09-25 6:58 ` Huang Shijie
2013-09-25 6:58 ` [PATCH v4 11/12] jffs2: do not support the MLC nand Huang Shijie
` (2 subsequent siblings)
12 siblings, 0 replies; 20+ messages in thread
From: Huang Shijie @ 2013-09-25 6:58 UTC (permalink / raw)
To: dwmw2; +Cc: Huang Shijie, computersforpeace, linux-mtd, dedekind1
The current mtd_type_show() misses the MTD_MLCNANDFLASH case.
This patch adds the case for it, and also updates the ABI.
Signed-off-by: Huang Shijie <b32955@freescale.com>
---
Documentation/ABI/testing/sysfs-class-mtd | 2 +-
drivers/mtd/mtdcore.c | 3 +++
2 files changed, 4 insertions(+), 1 deletions(-)
diff --git a/Documentation/ABI/testing/sysfs-class-mtd b/Documentation/ABI/testing/sysfs-class-mtd
index bfd119a..1399bb2 100644
--- a/Documentation/ABI/testing/sysfs-class-mtd
+++ b/Documentation/ABI/testing/sysfs-class-mtd
@@ -104,7 +104,7 @@ Description:
One of the following ASCII strings, representing the device
type:
- absent, ram, rom, nor, nand, dataflash, ubi, unknown
+ absent, ram, rom, nor, nand, mlc-nand, dataflash, ubi, unknown
What: /sys/class/mtd/mtdX/writesize
Date: April 2009
diff --git a/drivers/mtd/mtdcore.c b/drivers/mtd/mtdcore.c
index 5e14d54..92311a5 100644
--- a/drivers/mtd/mtdcore.c
+++ b/drivers/mtd/mtdcore.c
@@ -157,6 +157,9 @@ static ssize_t mtd_type_show(struct device *dev,
case MTD_UBIVOLUME:
type = "ubi";
break;
+ case MTD_MLCNANDFLASH:
+ type = "mlc-nand";
+ break;
default:
type = "unknown";
}
--
1.7.1
^ permalink raw reply related [flat|nested] 20+ messages in thread* [PATCH v4 11/12] jffs2: do not support the MLC nand
2013-09-25 6:58 [PATCH v4 00/12] About the SLC/MLC Huang Shijie
` (9 preceding siblings ...)
2013-09-25 6:58 ` [PATCH v4 10/12] mtd: add MTD_MLCNANDFLASH case for mtd_type_show() Huang Shijie
@ 2013-09-25 6:58 ` Huang Shijie
2013-09-25 6:58 ` [PATCH v4 12/12] mtd: nand: fix the wrong mtd->type for nand chip Huang Shijie
2013-09-30 23:15 ` [PATCH v4 00/12] About the SLC/MLC Brian Norris
12 siblings, 0 replies; 20+ messages in thread
From: Huang Shijie @ 2013-09-25 6:58 UTC (permalink / raw)
To: dwmw2; +Cc: Huang Shijie, computersforpeace, linux-mtd, dedekind1
We should not support the MLC nand for jffs2. So if the nand type is
MLC, we quit immediatly.
Signed-off-by: Huang Shijie <b32955@freescale.com>
---
fs/jffs2/fs.c | 4 ++++
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/fs/jffs2/fs.c b/fs/jffs2/fs.c
index fe3c052..09b3ed4 100644
--- a/fs/jffs2/fs.c
+++ b/fs/jffs2/fs.c
@@ -515,6 +515,10 @@ int jffs2_do_fill_super(struct super_block *sb, void *data, int silent)
c = JFFS2_SB_INFO(sb);
+ /* Do not support the MLC nand */
+ if (c->mtd->type == MTD_MLCNANDFLASH)
+ return -EINVAL;
+
#ifndef CONFIG_JFFS2_FS_WRITEBUFFER
if (c->mtd->type == MTD_NANDFLASH) {
pr_err("Cannot operate on NAND flash unless jffs2 NAND support is compiled in\n");
--
1.7.1
^ permalink raw reply related [flat|nested] 20+ messages in thread* [PATCH v4 12/12] mtd: nand: fix the wrong mtd->type for nand chip
2013-09-25 6:58 [PATCH v4 00/12] About the SLC/MLC Huang Shijie
` (10 preceding siblings ...)
2013-09-25 6:58 ` [PATCH v4 11/12] jffs2: do not support the MLC nand Huang Shijie
@ 2013-09-25 6:58 ` Huang Shijie
2013-09-30 23:15 ` [PATCH v4 00/12] About the SLC/MLC Brian Norris
12 siblings, 0 replies; 20+ messages in thread
From: Huang Shijie @ 2013-09-25 6:58 UTC (permalink / raw)
To: dwmw2; +Cc: Huang Shijie, computersforpeace, linux-mtd, dedekind1
Current code sets the mtd->type with MTD_NANDFLASH for both
SLC and MLC. So the jffs2 may supports the MLC nand, but in actually,
the jffs2 should not support the MLC.
This patch uses the nand_is_slc() to check the nand cell type,
and set the mtd->type with the right nand type.
After this patch, the jffs2 only supports the SLC nand.
The side-effect of this patch:
Before this patch, the ioctl(MEMGETINFO) can only return with the
MTD_NANDFLASH; but after this patch, the ioctl(MEMGETINFO) will
return with the MTD_NANDFLASH for SLC, and MTD_MLCNANDFLASH for MLC.
So the user applictions(such as mtd-utils) should also changes a little
for this.
Signed-off-by: Huang Shijie <b32955@freescale.com>
---
drivers/mtd/nand/nand_base.c | 2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/drivers/mtd/nand/nand_base.c b/drivers/mtd/nand/nand_base.c
index b94309b..2e5b50f 100644
--- a/drivers/mtd/nand/nand_base.c
+++ b/drivers/mtd/nand/nand_base.c
@@ -3785,7 +3785,7 @@ int nand_scan_tail(struct mtd_info *mtd)
chip->options |= NAND_SUBPAGE_READ;
/* Fill in remaining MTD driver data */
- mtd->type = MTD_NANDFLASH;
+ mtd->type = nand_is_slc(chip) ? MTD_NANDFLASH : MTD_MLCNANDFLASH;
mtd->flags = (chip->options & NAND_ROM) ? MTD_CAP_ROM :
MTD_CAP_NANDFLASH;
mtd->_erase = nand_erase;
--
1.7.1
^ permalink raw reply related [flat|nested] 20+ messages in thread* Re: [PATCH v4 00/12] About the SLC/MLC
2013-09-25 6:58 [PATCH v4 00/12] About the SLC/MLC Huang Shijie
` (11 preceding siblings ...)
2013-09-25 6:58 ` [PATCH v4 12/12] mtd: nand: fix the wrong mtd->type for nand chip Huang Shijie
@ 2013-09-30 23:15 ` Brian Norris
2013-10-08 2:38 ` Huang Shijie
12 siblings, 1 reply; 20+ messages in thread
From: Brian Norris @ 2013-09-30 23:15 UTC (permalink / raw)
To: Huang Shijie; +Cc: linux-mtd, dwmw2, dedekind1
On Wed, Sep 25, 2013 at 02:58:09PM +0800, Huang Shijie wrote:
> In current mtd code, the MTD_NANDFLASH is used to represent both the
> SLC nand MLC(including the TLC). But we already have the MTD_MLCNANDFLASH
> to stand for the MLC. What is worse is that the JFFS2 may run on the MLC
> nand with current code. For the reason of READ/WRITE disturbance, the JFFS2
> should runs on the SLC only,
>
> This patch set tries to make clear what is the SLC/MLC by adding the macros,
> adding helpers, adding the comments. ..
>
> After this patch set, the gpmi can support the JFFS2 for some SLC NAND now
> (only when the left oob area is big enough).
...
> Huang Shijie (12):
> mtd: nand: add a helper to check the SLC/MLC nand chip
> mtd: nand: rename the cellinfo to bits_per_cell
> mtd: nand: add the "bits per cell" info for legacy ID NAND
> mtd: nand: set the cell information for ONFI nand
> mtd: nand: print out the cell information for nand chip
> mtd: gpmi: rewrite the gpmi_ecc_write_oob() to support the jffs2
> mtd: nand: add more comment for MTD_NANDFLASH/MTD_MLCNANDFLASH
> mtd: nand: add a helper to detect the nand type
> mtd: mtd-abi: add a helper to detect the nand type
> mtd: add MTD_MLCNANDFLASH case for mtd_type_show()
> jffs2: do not support the MLC nand
> mtd: nand: fix the wrong mtd->type for nand chip
Pushed the whole series to l2-mtd.git, with a few amendments of my own
to fix trivial issues.
Are the mtd-utils fixes still applicable as-is? I'll review them next,
since we should get mtd-utils updated before the kernel changes make it
to mainline.
Brian
^ permalink raw reply [flat|nested] 20+ messages in thread