coding style: avoid unnecessary line continuations
[openocd.git] / src / flash / nor / cfi.c
index 248f76005e59ae2de2b8fc62bf58b478f02b4bee..50ab207c14d689b3556030d47c61274f05401ee1 100644 (file)
@@ -34,9 +34,6 @@
 #include <helper/binarybuffer.h>
 #include <target/algorithm.h>
 
-#define CFI_MAX_BUS_WIDTH       4
-#define CFI_MAX_CHIP_WIDTH      4
-
 /* defines internal maximum size for code fragment in cfi_intel_write_block() */
 #define CFI_MAX_INTEL_CODESIZE 256
 
@@ -103,16 +100,15 @@ static const struct cfi_fixup cfi_0001_fixups[] = {
 static void cfi_fixup(struct flash_bank *bank, const struct cfi_fixup *fixups)
 {
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
-       const struct cfi_fixup *f;
 
-       for (f = fixups; f->fixup; f++) {
+       for (const struct cfi_fixup *f = fixups; f->fixup; f++) {
                if (((f->mfr == CFI_MFR_ANY) || (f->mfr == cfi_info->manufacturer)) &&
                                ((f->id  == CFI_ID_ANY)  || (f->id  == cfi_info->device_id)))
                        f->fixup(bank, f->param);
        }
 }
 
-static inline uint32_t flash_address(struct flash_bank *bank, int sector, uint32_t offset)
+uint32_t cfi_flash_address(struct flash_bank *bank, int sector, uint32_t offset)
 {
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
 
@@ -131,32 +127,55 @@ static inline uint32_t flash_address(struct flash_bank *bank, int sector, uint32
        }
 }
 
+static int cfi_target_write_memory(struct flash_bank *bank, target_addr_t addr,
+                                  uint32_t count, const uint8_t *buffer)
+{
+       struct cfi_flash_bank *cfi_info = bank->driver_priv;
+       if (cfi_info->write_mem) {
+               return cfi_info->write_mem(bank, addr, count, buffer);
+       } else {
+               return target_write_memory(bank->target, addr, bank->bus_width,
+                                          count, buffer);
+       }
+}
+
+int cfi_target_read_memory(struct flash_bank *bank, target_addr_t addr,
+                          uint32_t count, uint8_t *buffer)
+{
+       struct cfi_flash_bank *cfi_info = bank->driver_priv;
+       if (cfi_info->read_mem) {
+               return cfi_info->read_mem(bank, addr, count, buffer);
+       } else {
+               return target_read_memory(bank->target, addr, bank->bus_width,
+                                         count, buffer);
+       }
+}
+
 static void cfi_command(struct flash_bank *bank, uint8_t cmd, uint8_t *cmd_buf)
 {
-       int i;
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
 
        /* clear whole buffer, to ensure bits that exceed the bus_width
         * are set to zero
         */
-       for (i = 0; i < CFI_MAX_BUS_WIDTH; i++)
+       for (size_t i = 0; i < CFI_MAX_BUS_WIDTH; i++)
                cmd_buf[i] = 0;
 
        if (cfi_info->endianness == TARGET_LITTLE_ENDIAN) {
-               for (i = bank->bus_width; i > 0; i--)
+               for (int i = bank->bus_width; i > 0; i--)
                        *cmd_buf++ = (i & (bank->chip_width - 1)) ? 0x0 : cmd;
        } else {
-               for (i = 1; i <= bank->bus_width; i++)
+               for (int i = 1; i <= bank->bus_width; i++)
                        *cmd_buf++ = (i & (bank->chip_width - 1)) ? 0x0 : cmd;
        }
 }
 
-static int cfi_send_command(struct flash_bank *bank, uint8_t cmd, uint32_t address)
+int cfi_send_command(struct flash_bank *bank, uint8_t cmd, uint32_t address)
 {
        uint8_t command[CFI_MAX_BUS_WIDTH];
 
        cfi_command(bank, cmd, command);
-       return target_write_memory(bank->target, address, bank->bus_width, 1, command);
+       return cfi_target_write_memory(bank, address, 1, command);
 }
 
 /* read unsigned 8-bit value from the bank
@@ -165,13 +184,12 @@ static int cfi_send_command(struct flash_bank *bank, uint8_t cmd, uint32_t addre
  */
 static int cfi_query_u8(struct flash_bank *bank, int sector, uint32_t offset, uint8_t *val)
 {
-       struct target *target = bank->target;
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
        uint8_t data[CFI_MAX_BUS_WIDTH];
 
        int retval;
-       retval = target_read_memory(target, flash_address(bank, sector, offset),
-                       bank->bus_width, 1, data);
+       retval = cfi_target_read_memory(bank, cfi_flash_address(bank, sector, offset),
+                                       1, data);
        if (retval != ERROR_OK)
                return retval;
 
@@ -189,25 +207,23 @@ static int cfi_query_u8(struct flash_bank *bank, int sector, uint32_t offset, ui
  */
 static int cfi_get_u8(struct flash_bank *bank, int sector, uint32_t offset, uint8_t *val)
 {
-       struct target *target = bank->target;
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
        uint8_t data[CFI_MAX_BUS_WIDTH];
-       int i;
 
        int retval;
-       retval = target_read_memory(target, flash_address(bank, sector, offset),
-                       bank->bus_width, 1, data);
+       retval = cfi_target_read_memory(bank, cfi_flash_address(bank, sector, offset),
+                                       1, data);
        if (retval != ERROR_OK)
                return retval;
 
        if (cfi_info->endianness == TARGET_LITTLE_ENDIAN) {
-               for (i = 0; i < bank->bus_width / bank->chip_width; i++)
+               for (int i = 0; i < bank->bus_width / bank->chip_width; i++)
                        data[0] |= data[i];
 
                *val = data[0];
        } else {
                uint8_t value = 0;
-               for (i = 0; i < bank->bus_width / bank->chip_width; i++)
+               for (int i = 0; i < bank->bus_width / bank->chip_width; i++)
                        value |= data[bank->bus_width - 1 - i];
 
                *val = value;
@@ -217,22 +233,20 @@ static int cfi_get_u8(struct flash_bank *bank, int sector, uint32_t offset, uint
 
 static int cfi_query_u16(struct flash_bank *bank, int sector, uint32_t offset, uint16_t *val)
 {
-       struct target *target = bank->target;
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
        uint8_t data[CFI_MAX_BUS_WIDTH * 2];
        int retval;
 
        if (cfi_info->x16_as_x8) {
-               uint8_t i;
-               for (i = 0; i < 2; i++) {
-                       retval = target_read_memory(target, flash_address(bank, sector, offset + i),
-                                       bank->bus_width, 1, &data[i * bank->bus_width]);
+               for (uint8_t i = 0; i < 2; i++) {
+                       retval = cfi_target_read_memory(bank, cfi_flash_address(bank, sector, offset + i),
+                                                       1, &data[i * bank->bus_width]);
                        if (retval != ERROR_OK)
                                return retval;
                }
        } else {
-               retval = target_read_memory(target, flash_address(bank, sector, offset),
-                               bank->bus_width, 2, data);
+               retval = cfi_target_read_memory(bank, cfi_flash_address(bank, sector, offset),
+                                               2, data);
                if (retval != ERROR_OK)
                        return retval;
        }
@@ -247,22 +261,20 @@ static int cfi_query_u16(struct flash_bank *bank, int sector, uint32_t offset, u
 
 static int cfi_query_u32(struct flash_bank *bank, int sector, uint32_t offset, uint32_t *val)
 {
-       struct target *target = bank->target;
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
        uint8_t data[CFI_MAX_BUS_WIDTH * 4];
        int retval;
 
        if (cfi_info->x16_as_x8) {
-               uint8_t i;
-               for (i = 0; i < 4; i++) {
-                       retval = target_read_memory(target, flash_address(bank, sector, offset + i),
-                                       bank->bus_width, 1, &data[i * bank->bus_width]);
+               for (uint8_t i = 0; i < 4; i++) {
+                       retval = cfi_target_read_memory(bank, cfi_flash_address(bank, sector, offset + i),
+                                                       1, &data[i * bank->bus_width]);
                        if (retval != ERROR_OK)
                                return retval;
                }
        } else {
-               retval = target_read_memory(target, flash_address(bank, sector, offset),
-                               bank->bus_width, 4, data);
+               retval = cfi_target_read_memory(bank, cfi_flash_address(bank, sector, offset),
+                                               4, data);
                if (retval != ERROR_OK)
                        return retval;
        }
@@ -278,16 +290,16 @@ static int cfi_query_u32(struct flash_bank *bank, int sector, uint32_t offset, u
        return ERROR_OK;
 }
 
-static int cfi_reset(struct flash_bank *bank)
+int cfi_reset(struct flash_bank *bank)
 {
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
        int retval = ERROR_OK;
 
-       retval = cfi_send_command(bank, 0xf0, flash_address(bank, 0, 0x0));
+       retval = cfi_send_command(bank, 0xf0, cfi_flash_address(bank, 0, 0x0));
        if (retval != ERROR_OK)
                return retval;
 
-       retval = cfi_send_command(bank, 0xff, flash_address(bank, 0, 0x0));
+       retval = cfi_send_command(bank, 0xff, cfi_flash_address(bank, 0, 0x0));
        if (retval != ERROR_OK)
                return retval;
 
@@ -295,7 +307,7 @@ static int cfi_reset(struct flash_bank *bank)
                        (cfi_info->device_id == 0x227E || cfi_info->device_id == 0x7E)) {
                /* Numonix M29W128G is cmd 0xFF intolerant - causes internal undefined state
                 * so we send an extra 0xF0 reset to fix the bug */
-               retval = cfi_send_command(bank, 0xf0, flash_address(bank, 0, 0x00));
+               retval = cfi_send_command(bank, 0xf0, cfi_flash_address(bank, 0, 0x00));
                if (retval != ERROR_OK)
                        return retval;
        }
@@ -305,7 +317,7 @@ static int cfi_reset(struct flash_bank *bank)
 
 static void cfi_intel_clear_status_register(struct flash_bank *bank)
 {
-       cfi_send_command(bank, 0x50, flash_address(bank, 0, 0x0));
+       cfi_send_command(bank, 0x50, cfi_flash_address(bank, 0, 0x0));
 }
 
 static int cfi_intel_wait_status_busy(struct flash_bank *bank, int timeout, uint8_t *val)
@@ -359,7 +371,7 @@ static int cfi_intel_wait_status_busy(struct flash_bank *bank, int timeout, uint
        return retval;
 }
 
-static int cfi_spansion_wait_status_busy(struct flash_bank *bank, int timeout)
+int cfi_spansion_wait_status_busy(struct flash_bank *bank, int timeout)
 {
        uint8_t status, oldstatus;
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
@@ -534,7 +546,7 @@ static int cfi_read_spansion_pri_ext(struct flash_bank *bank)
        pri_ext->_reversed_geometry = 0;
 
        if ((pri_ext->pri[0] != 'P') || (pri_ext->pri[1] != 'R') || (pri_ext->pri[2] != 'I')) {
-               retval = cfi_send_command(bank, 0xf0, flash_address(bank, 0, 0x0));
+               retval = cfi_send_command(bank, 0xf0, cfi_flash_address(bank, 0, 0x0));
                if (retval != ERROR_OK)
                        return retval;
                LOG_ERROR("Could not read spansion bank information");
@@ -641,7 +653,7 @@ static int cfi_read_atmel_pri_ext(struct flash_bank *bank)
 
        if ((atmel_pri_ext.pri[0] != 'P') || (atmel_pri_ext.pri[1] != 'R')
                        || (atmel_pri_ext.pri[2] != 'I')) {
-               retval = cfi_send_command(bank, 0xf0, flash_address(bank, 0, 0x0));
+               retval = cfi_send_command(bank, 0xf0, cfi_flash_address(bank, 0, 0x0));
                if (retval != ERROR_OK)
                        return retval;
                LOG_ERROR("Could not read atmel bank information");
@@ -799,14 +811,12 @@ static int cfi_intel_info(struct flash_bank *bank, char *buf, int buf_size)
        return ERROR_OK;
 }
 
-/* flash_bank cfi <base> <size> <chip_width> <bus_width> <target#> [options]
- */
-FLASH_BANK_COMMAND_HANDLER(cfi_flash_bank_command)
+int cfi_flash_bank_cmd(struct flash_bank *bank, unsigned int argc, const char **argv)
 {
        struct cfi_flash_bank *cfi_info;
        int bus_swap = 0;
 
-       if (CMD_ARGC < 6)
+       if (argc < 6)
                return ERROR_COMMAND_SYNTAX_ERROR;
 
        /* both widths must:
@@ -826,7 +836,7 @@ FLASH_BANK_COMMAND_HANDLER(cfi_flash_bank_command)
        }
 
        cfi_info = malloc(sizeof(struct cfi_flash_bank));
-       cfi_info->probed = 0;
+       cfi_info->probed = false;
        cfi_info->erase_region_info = NULL;
        cfi_info->pri_ext = NULL;
        bank->driver_priv = cfi_info;
@@ -834,13 +844,16 @@ FLASH_BANK_COMMAND_HANDLER(cfi_flash_bank_command)
        cfi_info->x16_as_x8 = 0;
        cfi_info->jedec_probe = 0;
        cfi_info->not_cfi = 0;
+       cfi_info->data_swap = 0;
 
-       for (unsigned i = 6; i < CMD_ARGC; i++) {
-               if (strcmp(CMD_ARGV[i], "x16_as_x8") == 0)
+       for (unsigned i = 6; i < argc; i++) {
+               if (strcmp(argv[i], "x16_as_x8") == 0)
                        cfi_info->x16_as_x8 = 1;
-               else if (strcmp(CMD_ARGV[i], "bus_swap") == 0)
+               else if (strcmp(argv[i], "data_swap") == 0)
+                       cfi_info->data_swap = 1;
+               else if (strcmp(argv[i], "bus_swap") == 0)
                        bus_swap = 1;
-               else if (strcmp(CMD_ARGV[i], "jedec_probe") == 0)
+               else if (strcmp(argv[i], "jedec_probe") == 0)
                        cfi_info->jedec_probe = 1;
        }
 
@@ -857,20 +870,26 @@ FLASH_BANK_COMMAND_HANDLER(cfi_flash_bank_command)
        return ERROR_OK;
 }
 
+/* flash_bank cfi <base> <size> <chip_width> <bus_width> <target#> [options]
+ */
+FLASH_BANK_COMMAND_HANDLER(cfi_flash_bank_command)
+{
+       return cfi_flash_bank_cmd(bank, CMD_ARGC, CMD_ARGV);
+}
+
 static int cfi_intel_erase(struct flash_bank *bank, int first, int last)
 {
        int retval;
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
-       int i;
 
        cfi_intel_clear_status_register(bank);
 
-       for (i = first; i <= last; i++) {
-               retval = cfi_send_command(bank, 0x20, flash_address(bank, i, 0x0));
+       for (int i = first; i <= last; i++) {
+               retval = cfi_send_command(bank, 0x20, cfi_flash_address(bank, i, 0x0));
                if (retval != ERROR_OK)
                        return retval;
 
-               retval = cfi_send_command(bank, 0xd0, flash_address(bank, i, 0x0));
+               retval = cfi_send_command(bank, 0xd0, cfi_flash_address(bank, i, 0x0));
                if (retval != ERROR_OK)
                        return retval;
 
@@ -882,68 +901,76 @@ static int cfi_intel_erase(struct flash_bank *bank, int first, int last)
                if (status == 0x80)
                        bank->sectors[i].is_erased = 1;
                else {
-                       retval = cfi_send_command(bank, 0xff, flash_address(bank, 0, 0x0));
+                       retval = cfi_send_command(bank, 0xff, cfi_flash_address(bank, 0, 0x0));
                        if (retval != ERROR_OK)
                                return retval;
 
-                       LOG_ERROR("couldn't erase block %i of flash bank at base 0x%"
-                                       PRIx32, i, bank->base);
+                       LOG_ERROR("couldn't erase block %i of flash bank at base "
+                                       TARGET_ADDR_FMT, i, bank->base);
                        return ERROR_FLASH_OPERATION_FAILED;
                }
        }
 
-       return cfi_send_command(bank, 0xff, flash_address(bank, 0, 0x0));
+       return cfi_send_command(bank, 0xff, cfi_flash_address(bank, 0, 0x0));
 }
 
-static int cfi_spansion_erase(struct flash_bank *bank, int first, int last)
+int cfi_spansion_unlock_seq(struct flash_bank *bank)
 {
        int retval;
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
        struct cfi_spansion_pri_ext *pri_ext = cfi_info->pri_ext;
-       int i;
 
-       for (i = first; i <= last; i++) {
-               retval = cfi_send_command(bank, 0xaa, flash_address(bank, 0, pri_ext->_unlock1));
-               if (retval != ERROR_OK)
-                       return retval;
+       retval = cfi_send_command(bank, 0xaa, cfi_flash_address(bank, 0, pri_ext->_unlock1));
+       if (retval != ERROR_OK)
+               return retval;
 
-               retval = cfi_send_command(bank, 0x55, flash_address(bank, 0, pri_ext->_unlock2));
-               if (retval != ERROR_OK)
-                       return retval;
+       retval = cfi_send_command(bank, 0x55, cfi_flash_address(bank, 0, pri_ext->_unlock2));
+       if (retval != ERROR_OK)
+               return retval;
+
+       return ERROR_OK;
+}
 
-               retval = cfi_send_command(bank, 0x80, flash_address(bank, 0, pri_ext->_unlock1));
+static int cfi_spansion_erase(struct flash_bank *bank, int first, int last)
+{
+       int retval;
+       struct cfi_flash_bank *cfi_info = bank->driver_priv;
+       struct cfi_spansion_pri_ext *pri_ext = cfi_info->pri_ext;
+
+       for (int i = first; i <= last; i++) {
+               retval = cfi_spansion_unlock_seq(bank);
                if (retval != ERROR_OK)
                        return retval;
 
-               retval = cfi_send_command(bank, 0xaa, flash_address(bank, 0, pri_ext->_unlock1));
+               retval = cfi_send_command(bank, 0x80, cfi_flash_address(bank, 0, pri_ext->_unlock1));
                if (retval != ERROR_OK)
                        return retval;
 
-               retval = cfi_send_command(bank, 0x55, flash_address(bank, 0, pri_ext->_unlock2));
+               retval = cfi_spansion_unlock_seq(bank);
                if (retval != ERROR_OK)
                        return retval;
 
-               retval = cfi_send_command(bank, 0x30, flash_address(bank, i, 0x0));
+               retval = cfi_send_command(bank, 0x30, cfi_flash_address(bank, i, 0x0));
                if (retval != ERROR_OK)
                        return retval;
 
                if (cfi_spansion_wait_status_busy(bank, cfi_info->block_erase_timeout) == ERROR_OK)
                        bank->sectors[i].is_erased = 1;
                else {
-                       retval = cfi_send_command(bank, 0xf0, flash_address(bank, 0, 0x0));
+                       retval = cfi_send_command(bank, 0xf0, cfi_flash_address(bank, 0, 0x0));
                        if (retval != ERROR_OK)
                                return retval;
 
-                       LOG_ERROR("couldn't erase block %i of flash bank at base 0x%"
-                               PRIx32, i, bank->base);
+                       LOG_ERROR("couldn't erase block %i of flash bank at base "
+                               TARGET_ADDR_FMT, i, bank->base);
                        return ERROR_FLASH_OPERATION_FAILED;
                }
        }
 
-       return cfi_send_command(bank, 0xf0, flash_address(bank, 0, 0x0));
+       return cfi_send_command(bank, 0xf0, cfi_flash_address(bank, 0, 0x0));
 }
 
-static int cfi_erase(struct flash_bank *bank, int first, int last)
+int cfi_erase(struct flash_bank *bank, int first, int last)
 {
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
 
@@ -962,10 +989,8 @@ static int cfi_erase(struct flash_bank *bank, int first, int last)
                case 1:
                case 3:
                        return cfi_intel_erase(bank, first, last);
-                       break;
                case 2:
                        return cfi_spansion_erase(bank, first, last);
-                       break;
                default:
                        LOG_ERROR("cfi primary command set %i unsupported", cfi_info->pri_id);
                        break;
@@ -980,7 +1005,6 @@ static int cfi_intel_protect(struct flash_bank *bank, int set, int first, int la
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
        struct cfi_intel_pri_ext *pri_ext = cfi_info->pri_ext;
        int retry = 0;
-       int i;
 
        /* if the device supports neither legacy lock/unlock (bit 3) nor
         * instant individual block locking (bit 5).
@@ -992,17 +1016,17 @@ static int cfi_intel_protect(struct flash_bank *bank, int set, int first, int la
 
        cfi_intel_clear_status_register(bank);
 
-       for (i = first; i <= last; i++) {
-               retval = cfi_send_command(bank, 0x60, flash_address(bank, i, 0x0));
+       for (int i = first; i <= last; i++) {
+               retval = cfi_send_command(bank, 0x60, cfi_flash_address(bank, i, 0x0));
                if (retval != ERROR_OK)
                        return retval;
                if (set) {
-                       retval = cfi_send_command(bank, 0x01, flash_address(bank, i, 0x0));
+                       retval = cfi_send_command(bank, 0x01, cfi_flash_address(bank, i, 0x0));
                        if (retval != ERROR_OK)
                                return retval;
                        bank->sectors[i].is_protected = 1;
                } else {
-                       retval = cfi_send_command(bank, 0xd0, flash_address(bank, i, 0x0));
+                       retval = cfi_send_command(bank, 0xd0, cfi_flash_address(bank, i, 0x0));
                        if (retval != ERROR_OK)
                                return retval;
                        bank->sectors[i].is_protected = 0;
@@ -1019,7 +1043,7 @@ static int cfi_intel_protect(struct flash_bank *bank, int set, int first, int la
                } else {
                        uint8_t block_status;
                        /* read block lock bit, to verify status */
-                       retval = cfi_send_command(bank, 0x90, flash_address(bank, 0, 0x55));
+                       retval = cfi_send_command(bank, 0x90, cfi_flash_address(bank, 0, 0x55));
                        if (retval != ERROR_OK)
                                return retval;
                        retval = cfi_get_u8(bank, i, 0x2, &block_status);
@@ -1030,7 +1054,7 @@ static int cfi_intel_protect(struct flash_bank *bank, int set, int first, int la
                                LOG_ERROR(
                                        "couldn't change block lock status (set = %i, block_status = 0x%2.2x)",
                                        set, block_status);
-                               retval = cfi_send_command(bank, 0x70, flash_address(bank, 0, 0x55));
+                               retval = cfi_send_command(bank, 0x70, cfi_flash_address(bank, 0, 0x55));
                                if (retval != ERROR_OK)
                                        return retval;
                                uint8_t status;
@@ -1063,15 +1087,15 @@ static int cfi_intel_protect(struct flash_bank *bank, int set, int first, int la
                 * 3. re-protect what should be protected.
                 *
                 */
-               for (i = 0; i < bank->num_sectors; i++) {
+               for (int i = 0; i < bank->num_sectors; i++) {
                        if (bank->sectors[i].is_protected == 1) {
                                cfi_intel_clear_status_register(bank);
 
-                               retval = cfi_send_command(bank, 0x60, flash_address(bank, i, 0x0));
+                               retval = cfi_send_command(bank, 0x60, cfi_flash_address(bank, i, 0x0));
                                if (retval != ERROR_OK)
                                        return retval;
 
-                               retval = cfi_send_command(bank, 0x01, flash_address(bank, i, 0x0));
+                               retval = cfi_send_command(bank, 0x01, cfi_flash_address(bank, i, 0x0));
                                if (retval != ERROR_OK)
                                        return retval;
 
@@ -1083,10 +1107,10 @@ static int cfi_intel_protect(struct flash_bank *bank, int set, int first, int la
                }
        }
 
-       return cfi_send_command(bank, 0xff, flash_address(bank, 0, 0x0));
+       return cfi_send_command(bank, 0xff, cfi_flash_address(bank, 0, 0x0));
 }
 
-static int cfi_protect(struct flash_bank *bank, int set, int first, int last)
+int cfi_protect(struct flash_bank *bank, int set, int first, int last)
 {
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
 
@@ -1095,11 +1119,6 @@ static int cfi_protect(struct flash_bank *bank, int set, int first, int last)
                return ERROR_TARGET_NOT_HALTED;
        }
 
-       if ((first < 0) || (last < first) || (last >= bank->num_sectors)) {
-               LOG_ERROR("Invalid sector range");
-               return ERROR_FLASH_SECTOR_INVALID;
-       }
-
        if (cfi_info->qry[0] != 'Q')
                return ERROR_FLASH_BANK_NOT_PROBED;
 
@@ -1107,7 +1126,6 @@ static int cfi_protect(struct flash_bank *bank, int set, int first, int last)
                case 1:
                case 3:
                        return cfi_intel_protect(bank, set, first, last);
-                       break;
                default:
                        LOG_WARNING("protect: cfi primary command set %i unsupported", cfi_info->pri_id);
                        return ERROR_OK;
@@ -1123,13 +1141,10 @@ static uint32_t cfi_command_val(struct flash_bank *bank, uint8_t cmd)
        switch (bank->bus_width) {
                case 1:
                        return buf[0];
-                       break;
                case 2:
                        return target_buffer_get_u16(target, buf);
-                       break;
                case 4:
                        return target_buffer_get_u32(target, buf);
-                       break;
                default:
                        LOG_ERROR("Unsupported bank buswidth %d, can't do block memory writes",
                                        bank->bus_width);
@@ -1309,7 +1324,7 @@ static int cfi_intel_write_block(struct flash_bank *bank, const uint8_t *buffer,
        busy_pattern_val  = cfi_command_val(bank, 0x80);
        error_pattern_val = cfi_command_val(bank, 0x7e);
 
-       LOG_DEBUG("Using target buffer at 0x%08" PRIx32 " and of size 0x%04" PRIx32,
+       LOG_DEBUG("Using target buffer at " TARGET_ADDR_FMT " and of size 0x%04" PRIx32,
                source->address, buffer_size);
 
        /* Programming main loop */
@@ -1421,50 +1436,50 @@ static int cfi_spansion_write_block_mips(struct flash_bank *bank, const uint8_t
 
        static const uint32_t mips_word_16_code[] = {
                /* start:       */
-               MIPS32_LHU(9, 0, 4),                    /* lhu $t1, ($a0)               ; out = &saddr */
-               MIPS32_ADDI(4, 4, 2),                   /* addi $a0, $a0, 2             ; saddr += 2 */
-               MIPS32_SH(13, 0, 12),                   /* sh $t5, ($t4)                ; *fl_unl_addr1 = fl_unl_cmd1 */
-               MIPS32_SH(15, 0, 14),                   /* sh $t7, ($t6)                ; *fl_unl_addr2 = fl_unl_cmd2 */
-               MIPS32_SH(7, 0, 12),                    /* sh $a3, ($t4)                ; *fl_unl_addr1 = fl_write_cmd */
-               MIPS32_SH(9, 0, 5),                             /* sh $t1, ($a1)                ; *daddr = out */
+               MIPS32_LHU(0, 9, 0, 4),         /* lhu $t1, ($a0)               ; out = &saddr */
+               MIPS32_ADDI(0, 4, 4, 2),        /* addi $a0, $a0, 2             ; saddr += 2 */
+               MIPS32_SH(0, 13, 0, 12),        /* sh $t5, ($t4)                ; *fl_unl_addr1 = fl_unl_cmd1 */
+               MIPS32_SH(0, 15, 0, 14),        /* sh $t7, ($t6)                ; *fl_unl_addr2 = fl_unl_cmd2 */
+               MIPS32_SH(0, 7, 0, 12),         /* sh $a3, ($t4)                ; *fl_unl_addr1 = fl_write_cmd */
+               MIPS32_SH(0, 9, 0, 5),          /* sh $t1, ($a1)                ; *daddr = out */
                MIPS32_NOP,                                             /* nop */
                /* busy:        */
-               MIPS32_LHU(10, 0, 5),                   /* lhu $t2, ($a1)               ; temp1 = *daddr */
-               MIPS32_XOR(11, 9, 10),                  /* xor $t3, $a0, $t2    ; temp2 = out ^ temp1; */
-               MIPS32_AND(11, 8, 11),                  /* and $t3, $t0, $t3    ; temp2 = temp2 & DQ7mask */
-               MIPS32_BNE(11, 8, 13),                  /* bne $t3, $t0, cont   ; if (temp2 != DQ7mask) goto cont */
-               MIPS32_NOP,                                             /* nop                                                                  */
-
-               MIPS32_SRL(10, 8, 2),                   /* srl $t2,$t0,2                ; temp1 = DQ7mask >> 2 */
-               MIPS32_AND(11, 10, 11),                 /* and $t3, $t2, $t3    ; temp2 = temp2 & temp1 */
-               MIPS32_BNE(11, 10, NEG16(8)),   /* bne $t3, $t2, busy   ; if (temp2 != temp1) goto busy */
-               MIPS32_NOP,                                             /* nop                                                                  */
-
-               MIPS32_LHU(10, 0, 5),                   /* lhu $t2, ($a1)               ; temp1 = *daddr */
-               MIPS32_XOR(11, 9, 10),                  /* xor $t3, $a0, $t2    ; temp2 = out ^ temp1; */
-               MIPS32_AND(11, 8, 11),                  /* and $t3, $t0, $t3    ; temp2 = temp2 & DQ7mask */
-               MIPS32_BNE(11, 8, 4),                   /* bne $t3, $t0, cont   ; if (temp2 != DQ7mask) goto cont */
+               MIPS32_LHU(0, 10, 0, 5),                /* lhu $t2, ($a1)               ; temp1 = *daddr */
+               MIPS32_XOR(0, 11, 9, 10),               /* xor $t3, $a0, $t2    ; temp2 = out ^ temp1; */
+               MIPS32_AND(0, 11, 8, 11),               /* and $t3, $t0, $t3    ; temp2 = temp2 & DQ7mask */
+               MIPS32_BNE(0, 11, 8, 13),               /* bne $t3, $t0, cont   ; if (temp2 != DQ7mask) goto cont */
+               MIPS32_NOP,                                             /* nop                                  */
+
+               MIPS32_SRL(0, 10, 8, 2),                /* srl $t2,$t0,2                ; temp1 = DQ7mask >> 2 */
+               MIPS32_AND(0, 11, 10, 11),                      /* and $t3, $t2, $t3    ; temp2 = temp2 & temp1 */
+               MIPS32_BNE(0, 11, 10, NEG16(8)),        /* bne $t3, $t2, busy   ; if (temp2 != temp1) goto busy */
+               MIPS32_NOP,                                             /* nop                                  */
+
+               MIPS32_LHU(0, 10, 0, 5),                /* lhu $t2, ($a1)               ; temp1 = *daddr */
+               MIPS32_XOR(0, 11, 9, 10),               /* xor $t3, $a0, $t2    ; temp2 = out ^ temp1; */
+               MIPS32_AND(0, 11, 8, 11),               /* and $t3, $t0, $t3    ; temp2 = temp2 & DQ7mask */
+               MIPS32_BNE(0, 11, 8, 4),                /* bne $t3, $t0, cont   ; if (temp2 != DQ7mask) goto cont */
                MIPS32_NOP,                                             /* nop */
 
-               MIPS32_XOR(9, 9, 9),                    /* xor $t1, $t1, $t1    ; out = 0 */
-               MIPS32_BEQ(9, 0, 11),                   /* beq $t1, $zero, done ; if (out == 0) goto done */
+               MIPS32_XOR(0, 9, 9, 9),                 /* xor $t1, $t1, $t1    ; out = 0 */
+               MIPS32_BEQ(0, 9, 0, 11),                        /* beq $t1, $zero, done ; if (out == 0) goto done */
                MIPS32_NOP,                                             /* nop */
                /* cont:        */
-               MIPS32_ADDI(6, 6, NEG16(1)),    /* addi, $a2, $a2, -1   ; numwrites-- */
-               MIPS32_BNE(6, 0, 5),                    /* bne $a2, $zero, cont2        ; if (numwrite != 0) goto cont2 */
+               MIPS32_ADDI(0, 6, 6, NEG16(1)), /* addi, $a2, $a2, -1   ; numwrites-- */
+               MIPS32_BNE(0, 6, 0, 5),         /* bne $a2, $zero, cont2        ; if (numwrite != 0) goto cont2 */
                MIPS32_NOP,                                             /* nop */
 
-               MIPS32_LUI(9, 0),                               /* lui $t1, 0 */
-               MIPS32_ORI(9, 9, 0x80),                 /* ori $t1, $t1, 0x80   ; out = 0x80 */
+               MIPS32_LUI(0, 9, 0),                            /* lui $t1, 0 */
+               MIPS32_ORI(0, 9, 9, 0x80),                      /* ori $t1, $t1, 0x80   ; out = 0x80 */
 
-               MIPS32_B(4),                                    /* b done                       ; goto done */
+               MIPS32_B(0, 4),                                 /* b done                       ; goto done */
                MIPS32_NOP,                                             /* nop */
                /* cont2:       */
-               MIPS32_ADDI(5, 5, 2),                   /* addi $a0, $a0, 2     ; daddr += 2 */
-               MIPS32_B(NEG16(33)),                    /* b start                      ; goto start */
+               MIPS32_ADDI(0, 5, 5, 2),                        /* addi $a0, $a0, 2     ; daddr += 2 */
+               MIPS32_B(0, NEG16(33)),                 /* b start                      ; goto start */
                MIPS32_NOP,                                             /* nop */
                /* done: */
-               MIPS32_SDBBP,                                   /* sdbbp                        ; break(); */
+               MIPS32_SDBBP(0),                                        /* sdbbp                        ; break(); */
        };
 
        mips32_info.common_magic = MIPS32_COMMON_MAGIC;
@@ -1560,9 +1575,9 @@ static int cfi_spansion_write_block_mips(struct flash_bank *bank, const uint8_t
                buf_set_u32(reg_params[2].value, 0, 32, thisrun_count / bank->bus_width);
                buf_set_u32(reg_params[3].value, 0, 32, cfi_command_val(bank, 0xA0));
                buf_set_u32(reg_params[4].value, 0, 32, cfi_command_val(bank, 0x80));
-               buf_set_u32(reg_params[6].value, 0, 32, flash_address(bank, 0, pri_ext->_unlock1));
+               buf_set_u32(reg_params[6].value, 0, 32, cfi_flash_address(bank, 0, pri_ext->_unlock1));
                buf_set_u32(reg_params[7].value, 0, 32, 0xaaaaaaaa);
-               buf_set_u32(reg_params[8].value, 0, 32, flash_address(bank, 0, pri_ext->_unlock2));
+               buf_set_u32(reg_params[8].value, 0, 32, cfi_flash_address(bank, 0, pri_ext->_unlock2));
                buf_set_u32(reg_params[9].value, 0, 32, 0x55555555);
 
                retval = target_run_algorithm(target, 0, NULL, 10, reg_params,
@@ -1939,9 +1954,9 @@ static int cfi_spansion_write_block(struct flash_bank *bank, const uint8_t *buff
                buf_set_u32(reg_params[2].value, 0, 32, thisrun_count / bank->bus_width);
                buf_set_u32(reg_params[3].value, 0, 32, cfi_command_val(bank, 0xA0));
                buf_set_u32(reg_params[4].value, 0, 32, cfi_command_val(bank, 0x80));
-               buf_set_u32(reg_params[6].value, 0, 32, flash_address(bank, 0, pri_ext->_unlock1));
+               buf_set_u32(reg_params[6].value, 0, 32, cfi_flash_address(bank, 0, pri_ext->_unlock1));
                buf_set_u32(reg_params[7].value, 0, 32, 0xaaaaaaaa);
-               buf_set_u32(reg_params[8].value, 0, 32, flash_address(bank, 0, pri_ext->_unlock2));
+               buf_set_u32(reg_params[8].value, 0, 32, cfi_flash_address(bank, 0, pri_ext->_unlock2));
                buf_set_u32(reg_params[9].value, 0, 32, 0x55555555);
 
                retval = target_run_algorithm(target, 0, NULL, 10, reg_params,
@@ -1983,14 +1998,13 @@ static int cfi_intel_write_word(struct flash_bank *bank, uint8_t *word, uint32_t
 {
        int retval;
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
-       struct target *target = bank->target;
 
        cfi_intel_clear_status_register(bank);
        retval = cfi_send_command(bank, 0x40, address);
        if (retval != ERROR_OK)
                return retval;
 
-       retval = target_write_memory(target, address, bank->bus_width, 1, word);
+       retval = cfi_target_write_memory(bank, address, 1, word);
        if (retval != ERROR_OK)
                return retval;
 
@@ -1999,12 +2013,13 @@ static int cfi_intel_write_word(struct flash_bank *bank, uint8_t *word, uint32_t
        if (retval != ERROR_OK)
                return retval;
        if (status != 0x80) {
-               retval = cfi_send_command(bank, 0xff, flash_address(bank, 0, 0x0));
+               retval = cfi_send_command(bank, 0xff, cfi_flash_address(bank, 0, 0x0));
                if (retval != ERROR_OK)
                        return retval;
 
-               LOG_ERROR("couldn't write word at base 0x%" PRIx32 ", address 0x%" PRIx32,
-                       bank->base, address);
+               LOG_ERROR("couldn't write word at base " TARGET_ADDR_FMT
+                               ", address 0x%" PRIx32,
+                               bank->base, address);
                return ERROR_FLASH_OPERATION_FAILED;
        }
 
@@ -2016,7 +2031,6 @@ static int cfi_intel_write_words(struct flash_bank *bank, const uint8_t *word,
 {
        int retval;
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
-       struct target *target = bank->target;
 
        /* Calculate buffer size and boundary mask
         * buffersize is (buffer size per chip) * (number of chips)
@@ -2028,9 +2042,9 @@ static int cfi_intel_write_words(struct flash_bank *bank, const uint8_t *word,
 
        /* Check for valid range */
        if (address & buffermask) {
-               LOG_ERROR("Write address at base 0x%" PRIx32 ", address 0x%" PRIx32
-                       " not aligned to 2^%d boundary",
-                       bank->base, address, cfi_info->max_buf_write_size);
+               LOG_ERROR("Write address at base " TARGET_ADDR_FMT ", address 0x%"
+                               PRIx32 " not aligned to 2^%d boundary",
+                               bank->base, address, cfi_info->max_buf_write_size);
                return ERROR_FLASH_OPERATION_FAILED;
        }
 
@@ -2053,12 +2067,13 @@ static int cfi_intel_write_words(struct flash_bank *bank, const uint8_t *word,
        if (retval != ERROR_OK)
                return retval;
        if (status != 0x80) {
-               retval = cfi_send_command(bank, 0xff, flash_address(bank, 0, 0x0));
+               retval = cfi_send_command(bank, 0xff, cfi_flash_address(bank, 0, 0x0));
                if (retval != ERROR_OK)
                        return retval;
 
                LOG_ERROR(
-                       "couldn't start buffer write operation at base 0x%" PRIx32 ", address 0x%" PRIx32,
+                       "couldn't start buffer write operation at base " TARGET_ADDR_FMT
+                       ", address 0x%" PRIx32,
                        bank->base,
                        address);
                return ERROR_FLASH_OPERATION_FAILED;
@@ -2069,7 +2084,7 @@ static int cfi_intel_write_words(struct flash_bank *bank, const uint8_t *word,
        if (retval != ERROR_OK)
                return retval;
 
-       retval = target_write_memory(target, address, bank->bus_width, bufferwsize, word);
+       retval = cfi_target_write_memory(bank, address, bufferwsize, word);
        if (retval != ERROR_OK)
                return retval;
 
@@ -2083,11 +2098,11 @@ static int cfi_intel_write_words(struct flash_bank *bank, const uint8_t *word,
                return retval;
 
        if (status != 0x80) {
-               retval = cfi_send_command(bank, 0xff, flash_address(bank, 0, 0x0));
+               retval = cfi_send_command(bank, 0xff, cfi_flash_address(bank, 0, 0x0));
                if (retval != ERROR_OK)
                        return retval;
 
-               LOG_ERROR("Buffer write at base 0x%" PRIx32
+               LOG_ERROR("Buffer write at base " TARGET_ADDR_FMT
                        ", address 0x%" PRIx32 " failed.", bank->base, address);
                return ERROR_FLASH_OPERATION_FAILED;
        }
@@ -2100,30 +2115,25 @@ static int cfi_spansion_write_word(struct flash_bank *bank, uint8_t *word, uint3
        int retval;
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
        struct cfi_spansion_pri_ext *pri_ext = cfi_info->pri_ext;
-       struct target *target = bank->target;
-
-       retval = cfi_send_command(bank, 0xaa, flash_address(bank, 0, pri_ext->_unlock1));
-       if (retval != ERROR_OK)
-               return retval;
 
-       retval = cfi_send_command(bank, 0x55, flash_address(bank, 0, pri_ext->_unlock2));
+       retval = cfi_spansion_unlock_seq(bank);
        if (retval != ERROR_OK)
                return retval;
 
-       retval = cfi_send_command(bank, 0xa0, flash_address(bank, 0, pri_ext->_unlock1));
+       retval = cfi_send_command(bank, 0xa0, cfi_flash_address(bank, 0, pri_ext->_unlock1));
        if (retval != ERROR_OK)
                return retval;
 
-       retval = target_write_memory(target, address, bank->bus_width, 1, word);
+       retval = cfi_target_write_memory(bank, address, 1, word);
        if (retval != ERROR_OK)
                return retval;
 
        if (cfi_spansion_wait_status_busy(bank, cfi_info->word_write_timeout) != ERROR_OK) {
-               retval = cfi_send_command(bank, 0xf0, flash_address(bank, 0, 0x0));
+               retval = cfi_send_command(bank, 0xf0, cfi_flash_address(bank, 0, 0x0));
                if (retval != ERROR_OK)
                        return retval;
 
-               LOG_ERROR("couldn't write word at base 0x%" PRIx32
+               LOG_ERROR("couldn't write word at base " TARGET_ADDR_FMT
                        ", address 0x%" PRIx32, bank->base, address);
                return ERROR_FLASH_OPERATION_FAILED;
        }
@@ -2136,8 +2146,6 @@ static int cfi_spansion_write_words(struct flash_bank *bank, const uint8_t *word
 {
        int retval;
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
-       struct target *target = bank->target;
-       struct cfi_spansion_pri_ext *pri_ext = cfi_info->pri_ext;
 
        /* Calculate buffer size and boundary mask
         * buffersize is (buffer size per chip) * (number of chips)
@@ -2149,7 +2157,7 @@ static int cfi_spansion_write_words(struct flash_bank *bank, const uint8_t *word
 
        /* Check for valid range */
        if (address & buffermask) {
-               LOG_ERROR("Write address at base 0x%" PRIx32
+               LOG_ERROR("Write address at base " TARGET_ADDR_FMT
                        ", address 0x%" PRIx32 " not aligned to 2^%d boundary",
                        bank->base, address, cfi_info->max_buf_write_size);
                return ERROR_FLASH_OPERATION_FAILED;
@@ -2163,11 +2171,7 @@ static int cfi_spansion_write_words(struct flash_bank *bank, const uint8_t *word
        }
 
        /* Unlock */
-       retval = cfi_send_command(bank, 0xaa, flash_address(bank, 0, pri_ext->_unlock1));
-       if (retval != ERROR_OK)
-               return retval;
-
-       retval = cfi_send_command(bank, 0x55, flash_address(bank, 0, pri_ext->_unlock2));
+       retval = cfi_spansion_unlock_seq(bank);
        if (retval != ERROR_OK)
                return retval;
 
@@ -2181,7 +2185,7 @@ static int cfi_spansion_write_words(struct flash_bank *bank, const uint8_t *word
        if (retval != ERROR_OK)
                return retval;
 
-       retval = target_write_memory(target, address, bank->bus_width, bufferwsize, word);
+       retval = cfi_target_write_memory(bank, address, bufferwsize, word);
        if (retval != ERROR_OK)
                return retval;
 
@@ -2191,11 +2195,11 @@ static int cfi_spansion_write_words(struct flash_bank *bank, const uint8_t *word
                return retval;
 
        if (cfi_spansion_wait_status_busy(bank, cfi_info->buf_write_timeout) != ERROR_OK) {
-               retval = cfi_send_command(bank, 0xf0, flash_address(bank, 0, 0x0));
+               retval = cfi_send_command(bank, 0xf0, cfi_flash_address(bank, 0, 0x0));
                if (retval != ERROR_OK)
                        return retval;
 
-               LOG_ERROR("couldn't write block at base 0x%" PRIx32
+               LOG_ERROR("couldn't write block at base " TARGET_ADDR_FMT
                        ", address 0x%" PRIx32 ", size 0x%" PRIx32, bank->base, address,
                        bufferwsize);
                return ERROR_FLASH_OPERATION_FAILED;
@@ -2204,7 +2208,7 @@ static int cfi_spansion_write_words(struct flash_bank *bank, const uint8_t *word
        return ERROR_OK;
 }
 
-static int cfi_write_word(struct flash_bank *bank, uint8_t *word, uint32_t address)
+int cfi_write_word(struct flash_bank *bank, uint8_t *word, uint32_t address)
 {
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
 
@@ -2212,10 +2216,8 @@ static int cfi_write_word(struct flash_bank *bank, uint8_t *word, uint32_t addre
                case 1:
                case 3:
                        return cfi_intel_write_word(bank, word, address);
-                       break;
                case 2:
                        return cfi_spansion_write_word(bank, word, address);
-                       break;
                default:
                        LOG_ERROR("cfi primary command set %i unsupported", cfi_info->pri_id);
                        break;
@@ -2239,10 +2241,8 @@ static int cfi_write_words(struct flash_bank *bank, const uint8_t *word,
                case 1:
                case 3:
                        return cfi_intel_write_words(bank, word, wordcount, address);
-                       break;
                case 2:
                        return cfi_spansion_write_words(bank, word, wordcount, address);
-                       break;
                default:
                        LOG_ERROR("cfi primary command set %i unsupported", cfi_info->pri_id);
                        break;
@@ -2254,12 +2254,10 @@ static int cfi_write_words(struct flash_bank *bank, const uint8_t *word,
 static int cfi_read(struct flash_bank *bank, uint8_t *buffer, uint32_t offset, uint32_t count)
 {
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
-       struct target *target = bank->target;
        uint32_t address = bank->base + offset;
        uint32_t read_p;
        int align;      /* number of unaligned bytes */
        uint8_t current_word[CFI_MAX_BUS_WIDTH];
-       int i;
        int retval;
 
        LOG_DEBUG("reading buffer of %i byte at 0x%8.8x",
@@ -2283,12 +2281,12 @@ static int cfi_read(struct flash_bank *bank, uint8_t *buffer, uint32_t offset, u
                LOG_INFO("Fixup %d unaligned read head bytes", align);
 
                /* read a complete word from flash */
-               retval = target_read_memory(target, read_p, bank->bus_width, 1, current_word);
+               retval = cfi_target_read_memory(bank, read_p, 1, current_word);
                if (retval != ERROR_OK)
                        return retval;
 
                /* take only bytes we need */
-               for (i = align; (i < bank->bus_width) && (count > 0); i++, count--)
+               for (int i = align; (i < bank->bus_width) && (count > 0); i++, count--)
                        *buffer++ = current_word[i];
 
                read_p += bank->bus_width;
@@ -2296,7 +2294,7 @@ static int cfi_read(struct flash_bank *bank, uint8_t *buffer, uint32_t offset, u
 
        align = count / bank->bus_width;
        if (align) {
-               retval = target_read_memory(target, read_p, bank->bus_width, align, buffer);
+               retval = cfi_target_read_memory(bank, read_p, align, buffer);
                if (retval != ERROR_OK)
                        return retval;
 
@@ -2309,12 +2307,12 @@ static int cfi_read(struct flash_bank *bank, uint8_t *buffer, uint32_t offset, u
                LOG_INFO("Fixup %" PRIu32 " unaligned read tail bytes", count);
 
                /* read a complete word from flash */
-               retval = target_read_memory(target, read_p, bank->bus_width, 1, current_word);
+               retval = cfi_target_read_memory(bank, read_p, 1, current_word);
                if (retval != ERROR_OK)
                        return retval;
 
                /* take only bytes we need */
-               for (i = 0; (i < bank->bus_width) && (count > 0); i++, count--)
+               for (int i = 0; (i < bank->bus_width) && (count > 0); i++, count--)
                        *buffer++ = current_word[i];
        }
 
@@ -2324,14 +2322,14 @@ static int cfi_read(struct flash_bank *bank, uint8_t *buffer, uint32_t offset, u
 static int cfi_write(struct flash_bank *bank, const uint8_t *buffer, uint32_t offset, uint32_t count)
 {
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
-       struct target *target = bank->target;
        uint32_t address = bank->base + offset; /* address of first byte to be programmed */
        uint32_t write_p;
        int align;      /* number of unaligned bytes */
        int blk_count;  /* number of bus_width bytes for block copy */
        uint8_t current_word[CFI_MAX_BUS_WIDTH * 4];    /* word (bus_width size) currently being
                                                         *programmed */
-       int i;
+       uint8_t *swapped_buffer = NULL;
+       const uint8_t *real_buffer = NULL;
        int retval;
 
        if (bank->target->state != TARGET_HALTED) {
@@ -2352,13 +2350,19 @@ static int cfi_write(struct flash_bank *bank, const uint8_t *buffer, uint32_t of
                LOG_INFO("Fixup %d unaligned head bytes", align);
 
                /* read a complete word from flash */
-               retval = target_read_memory(target, write_p, bank->bus_width, 1, current_word);
+               retval = cfi_target_read_memory(bank, write_p, 1, current_word);
                if (retval != ERROR_OK)
                        return retval;
 
                /* replace only bytes that must be written */
-               for (i = align; (i < bank->bus_width) && (count > 0); i++, count--)
-                       current_word[i] = *buffer++;
+               for (int i = align;
+                    (i < bank->bus_width) && (count > 0);
+                    i++, count--)
+                       if (cfi_info->data_swap)
+                               /* data bytes are swapped (reverse endianness) */
+                               current_word[bank->bus_width - i] = *buffer++;
+                       else
+                               current_word[i] = *buffer++;
 
                retval = cfi_write_word(bank, current_word, write_p);
                if (retval != ERROR_OK)
@@ -2366,6 +2370,22 @@ static int cfi_write(struct flash_bank *bank, const uint8_t *buffer, uint32_t of
                write_p += bank->bus_width;
        }
 
+       if (cfi_info->data_swap && count) {
+               swapped_buffer = malloc(count & ~(bank->bus_width - 1));
+               switch (bank->bus_width) {
+               case 2:
+                       buf_bswap16(swapped_buffer, buffer,
+                                   count & ~(bank->bus_width - 1));
+                       break;
+               case 4:
+                       buf_bswap32(swapped_buffer, buffer,
+                                   count & ~(bank->bus_width - 1));
+                       break;
+               }
+               real_buffer = buffer;
+               buffer = swapped_buffer;
+       }
+
        /* handle blocks of bus_size aligned bytes */
        blk_count = count & ~(bank->bus_width - 1);     /* round down, leave tail bytes */
        switch (cfi_info->pri_id) {
@@ -2401,12 +2421,12 @@ static int cfi_write(struct flash_bank *bank, const uint8_t *buffer, uint32_t of
 
                        /* fall back to memory writes */
                        while (count >= (uint32_t)bank->bus_width) {
-                               int fallback;
+                               bool fallback;
                                if ((write_p & 0xff) == 0) {
                                        LOG_INFO("Programming at 0x%08" PRIx32 ", count 0x%08"
                                                PRIx32 " bytes remaining", write_p, count);
                                }
-                               fallback = 1;
+                               fallback = true;
                                if ((bufferwsize > 0) && (count >= buffersize) &&
                                                !(write_p & buffermask)) {
                                        retval = cfi_write_words(bank, buffer, bufferwsize, write_p);
@@ -2414,13 +2434,13 @@ static int cfi_write(struct flash_bank *bank, const uint8_t *buffer, uint32_t of
                                                buffer += buffersize;
                                                write_p += buffersize;
                                                count -= buffersize;
-                                               fallback = 0;
+                                               fallback = false;
                                        } else if (retval != ERROR_FLASH_OPER_UNSUPPORTED)
                                                return retval;
                                }
                                /* try the slow way? */
                                if (fallback) {
-                                       for (i = 0; i < bank->bus_width; i++)
+                                       for (int i = 0; i < bank->bus_width; i++)
                                                current_word[i] = *buffer++;
 
                                        retval = cfi_write_word(bank, current_word, write_p);
@@ -2435,6 +2455,11 @@ static int cfi_write(struct flash_bank *bank, const uint8_t *buffer, uint32_t of
                        return retval;
        }
 
+       if (swapped_buffer) {
+               buffer = real_buffer + (buffer - swapped_buffer);
+               free(swapped_buffer);
+       }
+
        /* return to read array mode, so we can read from flash again for padding */
        retval = cfi_reset(bank);
        if (retval != ERROR_OK)
@@ -2445,13 +2470,17 @@ static int cfi_write(struct flash_bank *bank, const uint8_t *buffer, uint32_t of
                LOG_INFO("Fixup %" PRId32 " unaligned tail bytes", count);
 
                /* read a complete word from flash */
-               retval = target_read_memory(target, write_p, bank->bus_width, 1, current_word);
+               retval = cfi_target_read_memory(bank, write_p, 1, current_word);
                if (retval != ERROR_OK)
                        return retval;
 
                /* replace only bytes that must be written */
-               for (i = 0; (i < bank->bus_width) && (count > 0); i++, count--)
-                       current_word[i] = *buffer++;
+               for (int i = 0; (i < bank->bus_width) && (count > 0); i++, count--)
+                       if (cfi_info->data_swap)
+                               /* data bytes are swapped (reverse endianness) */
+                               current_word[bank->bus_width - i] = *buffer++;
+                       else
+                               current_word[i] = *buffer++;
 
                retval = cfi_write_word(bank, current_word, write_p);
                if (retval != ERROR_OK)
@@ -2473,7 +2502,6 @@ static void cfi_fixup_reversed_erase_regions(struct flash_bank *bank, const void
 
 static void cfi_fixup_0002_erase_regions(struct flash_bank *bank, const void *param)
 {
-       int i;
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
        struct cfi_spansion_pri_ext *pri_ext = cfi_info->pri_ext;
        (void) param;
@@ -2481,7 +2509,7 @@ static void cfi_fixup_0002_erase_regions(struct flash_bank *bank, const void *pa
        if ((pri_ext->_reversed_geometry) || (pri_ext->TopBottom == 3)) {
                LOG_DEBUG("swapping reversed erase region information on cmdset 0002 device");
 
-               for (i = 0; i < cfi_info->num_erase_regions / 2; i++) {
+               for (unsigned int i = 0; i < cfi_info->num_erase_regions / 2; i++) {
                        int j = (cfi_info->num_erase_regions - 1) - i;
                        uint32_t swap;
 
@@ -2516,7 +2544,7 @@ static int cfi_query_string(struct flash_bank *bank, int address)
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
        int retval;
 
-       retval = cfi_send_command(bank, 0x98, flash_address(bank, 0, address));
+       retval = cfi_send_command(bank, 0x98, cfi_flash_address(bank, 0, address));
        if (retval != ERROR_OK)
                return retval;
 
@@ -2544,12 +2572,11 @@ static int cfi_query_string(struct flash_bank *bank, int address)
        return ERROR_OK;
 }
 
-static int cfi_probe(struct flash_bank *bank)
+int cfi_probe(struct flash_bank *bank)
 {
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
        struct target *target = bank->target;
        int num_sectors = 0;
-       int i;
        int sector = 0;
        uint32_t unlock1 = 0x555;
        uint32_t unlock2 = 0x2aa;
@@ -2561,7 +2588,7 @@ static int cfi_probe(struct flash_bank *bank)
                return ERROR_TARGET_NOT_HALTED;
        }
 
-       cfi_info->probed = 0;
+       cfi_info->probed = false;
        cfi_info->num_erase_regions = 0;
        if (bank->sectors) {
                free(bank->sectors);
@@ -2581,22 +2608,22 @@ static int cfi_probe(struct flash_bank *bank)
        }
 
        /* switch to read identifier codes mode ("AUTOSELECT") */
-       retval = cfi_send_command(bank, 0xaa, flash_address(bank, 0, unlock1));
+       retval = cfi_send_command(bank, 0xaa, cfi_flash_address(bank, 0, unlock1));
        if (retval != ERROR_OK)
                return retval;
-       retval = cfi_send_command(bank, 0x55, flash_address(bank, 0, unlock2));
+       retval = cfi_send_command(bank, 0x55, cfi_flash_address(bank, 0, unlock2));
        if (retval != ERROR_OK)
                return retval;
-       retval = cfi_send_command(bank, 0x90, flash_address(bank, 0, unlock1));
+       retval = cfi_send_command(bank, 0x90, cfi_flash_address(bank, 0, unlock1));
        if (retval != ERROR_OK)
                return retval;
 
-       retval = target_read_memory(target, flash_address(bank, 0, 0x00),
-                       bank->bus_width, 1, value_buf0);
+       retval = cfi_target_read_memory(bank, cfi_flash_address(bank, 0, 0x00),
+                                       1, value_buf0);
        if (retval != ERROR_OK)
                return retval;
-       retval = target_read_memory(target, flash_address(bank, 0, 0x01),
-                       bank->bus_width, 1, value_buf1);
+       retval = cfi_target_read_memory(bank, cfi_flash_address(bank, 0, 0x01),
+                                       1, value_buf1);
        if (retval != ERROR_OK)
                return retval;
        switch (bank->chip_width) {
@@ -2733,7 +2760,7 @@ static int cfi_probe(struct flash_bank *bank)
                if (cfi_info->num_erase_regions) {
                        cfi_info->erase_region_info = malloc(sizeof(*cfi_info->erase_region_info)
                                        * cfi_info->num_erase_regions);
-                       for (i = 0; i < cfi_info->num_erase_regions; i++) {
+                       for (unsigned int i = 0; i < cfi_info->num_erase_regions; i++) {
                                retval = cfi_query_u32(bank,
                                                0,
                                                0x2d + (4 * i),
@@ -2848,15 +2875,14 @@ static int cfi_probe(struct flash_bank *bank)
        } else {
                uint32_t offset = 0;
 
-               for (i = 0; i < cfi_info->num_erase_regions; i++)
+               for (unsigned int i = 0; i < cfi_info->num_erase_regions; i++)
                        num_sectors += (cfi_info->erase_region_info[i] & 0xffff) + 1;
 
                bank->num_sectors = num_sectors;
                bank->sectors = malloc(sizeof(struct flash_sector) * num_sectors);
 
-               for (i = 0; i < cfi_info->num_erase_regions; i++) {
-                       uint32_t j;
-                       for (j = 0; j < (cfi_info->erase_region_info[i] & 0xffff) + 1; j++) {
+               for (unsigned int i = 0; i < cfi_info->num_erase_regions; i++) {
+                       for (uint32_t j = 0; j < (cfi_info->erase_region_info[i] & 0xffff) + 1; j++) {
                                bank->sectors[sector].offset = offset;
                                bank->sectors[sector].size =
                                        ((cfi_info->erase_region_info[i] >> 16) * 256)
@@ -2869,18 +2895,18 @@ static int cfi_probe(struct flash_bank *bank)
                }
                if (offset != (cfi_info->dev_size * bank->bus_width / bank->chip_width)) {
                        LOG_WARNING(
-                               "CFI size is 0x%" PRIx32 ", but total sector size is 0x%" PRIx32 "", \
+                               "CFI size is 0x%" PRIx32 ", but total sector size is 0x%" PRIx32 "",
                                (cfi_info->dev_size * bank->bus_width / bank->chip_width),
                                offset);
                }
        }
 
-       cfi_info->probed = 1;
+       cfi_info->probed = true;
 
        return ERROR_OK;
 }
 
-static int cfi_auto_probe(struct flash_bank *bank)
+int cfi_auto_probe(struct flash_bank *bank)
 {
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
        if (cfi_info->probed)
@@ -2893,17 +2919,16 @@ static int cfi_intel_protect_check(struct flash_bank *bank)
        int retval;
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
        struct cfi_intel_pri_ext *pri_ext = cfi_info->pri_ext;
-       int i;
 
        /* check if block lock bits are supported on this device */
        if (!(pri_ext->blk_status_reg_mask & 0x1))
                return ERROR_FLASH_OPERATION_FAILED;
 
-       retval = cfi_send_command(bank, 0x90, flash_address(bank, 0, 0x55));
+       retval = cfi_send_command(bank, 0x90, cfi_flash_address(bank, 0, 0x55));
        if (retval != ERROR_OK)
                return retval;
 
-       for (i = 0; i < bank->num_sectors; i++) {
+       for (int i = 0; i < bank->num_sectors; i++) {
                uint8_t block_status;
                retval = cfi_get_u8(bank, i, 0x2, &block_status);
                if (retval != ERROR_OK)
@@ -2915,7 +2940,7 @@ static int cfi_intel_protect_check(struct flash_bank *bank)
                        bank->sectors[i].is_protected = 0;
        }
 
-       return cfi_send_command(bank, 0xff, flash_address(bank, 0, 0x0));
+       return cfi_send_command(bank, 0xff, cfi_flash_address(bank, 0, 0x0));
 }
 
 static int cfi_spansion_protect_check(struct flash_bank *bank)
@@ -2923,21 +2948,16 @@ static int cfi_spansion_protect_check(struct flash_bank *bank)
        int retval;
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
        struct cfi_spansion_pri_ext *pri_ext = cfi_info->pri_ext;
-       int i;
 
-       retval = cfi_send_command(bank, 0xaa, flash_address(bank, 0, pri_ext->_unlock1));
+       retval = cfi_spansion_unlock_seq(bank);
        if (retval != ERROR_OK)
                return retval;
 
-       retval = cfi_send_command(bank, 0x55, flash_address(bank, 0, pri_ext->_unlock2));
+       retval = cfi_send_command(bank, 0x90, cfi_flash_address(bank, 0, pri_ext->_unlock1));
        if (retval != ERROR_OK)
                return retval;
 
-       retval = cfi_send_command(bank, 0x90, flash_address(bank, 0, pri_ext->_unlock1));
-       if (retval != ERROR_OK)
-               return retval;
-
-       for (i = 0; i < bank->num_sectors; i++) {
+       for (int i = 0; i < bank->num_sectors; i++) {
                uint8_t block_status;
                retval = cfi_get_u8(bank, i, 0x2, &block_status);
                if (retval != ERROR_OK)
@@ -2949,10 +2969,10 @@ static int cfi_spansion_protect_check(struct flash_bank *bank)
                        bank->sectors[i].is_protected = 0;
        }
 
-       return cfi_send_command(bank, 0xf0, flash_address(bank, 0, 0x0));
+       return cfi_send_command(bank, 0xf0, cfi_flash_address(bank, 0, 0x0));
 }
 
-static int cfi_protect_check(struct flash_bank *bank)
+int cfi_protect_check(struct flash_bank *bank)
 {
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
 
@@ -2968,10 +2988,8 @@ static int cfi_protect_check(struct flash_bank *bank)
                case 1:
                case 3:
                        return cfi_intel_protect_check(bank);
-                       break;
                case 2:
                        return cfi_spansion_protect_check(bank);
-                       break;
                default:
                        LOG_ERROR("cfi primary command set %i unsupported", cfi_info->pri_id);
                        break;
@@ -2980,7 +2998,7 @@ static int cfi_protect_check(struct flash_bank *bank)
        return ERROR_OK;
 }
 
-static int get_cfi_info(struct flash_bank *bank, char *buf, int buf_size)
+int cfi_get_info(struct flash_bank *bank, char *buf, int buf_size)
 {
        int printed;
        struct cfi_flash_bank *cfi_info = bank->driver_priv;
@@ -3079,7 +3097,7 @@ static void cfi_fixup_0002_write_buffer(struct flash_bank *bank, const void *par
        cfi_info->buf_write_timeout_typ = 0;
 }
 
-struct flash_driver cfi_flash = {
+const struct flash_driver cfi_flash = {
        .name = "cfi",
        .flash_bank_command = cfi_flash_bank_command,
        .erase = cfi_erase,
@@ -3091,5 +3109,6 @@ struct flash_driver cfi_flash = {
        /* FIXME: access flash at bus_width size */
        .erase_check = default_flash_blank_check,
        .protect_check = cfi_protect_check,
-       .info = get_cfi_info,
+       .info = cfi_get_info,
+       .free_driver_priv = default_flash_free_driver_priv,
 };

Linking to existing account procedure

If you already have an account and want to add another login method you MUST first sign in with your existing account and then change URL to read https://review.openocd.org/login/?link to get to this page again but this time it'll work for linking. Thank you.

SSH host keys fingerprints

1024 SHA256:YKx8b7u5ZWdcbp7/4AeXNaqElP49m6QrwfXaqQGJAOk gerrit-code-review@openocd.zylin.com (DSA)
384 SHA256:jHIbSQa4REvwCFG4cq5LBlBLxmxSqelQPem/EXIrxjk gerrit-code-review@openocd.org (ECDSA)
521 SHA256:UAOPYkU9Fjtcao0Ul/Rrlnj/OsQvt+pgdYSZ4jOYdgs gerrit-code-review@openocd.org (ECDSA)
256 SHA256:A13M5QlnozFOvTllybRZH6vm7iSt0XLxbA48yfc2yfY gerrit-code-review@openocd.org (ECDSA)
256 SHA256:spYMBqEYoAOtK7yZBrcwE8ZpYt6b68Cfh9yEVetvbXg gerrit-code-review@openocd.org (ED25519)
+--[ED25519 256]--+
|=..              |
|+o..   .         |
|*.o   . .        |
|+B . . .         |
|Bo. = o S        |
|Oo.+ + =         |
|oB=.* = . o      |
| =+=.+   + E     |
|. .=o   . o      |
+----[SHA256]-----+
2048 SHA256:0Onrb7/PHjpo6iVZ7xQX2riKN83FJ3KGU0TvI0TaFG4 gerrit-code-review@openocd.zylin.com (RSA)