[flashrom] [commit] r1321 - trunk
repository service
svn at flashrom.org
Sat May 28 04:37:14 CEST 2011
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 at student.tuwien.ac.at>
Acked-by: Stefan Tauner <stefan.tauner at 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];
More information about the flashrom
mailing list