[flashrom] [commit] r1333 - trunk
repository service
svn at flashrom.org
Sat Jun 11 21:44:32 CEST 2011
Author: stefanct
Date: Sat Jun 11 21:44:31 2011
New Revision: 1333
URL: http://flashrom.org/trac/flashrom/changeset/1333
Log:
use the max_data_read field of the new spi_programmer struct to simplify run_opcode
also, explain the transformation of the read/write array/count in
ich_spi_send_command better.
Signed-off-by: Stefan Tauner <stefan.tauner at student.tuwien.ac.at>
Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006 at gmx.net>
Modified:
trunk/ichspi.c
Modified: trunk/ichspi.c
==============================================================================
--- trunk/ichspi.c Sat Jun 11 20:16:50 2011 (r1332)
+++ trunk/ichspi.c Sat Jun 11 21:44:31 2011 (r1333)
@@ -922,37 +922,31 @@
static int run_opcode(OPCODE op, uint32_t offset,
uint8_t datalength, uint8_t * data)
{
+ /* max_data_read == max_data_write for all Intel/VIA SPI masters */
+ uint8_t maxlength = spi_programmer->max_data_read;
+
+ if (spi_programmer->type == SPI_CONTROLLER_NONE) {
+ msg_perr("%s: unsupported chipset\n", __func__);
+ return -1;
+ }
+
+ if (datalength > maxlength) {
+ msg_perr("%s: Internal command size error for "
+ "opcode 0x%02x, got datalength=%i, want <=%i\n",
+ __func__, op.opcode, datalength, maxlength);
+ return SPI_INVALID_LENGTH;
+ }
+
switch (spi_programmer->type) {
case SPI_CONTROLLER_VIA:
- if (datalength > 16) {
- msg_perr("%s: Internal command size error for "
- "opcode 0x%02x, got datalength=%i, want <=16\n",
- __func__, op.opcode, datalength);
- return SPI_INVALID_LENGTH;
- }
- return ich7_run_opcode(op, offset, datalength, data, 16);
case SPI_CONTROLLER_ICH7:
- if (datalength > 64) {
- msg_perr("%s: Internal command size error for "
- "opcode 0x%02x, got datalength=%i, want <=16\n",
- __func__, op.opcode, datalength);
- return SPI_INVALID_LENGTH;
- }
- return ich7_run_opcode(op, offset, datalength, data, 64);
+ return ich7_run_opcode(op, offset, datalength, data, maxlength);
case SPI_CONTROLLER_ICH9:
- if (datalength > 64) {
- msg_perr("%s: Internal command size error for "
- "opcode 0x%02x, got datalength=%i, want <=16\n",
- __func__, op.opcode, datalength);
- return SPI_INVALID_LENGTH;
- }
return ich9_run_opcode(op, offset, datalength, data);
default:
- msg_perr("%s: unsupported chipset\n", __func__);
+ /* If we ever get here, something really weird happened */
+ return -1;
}
-
- /* If we ever get here, something really weird happened */
- return -1;
}
static int ich_spi_send_command(unsigned int writecnt, unsigned int readcnt,
@@ -1038,7 +1032,13 @@
}
}
- /* translate read/write array/count */
+ /* Translate read/write array/count.
+ * The maximum data length is identical for the maximum read length and
+ * for the maximum write length excluding opcode and address. Opcode and
+ * address are stored in separate registers, not in the data registers
+ * and are thus not counted towards data length. The only exception
+ * applies if the opcode definition (un)intentionally classifies said
+ * opcode incorrectly as non-address opcode or vice versa. */
if (opcode->spi_type == SPI_OPCODE_TYPE_WRITE_NO_ADDRESS) {
data = (uint8_t *) (writearr + 1);
count = writecnt - 1;
More information about the flashrom
mailing list