X-Git-Url: https://review.openocd.org/gitweb?a=blobdiff_plain;f=src%2Fflash%2Fnon_cfi.c;h=34acd2a40ccf74fcfe21a483ffd11b9319030729;hb=9262e0dbdfeb6a4c7fa27aa64cfca19379981d6c;hp=3e86e0da30ed80cd9d639d906eb81c7793f2b607;hpb=fbcb57baf842a84898dcc1cbe9a8b56d2a0028e1;p=openocd.git

diff --git a/src/flash/non_cfi.c b/src/flash/non_cfi.c
index 3e86e0da30..34acd2a40c 100644
--- a/src/flash/non_cfi.c
+++ b/src/flash/non_cfi.c
@@ -29,10 +29,10 @@
 
 #define KB 1024
 #define MB (1024*1024)
-#define ERASE_REGION(num, size) (((size/256)<<16)|(num-1))
+#define ERASE_REGION(num, size) (((size/256) << 16) | (num-1))
 
 /* non-CFI compatible flashes */
-non_cfi_t non_cfi_flashes[] = {
+static non_cfi_t non_cfi_flashes[] = {
 	{
 		.mfr = CFI_MFR_SST,
 		.id = 0xd4,
@@ -114,10 +114,10 @@ non_cfi_t non_cfi_flashes[] = {
 		.num_erase_regions = 4,
 		.erase_region_info =
 		{
-			ERASE_REGION( 1, 16*KB),
-			ERASE_REGION( 2,  8*KB),
-			ERASE_REGION( 1, 32*KB),
-			ERASE_REGION( 7, 64*KB)
+			ERASE_REGION(1, 16*KB),
+			ERASE_REGION(2,  8*KB),
+			ERASE_REGION(1, 32*KB),
+			ERASE_REGION(7, 64*KB)
 		}
 	},
 	{
@@ -131,16 +131,19 @@ non_cfi_t non_cfi_flashes[] = {
 		.num_erase_regions = 4,
 		.erase_region_info =
 		{
-			ERASE_REGION( 7, 64*KB),
-			ERASE_REGION( 1, 32*KB),
-			ERASE_REGION( 2,  8*KB),
-			ERASE_REGION( 1, 16*KB)
+			ERASE_REGION(7, 64*KB),
+			ERASE_REGION(1, 32*KB),
+			ERASE_REGION(2,  8*KB),
+			ERASE_REGION(1, 16*KB)
 		}
 	},
 
 	/* SST 39VF* do not support DQ5 status polling - this currently is
 	   only supported by the host algorithm, not by the target code using
-	   the work area. */
+	   the work area.
+           Only true for 8-bit and 32-bit wide memories. 16-bit wide memories
+           without DQ5 status polling are supported by the target code.
+        */
 	{
 		.mfr = CFI_MFR_SST,
 		.id = 0x2782,				/* SST39xF160 */
@@ -236,10 +239,10 @@ non_cfi_t non_cfi_flashes[] = {
 		.num_erase_regions = 4,
 		.erase_region_info =
 		{
-			ERASE_REGION( 1, 16*KB),
-			ERASE_REGION( 2,  8*KB),
-			ERASE_REGION( 1, 32*KB),
-			ERASE_REGION( 7, 64*KB)
+			ERASE_REGION(1, 16*KB),
+			ERASE_REGION(2,  8*KB),
+			ERASE_REGION(1, 32*KB),
+			ERASE_REGION(7, 64*KB)
 		}
 	},
 	{
@@ -253,10 +256,10 @@ non_cfi_t non_cfi_flashes[] = {
 		.num_erase_regions = 4,
 		.erase_region_info =
 		{
-			ERASE_REGION( 7, 64*KB),
-			ERASE_REGION( 1, 32*KB),
-			ERASE_REGION( 2,  8*KB),
-			ERASE_REGION( 1, 16*KB)
+			ERASE_REGION(7, 64*KB),
+			ERASE_REGION(1, 32*KB),
+			ERASE_REGION(2,  8*KB),
+			ERASE_REGION(1, 16*KB)
 		}
 	},
 	{
@@ -270,9 +273,9 @@ non_cfi_t non_cfi_flashes[] = {
 		.num_erase_regions = 4,
 		.erase_region_info =
 		{
-			ERASE_REGION( 1, 16*KB),
-			ERASE_REGION( 2,  8*KB),
-			ERASE_REGION( 1, 32*KB),
+			ERASE_REGION(1, 16*KB),
+			ERASE_REGION(2,  8*KB),
+			ERASE_REGION(1, 32*KB),
 			ERASE_REGION(15, 64*KB)
 		}
 	},
@@ -287,9 +290,9 @@ non_cfi_t non_cfi_flashes[] = {
 		.num_erase_regions = 4,
 		.erase_region_info =
 		{
-			ERASE_REGION( 1, 16*KB),
-			ERASE_REGION( 2,  8*KB),
-			ERASE_REGION( 1, 32*KB),
+			ERASE_REGION(1, 16*KB),
+			ERASE_REGION(2,  8*KB),
+			ERASE_REGION(1, 32*KB),
 			ERASE_REGION(15, 64*KB)
 		}
 	},
@@ -304,9 +307,9 @@ non_cfi_t non_cfi_flashes[] = {
 		.num_erase_regions = 4,
 		.erase_region_info =
 		{
-			ERASE_REGION( 1, 16*KB),
-			ERASE_REGION( 2, 8*KB),
-			ERASE_REGION( 1, 32*KB),
+			ERASE_REGION(1, 16*KB),
+			ERASE_REGION(2, 8*KB),
+			ERASE_REGION(1, 32*KB),
 			ERASE_REGION(15, 64*KB)
 		}
 	},
@@ -322,9 +325,9 @@ non_cfi_t non_cfi_flashes[] = {
 		.num_erase_regions = 4,
 		.erase_region_info =
 		{
-			ERASE_REGION( 1, 16*KB),
-			ERASE_REGION( 2, 8*KB),
-			ERASE_REGION( 1, 32*KB),
+			ERASE_REGION(1, 16*KB),
+			ERASE_REGION(2, 8*KB),
+			ERASE_REGION(1, 32*KB),
 			ERASE_REGION(31, 64*KB)
 		}
 	},
@@ -340,9 +343,9 @@ non_cfi_t non_cfi_flashes[] = {
 		.erase_region_info =
 		{
 			ERASE_REGION(31, 64*KB),
-			ERASE_REGION( 1, 32*KB),
-			ERASE_REGION( 2, 8*KB),
-			ERASE_REGION( 1, 16*KB)
+			ERASE_REGION(1, 32*KB),
+			ERASE_REGION(2, 8*KB),
+			ERASE_REGION(1, 16*KB)
 		}
 	},
 	{
@@ -356,8 +359,8 @@ non_cfi_t non_cfi_flashes[] = {
 		.num_erase_regions = 3,
 		.erase_region_info =
 		{
-			ERASE_REGION( 8,  8*KB),
-			ERASE_REGION( 2, 32*KB),
+			ERASE_REGION(8,  8*KB),
+			ERASE_REGION(2, 32*KB),
 			ERASE_REGION(30, 64*KB)
 		}
 	},
@@ -373,8 +376,8 @@ non_cfi_t non_cfi_flashes[] = {
 		.erase_region_info =
 		{
 			ERASE_REGION(30, 64*KB),
-			ERASE_REGION( 2, 32*KB),
-			ERASE_REGION( 8,  8*KB)
+			ERASE_REGION(2, 32*KB),
+			ERASE_REGION(8,  8*KB)
 		}
 	},
 	{
@@ -388,9 +391,9 @@ non_cfi_t non_cfi_flashes[] = {
 		.num_erase_regions = 4,
 		.erase_region_info =
 		{
-			ERASE_REGION( 1, 16*KB),
-			ERASE_REGION( 2, 8*KB),
-			ERASE_REGION( 1, 32*KB),
+			ERASE_REGION(1, 16*KB),
+			ERASE_REGION(2, 8*KB),
+			ERASE_REGION(1, 32*KB),
 			ERASE_REGION(15, 64*KB)
 		}
 	},