summaryrefslogtreecommitdiffstats
path: root/target/linux/generic-2.6/patches/050-mtdpart_redboot_byteswap_partition_truncate.patch
blob: 2d05792405e3c742a65cc4468feb792a049f944f (plain)
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
diff -urN linux-2.6.19.old/drivers/mtd/redboot.c linux-2.6.19.dev/drivers/mtd/redboot.c
--- linux-2.6.19.old/drivers/mtd/redboot.c	2006-11-29 22:57:37.000000000 +0100
+++ linux-2.6.19.dev/drivers/mtd/redboot.c	2006-12-14 03:13:35.000000000 +0100
@@ -92,22 +92,47 @@
 			 * swab32(erasesize) then we know we are looking at
 			 * a byte swapped FIS directory - swap all the entries!
 			 * (NOTE: this is 'size' not 'data_length'; size is
-			 * the full size of the entry.)
+			 * the full size of the entry.) 
+			 *
+			 * Handle cases where the FIS directory is less than
+			 * a full erase block (like combine FIS directory
+			 * and RedBoot config).
+			 *
+			 * IMHO the best solution would be to compute the
+			 * flash address of the RedBoot FIS directory and
+			 * compare that with the entry in the FIS directory
+			 * entry swabbed.  However, I haven't yet figured out
+			 * how to compute that.
 			 */
-			if (swab32(buf[i].size) == master->erasesize) {
+
+			unsigned long erasesize_mask = master->erasesize -1;
+			unsigned long eraseaddr_mask = 0xFFFFFFFF ^ erasesize_mask;
+			
+			if (((swab32(buf[i].size)+erasesize_mask) 
+				& eraseaddr_mask) == master->erasesize) {
 				int j;
-				for (j = 0; j < numslots && buf[j].name[0] != 0xff; ++j) {
+
+				/* N.B. The full table being processed so adjust size now */
+				numslots = swab32(buf[i].size) / sizeof (struct fis_image_desc);
+				for (j = 0; j < numslots; ++j) {
 					/* The unsigned long fields were written with the
 					 * wrong byte sex, name and pad have no byte sex.
+					 *
+					 * Only process non-deleted entries. Don't exit early.
 					 */
-					swab32s(&buf[j].flash_base);
-					swab32s(&buf[j].mem_base);
-					swab32s(&buf[j].size);
-					swab32s(&buf[j].entry_point);
-					swab32s(&buf[j].data_length);
-					swab32s(&buf[j].desc_cksum);
-					swab32s(&buf[j].file_cksum);
+					if (buf[j].name[0] != 0xff) {
+						swab32s(&buf[j].flash_base);
+						swab32s(&buf[j].mem_base);
+						swab32s(&buf[j].size);
+						swab32s(&buf[j].entry_point);
+						swab32s(&buf[j].data_length);
+						swab32s(&buf[j].desc_cksum);
+						swab32s(&buf[j].file_cksum);
+					}
 				}
+			} else {
+				/* Update numslots based on actual FIS directory size */
+				numslots = buf[i].size / sizeof (struct fis_image_desc);
 			}
 			break;
 		}