mtd: chips: add SST38VF640x nor-flashes

Submitted by zhangyd on Oct. 22, 2010, 2:53 a.m.

Details

Message ID 1287716022-2792-1-git-send-email-zhangyd6@gmail.com
State New, archived
Headers show

Commit Message

zhangyd Oct. 22, 2010, 2:53 a.m.
Due to broken CFI, these flashes should be operated in jedec probe mode.
When you operate them in the CFI probe mode, you will get the wrong sector
size.And it will fail to set up the flash during the probe(CFI mode).
And i have tested it on my platform.

Signed-off-by: yidong zhang <zhangyd6@gmail.com>
---
 drivers/mtd/chips/jedec_probe.c |   13 +++++++++++++
 1 files changed, 13 insertions(+), 0 deletions(-)

Comments

Wolfram Sang Oct. 21, 2010, 5:10 p.m.
On Thu, Oct 21, 2010 at 10:53:42PM -0400, zhangyd wrote:
> Due to broken CFI, these flashes should be operated in jedec probe mode.
> When you operate them in the CFI probe mode, you will get the wrong sector
> size.And it will fail to set up the flash during the probe(CFI mode).
> And i have tested it on my platform.
> 
> Signed-off-by: yidong zhang <zhangyd6@gmail.com>

Still NACK. Unless Guillaume doesn't come to the conclusion that JEDEC is
better, supporting him is the preferred option IMO.

Patch hide | download patch | download mbox

diff --git a/drivers/mtd/chips/jedec_probe.c b/drivers/mtd/chips/jedec_probe.c
index d72a5fb..21f07da 100644
--- a/drivers/mtd/chips/jedec_probe.c
+++ b/drivers/mtd/chips/jedec_probe.c
@@ -163,6 +163,7 @@ 
 #define SST49LF030A	0x001C
 #define SST49LF040A	0x0051
 #define SST49LF080A	0x005B
+#define SST38VF6401	0x536B
 #define SST36VF3203	0x7354
 
 /* Toshiba */
@@ -1557,6 +1558,18 @@  static const struct amd_flash_info jedec_table[] = {
 		}
 	}, {
 		.mfr_id		= CFI_MFR_SST,
+		.dev_id		= SST38VF6401,
+		.name		= "SST 38VF6401",
+		.devtypes	= CFI_DEVICETYPE_X16,
+		.uaddr		= MTD_UADDR_0x0AAA_0x0555,
+		.dev_size	= SIZE_8MiB,
+		.cmd_set	= P_ID_AMD_STD,
+		.nr_regions	= 1,
+		.regions	= {
+			ERASEINFO(0x10000,128),
+		}
+	}, {
+		.mfr_id		= CFI_MFR_SST,
 		.dev_id		= SST36VF3203,
 		.name		= "SST 36VF3203",
 		.devtypes	= CFI_DEVICETYPE_X16|CFI_DEVICETYPE_X8,