[LinuxBIOS] r2873 - trunk/util/flashrom

svn at openbios.org svn at openbios.org
Thu Oct 18 01:55:15 CEST 2007


Author: uwe
Date: 2007-10-18 01:55:15 +0200 (Thu, 18 Oct 2007)
New Revision: 2873

Modified:
   trunk/util/flashrom/82802ab.c
   trunk/util/flashrom/board_enable.c
   trunk/util/flashrom/chipset_enable.c
   trunk/util/flashrom/flashrom.c
   trunk/util/flashrom/jedec.c
   trunk/util/flashrom/layout.c
   trunk/util/flashrom/lbtable.c
   trunk/util/flashrom/m29f400bt.c
   trunk/util/flashrom/msys_doc.c
   trunk/util/flashrom/mx29f002.c
   trunk/util/flashrom/pm49fl004.c
   trunk/util/flashrom/sharplhf00l04.c
   trunk/util/flashrom/spi.c
   trunk/util/flashrom/sst28sf040.c
   trunk/util/flashrom/sst39sf020.c
   trunk/util/flashrom/sst49lf040.c
   trunk/util/flashrom/sst49lfxxxc.c
   trunk/util/flashrom/sst_fwhub.c
   trunk/util/flashrom/udelay.c
   trunk/util/flashrom/w49f002u.c
Log:
Some cosmetic cleanups in the flashrom code and output.

Signed-off-by: Uwe Hermann <uwe at hermann-uwe.de>
Acked-by: Stefan Reinauer <stepan at coresystems.de>



Modified: trunk/util/flashrom/82802ab.c
===================================================================
--- trunk/util/flashrom/82802ab.c	2007-10-17 23:43:59 UTC (rev 2872)
+++ trunk/util/flashrom/82802ab.c	2007-10-17 23:55:15 UTC (rev 2873)
@@ -172,7 +172,7 @@
 		printf("ERASE FAILED\n");
 		return -1;
 	}
-	printf("Programming Page: ");
+	printf("Programming page: ");
 	for (i = 0; i < total_size / page_size; i++) {
 		printf("%04d at address: 0x%08x", i, i * page_size);
 		write_page_82802ab(bios, buf + i * page_size,

Modified: trunk/util/flashrom/board_enable.c
===================================================================
--- trunk/util/flashrom/board_enable.c	2007-10-17 23:43:59 UTC (rev 2872)
+++ trunk/util/flashrom/board_enable.c	2007-10-17 23:55:15 UTC (rev 2873)
@@ -161,7 +161,7 @@
 
 	dev = pci_dev_find(0x1106, 0x3177);	/* VT8235 ISA bridge */
 	if (!dev) {
-		fprintf(stderr, "\nERROR: VT8235 ISA Bridge not found.\n");
+		fprintf(stderr, "\nERROR: VT8235 ISA bridge not found.\n");
 		return -1;
 	}
 
@@ -355,9 +355,9 @@
 
 struct board_pciid_enable board_pciid_enables[] = {
 	{0x10de, 0x0360, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000,
-	 "gigabyte", "m57sli", "GIGABYTE GA-M57SLI", it87xx_probe_spi_flash},
+	 "gigabyte", "m57sli", "GIGABYTE GA-M57SLI-S4", it87xx_probe_spi_flash},
 	{0x10de, 0x03e0, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000,
-	 "gigabyte", "m61ps3", "GIGABYTE GA-M61P-S3", it87xx_probe_spi_flash},
+	 "gigabyte", "m61p", "GIGABYTE GA-M61P-S3", it87xx_probe_spi_flash},
 	{0x1022, 0x7468, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000,
 	 "iwill", "dk8_htx", "IWILL DK8-HTX", w83627hf_gpio24_raise_2e},
 	{0x10de, 0x005e, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000,
@@ -456,12 +456,12 @@
 		board = board_match_pci_card_ids();
 
 	if (board) {
-		printf("Found board \"%s\": Enabling flash write... ",
+		printf("Found board \"%s\": enabling flash write... ",
 			board->name);
 
 		ret = board->enable(board->name);
 		if (ret)
-			printf("Failed!\n");
+			printf("FAILED!\n");
 		else
 			printf("OK.\n");
 	}

Modified: trunk/util/flashrom/chipset_enable.c
===================================================================
--- trunk/util/flashrom/chipset_enable.c	2007-10-17 23:43:59 UTC (rev 2872)
+++ trunk/util/flashrom/chipset_enable.c	2007-10-17 23:55:15 UTC (rev 2873)
@@ -499,12 +499,12 @@
 	}
 
 	if (dev) {
-		printf("Found chipset \"%s\": Enabling flash write... ",
+		printf("Found chipset \"%s\", enabling flash write... ",
 		       enables[i].name);
 
 		ret = enables[i].doit(dev, enables[i].name);
 		if (ret)
-			printf("Failed!\n");
+			printf("FAILED!\n");
 		else
 			printf("OK.\n");
 	}

Modified: trunk/util/flashrom/flashrom.c
===================================================================
--- trunk/util/flashrom/flashrom.c	2007-10-17 23:43:59 UTC (rev 2872)
+++ trunk/util/flashrom/flashrom.c	2007-10-17 23:55:15 UTC (rev 2873)
@@ -143,7 +143,7 @@
 		flash->virtual_memory = bios;
 
 		if (flash->probe(flash) == 1) {
-			printf("%s found at physical address: 0x%lx\n",
+			printf("%s found at physical address 0x%lx.\n",
 			       flash->name, flash_baseaddr);
 			return flash;
 		}
@@ -161,7 +161,7 @@
 	int total_size = flash->total_size * 1024;
 	volatile uint8_t *bios = flash->virtual_memory;
 
-	printf("Verifying flash ");
+	printf("Verifying flash... ");
 
 	if (verbose)
 		printf("address: 0x00000000\b\b\b\b\b\b\b\b\b\b");
@@ -174,7 +174,7 @@
 			if (verbose) {
 				printf("0x%08x ", idx);
 			}
-			printf("- FAILED\n");
+			printf("FAILED!\n");
 			return 1;
 		}
 
@@ -184,7 +184,7 @@
 	if (verbose)
 		printf("\b\b\b\b\b\b\b\b\b\b ");
 
-	printf("- VERIFIED         \n");
+	printf("VERIFIED.          \n");
 
 	return 0;
 }
@@ -369,7 +369,7 @@
 		exit(1);
 	}
 
-	printf("Flash part is %s (%d KB)\n", flash->name, flash->total_size);
+	printf("Flash part is %s (%d KB).\n", flash->name, flash->total_size);
 
 	if (!(read_it | write_it | verify_it | erase_it)) {
 		printf("No operations were specified.\n");

Modified: trunk/util/flashrom/jedec.c
===================================================================
--- trunk/util/flashrom/jedec.c	2007-10-17 23:43:59 UTC (rev 2872)
+++ trunk/util/flashrom/jedec.c	2007-10-17 23:55:15 UTC (rev 2873)
@@ -281,12 +281,12 @@
 	// dumb check if erase was successful.
 	for (i = 0; i < total_size; i++) {
 		if (bios[i] != (uint8_t) 0xff) {
-			printf("ERASE FAILED @%d, val %02x\n", i, bios[i]);
+			printf("ERASE FAILED @%d, val %02x!\n", i, bios[i]);
 			return -1;
 		}
 	}
 
-	printf("Programming Page: ");
+	printf("Programming page: ");
 	for (i = 0; i < total_size / page_size; i++) {
 		printf("%04d at address: 0x%08x", i, i * page_size);
 		write_page_write_jedec(bios, buf + i * page_size,

Modified: trunk/util/flashrom/layout.c
===================================================================
--- trunk/util/flashrom/layout.c	2007-10-17 23:43:59 UTC (rev 2872)
+++ trunk/util/flashrom/layout.c	2007-10-17 23:55:15 UTC (rev 2873)
@@ -47,14 +47,14 @@
 	}
 
 	printf_debug("LinuxBIOS last image size "
-		     "(not rom size) is %d bytes.\n", *walk);
+		     "(not ROM size) is %d bytes.\n", *walk);
 
 	walk--;
 	mainboard_part = strdup((const char *)(bios + size - *walk));
 	walk--;
 	mainboard_vendor = strdup((const char *)(bios + size - *walk));
-	printf_debug("MANUFACTURER: %s\n", mainboard_vendor);
-	printf_debug("MAINBOARD ID: %s\n", mainboard_part);
+	printf_debug("Manufacturer: %s\n", mainboard_vendor);
+	printf_debug("Mainboard ID: %s\n", mainboard_part);
 
 	/*
 	 * If lb_vendor is not set, the linuxbios table was
@@ -63,7 +63,7 @@
 
 	if (!lb_vendor || !lb_part) {
 		printf("Note: If the following flash access fails, "
-		       "you might need to specify -m <vendor>:<mainboard>\n");
+		       "you might need to specify -m <vendor>:<mainboard>.\n");
 		return 0;
 	}
 
@@ -104,7 +104,7 @@
 	romlayout = fopen(name, "r");
 
 	if (!romlayout) {
-		fprintf(stderr, "ERROR: Could not open rom layout (%s).\n",
+		fprintf(stderr, "ERROR: Could not open ROM layout (%s).\n",
 			name);
 		return -1;
 	}
@@ -154,8 +154,7 @@
 			return i;
 		}
 	}
-	printf("not found.\n");
-	// Not found. Error.
+	printf("not found.\n"); // Not found. Error.
 
 	return -1;
 }

Modified: trunk/util/flashrom/lbtable.c
===================================================================
--- trunk/util/flashrom/lbtable.c	2007-10-17 23:43:59 UTC (rev 2872)
+++ trunk/util/flashrom/lbtable.c	2007-10-17 23:55:15 UTC (rev 2873)
@@ -105,26 +105,26 @@
 		printf_debug("Found canidate at: %08lx-%08lx\n",
 			     addr, addr + head->table_bytes);
 		if (head->header_bytes != sizeof(*head)) {
-			fprintf(stderr, "Header bytes of %d are incorrect\n",
+			fprintf(stderr, "Header bytes of %d are incorrect.\n",
 				head->header_bytes);
 			continue;
 		}
 		if (count_lb_records(head) != head->table_entries) {
-			fprintf(stderr, "bad record count: %d\n",
+			fprintf(stderr, "Bad record count: %d.\n",
 				head->table_entries);
 			continue;
 		}
 		if (compute_checksum((uint8_t *) head, sizeof(*head)) != 0) {
-			fprintf(stderr, "bad header checksum\n");
+			fprintf(stderr, "Bad header checksum.\n");
 			continue;
 		}
 		if (compute_checksum(recs, head->table_bytes)
 		    != head->table_checksum) {
-			fprintf(stderr, "bad table checksum: %04x\n",
+			fprintf(stderr, "Bad table checksum: %04x.\n",
 				head->table_checksum);
 			continue;
 		}
-		fprintf(stdout, "Found LinuxBIOS table at: %08lx\n", addr);
+		fprintf(stdout, "Found LinuxBIOS table at 0x%08lx.\n", addr);
 		return head;
 
 	};
@@ -140,7 +140,7 @@
 
 	rec = (struct lb_mainboard *)ptr;
 	max_size = rec->size - sizeof(*rec);
-	printf("vendor id: %.*s part id: %.*s\n",
+	printf("Vendor ID: %.*s, part ID: %.*s\n",
 	       max_size - rec->vendor_idx,
 	       rec->strings + rec->vendor_idx,
 	       max_size - rec->part_number_idx,
@@ -151,7 +151,7 @@
 		 rec->strings + rec->part_number_idx);
 
 	if (lb_part) {
-		printf("overwritten by command line, vendor id: %s part id: %s\n", lb_vendor, lb_part);
+		printf("Overwritten by command line, vendor ID: %s, part ID: %s.\n", lb_vendor, lb_part);
 	} else {
 		lb_part = strdup(part);
 		lb_vendor = strdup(vendor);
@@ -201,7 +201,7 @@
 	if (lb_table) {
 		unsigned long addr;
 		addr = ((char *)lb_table) - ((char *)low_1MB);
-		printf_debug("lb_table found at address %p\n", lb_table);
+		printf_debug("LinuxBIOS table found at %p.\n", lb_table);
 		rec = (struct lb_record *)(((char *)lb_table) + lb_table->header_bytes);
 		last = (struct lb_record *)(((char *)rec) + lb_table->table_bytes);
 		printf_debug("LinuxBIOS header(%d) checksum: %04x table(%d) checksum: %04x entries: %d\n",

Modified: trunk/util/flashrom/m29f400bt.c
===================================================================
--- trunk/util/flashrom/m29f400bt.c	2007-10-17 23:43:59 UTC (rev 2872)
+++ trunk/util/flashrom/m29f400bt.c	2007-10-17 23:55:15 UTC (rev 2873)
@@ -124,7 +124,7 @@
 	volatile uint8_t *bios = flash->virtual_memory;
 
 	//erase_m29f400bt (flash);
-	printf("Programming Page:\n ");
+	printf("Programming page:\n ");
 	/*********************************
 	*Pages for M29F400BT:
 	* 16	0x7c000		0x7ffff		TOP
@@ -175,7 +175,7 @@
 {
 	volatile uint8_t *bios = flash->virtual_memory;
 
-	printf("Programming Page:\n ");
+	printf("Programming page:\n ");
 	/*********************************
 	*Pages for M29F400BT:
 	* 16	0x7c000		0x7ffff		TOP

Modified: trunk/util/flashrom/msys_doc.c
===================================================================
--- trunk/util/flashrom/msys_doc.c	2007-10-17 23:43:59 UTC (rev 2872)
+++ trunk/util/flashrom/msys_doc.c	2007-10-17 23:55:15 UTC (rev 2873)
@@ -169,10 +169,10 @@
 	return (1);
 	erase_md2802(flash);
 	if (*bios != (uint8_t) 0xff) {
-		printf("ERASE FAILED\n");
+		printf("ERASE FAILED!\n");
 		return -1;
 	}
-	printf("Programming Page: ");
+	printf("Programming page: ");
 	for (i = 0; i < total_size / page_size; i++) {
 		printf("%04d at address: 0x%08x", i, i * page_size);
 		//write_page_md2802(bios, buf + i * page_size, bios + i * page_size, page_size);

Modified: trunk/util/flashrom/mx29f002.c
===================================================================
--- trunk/util/flashrom/mx29f002.c	2007-10-17 23:43:59 UTC (rev 2872)
+++ trunk/util/flashrom/mx29f002.c	2007-10-17 23:55:15 UTC (rev 2873)
@@ -88,7 +88,7 @@
 	erase_29f002(flash);
 	//*bios = 0xF0;
 #if 1
-	printf("Programming Page: ");
+	printf("Programming page: ");
 	for (i = 0; i < total_size; i++) {
 		/* write to the sector */
 		if ((i & 0xfff) == 0)

Modified: trunk/util/flashrom/pm49fl004.c
===================================================================
--- trunk/util/flashrom/pm49fl004.c	2007-10-17 23:43:59 UTC (rev 2872)
+++ trunk/util/flashrom/pm49fl004.c	2007-10-17 23:55:15 UTC (rev 2873)
@@ -30,7 +30,7 @@
 	int page_size = flash->page_size;
 	volatile uint8_t *bios = flash->virtual_memory;
 
-	printf("Programming Page: ");
+	printf("Programming page: ");
 	for (i = 0; i < total_size / page_size; i++) {
 		if ((i >= exclude_start_page) && (i < exclude_end_page))
 			continue;

Modified: trunk/util/flashrom/sharplhf00l04.c
===================================================================
--- trunk/util/flashrom/sharplhf00l04.c	2007-10-17 23:43:59 UTC (rev 2872)
+++ trunk/util/flashrom/sharplhf00l04.c	2007-10-17 23:55:15 UTC (rev 2873)
@@ -164,10 +164,10 @@
 
 	erase_lhf00l04(flash);
 	if (*bios != 0xff) {
-		printf("ERASE FAILED\n");
+		printf("ERASE FAILED!\n");
 		return -1;
 	}
-	printf("Programming Page: ");
+	printf("Programming page: ");
 	for (i = 0; i < total_size / page_size; i++) {
 		printf("%04d at address: 0x%08x", i, i * page_size);
 		write_page_lhf00l04(bios, buf + i * page_size,

Modified: trunk/util/flashrom/spi.c
===================================================================
--- trunk/util/flashrom/spi.c	2007-10-17 23:43:59 UTC (rev 2872)
+++ trunk/util/flashrom/spi.c	2007-10-17 23:55:15 UTC (rev 2873)
@@ -123,7 +123,7 @@
 		busy = inb(port) & 0x80;
 	} while (busy);
 	if (readcnt > 3) {
-		printf("%s called with unsupported readcnt %i\n",
+		printf("%s called with unsupported readcnt %i.\n",
 			__FUNCTION__, readcnt);
 		return 1;
 	}
@@ -153,7 +153,7 @@
 		writeenc = 0x3;
 		break;
 	default:
-		printf("%s called with unsupported writecnt %i\n",
+		printf("%s called with unsupported writecnt %i.\n",
 			__FUNCTION__, writecnt);
 		return 1;
 	}
@@ -184,7 +184,7 @@
 
 	if (generic_spi_command(JEDEC_RDID_OUTSIZE, JEDEC_RDID_INSIZE, cmd, readarr))
 		return 1;
-	printf("RDID returned %02x %02x %02x\n", readarr[0], readarr[1], readarr[2]);
+	printf("RDID returned %02x %02x %02x.\n", readarr[0], readarr[1], readarr[2]);
 	return 0;
 }
 

Modified: trunk/util/flashrom/sst28sf040.c
===================================================================
--- trunk/util/flashrom/sst28sf040.c	2007-10-17 23:43:59 UTC (rev 2872)
+++ trunk/util/flashrom/sst28sf040.c	2007-10-17 23:55:15 UTC (rev 2873)
@@ -143,7 +143,7 @@
 
 	unprotect_28sf040(bios);
 
-	printf("Programming Page: ");
+	printf("Programming page: ");
 	for (i = 0; i < total_size / page_size; i++) {
 		/* erase the page before programming */
 		erase_sector_28sf040(bios, i * page_size);

Modified: trunk/util/flashrom/sst39sf020.c
===================================================================
--- trunk/util/flashrom/sst39sf020.c	2007-10-17 23:43:59 UTC (rev 2872)
+++ trunk/util/flashrom/sst39sf020.c	2007-10-17 23:55:15 UTC (rev 2873)
@@ -46,7 +46,7 @@
 
 	erase_chip_jedec(flash);
 
-	printf("Programming Page: ");
+	printf("Programming page: ");
 	for (i = 0; i < total_size / page_size; i++) {
 		/* write to the sector */
 		printf("%04d at address: 0x%08x", i, i * page_size);

Modified: trunk/util/flashrom/sst49lf040.c
===================================================================
--- trunk/util/flashrom/sst49lf040.c	2007-10-17 23:43:59 UTC (rev 2872)
+++ trunk/util/flashrom/sst49lf040.c	2007-10-17 23:55:15 UTC (rev 2873)
@@ -44,7 +44,7 @@
 	int page_size = flash->page_size;
 	volatile uint8_t *bios = flash->virtual_memory;
 
-	printf("Programming Page: ");
+	printf("Programming page: ");
 	for (i = 0; i < total_size / page_size; i++) {
 		/* erase the page before programming
 		 * Chip erase only works in parallel programming mode

Modified: trunk/util/flashrom/sst49lfxxxc.c
===================================================================
--- trunk/util/flashrom/sst49lfxxxc.c	2007-10-17 23:43:59 UTC (rev 2872)
+++ trunk/util/flashrom/sst49lfxxxc.c	2007-10-17 23:55:15 UTC (rev 2873)
@@ -170,7 +170,7 @@
 	volatile uint8_t *bios = flash->virtual_memory;
 
 	write_lockbits_49lfxxxc(flash->virtual_registers, total_size, 0);
-	printf("Programming Page: ");
+	printf("Programming page: ");
 	for (i = 0; i < total_size / page_size; i++) {
 		/* erase the page before programming */
 		erase_sector_49lfxxxc(bios, i * page_size);

Modified: trunk/util/flashrom/sst_fwhub.c
===================================================================
--- trunk/util/flashrom/sst_fwhub.c	2007-10-17 23:43:59 UTC (rev 2872)
+++ trunk/util/flashrom/sst_fwhub.c	2007-10-17 23:55:15 UTC (rev 2873)
@@ -83,12 +83,12 @@
 	// dumb check if erase was successful.
 	for (i = 0; i < total_size; i++) {
 		if (bios[i] != 0xff) {
-			printf("ERASE FAILED\n");
+			printf("ERASE FAILED!\n");
 			return -1;
 		}
 	}
 
-	printf("Programming Page: ");
+	printf("Programming page: ");
 	for (i = 0; i < total_size / page_size; i++) {
 		printf("%04d at address: 0x%08x", i, i * page_size);
 		write_sector_jedec(bios, buf + i * page_size,

Modified: trunk/util/flashrom/udelay.c
===================================================================
--- trunk/util/flashrom/udelay.c	2007-10-17 23:43:59 UTC (rev 2872)
+++ trunk/util/flashrom/udelay.c	2007-10-17 23:55:15 UTC (rev 2873)
@@ -56,5 +56,5 @@
 	micro = count / timeusec;
 
 	printf_debug("%ldM loops per second. ", (unsigned long)micro);
-	printf("ok\n");
+	printf("OK.\n");
 }

Modified: trunk/util/flashrom/w49f002u.c
===================================================================
--- trunk/util/flashrom/w49f002u.c	2007-10-17 23:43:59 UTC (rev 2872)
+++ trunk/util/flashrom/w49f002u.c	2007-10-17 23:55:15 UTC (rev 2873)
@@ -30,7 +30,7 @@
 
 	erase_chip_jedec(flash);
 
-	printf("Programming Page: ");
+	printf("Programming page: ");
 	for (i = 0; i < total_size / page_size; i++) {
 		/* write to the sector */
 		printf("%04d at address: 0x%08x ", i, i * page_size);





More information about the coreboot mailing list