Author: stefanct Date: Sat May 28 04:37:14 2011 New Revision: 1321 URL: http://flashrom.org/trac/flashrom/changeset/1321
Log: small fixes
- missing spaces in code and output - improved documentation/naming/output - missing line breaks in spi probing functions
Signed-off-by: Stefan Tauner stefan.tauner@student.tuwien.ac.at Acked-by: Stefan Tauner stefan.tauner@student.tuwien.ac.at
Modified: trunk/flash.h trunk/flashrom.c trunk/ichspi.c trunk/print.c trunk/spi25.c
Modified: trunk/flash.h ============================================================================== --- trunk/flash.h Thu May 26 16:28:51 2011 (r1320) +++ trunk/flash.h Sat May 28 04:37:14 2011 (r1321) @@ -136,6 +136,8 @@ unsigned int size; /* Eraseblock size */ unsigned int count; /* Number of contiguous blocks with that size */ } eraseblocks[NUM_ERASEREGIONS]; + /* a block_erase function should try to erase one block of size + * 'blocklen' at address 'blockaddr' and return 0 on success. */ int (*block_erase) (struct flashchip *flash, unsigned int blockaddr, unsigned int blocklen); } block_erasers[NUM_ERASEFUNCTIONS];
Modified: trunk/flashrom.c ============================================================================== --- trunk/flashrom.c Thu May 26 16:28:51 2011 (r1320) +++ trunk/flashrom.c Sat May 28 04:37:14 2011 (r1321) @@ -1312,7 +1312,7 @@ goto out_free; }
- ret = write_buf_to_file(buf, flash->total_size * 1024, filename); + ret = write_buf_to_file(buf, size, filename); out_free: free(buf); msg_cinfo("%s.\n", ret ? "FAILED" : "done"); @@ -1489,13 +1489,13 @@ if (!eraser.block_erase && eraser.eraseblocks[0].count) { if (log) msg_cdbg("eraseblock layout is known, but matching " - "block erase function is not implemented. "); + "block erase function is not implemented. "); return 1; } if (eraser.block_erase && !eraser.eraseblocks[0].count) { if (log) msg_cdbg("block erase function found, but " - "eraseblock layout is not defined. "); + "eraseblock layout is not defined. "); return 1; } return 0;
Modified: trunk/ichspi.c ============================================================================== --- trunk/ichspi.c Thu May 26 16:28:51 2011 (r1320) +++ trunk/ichspi.c Sat May 28 04:37:14 2011 (r1321) @@ -159,9 +159,9 @@ return mmio_readb(ich_spibar + X); }
-#define REGWRITE32(X,Y) mmio_writel(Y, ich_spibar+X) -#define REGWRITE16(X,Y) mmio_writew(Y, ich_spibar+X) -#define REGWRITE8(X,Y) mmio_writeb(Y, ich_spibar+X) +#define REGWRITE32(off,val) mmio_writel(val, ich_spibar+off) +#define REGWRITE16(off,val) mmio_writew(val, ich_spibar+off) +#define REGWRITE8(off,val) mmio_writeb(val, ich_spibar+off)
/* Common SPI functions */ static int find_opcode(OPCODES *op, uint8_t opcode); @@ -856,7 +856,8 @@ if (!ichspi_lock) opcode_index = reprogram_opcode_on_the_fly(cmd, writecnt, readcnt); if (opcode_index == -1) { - msg_pdbg("Invalid OPCODE 0x%02x\n", cmd); + msg_pdbg("Invalid OPCODE 0x%02x, will not execute.\n", + cmd); return SPI_INVALID_OPCODE; } }
Modified: trunk/print.c ============================================================================== --- trunk/print.c Thu May 26 16:28:51 2011 (r1320) +++ trunk/print.c Sat May 28 04:37:14 2011 (r1321) @@ -43,18 +43,18 @@ ret = strcat_realloc(ret, "Non-SPI,"); } else { if (bustype & CHIP_BUSTYPE_PARALLEL) - ret = strcat_realloc(ret, "Parallel,"); + ret = strcat_realloc(ret, "Parallel, "); if (bustype & CHIP_BUSTYPE_LPC) - ret = strcat_realloc(ret, "LPC,"); + ret = strcat_realloc(ret, "LPC, "); if (bustype & CHIP_BUSTYPE_FWH) - ret = strcat_realloc(ret, "FWH,"); + ret = strcat_realloc(ret, "FWH, "); if (bustype & CHIP_BUSTYPE_SPI) - ret = strcat_realloc(ret, "SPI,"); + ret = strcat_realloc(ret, "SPI, "); if (bustype == CHIP_BUSTYPE_NONE) - ret = strcat_realloc(ret, "None,"); + ret = strcat_realloc(ret, "None, "); } /* Kill last comma. */ - ret[strlen(ret) - 1] = '\0'; + ret[strlen(ret) - 2] = '\0'; ret = realloc(ret, strlen(ret) + 1); return ret; }
Modified: trunk/spi25.c ============================================================================== --- trunk/spi25.c Thu May 26 16:28:51 2011 (r1320) +++ trunk/spi25.c Sat May 28 04:37:14 2011 (r1321) @@ -119,8 +119,10 @@ uint32_t id1; uint32_t id2;
- if (spi_rdid(readarr, bytes)) + if (spi_rdid(readarr, bytes)) { + msg_cdbg("\n"); return 0; + }
if (!oddparity(readarr[0])) msg_cdbg("RDID byte 0 parity violation. "); @@ -198,8 +200,10 @@ unsigned char readarr[JEDEC_REMS_INSIZE]; uint32_t id1, id2;
- if (spi_rems(readarr)) + if (spi_rems(readarr)) { + msg_cdbg("\n"); return 0; + }
id1 = readarr[0]; id2 = readarr[1]; @@ -254,15 +258,19 @@ return 0; }
- if (spi_res(readarr, 1)) + if (spi_res(readarr, 1)) { + msg_cdbg("\n"); return 0; + }
id2 = readarr[0];
msg_cdbg("%s: id 0x%x\n", __func__, id2);
- if (id2 != flash->model_id) + if (id2 != flash->model_id) { + msg_cdbg("\n"); return 0; + }
/* Print the status register to tell the * user about possible write protection. @@ -276,8 +284,10 @@ unsigned char readarr[2]; uint32_t id1, id2;
- if (spi_res(readarr, 2)) + if (spi_res(readarr, 2)) { + msg_cdbg("\n"); return 0; + }
id1 = readarr[0]; id2 = readarr[1];