Some cosmetic cleanups in the flashrom code and output.

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



git-svn-id: svn://svn.coreboot.org/coreboot/trunk@2873 2b7e53f0-3cfb-0310-b3e9-8179ed1497e1
This commit is contained in:
Uwe Hermann 2007-10-17 23:55:15 +00:00
parent 7dabe5ea0d
commit dcb9abdf4c
20 changed files with 48 additions and 49 deletions

View File

@ -172,7 +172,7 @@ int write_82802ab(struct flashchip *flash, uint8_t *buf)
printf("ERASE FAILED\n"); printf("ERASE FAILED\n");
return -1; return -1;
} }
printf("Programming Page: "); printf("Programming page: ");
for (i = 0; i < total_size / page_size; i++) { for (i = 0; i < total_size / page_size; i++) {
printf("%04d at address: 0x%08x", i, i * page_size); printf("%04d at address: 0x%08x", i, i * page_size);
write_page_82802ab(bios, buf + i * page_size, write_page_82802ab(bios, buf + i * page_size,

View File

@ -161,7 +161,7 @@ static int board_via_epia_m(const char *name)
dev = pci_dev_find(0x1106, 0x3177); /* VT8235 ISA bridge */ dev = pci_dev_find(0x1106, 0x3177); /* VT8235 ISA bridge */
if (!dev) { if (!dev) {
fprintf(stderr, "\nERROR: VT8235 ISA Bridge not found.\n"); fprintf(stderr, "\nERROR: VT8235 ISA bridge not found.\n");
return -1; return -1;
} }
@ -355,9 +355,9 @@ struct board_pciid_enable {
struct board_pciid_enable board_pciid_enables[] = { struct board_pciid_enable board_pciid_enables[] = {
{0x10de, 0x0360, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, {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, {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, {0x1022, 0x7468, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000,
"iwill", "dk8_htx", "IWILL DK8-HTX", w83627hf_gpio24_raise_2e}, "iwill", "dk8_htx", "IWILL DK8-HTX", w83627hf_gpio24_raise_2e},
{0x10de, 0x005e, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, {0x10de, 0x005e, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000, 0x0000,
@ -456,12 +456,12 @@ int board_flash_enable(char *vendor, char *part)
board = board_match_pci_card_ids(); board = board_match_pci_card_ids();
if (board) { if (board) {
printf("Found board \"%s\": Enabling flash write... ", printf("Found board \"%s\": enabling flash write... ",
board->name); board->name);
ret = board->enable(board->name); ret = board->enable(board->name);
if (ret) if (ret)
printf("Failed!\n"); printf("FAILED!\n");
else else
printf("OK.\n"); printf("OK.\n");
} }

View File

@ -499,12 +499,12 @@ int chipset_flash_enable(void)
} }
if (dev) { if (dev) {
printf("Found chipset \"%s\": Enabling flash write... ", printf("Found chipset \"%s\", enabling flash write... ",
enables[i].name); enables[i].name);
ret = enables[i].doit(dev, enables[i].name); ret = enables[i].doit(dev, enables[i].name);
if (ret) if (ret)
printf("Failed!\n"); printf("FAILED!\n");
else else
printf("OK.\n"); printf("OK.\n");
} }

View File

@ -143,7 +143,7 @@ struct flashchip *probe_flash(struct flashchip *flash)
flash->virtual_memory = bios; flash->virtual_memory = bios;
if (flash->probe(flash) == 1) { 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); flash->name, flash_baseaddr);
return flash; return flash;
} }
@ -161,7 +161,7 @@ int verify_flash(struct flashchip *flash, uint8_t *buf)
int total_size = flash->total_size * 1024; int total_size = flash->total_size * 1024;
volatile uint8_t *bios = flash->virtual_memory; volatile uint8_t *bios = flash->virtual_memory;
printf("Verifying flash "); printf("Verifying flash... ");
if (verbose) if (verbose)
printf("address: 0x00000000\b\b\b\b\b\b\b\b\b\b"); printf("address: 0x00000000\b\b\b\b\b\b\b\b\b\b");
@ -174,7 +174,7 @@ int verify_flash(struct flashchip *flash, uint8_t *buf)
if (verbose) { if (verbose) {
printf("0x%08x ", idx); printf("0x%08x ", idx);
} }
printf("- FAILED\n"); printf("FAILED!\n");
return 1; return 1;
} }
@ -184,7 +184,7 @@ int verify_flash(struct flashchip *flash, uint8_t *buf)
if (verbose) if (verbose)
printf("\b\b\b\b\b\b\b\b\b\b "); printf("\b\b\b\b\b\b\b\b\b\b ");
printf("- VERIFIED \n"); printf("VERIFIED. \n");
return 0; return 0;
} }
@ -369,7 +369,7 @@ int main(int argc, char *argv[])
exit(1); 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)) { if (!(read_it | write_it | verify_it | erase_it)) {
printf("No operations were specified.\n"); printf("No operations were specified.\n");

View File

@ -281,12 +281,12 @@ int write_jedec(struct flashchip *flash, uint8_t *buf)
// dumb check if erase was successful. // dumb check if erase was successful.
for (i = 0; i < total_size; i++) { for (i = 0; i < total_size; i++) {
if (bios[i] != (uint8_t) 0xff) { 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; return -1;
} }
} }
printf("Programming Page: "); printf("Programming page: ");
for (i = 0; i < total_size / page_size; i++) { for (i = 0; i < total_size / page_size; i++) {
printf("%04d at address: 0x%08x", i, i * page_size); printf("%04d at address: 0x%08x", i, i * page_size);
write_page_write_jedec(bios, buf + i * page_size, write_page_write_jedec(bios, buf + i * page_size,

View File

@ -47,14 +47,14 @@ int show_id(uint8_t *bios, int size)
} }
printf_debug("LinuxBIOS last image size " printf_debug("LinuxBIOS last image size "
"(not rom size) is %d bytes.\n", *walk); "(not ROM size) is %d bytes.\n", *walk);
walk--; walk--;
mainboard_part = strdup((const char *)(bios + size - *walk)); mainboard_part = strdup((const char *)(bios + size - *walk));
walk--; walk--;
mainboard_vendor = strdup((const char *)(bios + size - *walk)); mainboard_vendor = strdup((const char *)(bios + size - *walk));
printf_debug("MANUFACTURER: %s\n", mainboard_vendor); printf_debug("Manufacturer: %s\n", mainboard_vendor);
printf_debug("MAINBOARD ID: %s\n", mainboard_part); printf_debug("Mainboard ID: %s\n", mainboard_part);
/* /*
* If lb_vendor is not set, the linuxbios table was * If lb_vendor is not set, the linuxbios table was
@ -63,7 +63,7 @@ int show_id(uint8_t *bios, int size)
if (!lb_vendor || !lb_part) { if (!lb_vendor || !lb_part) {
printf("Note: If the following flash access fails, " 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; return 0;
} }
@ -104,7 +104,7 @@ int read_romlayout(char *name)
romlayout = fopen(name, "r"); romlayout = fopen(name, "r");
if (!romlayout) { if (!romlayout) {
fprintf(stderr, "ERROR: Could not open rom layout (%s).\n", fprintf(stderr, "ERROR: Could not open ROM layout (%s).\n",
name); name);
return -1; return -1;
} }
@ -154,8 +154,7 @@ int find_romentry(char *name)
return i; return i;
} }
} }
printf("not found.\n"); printf("not found.\n"); // Not found. Error.
// Not found. Error.
return -1; return -1;
} }

View File

@ -105,26 +105,26 @@ static struct lb_header *find_lb_table(void *base, unsigned long start,
printf_debug("Found canidate at: %08lx-%08lx\n", printf_debug("Found canidate at: %08lx-%08lx\n",
addr, addr + head->table_bytes); addr, addr + head->table_bytes);
if (head->header_bytes != sizeof(*head)) { 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); head->header_bytes);
continue; continue;
} }
if (count_lb_records(head) != head->table_entries) { 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); head->table_entries);
continue; continue;
} }
if (compute_checksum((uint8_t *) head, sizeof(*head)) != 0) { if (compute_checksum((uint8_t *) head, sizeof(*head)) != 0) {
fprintf(stderr, "bad header checksum\n"); fprintf(stderr, "Bad header checksum.\n");
continue; continue;
} }
if (compute_checksum(recs, head->table_bytes) if (compute_checksum(recs, head->table_bytes)
!= head->table_checksum) { != head->table_checksum) {
fprintf(stderr, "bad table checksum: %04x\n", fprintf(stderr, "Bad table checksum: %04x.\n",
head->table_checksum); head->table_checksum);
continue; continue;
} }
fprintf(stdout, "Found LinuxBIOS table at: %08lx\n", addr); fprintf(stdout, "Found LinuxBIOS table at 0x%08lx.\n", addr);
return head; return head;
}; };
@ -140,7 +140,7 @@ static void find_mainboard(struct lb_record *ptr, unsigned long addr)
rec = (struct lb_mainboard *)ptr; rec = (struct lb_mainboard *)ptr;
max_size = rec->size - sizeof(*rec); 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, max_size - rec->vendor_idx,
rec->strings + rec->vendor_idx, rec->strings + rec->vendor_idx,
max_size - rec->part_number_idx, max_size - rec->part_number_idx,
@ -151,7 +151,7 @@ static void find_mainboard(struct lb_record *ptr, unsigned long addr)
rec->strings + rec->part_number_idx); rec->strings + rec->part_number_idx);
if (lb_part) { 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 { } else {
lb_part = strdup(part); lb_part = strdup(part);
lb_vendor = strdup(vendor); lb_vendor = strdup(vendor);
@ -201,7 +201,7 @@ int linuxbios_init(void)
if (lb_table) { if (lb_table) {
unsigned long addr; unsigned long addr;
addr = ((char *)lb_table) - ((char *)low_1MB); 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); rec = (struct lb_record *)(((char *)lb_table) + lb_table->header_bytes);
last = (struct lb_record *)(((char *)rec) + lb_table->table_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", printf_debug("LinuxBIOS header(%d) checksum: %04x table(%d) checksum: %04x entries: %d\n",

View File

@ -124,7 +124,7 @@ int write_m29f400bt(struct flashchip *flash, uint8_t *buf)
volatile uint8_t *bios = flash->virtual_memory; volatile uint8_t *bios = flash->virtual_memory;
//erase_m29f400bt (flash); //erase_m29f400bt (flash);
printf("Programming Page:\n "); printf("Programming page:\n ");
/********************************* /*********************************
*Pages for M29F400BT: *Pages for M29F400BT:
* 16 0x7c000 0x7ffff TOP * 16 0x7c000 0x7ffff TOP
@ -175,7 +175,7 @@ int write_linuxbios_m29f400bt(struct flashchip *flash, uint8_t *buf)
{ {
volatile uint8_t *bios = flash->virtual_memory; volatile uint8_t *bios = flash->virtual_memory;
printf("Programming Page:\n "); printf("Programming page:\n ");
/********************************* /*********************************
*Pages for M29F400BT: *Pages for M29F400BT:
* 16 0x7c000 0x7ffff TOP * 16 0x7c000 0x7ffff TOP

View File

@ -169,10 +169,10 @@ int write_md2802(struct flashchip *flash, uint8_t *buf)
return (1); return (1);
erase_md2802(flash); erase_md2802(flash);
if (*bios != (uint8_t) 0xff) { if (*bios != (uint8_t) 0xff) {
printf("ERASE FAILED\n"); printf("ERASE FAILED!\n");
return -1; return -1;
} }
printf("Programming Page: "); printf("Programming page: ");
for (i = 0; i < total_size / page_size; i++) { for (i = 0; i < total_size / page_size; i++) {
printf("%04d at address: 0x%08x", i, i * page_size); printf("%04d at address: 0x%08x", i, i * page_size);
//write_page_md2802(bios, buf + i * page_size, bios + i * page_size, page_size); //write_page_md2802(bios, buf + i * page_size, bios + i * page_size, page_size);

View File

@ -88,7 +88,7 @@ int write_29f002(struct flashchip *flash, uint8_t *buf)
erase_29f002(flash); erase_29f002(flash);
//*bios = 0xF0; //*bios = 0xF0;
#if 1 #if 1
printf("Programming Page: "); printf("Programming page: ");
for (i = 0; i < total_size; i++) { for (i = 0; i < total_size; i++) {
/* write to the sector */ /* write to the sector */
if ((i & 0xfff) == 0) if ((i & 0xfff) == 0)

View File

@ -30,7 +30,7 @@ int write_49fl004(struct flashchip *flash, uint8_t *buf)
int page_size = flash->page_size; int page_size = flash->page_size;
volatile uint8_t *bios = flash->virtual_memory; volatile uint8_t *bios = flash->virtual_memory;
printf("Programming Page: "); printf("Programming page: ");
for (i = 0; i < total_size / page_size; i++) { for (i = 0; i < total_size / page_size; i++) {
if ((i >= exclude_start_page) && (i < exclude_end_page)) if ((i >= exclude_start_page) && (i < exclude_end_page))
continue; continue;

View File

@ -164,10 +164,10 @@ int write_lhf00l04(struct flashchip *flash, uint8_t *buf)
erase_lhf00l04(flash); erase_lhf00l04(flash);
if (*bios != 0xff) { if (*bios != 0xff) {
printf("ERASE FAILED\n"); printf("ERASE FAILED!\n");
return -1; return -1;
} }
printf("Programming Page: "); printf("Programming page: ");
for (i = 0; i < total_size / page_size; i++) { for (i = 0; i < total_size / page_size; i++) {
printf("%04d at address: 0x%08x", i, i * page_size); printf("%04d at address: 0x%08x", i, i * page_size);
write_page_lhf00l04(bios, buf + i * page_size, write_page_lhf00l04(bios, buf + i * page_size,

View File

@ -123,7 +123,7 @@ static int it8716f_spi_command(uint16_t port, unsigned char writecnt, unsigned c
busy = inb(port) & 0x80; busy = inb(port) & 0x80;
} while (busy); } while (busy);
if (readcnt > 3) { if (readcnt > 3) {
printf("%s called with unsupported readcnt %i\n", printf("%s called with unsupported readcnt %i.\n",
__FUNCTION__, readcnt); __FUNCTION__, readcnt);
return 1; return 1;
} }
@ -153,7 +153,7 @@ static int it8716f_spi_command(uint16_t port, unsigned char writecnt, unsigned c
writeenc = 0x3; writeenc = 0x3;
break; break;
default: default:
printf("%s called with unsupported writecnt %i\n", printf("%s called with unsupported writecnt %i.\n",
__FUNCTION__, writecnt); __FUNCTION__, writecnt);
return 1; return 1;
} }
@ -184,7 +184,7 @@ static int generic_spi_rdid(unsigned char *readarr)
if (generic_spi_command(JEDEC_RDID_OUTSIZE, JEDEC_RDID_INSIZE, cmd, readarr)) if (generic_spi_command(JEDEC_RDID_OUTSIZE, JEDEC_RDID_INSIZE, cmd, readarr))
return 1; 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; return 0;
} }

View File

@ -143,7 +143,7 @@ int write_28sf040(struct flashchip *flash, uint8_t *buf)
unprotect_28sf040(bios); unprotect_28sf040(bios);
printf("Programming Page: "); printf("Programming page: ");
for (i = 0; i < total_size / page_size; i++) { for (i = 0; i < total_size / page_size; i++) {
/* erase the page before programming */ /* erase the page before programming */
erase_sector_28sf040(bios, i * page_size); erase_sector_28sf040(bios, i * page_size);

View File

@ -46,7 +46,7 @@ int write_39sf020(struct flashchip *flash, uint8_t *buf)
erase_chip_jedec(flash); erase_chip_jedec(flash);
printf("Programming Page: "); printf("Programming page: ");
for (i = 0; i < total_size / page_size; i++) { for (i = 0; i < total_size / page_size; i++) {
/* write to the sector */ /* write to the sector */
printf("%04d at address: 0x%08x", i, i * page_size); printf("%04d at address: 0x%08x", i, i * page_size);

View File

@ -44,7 +44,7 @@ int write_49lf040(struct flashchip *flash, uint8_t *buf)
int page_size = flash->page_size; int page_size = flash->page_size;
volatile uint8_t *bios = flash->virtual_memory; volatile uint8_t *bios = flash->virtual_memory;
printf("Programming Page: "); printf("Programming page: ");
for (i = 0; i < total_size / page_size; i++) { for (i = 0; i < total_size / page_size; i++) {
/* erase the page before programming /* erase the page before programming
* Chip erase only works in parallel programming mode * Chip erase only works in parallel programming mode

View File

@ -170,7 +170,7 @@ int write_49lfxxxc(struct flashchip *flash, uint8_t *buf)
volatile uint8_t *bios = flash->virtual_memory; volatile uint8_t *bios = flash->virtual_memory;
write_lockbits_49lfxxxc(flash->virtual_registers, total_size, 0); write_lockbits_49lfxxxc(flash->virtual_registers, total_size, 0);
printf("Programming Page: "); printf("Programming page: ");
for (i = 0; i < total_size / page_size; i++) { for (i = 0; i < total_size / page_size; i++) {
/* erase the page before programming */ /* erase the page before programming */
erase_sector_49lfxxxc(bios, i * page_size); erase_sector_49lfxxxc(bios, i * page_size);

View File

@ -83,12 +83,12 @@ int write_sst_fwhub(struct flashchip *flash, uint8_t *buf)
// dumb check if erase was successful. // dumb check if erase was successful.
for (i = 0; i < total_size; i++) { for (i = 0; i < total_size; i++) {
if (bios[i] != 0xff) { if (bios[i] != 0xff) {
printf("ERASE FAILED\n"); printf("ERASE FAILED!\n");
return -1; return -1;
} }
} }
printf("Programming Page: "); printf("Programming page: ");
for (i = 0; i < total_size / page_size; i++) { for (i = 0; i < total_size / page_size; i++) {
printf("%04d at address: 0x%08x", i, i * page_size); printf("%04d at address: 0x%08x", i, i * page_size);
write_sector_jedec(bios, buf + i * page_size, write_sector_jedec(bios, buf + i * page_size,

View File

@ -56,5 +56,5 @@ void myusec_calibrate_delay()
micro = count / timeusec; micro = count / timeusec;
printf_debug("%ldM loops per second. ", (unsigned long)micro); printf_debug("%ldM loops per second. ", (unsigned long)micro);
printf("ok\n"); printf("OK.\n");
} }

View File

@ -30,7 +30,7 @@ int write_49f002(struct flashchip *flash, uint8_t *buf)
erase_chip_jedec(flash); erase_chip_jedec(flash);
printf("Programming Page: "); printf("Programming page: ");
for (i = 0; i < total_size / page_size; i++) { for (i = 0; i < total_size / page_size; i++) {
/* write to the sector */ /* write to the sector */
printf("%04d at address: 0x%08x ", i, i * page_size); printf("%04d at address: 0x%08x ", i, i * page_size);