[coreboot] [flashrom] r599 - trunk

[svn at coreboot.org]

Author: uwe
Date: 2009-06-17 01:15:10 +0200 (Wed, 17 Jun 2009)
New Revision: 599

Modified:
   trunk/board_enable.c
   trunk/chipset_enable.c
   trunk/flashrom.c
Log:
List the size (in KB) and type of supported flash chips in 'flashrom -L'.

Also, list how many chips/chipsets/boards we support in 'flashrom -L'.

Signed-off-by: Uwe Hermann <uwe at hermann-uwe.de>
Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>



Modified: trunk/board_enable.c
===================================================================
--- trunk/board_enable.c	2009-06-16 21:08:06 UTC (rev 598)
+++ trunk/board_enable.c	2009-06-16 23:15:10 UTC (rev 599)
@@ -829,8 +829,11 @@
 
 void print_supported_boards_helper(const struct board_info *b)
 {
-	int i, j;
+	int i, j, boardcount = 0;
 
+	for (i = 0; b[i].vendor != NULL; i++)
+		boardcount++;
+
 	for (i = 0; b[i].vendor != NULL; i++) {
 		printf("%s", b[i].vendor);
 		for (j = 0; j < 25 - strlen(b[i].vendor); j++)
@@ -844,12 +847,16 @@
 
 void print_supported_boards(void)
 {
-	int i, j;
+	int i, j, boardcount = 0;
 	struct board_pciid_enable *b = board_pciid_enables;
 
-	printf("\nSupported boards which need write-enable code:\n\nVendor:   "
-	       "               Board:                   Required option:\n\n");
+	for (i = 0; b[i].vendor_name != NULL; i++)
+		boardcount++;
 
+	printf("\nSupported boards which need write-enable code (total: %d):"
+	       "\n\nVendor:                  Board:                   "
+	       "Required option:\n\n", boardcount);
+
 	for (i = 0; b[i].vendor_name != NULL; i++) {
 		printf("%s", b[i].vendor_name);
 		for (j = 0; j < 25 - strlen(b[i].vendor_name); j++)
@@ -863,10 +870,16 @@
 			printf("(none, board is autodetected)\n");
 	}
 
-	printf("\nSupported boards which don't need write-enable code:\n\n");
+	for (i = 0, boardcount = 0; boards_ok[i].vendor != NULL; i++)
+		boardcount++;
+	printf("\nSupported boards which don't need write-enable code "
+	       "(total: %d):\n\n", boardcount);
 	print_supported_boards_helper(boards_ok);
 
-	printf("\nBoards which have been verified to NOT work (yet):\n\n");
+	for (i = 0, boardcount = 0; boards_bad[i].vendor != NULL; i++)
+		boardcount++;
+	printf("\nBoards which have been verified to NOT work yet "
+	       "(total: %d):\n\n", boardcount);
 	print_supported_boards_helper(boards_bad);
 }
 

Modified: trunk/chipset_enable.c
===================================================================
--- trunk/chipset_enable.c	2009-06-16 21:08:06 UTC (rev 598)
+++ trunk/chipset_enable.c	2009-06-16 23:15:10 UTC (rev 599)
@@ -1037,12 +1037,15 @@
 
 void print_supported_chipsets(void)
 {
-	int i, j;
+	int i, j, chipsetcount = 0;
 	const struct penable *c = chipset_enables;
 
-	printf("\nSupported chipsets:\n\nVendor:                  Chipset:"
-	       "                 PCI IDs:\n\n");
+	for (i = 0; c[i].vendor_name != NULL; i++)
+		chipsetcount++;
 
+	printf("\nSupported chipsets (total: %d):\n\nVendor:                  "
+	       "Chipset:                 PCI IDs:\n\n", chipsetcount);
+
 	for (i = 0; c[i].vendor_name != NULL; i++) {
 		printf("%s", c[i].vendor_name);
 		for (j = 0; j < 25 - strlen(c[i].vendor_name); j++)

Modified: trunk/flashrom.c
===================================================================
--- trunk/flashrom.c	2009-06-16 21:08:06 UTC (rev 598)
+++ trunk/flashrom.c	2009-06-16 23:15:10 UTC (rev 599)
@@ -516,9 +516,22 @@
 #endif
 #define POS_PRINT(x) do { pos += strlen(x); printf(x); } while (0)
 
+static int digits(int n)
+{
+	int i;
+
+	if (!n)
+		return 1;
+
+	for (i = 0; n; ++i)
+		n /= 10;
+
+	return i;
+}
+
 void print_supported_chips(void)
 {
-	int okcol = 0, pos = 0, i;
+	int okcol = 0, pos = 0, i, chipcount = 0;
 	struct flashchip *f;
 
 	for (f = flashchips; f->name != NULL; f++) {
@@ -528,14 +541,19 @@
 	}
 	okcol = (okcol + 7) & ~7;
 
-	printf("Supported flash chips:\n\n");
+	for (f = flashchips; f->name != NULL; f++)
+		chipcount++;
+
+	printf("\nSupported flash chips (total: %d):\n\n", chipcount);
 	POS_PRINT("Vendor:   Device:");
 	while (pos < okcol) {
 		printf("\t");
 		pos += 8 - (pos % 8);
 	}
-	printf("Tested OK operations:\tKnown BAD operations:\n\n");
 
+	printf("Tested OK:\tKnown BAD:  Size/KB:  Type:\n\n");
+	printf("(P = PROBE, R = READ, E = ERASE, W = WRITE)\n\n");
+
 	for (f = flashchips; f->name != NULL; f++) {
 		/* Don't print "unknown XXXX SPI chip" entries. */
 		if (!strncmp(f->name, "unknown", 7))
@@ -553,29 +571,33 @@
 		}
 		if ((f->tested & TEST_OK_MASK)) {
 			if ((f->tested & TEST_OK_PROBE))
-				POS_PRINT("PROBE ");
+				POS_PRINT("P ");
 			if ((f->tested & TEST_OK_READ))
-				POS_PRINT("READ ");
+				POS_PRINT("R ");
 			if ((f->tested & TEST_OK_ERASE))
-				POS_PRINT("ERASE ");
+				POS_PRINT("E ");
 			if ((f->tested & TEST_OK_WRITE))
-				POS_PRINT("WRITE");
+				POS_PRINT("W ");
 		}
-		while (pos < okcol + 24) {
+		while (pos < okcol + 9) {
 			printf("\t");
 			pos += 8 - (pos % 8);
 		}
 		if ((f->tested & TEST_BAD_MASK)) {
 			if ((f->tested & TEST_BAD_PROBE))
-				printf("PROBE ");
+				printf("P ");
 			if ((f->tested & TEST_BAD_READ))
-				printf("READ ");
+				printf("R ");
 			if ((f->tested & TEST_BAD_ERASE))
-				printf("ERASE ");
+				printf("E ");
 			if ((f->tested & TEST_BAD_WRITE))
-				printf("WRITE");
+				printf("W ");
 		}
-		printf("\n");
+
+		printf("\t    %d", f->total_size);
+		for (i = 0; i < 10 - digits(f->total_size); i++)
+			printf(" ");
+		printf("%s\n", flashbuses_to_text(f->bustype));
 	}
 }