Changeset 1546

Timestamp:

Jun 19, 2012 2:06:53 PM (11 months ago)

Author:

hailfinger

Message:

Add native SPI AAI write support to the Dediprog SF100 driver

To tell the programmer how to handle the data on the spi bus, a flag in
the fourth byte sent with the usb command is used. The second word was
mistaken for the size of the chunks sent over usb earlier. The third
byte (first of the second word) is now set to zero. This also adds some
checks for the size of data chunks sent over usb.

Signed-off-by: Nico Huber <nico.huber@…>
Acked-by: Carl-Daniel Hailfinger <c-d.hailfinger.devel.2006@…>

File:

• trunk/dediprog.c (modified) (9 diffs)

trunk/dediprog.c

@@ -32 +32 @@
 static int dediprog_endpoint;
 
+#define DEDI_SPI_CMD_PAGEWRITE  0x1
+#define DEDI_SPI_CMD_AAIWRITE   0x4
+
 #if 0
 /* Might be useful for other pieces of code as well. */
@@ -308 +311 @@
 }
 
-/* Bulk write interface, will write multiple page_size byte chunks aligned to page_size bytes.
- * @start       start address
- * @len         length
- * @return      0 on success, 1 on failure
- */
-static int dediprog_spi_bulk_write(struct flashctx *flash, uint8_t *buf,
-                                   unsigned int start, unsigned int len)
+/* Bulk write interface, will write multiple chunksize byte chunks aligned to chunksize bytes.
+ * @chunksize       length of data chunks, only 256 supported by now
+ * @start           start address
+ * @len             length
+ * @dedi_spi_cmd    dediprog specific write command for spi bus
+ * @return          0 on success, 1 on failure
+ */
+static int dediprog_spi_bulk_write(struct flashctx *flash, uint8_t *buf, unsigned int chunksize,
+                                   unsigned int start, unsigned int len, uint8_t dedi_spi_cmd)
 {
         int ret;
@@ -322 +327 @@
          * space in a USB bulk transfer must be filled with 0xff padding.
          */
-        const unsigned int chunksize = flash->page_size;
         const unsigned int count = len / chunksize;
-        const char count_and_chunk[] = {count & 0xff,
-                                        (count >> 8) & 0xff,
-                                        chunksize & 0xff,
-                                        (chunksize >> 8) & 0xff};
+        const char count_and_cmd[] = {count & 0xff, (count >> 8) & 0xff, 0x00, dedi_spi_cmd};
         char usbbuf[512];
+
+        /*
+         * We should change this check to
+         *   chunksize > 512
+         * once we know how to handle different chunk sizes.
+         */
+        if (chunksize != 256) {
+                msg_perr("%s: Chunk sizes other than 256 bytes are unsupported, chunksize=%u!\n"
+                         "Please report a bug at flashrom@flashrom.org\n", __func__, chunksize);
+                return 1;
+        }
 
         if ((start % chunksize) || (len % chunksize)) {
@@ -342 +354 @@
          * SPI side.
          */
-        ret = usb_control_msg(dediprog_handle, 0x42, 0x30, start % 0x10000,
-                              start / 0x10000, (char *)count_and_chunk,
-                              sizeof(count_and_chunk), DEFAULT_TIMEOUT);
-        if (ret != sizeof(count_and_chunk)) {
+        ret = usb_control_msg(dediprog_handle, 0x42, 0x30, start % 0x10000, start / 0x10000,
+                              (char *)count_and_cmd, sizeof(count_and_cmd), DEFAULT_TIMEOUT);
+        if (ret != sizeof(count_and_cmd)) {
                 msg_perr("Command Write SPI Bulk failed, %i %s!\n", ret,
                          usb_strerror());
@@ -367 +378 @@
 }
 
-static int dediprog_spi_write_256(struct flashctx *flash, uint8_t *buf,
-                                  unsigned int start, unsigned int len)
+static int dediprog_spi_write(struct flashctx *flash, uint8_t *buf,
+                              unsigned int start, unsigned int len, uint8_t dedi_spi_cmd)
 {
         int ret;
@@ -376 +387 @@
 
         dediprog_set_leds(PASS_OFF|BUSY_ON|ERROR_OFF);
+
+        if (chunksize != 256) {
+                msg_pdbg("Page sizes other than 256 bytes are unsupported as "
+                         "we don't know how dediprog\nhandles them.\n");
+                /* Write everything like it was residue. */
+                residue = len;
+        }
 
         if (residue) {
@@ -390 +408 @@
         /* Round down. */
         bulklen = (len - residue) / chunksize * chunksize;
-        ret = dediprog_spi_bulk_write(flash, buf + residue, start + residue,
-                                     bulklen);
+        ret = dediprog_spi_bulk_write(flash, buf + residue, chunksize, start + residue, bulklen, dedi_spi_cmd);
         if (ret) {
                 dediprog_set_leds(PASS_OFF|BUSY_OFF|ERROR_ON);
@@ -411 +428 @@
         dediprog_set_leds(PASS_ON|BUSY_OFF|ERROR_OFF);
         return 0;
+}
+
+static int dediprog_spi_write_256(struct flashctx *flash, uint8_t *buf, unsigned int start, unsigned int len)
+{
+        return dediprog_spi_write(flash, buf, start, len, DEDI_SPI_CMD_PAGEWRITE);
+}
+
+static int dediprog_spi_write_aai(struct flashctx *flash, uint8_t *buf, unsigned int start, unsigned int len)
+{
+        return dediprog_spi_write(flash, buf, start, len, DEDI_SPI_CMD_AAIWRITE);
 }
 
@@ -718 +745 @@
         .read           = dediprog_spi_read,
         .write_256      = dediprog_spi_write_256,
-        .write_aai      = default_spi_write_aai,
+        .write_aai      = dediprog_spi_write_aai,
 };