Patchwork mtd: chips: add SST38VF640x nor-flashes

login
register
mail settings
Submitter zhangyd
Date Oct. 22, 2010, 2:53 a.m.
Message ID <1287716022-2792-1-git-send-email-zhangyd6@gmail.com>
Download mbox | patch
Permalink /patch/68622/
State New
Headers show

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.
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(-)

Patch

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,