Patchwork Add support for AT49F040

login
register
about
Submitter Stefan Tauner
Date 2011-08-11 03:59:42
Message ID <1313035182-16538-1-git-send-email-stefan.tauner@student.tuwien.ac.at>
Download mbox | patch
Permalink /patch/3338/
State Accepted
Headers show

Comments

Stefan Tauner - 2011-08-11 03:59:42
Chip features an optional permanent boot block write protection.

Signed-off-by: Stefan Tauner <stefan.tauner@student.tuwien.ac.at>
Signed-off-by: David Borg <borg.db@gmail.com>

---
Uwe Hermann - 2012-04-15 13:16:27
On Thu, Aug 11, 2011 at 05:59:42AM +0200, Stefan Tauner wrote:
> Chip features an optional permanent boot block write protection.
> 
> Signed-off-by: Stefan Tauner <stefan.tauner@student.tuwien.ac.at>
> Signed-off-by: David Borg <borg.db@gmail.com>
> 
> ---
> diff of flashchips.c looks very odd, this is because i moved AT49F020 down.
>  - rebased
>  - voltages added
>  - FEATURE_ADDR_FULL flag removed
>  - printlock_at49f040 added (location dubios, but w39_idmode_readb
>    implements what is needed already (and is static...)
>  - AT49F020 does also feature a permanent lock (8 kB boot block), which uses
>    the same detection method...

Rebased (again) and committed in r1522, thanks.

I changed 'struct flashchip *flash' to 'struct flashctx *flash' so that
it compiles.


Uwe.

Patch

diff of flashchips.c looks very odd, this is because i moved AT49F020 down.
 - rebased
 - voltages added
 - FEATURE_ADDR_FULL flag removed
 - printlock_at49f040 added (location dubios, but w39_idmode_readb
   implements what is needed already (and is static...)
 - AT49F020 does also feature a permanent lock (8 kB boot block), which uses
   the same detection method...
---
 chipdrivers.h |    1 +
 flashchips.c  |   80 ++++++++++++++++++++++++++++++++++++++++++---------------
 flashchips.h  |    5 ++-
 w39.c         |    8 +++++
 4 files changed, 71 insertions(+), 23 deletions(-)

diff --git a/chipdrivers.h b/chipdrivers.h
index 92ddbea..4583ae6 100644
--- a/chipdrivers.h
+++ b/chipdrivers.h
@@ -136,6 +136,7 @@  int printlock_w39v080fa(struct flashchip *flash);
 int printlock_w39v080fa_dual(struct flashchip *flash);
 int unlock_w39v040fb(struct flashchip *flash);
 int unlock_w39v080fa(struct flashchip *flash);
+int printlock_at49f(struct flashchip *flash);
 
 /* w29ee011.c */
 int probe_w29ee011(struct flashchip *flash);
diff --git a/flashchips.c b/flashchips.c
index 1a5622f..3d260b9 100644
--- a/flashchips.c
+++ b/flashchips.c
@@ -2218,19 +2218,27 @@  const struct flashchip flashchips[] = {
 
 	{
 		.vendor		= "Atmel",
-		.name		= "AT49F020",
+		.name		= "AT49F002(N)",
 		.bustype	= BUS_PARALLEL,
 		.manufacture_id	= ATMEL_ID,
-		.model_id	= ATMEL_AT49F020,
+		.model_id	= ATMEL_AT49F002N,
 		.total_size	= 256,
 		.page_size	= 256,
 		.feature_bits	= FEATURE_EITHER_RESET,
-		.tested		= TEST_OK_PRE,
+		.tested		= TEST_UNTESTED,
 		.probe		= probe_jedec,
 		.probe_timing	= TIMING_ZERO,	/* Datasheet has no timing info specified */
 		.block_erasers	=
 		{
 			{
+				.eraseblocks = {
+					{16 * 1024, 1},
+					{8 * 1024, 2},
+					{96 * 1024, 1},
+					{128 * 1024, 1},
+				},
+				.block_erase = erase_sector_jedec,
+			}, {
 				.eraseblocks = { {256 * 1024, 1} },
 				.block_erase = erase_chip_block_jedec,
 			}
@@ -2242,24 +2250,24 @@  const struct flashchip flashchips[] = {
 
 	{
 		.vendor		= "Atmel",
-		.name		= "AT49F002(N)",
+		.name		= "AT49F002(N)T",
 		.bustype	= BUS_PARALLEL,
 		.manufacture_id	= ATMEL_ID,
-		.model_id	= ATMEL_AT49F002N,
+		.model_id	= ATMEL_AT49F002NT,
 		.total_size	= 256,
 		.page_size	= 256,
 		.feature_bits	= FEATURE_EITHER_RESET,
-		.tested		= TEST_UNTESTED,
+		.tested		= TEST_OK_PR,
 		.probe		= probe_jedec,
 		.probe_timing	= TIMING_ZERO,	/* Datasheet has no timing info specified */
 		.block_erasers	=
 		{
 			{
 				.eraseblocks = {
-					{16 * 1024, 1},
-					{8 * 1024, 2},
-					{96 * 1024, 1},
 					{128 * 1024, 1},
+					{96 * 1024, 1},
+					{8 * 1024, 2},
+					{16 * 1024, 1},
 				},
 				.block_erase = erase_sector_jedec,
 			}, {
@@ -2274,31 +2282,61 @@  const struct flashchip flashchips[] = {
 
 	{
 		.vendor		= "Atmel",
-		.name		= "AT49F002(N)T",
+		.name		= "AT49F020",
 		.bustype	= BUS_PARALLEL,
 		.manufacture_id	= ATMEL_ID,
-		.model_id	= ATMEL_AT49F002NT,
+		.model_id	= ATMEL_AT49F020,
 		.total_size	= 256,
 		.page_size	= 256,
 		.feature_bits	= FEATURE_EITHER_RESET,
-		.tested		= TEST_OK_PR,
+		.tested		= TEST_OK_PRE,
 		.probe		= probe_jedec,
 		.probe_timing	= TIMING_ZERO,	/* Datasheet has no timing info specified */
 		.block_erasers	=
 		{
 			{
-				.eraseblocks = {
-					{128 * 1024, 1},
-					{96 * 1024, 1},
-					{8 * 1024, 2},
-					{16 * 1024, 1},
-				},
-				.block_erase = erase_sector_jedec,
-			}, {
 				.eraseblocks = { {256 * 1024, 1} },
 				.block_erase = erase_chip_block_jedec,
 			}
+			/* Chip features an optional permanent write protection
+			 * of the first 8 kB. The erase function is the same as
+			 * above, but 00000H to 01FFFH will not be erased.
+			 * FIXME: add another eraser when partial erasers are
+			 * supported.
+			 */
+		},
+		.printlock	= printlock_at49f,
+		.write		= write_jedec_1,
+		.read		= read_memmapped,
+		.voltage	= {4500, 5500},
+	},
+
+	{
+		.vendor		= "Atmel",
+		.name		= "AT49F040",
+		.bustype	= BUS_PARALLEL,
+		.manufacture_id	= ATMEL_ID,
+		.model_id	= ATMEL_AT49F040,
+		.total_size	= 512,
+		.page_size	= 512,
+		.feature_bits	= FEATURE_EITHER_RESET,
+		.tested		= TEST_UNTESTED,
+		.probe		= probe_jedec,
+		.probe_timing	= TIMING_ZERO,  /* Datasheet has no timing info specified */
+		.block_erasers	=
+		{
+			{
+				.eraseblocks = { {512 * 1024, 1} },
+				.block_erase = erase_chip_block_jedec,
+			}
+			/* Chip features an optional permanent write protection
+			 * of the first 16 kB. The erase function is the same as
+			 * above, but 00000H to 03FFFH will not be erased.
+			 * FIXME: add another eraser when partial erasers are
+			 * supported.
+			 */
 		},
+		.printlock	= printlock_at49f,
 		.write		= write_jedec_1,
 		.read		= read_memmapped,
 		.voltage	= {4500, 5500},
diff --git a/flashchips.h b/flashchips.h
index ff49d31..c9685ac 100644
--- a/flashchips.h
+++ b/flashchips.h
@@ -181,9 +181,10 @@ 
 #define ATMEL_AT45DB642		/* No ID available */
 #define ATMEL_AT45DB642D	0x2800
 #define ATMEL_AT49BV512		0x03
-#define ATMEL_AT49F020		0x0B
 #define ATMEL_AT49F002N		0x07	/* for AT49F002(N)  */
-#define ATMEL_AT49F002NT		0x08	/* for AT49F002(N)T */
+#define ATMEL_AT49F002NT	0x08	/* for AT49F002(N)T */
+#define ATMEL_AT49F020		0x0B
+#define ATMEL_AT49F040		0x13
 
 /* Bright Microelectronics has the same manufacturer ID as Hyundai... */
 #define BRIGHT_ID		0xAD	/* Bright Microelectronics */
diff --git a/w39.c b/w39.c
index 920a43f..8cc71e8 100644
--- a/w39.c
+++ b/w39.c
@@ -279,3 +279,11 @@  int unlock_w39v080fa(struct flashchip *flash)
 
 	return 0;
 }
+
+int printlock_at49f(struct flashchip *flash)
+{
+	uint8_t lock = w39_idmode_readb(flash, 0x00002);
+	msg_cdbg("Hardware bootblock lockout is %sactive.\n",
+		 (lock & 0x01) ? "" : "not ");
+	return 0;
+}