mtd: spinand: Add initial support for Toshiba TC58CVG2S0F

RFC In review Clément Péron 2 participants 5 messages Nov 27, 2018

Subsystems (2)

Memory technology devices (MTD) NAND flash

lore

Share & tools

lore

Clément Péron

author

2018-11-27 13:59 UTC #0 lore raw

RFC 1/1 mtd: spinand: Add initial support for Toshiba TC58CVG2S0F

Add support for Toshiba chip TC58CVG2S0F

Signed-off-by Clément Péron <[email protected]>

---

Hi,

Thanks a lot for adding this framework to Linux it's very nice and easy to read.

This is my first SPI Nand/Nand board so I'm still unfamiliar with those BBM and OOB features.

In the ECC status maybe I should return the Maximum Bit Flip Count (page 34 of the DS) like it's done for Macronix.

Datasheet could be found here: https://media.digikey.com/pdf/Data%20Sheets/Toshiba%20PDFs/TC58CVG2S0HxAIx_Rev1.1_2016-11-08.pdf

Also this has not been fully tested and untested with a QuadSPI controller.

Thanks,
Clement

 drivers/mtd/nand/spi/Makefile  |   2 +-
 drivers/mtd/nand/spi/core.c    |   1 +
 drivers/mtd/nand/spi/toshiba.c | 112 +++++++++++++++++++++++++++++++++
 include/linux/mtd/spinand.h    |   1 +
 4 files changed, 115 insertions(+), 1 deletion(-)
 create mode 100644 drivers/mtd/nand/spi/toshiba.c

drivers/mtd/nand/spi/Makefile +1 -1

index b74e074b363a..be5f73512ece 100644
@@ -1,3 +1,3 @@
 # SPDX-License-Identifier: GPL-2.0
-spinand-objs := core.o macronix.o micron.o winbond.o
+spinand-objs := core.o macronix.o micron.o toshiba.o winbond.o
 obj-$(CONFIG_MTD_SPI_NAND) += spinand.o

drivers/mtd/nand/spi/core.c +1 -0

index 30f83649c481..87bdf2a7b724 100644
@@ -766,6 +766,7 @@ static const struct nand_ops spinand_ops = {
 static const struct spinand_manufacturer *spinand_manufacturers[] = {
 	&macronix_spinand_manufacturer,
 	&micron_spinand_manufacturer,
+	&toshiba_spinand_manufacturer,
 	&winbond_spinand_manufacturer,
 };

drivers/mtd/nand/spi/toshiba.c new file +112 -0

112 lines not on this page · raw message

include/linux/mtd/spinand.h +1 -0

index 088ff96c3eb6..816c4b00abca 100644
@@ -196,6 +196,7 @@ struct spinand_manufacturer {
 /* SPI NAND manufacturers */
 extern const struct spinand_manufacturer macronix_spinand_manufacturer;
 extern const struct spinand_manufacturer micron_spinand_manufacturer;
+extern const struct spinand_manufacturer toshiba_spinand_manufacturer;
 extern const struct spinand_manufacturer winbond_spinand_manufacturer;
 
 /**

Signature · 1 line

2.19.1

Schrempf Frieder

2018-11-27 14:48 UTC #1 lore raw

Hello Clément,

Quoted 9 lines · Clément Péron

Add support for Toshiba chip TC58CVG2S0F
Signed-off-by Clément Péron <[email protected]>
---
Hi,
Thanks a lot for adding this framework to Linux it's very nice and easy to read.

Quoted 8 lines

This is my first SPI Nand/Nand board so I'm still unfamiliar with those BBM and OOB features.
In the ECC status maybe I should return the Maximum Bit Flip Count (page 34 of the DS) like it's done for Macronix.
Datasheet could be found here: https://media.digikey.com/pdf/Data%20Sheets/Toshiba%20PDFs/TC58CVG2S0HxAIx_Rev1.1_2016-11-08.pdf
Also this has not been fully tested and untested with a QuadSPI controller.

I'm afraid there is a big overlap between your patch and what I posted some days ago. See here: Patchwork [1] or linux-next [2].

Please have a look at my patch and test it with your hardware. You seem to use the same chip as I am (except for the last letter in the part number, whose meaning I don't know).

Thanks,
Frieder

[1] https://patchwork.ozlabs.org/patch/994687/
[2]
https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/drivers/mtd/nand/spi?h=next-20181127&id=f85fd9a073f7a0f063d046c07f0558c03b1d90de

Quoted 164 lines

Thanks,
Clement

  drivers/mtd/nand/spi/Makefile  |   2 +-
  drivers/mtd/nand/spi/core.c    |   1 +
  drivers/mtd/nand/spi/toshiba.c | 112 +++++++++++++++++++++++++++++++++
  include/linux/mtd/spinand.h    |   1 +
  4 files changed, 115 insertions(+), 1 deletion(-)

create mode 100644 drivers/mtd/nand/spi/toshiba.c

drivers/mtd/nand/spi/Makefile +1 -1

index b74e074b363a..be5f73512ece 100644
@@ -1,3 +1,3 @@
  # SPDX-License-Identifier: GPL-2.0
-spinand-objs := core.o macronix.o micron.o winbond.o
+spinand-objs := core.o macronix.o micron.o toshiba.o winbond.o
  obj-$(CONFIG_MTD_SPI_NAND) += spinand.o

drivers/mtd/nand/spi/core.c +1 -0

index 30f83649c481..87bdf2a7b724 100644
@@ -766,6 +766,7 @@ static const struct nand_ops spinand_ops = {
  static const struct spinand_manufacturer *spinand_manufacturers[] = {
  	&macronix_spinand_manufacturer,
  	&micron_spinand_manufacturer,
+	&toshiba_spinand_manufacturer,
  	&winbond_spinand_manufacturer,
  };

drivers/mtd/nand/spi/toshiba.c new file +112 -0

new file mode 100644
index 000000000000..6ff65949ddd0
@@ -0,0 +1,112 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * Author: Clément Péron <[email protected]>
+ */
+
+#include <linux/device.h>
+#include <linux/kernel.h>
+#include <linux/mtd/spinand.h>
+
+#define SPINAND_MFR_TOSHIBA		0x98
+
+static SPINAND_OP_VARIANTS(read_cache_variants,
+		SPINAND_PAGE_READ_FROM_CACHE_X4_OP(0, 1, NULL, 0),
+		SPINAND_PAGE_READ_FROM_CACHE_X2_OP(0, 1, NULL, 0),
+		SPINAND_PAGE_READ_FROM_CACHE_OP(true, 0, 1, NULL, 0),
+		SPINAND_PAGE_READ_FROM_CACHE_OP(false, 0, 1, NULL, 0));
+
+static SPINAND_OP_VARIANTS(write_cache_variants,
+		SPINAND_PROG_LOAD(true, 0, NULL, 0));
+
+static SPINAND_OP_VARIANTS(update_cache_variants,
+		SPINAND_PROG_LOAD(false, 0, NULL, 0));
+
+static int tc58_ooblayout_ecc(struct mtd_info *mtd, int section,
+			      struct mtd_oob_region *region)
+{
+	/* ECC not accesible when Internal ECC is turned on. */
+	return -ERANGE;
+}
+
+static int tc58_ooblayout_free(struct mtd_info *mtd, int section,
+			       struct mtd_oob_region *region)
+{
+	if (section)
+		return -ERANGE;
+
+	/* Reserve 2 bytes for the BBM. */
+	region->offset = 2;
+	region->length = 126;
+
+	return 0;
+}
+
+static const struct mtd_ooblayout_ops tc58_ooblayout = {
+	.ecc = tc58_ooblayout_ecc,
+	.free = tc58_ooblayout_free,
+};
+
+static int tc58_ecc_get_status(struct spinand_device *spinand,
+			       u8 status)
+{
+	switch (status & STATUS_ECC_MASK) {
+	case STATUS_ECC_NO_BITFLIPS:
+		return 0;
+
+	case STATUS_ECC_HAS_BITFLIPS:
+		return 4;
+
+	case STATUS_ECC_UNCOR_ERROR:
+		return -EBADMSG;
+
+	default:
+		return 8;
+	}
+
+	return -EINVAL;
+}
+
+static const struct spinand_info toshiba_spinand_table[] = {
+	SPINAND_INFO("TC58CVG2S0F", 0xCD,
+		     NAND_MEMORG(1, SZ_4K, 256, 64, 2048, 1, 1, 1),
+		     NAND_ECCREQ(8, 512),
+		     SPINAND_INFO_OP_VARIANTS(&read_cache_variants,
+					      &write_cache_variants,
+					      &update_cache_variants),
+		     0,
+		     SPINAND_ECCINFO(&tc58_ooblayout, tc58_ecc_get_status)),
+};
+/**
+ * toshiba_spinand_detect - initialize device related part in spinand_device
+ * struct if it is a Toshiba device.
+ * @spinand: SPI NAND device structure
+ */
+static int toshiba_spinand_detect(struct spinand_device *spinand)
+{
+	u8 *id = spinand->id.data;
+	int ret;
+
+	/*
+	 * Toshiba SPI NAND read ID need a dummy byte,
+	 * so the first byte in raw_id is dummy.
+	 */
+	if (id[1] != SPINAND_MFR_TOSHIBA)
+		return 0;
+
+	ret = spinand_match_and_init(spinand, toshiba_spinand_table,
+				     ARRAY_SIZE(toshiba_spinand_table), id[2]);
+	if (ret)
+		return ret;
+
+	return 1;
+}
+
+static const struct spinand_manufacturer_ops toshiba_spinand_manuf_ops = {
+	.detect = toshiba_spinand_detect,
+};
+
+const struct spinand_manufacturer toshiba_spinand_manufacturer = {
+	.id = SPINAND_MFR_TOSHIBA,
+	.name = "Toshiba",
+	.ops = &toshiba_spinand_manuf_ops,
+};

include/linux/mtd/spinand.h +1 -0

index 088ff96c3eb6..816c4b00abca 100644
@@ -196,6 +196,7 @@ struct spinand_manufacturer {
  /* SPI NAND manufacturers */
  extern const struct spinand_manufacturer macronix_spinand_manufacturer;
  extern const struct spinand_manufacturer micron_spinand_manufacturer;
+extern const struct spinand_manufacturer toshiba_spinand_manufacturer;
  extern const struct spinand_manufacturer winbond_spinand_manufacturer;
 
  /**

Clément Péron

author

2018-11-27 15:00 UTC #2 lore raw

Hi Frieder,

Quoted 20 lines · Schrempf Frieder

Hello Clément,
Quoted 16 lines · Clément Péron
Add support for Toshiba chip TC58CVG2S0F
Signed-off-by Clément Péron <[email protected]>
---
Hi,
Thanks a lot for adding this framework to Linux it's very nice and easy to read.
This is my first SPI Nand/Nand board so I'm still unfamiliar with those BBM and OOB features.
In the ECC status maybe I should return the Maximum Bit Flip Count (page 34 of the DS) like it's done for Macronix.
Datasheet could be found here: https://media.digikey.com/pdf/Data%20Sheets/Toshiba%20PDFs/TC58CVG2S0HxAIx_Rev1.1_2016-11-08.pdf

Quoted 8 lines

Quoted 1 line
Also this has not been fully tested and untested with a QuadSPI controller.
I'm afraid there is a big overlap between your patch and what I posted some days ago. See here: Patchwork [1] or linux-next [2].
Please have a look at my patch and test it with your hardware. You seem to use the same chip as I am (except for the last letter in the part number, whose meaning I don't know).

Thanks for pointing that,

I'm not a linux-mtd subscriber could you reply and add my e-mail to your patch ?

Thanks
Clement

Quoted 172 lines

Thanks,
Frieder

Quoted 164 lines

Thanks,
Clement

  drivers/mtd/nand/spi/Makefile  |   2 +-
  drivers/mtd/nand/spi/core.c    |   1 +
  drivers/mtd/nand/spi/toshiba.c | 112 +++++++++++++++++++++++++++++++++
  include/linux/mtd/spinand.h    |   1 +
  4 files changed, 115 insertions(+), 1 deletion(-)

create mode 100644 drivers/mtd/nand/spi/toshiba.c

drivers/mtd/nand/spi/Makefile +1 -1

index b74e074b363a..be5f73512ece 100644
@@ -1,3 +1,3 @@
  # SPDX-License-Identifier: GPL-2.0
-spinand-objs := core.o macronix.o micron.o winbond.o
+spinand-objs := core.o macronix.o micron.o toshiba.o winbond.o
  obj-$(CONFIG_MTD_SPI_NAND) += spinand.o

drivers/mtd/nand/spi/core.c +1 -0

index 30f83649c481..87bdf2a7b724 100644
@@ -766,6 +766,7 @@ static const struct nand_ops spinand_ops = {
  static const struct spinand_manufacturer *spinand_manufacturers[] = {
      &macronix_spinand_manufacturer,
      &micron_spinand_manufacturer,
+     &toshiba_spinand_manufacturer,
      &winbond_spinand_manufacturer,
  };

drivers/mtd/nand/spi/toshiba.c new file +112 -0

new file mode 100644
index 000000000000..6ff65949ddd0
@@ -0,0 +1,112 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * Author: Clément Péron <[email protected]>
+ */
+
+#include <linux/device.h>
+#include <linux/kernel.h>
+#include <linux/mtd/spinand.h>
+
+#define SPINAND_MFR_TOSHIBA          0x98
+
+static SPINAND_OP_VARIANTS(read_cache_variants,
+             SPINAND_PAGE_READ_FROM_CACHE_X4_OP(0, 1, NULL, 0),
+             SPINAND_PAGE_READ_FROM_CACHE_X2_OP(0, 1, NULL, 0),
+             SPINAND_PAGE_READ_FROM_CACHE_OP(true, 0, 1, NULL, 0),
+             SPINAND_PAGE_READ_FROM_CACHE_OP(false, 0, 1, NULL, 0));
+
+static SPINAND_OP_VARIANTS(write_cache_variants,
+             SPINAND_PROG_LOAD(true, 0, NULL, 0));
+
+static SPINAND_OP_VARIANTS(update_cache_variants,
+             SPINAND_PROG_LOAD(false, 0, NULL, 0));
+
+static int tc58_ooblayout_ecc(struct mtd_info *mtd, int section,
+                           struct mtd_oob_region *region)
+{
+     /* ECC not accesible when Internal ECC is turned on. */
+     return -ERANGE;
+}
+
+static int tc58_ooblayout_free(struct mtd_info *mtd, int section,
+                            struct mtd_oob_region *region)
+{
+     if (section)
+             return -ERANGE;
+
+     /* Reserve 2 bytes for the BBM. */
+     region->offset = 2;
+     region->length = 126;
+
+     return 0;
+}
+
+static const struct mtd_ooblayout_ops tc58_ooblayout = {
+     .ecc = tc58_ooblayout_ecc,
+     .free = tc58_ooblayout_free,
+};
+
+static int tc58_ecc_get_status(struct spinand_device *spinand,
+                            u8 status)
+{
+     switch (status & STATUS_ECC_MASK) {
+     case STATUS_ECC_NO_BITFLIPS:
+             return 0;
+
+     case STATUS_ECC_HAS_BITFLIPS:
+             return 4;
+
+     case STATUS_ECC_UNCOR_ERROR:
+             return -EBADMSG;
+
+     default:
+             return 8;
+     }
+
+     return -EINVAL;
+}
+
+static const struct spinand_info toshiba_spinand_table[] = {
+     SPINAND_INFO("TC58CVG2S0F", 0xCD,
+                  NAND_MEMORG(1, SZ_4K, 256, 64, 2048, 1, 1, 1),
+                  NAND_ECCREQ(8, 512),
+                  SPINAND_INFO_OP_VARIANTS(&read_cache_variants,
+                                           &write_cache_variants,
+                                           &update_cache_variants),
+                  0,
+                  SPINAND_ECCINFO(&tc58_ooblayout, tc58_ecc_get_status)),
+};
+/**
+ * toshiba_spinand_detect - initialize device related part in spinand_device
+ * struct if it is a Toshiba device.
+ * @spinand: SPI NAND device structure
+ */
+static int toshiba_spinand_detect(struct spinand_device *spinand)
+{
+     u8 *id = spinand->id.data;
+     int ret;
+
+     /*
+      * Toshiba SPI NAND read ID need a dummy byte,
+      * so the first byte in raw_id is dummy.
+      */
+     if (id[1] != SPINAND_MFR_TOSHIBA)
+             return 0;
+
+     ret = spinand_match_and_init(spinand, toshiba_spinand_table,
+                                  ARRAY_SIZE(toshiba_spinand_table), id[2]);
+     if (ret)
+             return ret;
+
+     return 1;
+}
+
+static const struct spinand_manufacturer_ops toshiba_spinand_manuf_ops = {
+     .detect = toshiba_spinand_detect,
+};
+
+const struct spinand_manufacturer toshiba_spinand_manufacturer = {
+     .id = SPINAND_MFR_TOSHIBA,
+     .name = "Toshiba",
+     .ops = &toshiba_spinand_manuf_ops,
+};

include/linux/mtd/spinand.h +1 -0

index 088ff96c3eb6..816c4b00abca 100644
@@ -196,6 +196,7 @@ struct spinand_manufacturer {
  /* SPI NAND manufacturers */
  extern const struct spinand_manufacturer macronix_spinand_manufacturer;
  extern const struct spinand_manufacturer micron_spinand_manufacturer;
+extern const struct spinand_manufacturer toshiba_spinand_manufacturer;
  extern const struct spinand_manufacturer winbond_spinand_manufacturer;
 
  /**

Clément Péron

author

2018-11-27 15:08 UTC #3 lore raw

Quoted 30 lines · Clément Péron

Hi Frieder,
Quoted 25 lines · Schrempf Frieder
Hello Clément,
Quoted 9 lines · Clément Péron
Add support for Toshiba chip TC58CVG2S0F
Signed-off-by Clément Péron <[email protected]>
---
Hi,
Thanks a lot for adding this framework to Linux it's very nice and easy to read.
Quoted 8 lines
This is my first SPI Nand/Nand board so I'm still unfamiliar with those BBM and OOB features.
In the ECC status maybe I should return the Maximum Bit Flip Count (page 34 of the DS) like it's done for Macronix.
Datasheet could be found here: https://media.digikey.com/pdf/Data%20Sheets/Toshiba%20PDFs/TC58CVG2S0HxAIx_Rev1.1_2016-11-08.pdf
Also this has not been fully tested and untested with a QuadSPI controller.
I'm afraid there is a big overlap between your patch and what I posted some days ago. See here: Patchwork [1] or linux-next [2].

Quoted 7 lines

Quoted 3 lines
Please have a look at my patch and test it with your hardware. You seem to use the same chip as I am (except for the last letter in the part number, whose meaning I don't know).
Thanks for pointing that,
I'm not a linux-mtd subscriber could you reply and add my e-mail to your patch ?

Or maybe as it has already been merged maybe I should write a new e-mail with my remark no ?

Thanks,
Clement

Quoted 176 lines

Thanks
Clement

Quoted 172 lines

Thanks,
Frieder

Quoted 164 lines

Thanks,
Clement

  drivers/mtd/nand/spi/Makefile  |   2 +-
  drivers/mtd/nand/spi/core.c    |   1 +
  drivers/mtd/nand/spi/toshiba.c | 112 +++++++++++++++++++++++++++++++++
  include/linux/mtd/spinand.h    |   1 +
  4 files changed, 115 insertions(+), 1 deletion(-)

create mode 100644 drivers/mtd/nand/spi/toshiba.c

drivers/mtd/nand/spi/Makefile +1 -1

index b74e074b363a..be5f73512ece 100644
@@ -1,3 +1,3 @@
  # SPDX-License-Identifier: GPL-2.0
-spinand-objs := core.o macronix.o micron.o winbond.o
+spinand-objs := core.o macronix.o micron.o toshiba.o winbond.o
  obj-$(CONFIG_MTD_SPI_NAND) += spinand.o

drivers/mtd/nand/spi/core.c +1 -0

index 30f83649c481..87bdf2a7b724 100644
@@ -766,6 +766,7 @@ static const struct nand_ops spinand_ops = {
  static const struct spinand_manufacturer *spinand_manufacturers[] = {
      &macronix_spinand_manufacturer,
      &micron_spinand_manufacturer,
+     &toshiba_spinand_manufacturer,
      &winbond_spinand_manufacturer,
  };

drivers/mtd/nand/spi/toshiba.c new file +112 -0

new file mode 100644
index 000000000000..6ff65949ddd0
@@ -0,0 +1,112 @@
+// SPDX-License-Identifier: GPL-2.0
+/*
+ * Author: Clément Péron <[email protected]>
+ */
+
+#include <linux/device.h>
+#include <linux/kernel.h>
+#include <linux/mtd/spinand.h>
+
+#define SPINAND_MFR_TOSHIBA          0x98
+
+static SPINAND_OP_VARIANTS(read_cache_variants,
+             SPINAND_PAGE_READ_FROM_CACHE_X4_OP(0, 1, NULL, 0),
+             SPINAND_PAGE_READ_FROM_CACHE_X2_OP(0, 1, NULL, 0),
+             SPINAND_PAGE_READ_FROM_CACHE_OP(true, 0, 1, NULL, 0),
+             SPINAND_PAGE_READ_FROM_CACHE_OP(false, 0, 1, NULL, 0));
+
+static SPINAND_OP_VARIANTS(write_cache_variants,
+             SPINAND_PROG_LOAD(true, 0, NULL, 0));
+
+static SPINAND_OP_VARIANTS(update_cache_variants,
+             SPINAND_PROG_LOAD(false, 0, NULL, 0));
+
+static int tc58_ooblayout_ecc(struct mtd_info *mtd, int section,
+                           struct mtd_oob_region *region)
+{
+     /* ECC not accesible when Internal ECC is turned on. */
+     return -ERANGE;
+}
+
+static int tc58_ooblayout_free(struct mtd_info *mtd, int section,
+                            struct mtd_oob_region *region)
+{
+     if (section)
+             return -ERANGE;
+
+     /* Reserve 2 bytes for the BBM. */
+     region->offset = 2;
+     region->length = 126;
+
+     return 0;
+}
+
+static const struct mtd_ooblayout_ops tc58_ooblayout = {
+     .ecc = tc58_ooblayout_ecc,
+     .free = tc58_ooblayout_free,
+};
+
+static int tc58_ecc_get_status(struct spinand_device *spinand,
+                            u8 status)
+{
+     switch (status & STATUS_ECC_MASK) {
+     case STATUS_ECC_NO_BITFLIPS:
+             return 0;
+
+     case STATUS_ECC_HAS_BITFLIPS:
+             return 4;
+
+     case STATUS_ECC_UNCOR_ERROR:
+             return -EBADMSG;
+
+     default:
+             return 8;
+     }
+
+     return -EINVAL;
+}
+
+static const struct spinand_info toshiba_spinand_table[] = {
+     SPINAND_INFO("TC58CVG2S0F", 0xCD,
+                  NAND_MEMORG(1, SZ_4K, 256, 64, 2048, 1, 1, 1),
+                  NAND_ECCREQ(8, 512),
+                  SPINAND_INFO_OP_VARIANTS(&read_cache_variants,
+                                           &write_cache_variants,
+                                           &update_cache_variants),
+                  0,
+                  SPINAND_ECCINFO(&tc58_ooblayout, tc58_ecc_get_status)),
+};
+/**
+ * toshiba_spinand_detect - initialize device related part in spinand_device
+ * struct if it is a Toshiba device.
+ * @spinand: SPI NAND device structure
+ */
+static int toshiba_spinand_detect(struct spinand_device *spinand)
+{
+     u8 *id = spinand->id.data;
+     int ret;
+
+     /*
+      * Toshiba SPI NAND read ID need a dummy byte,
+      * so the first byte in raw_id is dummy.
+      */
+     if (id[1] != SPINAND_MFR_TOSHIBA)
+             return 0;
+
+     ret = spinand_match_and_init(spinand, toshiba_spinand_table,
+                                  ARRAY_SIZE(toshiba_spinand_table), id[2]);
+     if (ret)
+             return ret;
+
+     return 1;
+}
+
+static const struct spinand_manufacturer_ops toshiba_spinand_manuf_ops = {
+     .detect = toshiba_spinand_detect,
+};
+
+const struct spinand_manufacturer toshiba_spinand_manufacturer = {
+     .id = SPINAND_MFR_TOSHIBA,
+     .name = "Toshiba",
+     .ops = &toshiba_spinand_manuf_ops,
+};

include/linux/mtd/spinand.h +1 -0

index 088ff96c3eb6..816c4b00abca 100644
@@ -196,6 +196,7 @@ struct spinand_manufacturer {
  /* SPI NAND manufacturers */
  extern const struct spinand_manufacturer macronix_spinand_manufacturer;
  extern const struct spinand_manufacturer micron_spinand_manufacturer;
+extern const struct spinand_manufacturer toshiba_spinand_manufacturer;
  extern const struct spinand_manufacturer winbond_spinand_manufacturer;
 
  /**

Schrempf Frieder

2018-11-27 15:16 UTC #4 lore raw

Quoted 35 lines · Clément Péron

Quoted 34 lines · Clément Péron
Hi Frieder,
Quoted 28 lines · Schrempf Frieder
Hello Clément,
Quoted 9 lines · Clément Péron
Add support for Toshiba chip TC58CVG2S0F
Signed-off-by Clément Péron <[email protected]>
---
Hi,
Thanks a lot for adding this framework to Linux it's very nice and easy to read.
Quoted 8 lines
This is my first SPI Nand/Nand board so I'm still unfamiliar with those BBM and OOB features.
In the ECC status maybe I should return the Maximum Bit Flip Count (page 34 of the DS) like it's done for Macronix.
Datasheet could be found here: https://media.digikey.com/pdf/Data%20Sheets/Toshiba%20PDFs/TC58CVG2S0HxAIx_Rev1.1_2016-11-08.pdf
Also this has not been fully tested and untested with a QuadSPI controller.
I'm afraid there is a big overlap between your patch and what I posted some days ago. See here: Patchwork [1] or linux-next [2].
Please have a look at my patch and test it with your hardware. You seem to use the same chip as I am (except for the last letter in the part number, whose meaning I don't know).

Quoted 5 lines

Quoted 3 lines
Thanks for pointing that,
I'm not a linux-mtd subscriber could you reply and add my e-mail to your patch ?
Or maybe as it has already been merged maybe I should write a new e-mail with my remark no ?

Hm, I'm not sure. I have sent a copy of the original patch to you. I guess you can still reply with your comments and we will see how to deal with them.

If there are any issues, please let us know so we can apply fixes on top of the current patch.

Thanks,
Frieder