dediprog: Implement multi-i/o reads
This implements i/o-mode switches and opcode handling for multi-i/o
reads with protocol versions 2 and 3. The mode switching is done by
a simple command that takes an enum just as our internal `io_mode`
as argument.
The opcode handling differs between protocol versions. For protocol
v2, we keep the current behavior for single-i/o operations and only
set the matching opcode. Tests with an SF600Plus-G2 have shown that
the programmer automatically chooses the address length and number
of dummy cycles. It is unknown, however, if it chooses these para-
meters based on the opcode or the configured i/o mode. For dual-out
reads, it seems to choose the wrong number of dummy cycles. Hence,
we mask the respective support bit for the v2 case.
For protocol v3, a new `read mode' was discovered in traces of the
Dediprog Windows application. It allows to explicitly specify the
opcode, the address length, and the number of dummy cycles. We call
this READ_MODE_CONFIGURABLE. As this is the only way to make use of
the additional command bytes of the v3 protocol, we can assume that
this mode always works with v3.
For partial reads, i.e. not multiples of 512B blocks, that have to
go through dediprog_spi_send_command(), we temporarily disable the
chosen `.spi_fast_read` function. This is necessary, because multi-
io is not supported on this path.
We enable dual i/o by default for protocol v3 devices. This should
work out of the box with many compatible flash chips. The command-
line logic is a little convoluted this way, but can be refactored
once protocol v2 devices are tested.
Change-Id: Ib07b1b61eccc19c7ead9f64c980b37feabfa70a8
Signed-off-by: Nico Huber <nico.h@gmx.de>
Reviewed-on: https://review.sourcearcade.org/c/flashprog/+/114
Reviewed-by: Arthur Heymans <arthur@aheymans.xyz>
diff --git a/dediprog.c b/dediprog.c
index b8cc231..43d8b84 100644
--- a/dediprog.c
+++ b/dediprog.c
@@ -26,6 +26,7 @@
#include "chipdrivers.h"
#include "programmer.h"
#include "spi.h"
+#include "spi_command.h"
/* LIBUSB_CALL ensures the right calling conventions on libusb callbacks.
* However, the macro is not defined everywhere. m(
@@ -117,6 +118,7 @@
READ_MODE_ATMEL45 = 3,
READ_MODE_4B_ADDR_FAST = 4,
READ_MODE_4B_ADDR_FAST_0x0C = 5, /* New protocol only */
+ READ_MODE_CONFIGURABLE = 9, /* Not seen documented so far */
};
enum dediprog_writemode {
@@ -167,6 +169,7 @@
int (*prepare_rw_cmd)(
struct flashctx *, uint8_t cmd_buf[MAX_CMD_SIZE], uint16_t *value, uint16_t *idx,
bool is_read, uint8_t dp_spi_cmd, unsigned int start, unsigned int block_count);
+ enum io_mode io_mode;
};
#if defined(LIBUSB_MAJOR) && defined(LIBUSB_MINOR) && defined(LIBUSB_MICRO) && \
@@ -379,6 +382,40 @@
return 0;
}
+static int dediprog_set_io_mode(struct dediprog_data *const dp, const enum io_mode io_mode)
+{
+ const unsigned char dp_io_mode[] = {
+ [SINGLE_IO_1_1_1] = 0,
+ [DUAL_OUT_1_1_2] = 1,
+ [DUAL_IO_1_2_2] = 2,
+ [QUAD_OUT_1_1_4] = 3,
+ [QUAD_IO_1_4_4] = 4,
+ [QPI_4_4_4] = 5,
+ };
+
+ if (dp->devicetype < DEV_SF600)
+ return 0;
+
+ if (dp->io_mode == io_mode)
+ return 0;
+
+ if (io_mode >= ARRAY_SIZE(dp_io_mode)) {
+ msg_perr("%s: Unsupported I/O mode %d! "
+ "Please report a bug at flashprog@flashprog.org\n",
+ __func__, io_mode);
+ return 1;
+ }
+
+ const int ret = dediprog_write(dp->handle, CMD_IO_MODE, dp_io_mode[io_mode], 0, NULL, 0);
+ if (ret) {
+ msg_perr("Command I/O Mode 0x%x failed!\n", dp_io_mode[io_mode]);
+ return 1;
+ }
+
+ dp->io_mode = io_mode;
+ return 0;
+}
+
static int prepare_rw_cmd_common(uint8_t cmd_buf[MAX_CMD_SIZE], uint8_t dp_spi_cmd, unsigned int count)
{
if (count > MAX_BLOCK_COUNT) {
@@ -427,22 +464,42 @@
struct flashctx *flash, uint8_t cmd_buf[MAX_CMD_SIZE], uint16_t *value, uint16_t *idx,
bool is_read, uint8_t dp_spi_cmd, unsigned int start, unsigned int block_count)
{
+ struct dediprog_data *const dp = flash->mst.spi->data;
+
if (prepare_rw_cmd_common(cmd_buf, dp_spi_cmd, block_count) < 0)
return -1;
- if (is_read && flash->chip->feature_bits & FEATURE_4BA_FAST_READ) {
- cmd_buf[3] = READ_MODE_4B_ADDR_FAST_0x0C;
- cmd_buf[4] = JEDEC_FAST_READ_4BA;
- } else if (dp_spi_cmd == WRITE_MODE_PAGE_PGM
- && (flash->chip->feature_bits & FEATURE_4BA_WRITE)) {
- cmd_buf[3] = WRITE_MODE_4B_ADDR_256B_PAGE_PGM_0x12;
- cmd_buf[4] = JEDEC_BYTE_PROGRAM_4BA;
+ if (is_read) {
+ const struct spi_read_op *const read_op = get_spi_read_op(flash);
+ if (dediprog_set_io_mode(dp, read_op->io_mode))
+ return -1;
+
+ if (read_op->native_4ba)
+ cmd_buf[3] = READ_MODE_4B_ADDR_FAST_0x0C;
+ else if (read_op->opcode != JEDEC_READ)
+ cmd_buf[3] = READ_MODE_FAST;
+
+ if (read_op->opcode == JEDEC_READ_4BA)
+ cmd_buf[4] = JEDEC_FAST_READ_4BA;
+ else
+ cmd_buf[4] = read_op->opcode;
+ } else {
+ if (dediprog_set_io_mode(dp, SINGLE_IO_1_1_1))
+ return -1;
+
+ if (dp_spi_cmd == WRITE_MODE_PAGE_PGM
+ && (flash->chip->feature_bits & FEATURE_4BA_WRITE)) {
+ cmd_buf[3] = WRITE_MODE_4B_ADDR_256B_PAGE_PGM_0x12;
+ cmd_buf[4] = JEDEC_BYTE_PROGRAM_4BA;
+ }
}
+
cmd_buf[5] = 0; /* RFU */
cmd_buf[6] = (start >> 0) & 0xff;
cmd_buf[7] = (start >> 8) & 0xff;
cmd_buf[8] = (start >> 16) & 0xff;
cmd_buf[9] = (start >> 24) & 0xff;
+
return 10;
}
@@ -450,6 +507,8 @@
struct flashctx *flash, uint8_t cmd_buf[MAX_CMD_SIZE], uint16_t *value, uint16_t *idx,
bool is_read, uint8_t dp_spi_cmd, unsigned int start, unsigned int block_count)
{
+ struct dediprog_data *const dp = flash->mst.spi->data;
+
if (prepare_rw_cmd_common(cmd_buf, dp_spi_cmd, block_count) < 0)
return -1;
@@ -460,14 +519,19 @@
cmd_buf[9] = (start >> 24) & 0xff;
if (is_read) {
- if (flash->chip->feature_bits & FEATURE_4BA_FAST_READ) {
- cmd_buf[3] = READ_MODE_4B_ADDR_FAST_0x0C;
- cmd_buf[4] = JEDEC_FAST_READ_4BA;
- }
- cmd_buf[10] = 0x00; /* address length (3 or 4) */
- cmd_buf[11] = 0x00; /* dummy cycle / 2 */
+ const struct spi_read_op *const read_op = get_spi_read_op(flash);
+ if (dediprog_set_io_mode(dp, read_op->io_mode))
+ return -1;
+
+ cmd_buf[3] = READ_MODE_CONFIGURABLE;
+ cmd_buf[4] = read_op->opcode;
+ cmd_buf[10] = read_op->native_4ba || flash->in_4ba_mode ? 4 : 3;
+ cmd_buf[11] = spi_dummy_cycles(read_op) / 2;
return 12;
} else {
+ if (dediprog_set_io_mode(dp, SINGLE_IO_1_1_1))
+ return -1;
+
if (dp_spi_cmd == WRITE_MODE_PAGE_PGM
&& (flash->chip->feature_bits & FEATURE_4BA_WRITE)) {
cmd_buf[3] = WRITE_MODE_4B_ADDR_256B_PAGE_PGM;
@@ -577,6 +641,20 @@
return err;
}
+static int dediprog_slow_read(struct flashctx *flash, uint8_t *buf, unsigned int start, unsigned int len)
+{
+ msg_pdbg("Slow read for partial block from 0x%x, length 0x%x\n", start, len);
+
+ /* Override fast-read function for a moment: */
+ const struct spi_read_op *const backup = flash->spi_fast_read;
+ flash->spi_fast_read = NULL;
+
+ const int ret = default_spi_read(flash, buf, start, len);
+
+ flash->spi_fast_read = backup;
+ return ret;
+}
+
static int dediprog_spi_read(struct flashctx *flash, uint8_t *buf, unsigned int start, unsigned int len)
{
int ret;
@@ -589,9 +667,7 @@
dediprog_set_leds(LED_BUSY, dp_data);
if (residue) {
- msg_pdbg("Slow read for partial block from 0x%x, length 0x%x\n",
- start, residue);
- ret = default_spi_read(flash, buf, start, residue);
+ ret = dediprog_slow_read(flash, buf, start, residue);
if (ret)
goto err;
}
@@ -604,10 +680,8 @@
len -= residue + bulklen;
if (len != 0) {
- msg_pdbg("Slow read for partial block from 0x%x, length 0x%x\n",
- start, len);
- ret = default_spi_read(flash, buf + residue + bulklen,
- start + residue + bulklen, len);
+ ret = dediprog_slow_read(flash, buf + residue + bulklen,
+ start + residue + bulklen, len);
if (ret)
goto err;
}
@@ -775,8 +849,8 @@
const unsigned char *writearr,
unsigned char *readarr)
{
+ struct dediprog_data *const dp_data = flash->mst.spi->data;
int ret;
- const struct dediprog_data *dp_data = flash->mst.spi->data;
msg_pspew("%s, writecnt=%i, readcnt=%i\n", __func__, writecnt, readcnt);
if (writecnt > flash->mst.spi->max_data_write + 5) {
@@ -788,6 +862,9 @@
return 1;
}
+ if (dediprog_set_io_mode(dp_data, SINGLE_IO_1_1_1))
+ return 1;
+
unsigned int idx, value;
/* New protocol has options and timeout combined as value while the old one used the value field for
* timeout and the index field for options. */
@@ -1170,6 +1247,8 @@
int ret = 0;
struct dediprog_data *dp_data = data;
+ dediprog_set_io_mode(dp_data, SINGLE_IO_1_1_1);
+
/* URB 28. Command Set SPI Voltage to 0. */
if (dediprog_set_spi_voltage(dp_data->handle, 0x0)) {
ret = 1;
@@ -1191,6 +1270,12 @@
static int dediprog_init(struct flashprog_programmer *const prog)
{
char *voltage, *id_str, *device, *spispeed, *target_str;
+ enum {
+ DEFAULT,
+ SINGLE,
+ DUAL,
+ QUAD,
+ } io_mode = DEFAULT;
int spispeed_idx = 1;
int millivolt = 3500;
long id = -1; /* -1 defaults to enumeration order */
@@ -1199,6 +1284,21 @@
long target = FLASH_TYPE_APPLICATION_FLASH_1;
int i, ret;
+ char *const io_mode_str = extract_programmer_param("iomode");
+ if (io_mode_str) {
+ if (strcmp(io_mode_str, "single") == 0) {
+ io_mode = SINGLE;
+ } else if (strcmp(io_mode_str, "dual") == 0) {
+ io_mode = DUAL;
+ } else if (strcmp(io_mode_str, "quad") == 0) {
+ io_mode = QUAD;
+ } else {
+ msg_perr("Invalid iomode setting: %s\n", io_mode_str);
+ return SPI_GENERIC_ERROR;
+ }
+ }
+ free(io_mode_str);
+
spispeed = extract_programmer_param("spispeed");
if (spispeed) {
for (i = 0; spispeeds[i].name; ++i) {
@@ -1326,6 +1426,7 @@
}
dp_data->firmwareversion = FIRMWARE_VERSION(0, 0, 0);
dp_data->devicetype = DEV_UNKNOWN;
+ dp_data->io_mode = -1;
/* Here comes the USB stuff. */
ret = libusb_init(&dp_data->usb_ctx);
@@ -1397,6 +1498,29 @@
default: dp_data->prepare_rw_cmd = prepare_rw_cmd_v1; break;
}
+ if (io_mode == DEFAULT) {
+ if (protocol(dp_data) < PROTOCOL_V3) {
+ msg_pdbg("Multi i/o is only tested with protocol v3, not enabling by default.\n");
+ } else {
+ io_mode = DUAL;
+ }
+ } else if (io_mode > SINGLE &&
+ (dp_data->devicetype < DEV_SF600 || protocol(dp_data) < PROTOCOL_V2)) {
+ msg_pinfo("Multi i/o is only supported for SF600 and SF700 models w/ protocol v2 or later.\n");
+ io_mode = SINGLE;
+ }
+
+ switch (io_mode) {
+ case DUAL: spi_master_dediprog.features |= SPI_MASTER_DUAL; break;
+ case QUAD: spi_master_dediprog.features |= SPI_MASTER_DUAL | SPI_MASTER_QUAD; break;
+ default: break;
+ }
+
+ /* The v2, fixed-op JEDEC_FAST_READ_DUAL_DIO command
+ seems to use the wrong number of dummy cycles. */
+ if (protocol(dp_data) < PROTOCOL_V3)
+ spi_master_dediprog.features &= ~SPI_MASTER_DUAL_IO;
+
if ((dp_data->devicetype == DEV_SF100) ||
(dp_data->devicetype == DEV_SF600 && protocol(dp_data) == PROTOCOL_V3))
spi_master_dediprog.features &= ~SPI_MASTER_NO_4BA_MODES;
diff --git a/flashprog.8.tmpl b/flashprog.8.tmpl
index 70e21af..c8f35d7 100644
--- a/flashprog.8.tmpl
+++ b/flashprog.8.tmpl
@@ -323,7 +323,7 @@
.sp
.BR "* buspirate_spi" " (for SPI flash ROMs attached to a Bus Pirate)"
.sp
-.BR "* dediprog" " (for SPI flash ROMs attached to a Dediprog SF100)"
+.BR "* dediprog" " (for SPI flash ROMs attached to a Dediprog SF100/SF200/SF600/SF700)"
.sp
.BR "* rayer_spi" " (for SPI flash ROMs attached to a parallel port by one of various cable types)"
.sp
@@ -1105,6 +1105,16 @@
can be
.BR 1 " or " 2
to select target chip 1 or 2 respectively. The default is target chip 1.
+.sp
+Dediprog SF600 and SF700 programmer models support dual and quad i/o.
+The default is dual i/o on newer models with protocol v3 and single i/o
+otherwise. The mode can be set with the
+.B iomode
+parameter. Valid values are
+.BR single ", " dual ", or " quad .
+For instance, to enable quad i/o
+.sp
+.B " flashprog \-p dediprog:iomode=quad"
.SS
.BR "rayer_spi " programmer
.IP
diff --git a/include/spi_command.h b/include/spi_command.h
index b0daeee..3af0cd0 100644
--- a/include/spi_command.h
+++ b/include/spi_command.h
@@ -77,6 +77,15 @@
uint8_t dummy_len; /* dummy bytes (including optional mode byte) */
};
+const struct spi_read_op *get_spi_read_op(const struct flashctx *);
+
+static inline unsigned int spi_dummy_cycles(const struct spi_read_op *const op)
+{
+ return op->dummy_len * 8
+ / (op->io_mode == SINGLE_IO_1_1_1 ? 1
+ : (op->io_mode <= DUAL_IO_1_2_2 ? 2 : 4));
+}
+
struct spi_command {
enum io_mode io_mode;
size_t opcode_len; /* bytes to write in opcode i/o phase */
diff --git a/spi25.c b/spi25.c
index edcaf48..34f30df 100644
--- a/spi25.c
+++ b/spi25.c
@@ -644,7 +644,7 @@
return spi_write_cmd(flash, op, native_4ba, addr, bytes, len, 10);
}
-static const struct spi_read_op *get_spi_read_op(const struct flashctx *flash)
+const struct spi_read_op *get_spi_read_op(const struct flashctx *flash)
{
static const struct spi_read_op sio_read = { SINGLE_IO_1_1_1, false, JEDEC_READ, 0x00, 0 };
static const struct spi_read_op sio_read_4ba = { SINGLE_IO_1_1_1, true, JEDEC_READ_4BA, 0x00, 0 };