flash/nor: make all working area pointers local
[openocd.git] / src / flash / nor / fm3.c
index 1e2adf55f27a2631dcf16dbb5c2520ef87283798..224090db8c1f1a6d2570f2a51810b05d956dbf8b 100644 (file)
@@ -1,7 +1,6 @@
 /***************************************************************************
  *   Copyright (C) 2011 by Marc Willam, Holger Wech                        *
- *   openOCD.fseu(AT)de.fujitsu.com                                        *
- *                                                                         *
+ *       openOCD.fseu(AT)de.fujitsu.com                                    *
  *   Copyright (C) 2011 Ronny Strutz                                       *
  *                                                                         *
  *   This program is free software; you can redistribute it and/or modify  *
 #include <target/algorithm.h>
 #include <target/armv7m.h>
 
-#define FLASH_DQ6 0x00000040   /* Data toggle flag bit (TOGG) */
-#define FLASH_DQ5 0x00000020   /* Time limit exceeding flag bit (TLOV) */
+#define FLASH_DQ6 0x00000040   /* Data toggle flag bit (TOGG) position */
+#define FLASH_DQ5 0x00000020   /* Time limit exceeding flag bit (TLOV) position */
 
-enum fm3_variant
-{
+enum fm3_variant {
        mb9bfxx1,       /* Flash Type '1' */
        mb9bfxx2,
        mb9bfxx3,
@@ -48,16 +46,13 @@ enum fm3_variant
        mb9afxx6
 };
 
-enum fm3_flash_type
-{
+enum fm3_flash_type {
        fm3_no_flash_type = 0,
        fm3_flash_type1   = 1,
        fm3_flash_type2   = 2
 };
 
-struct fm3_flash_bank
-{
-       struct working_area *write_algorithm;
+struct fm3_flash_bank {
        enum fm3_variant variant;
        enum fm3_flash_type flashtype;
        int probed;
@@ -68,87 +63,57 @@ FLASH_BANK_COMMAND_HANDLER(fm3_flash_bank_command)
        struct fm3_flash_bank *fm3_info;
 
        if (CMD_ARGC < 6)
-       {
-               LOG_WARNING("incomplete flash_bank fm3 configuration");
-               return ERROR_FLASH_BANK_INVALID;
-       }
+               return ERROR_COMMAND_SYNTAX_ERROR;
 
        fm3_info = malloc(sizeof(struct fm3_flash_bank));
        bank->driver_priv = fm3_info;
 
        /* Flash type '1' */
-       if (strcmp(CMD_ARGV[5], "mb9bfxx1.cpu") == 0)
-       {
+       if (strcmp(CMD_ARGV[5], "mb9bfxx1.cpu") == 0) {
                fm3_info->variant = mb9bfxx1;
                fm3_info->flashtype = fm3_flash_type1;
-       }
-       else if (strcmp(CMD_ARGV[5], "mb9bfxx2.cpu") == 0)
-       {
+       } else if (strcmp(CMD_ARGV[5], "mb9bfxx2.cpu") == 0) {
                fm3_info->variant = mb9bfxx2;
                fm3_info->flashtype = fm3_flash_type1;
-       }
-       else if (strcmp(CMD_ARGV[5], "mb9bfxx3.cpu") == 0)
-       {
+       } else if (strcmp(CMD_ARGV[5], "mb9bfxx3.cpu") == 0) {
                fm3_info->variant = mb9bfxx3;
                fm3_info->flashtype = fm3_flash_type1;
-       }
-       else if (strcmp(CMD_ARGV[5], "mb9bfxx4.cpu") == 0)
-       {
+       } else if (strcmp(CMD_ARGV[5], "mb9bfxx4.cpu") == 0) {
                fm3_info->variant = mb9bfxx4;
                fm3_info->flashtype = fm3_flash_type1;
-       }
-       else if (strcmp(CMD_ARGV[5], "mb9bfxx5.cpu") == 0)
-       {
+       } else if (strcmp(CMD_ARGV[5], "mb9bfxx5.cpu") == 0) {
                fm3_info->variant = mb9bfxx5;
                fm3_info->flashtype = fm3_flash_type1;
-       }
-       else if (strcmp(CMD_ARGV[5], "mb9bfxx6.cpu") == 0)
-       {
+       } else if (strcmp(CMD_ARGV[5], "mb9bfxx6.cpu") == 0) {
                fm3_info->variant = mb9bfxx6;
                fm3_info->flashtype = fm3_flash_type1;
-       }
-
-       /* Flash type '2' */
-       else if (strcmp(CMD_ARGV[5], "mb9afxx1.cpu") == 0)
-       {
+       } else if (strcmp(CMD_ARGV[5], "mb9afxx1.cpu") == 0) {  /* Flash type '2' */
                fm3_info->variant = mb9afxx1;
                fm3_info->flashtype = fm3_flash_type2;
-       }
-       else if (strcmp(CMD_ARGV[5], "mb9afxx2.cpu") == 0)
-       {
+       } else if (strcmp(CMD_ARGV[5], "mb9afxx2.cpu") == 0) {
                fm3_info->variant = mb9afxx2;
                fm3_info->flashtype = fm3_flash_type2;
-       }
-       else if (strcmp(CMD_ARGV[5], "mb9afxx3.cpu") == 0)
-       {
+       } else if (strcmp(CMD_ARGV[5], "mb9afxx3.cpu") == 0) {
                fm3_info->variant = mb9afxx3;
                fm3_info->flashtype = fm3_flash_type2;
-       }
-       else if (strcmp(CMD_ARGV[5], "mb9afxx4.cpu") == 0)
-       {
+       } else if (strcmp(CMD_ARGV[5], "mb9afxx4.cpu") == 0) {
                fm3_info->variant = mb9afxx4;
                fm3_info->flashtype = fm3_flash_type2;
-       }
-       else if (strcmp(CMD_ARGV[5], "mb9afxx5.cpu") == 0)
-       {
+       } else if (strcmp(CMD_ARGV[5], "mb9afxx5.cpu") == 0) {
                fm3_info->variant = mb9afxx5;
                fm3_info->flashtype = fm3_flash_type2;
-       }
-       else if (strcmp(CMD_ARGV[5], "mb9afxx6.cpu") == 0)
-       {
+       } else if (strcmp(CMD_ARGV[5], "mb9afxx6.cpu") == 0) {
                fm3_info->variant = mb9afxx6;
                fm3_info->flashtype = fm3_flash_type2;
        }
 
        /* unknown Flash type */
-       else
-       {
+       else {
                LOG_ERROR("unknown fm3 variant: %s", CMD_ARGV[5]);
                free(fm3_info);
                return ERROR_FLASH_BANK_INVALID;
        }
 
-       fm3_info->write_algorithm = NULL;
        fm3_info->probed = 0;
 
        return ERROR_OK;
@@ -162,8 +127,7 @@ static int fm3_busy_wait(struct target *target, uint32_t offset, int timeout_ms)
        int ms = 0;
 
        /* While(1) loop exit via "break" and "return" on error */
-       while(1)
-       {
+       while (1) {
                /* dummy-read - see flash manual */
                retval = target_read_u16(target, offset, &state1);
                if (retval != ERROR_OK)
@@ -180,13 +144,10 @@ static int fm3_busy_wait(struct target *target, uint32_t offset, int timeout_ms)
                        return retval;
 
                /* Flash command finished via polled data equal? */
-               if ( (state1 & FLASH_DQ6) == (state2 & FLASH_DQ6) )
-               {
+               if ((state1 & FLASH_DQ6) == (state2 & FLASH_DQ6))
                        break;
-               }
                /* Timeout Flag? */
-               else if (state1 & FLASH_DQ5)
-               {
+               else if (state1 & FLASH_DQ5) {
                        /* Retry data polling */
 
                        /* Data polling 1 */
@@ -200,10 +161,8 @@ static int fm3_busy_wait(struct target *target, uint32_t offset, int timeout_ms)
                                return retval;
 
                        /* Flash command finished via polled data equal? */
-                       if ( (state1 & FLASH_DQ6) != (state2 & FLASH_DQ6) )
-                       {
+                       if ((state1 & FLASH_DQ6) != (state2 & FLASH_DQ6))
                                return ERROR_FLASH_OPERATION_FAILED;
-                       }
 
                        /* finish anyway */
                        break;
@@ -212,8 +171,7 @@ static int fm3_busy_wait(struct target *target, uint32_t offset, int timeout_ms)
                ++ms;
 
                /* Polling time exceeded? */
-               if (ms > timeout_ms)
-               {
+               if (ms > timeout_ms) {
                        LOG_ERROR("Polling data reading timed out!");
                        return ERROR_FLASH_OPERATION_FAILED;
                }
@@ -238,18 +196,13 @@ static int fm3_erase(struct flash_bank *bank, int first, int last)
 
        u32FlashType = (uint32_t) fm3_info->flashtype;
 
-       if (u32FlashType == fm3_flash_type1)
-       {
+       if (u32FlashType == fm3_flash_type1) {
                u32FlashSeqAddress1 = 0x00001550;
                u32FlashSeqAddress2 = 0x00000AA8;
-       }
-       else if (u32FlashType == fm3_flash_type2)
-       {
+       } else if (u32FlashType == fm3_flash_type2) {
                u32FlashSeqAddress1 = 0x00000AA8;
                u32FlashSeqAddress2 = 0x00000554;
-       }
-       else
-       {
+       } else {
                LOG_ERROR("Flash/Device type unknown!");
                return ERROR_FLASH_OPERATION_FAILED;
        }
@@ -271,12 +224,10 @@ static int fm3_erase(struct flash_bank *bank, int first, int last)
        if (retval != ERROR_OK)
                return retval;
 
-       for (sector = first ; sector <= last ; sector++)
-       {
+       for (sector = first ; sector <= last ; sector++) {
                uint32_t offset = bank->sectors[sector].offset;
 
-               for (odd = 0; odd < 2 ; odd++)
-               {
+               for (odd = 0; odd < 2 ; odd++) {
                        if (odd)
                                offset += 4;
 
@@ -318,8 +269,7 @@ static int fm3_erase(struct flash_bank *bank, int first, int last)
        if (retval != ERROR_OK)
                return retval;
 
-       /* dummy read of FASZR */
-       retval = target_read_u32(target, 0x40000000, &u32DummyRead);
+       retval = target_read_u32(target, 0x40000000, &u32DummyRead); /* dummy read of FASZR */
 
        return retval;
 }
@@ -330,6 +280,7 @@ static int fm3_write_block(struct flash_bank *bank, uint8_t *buffer,
        struct fm3_flash_bank *fm3_info = bank->driver_priv;
        struct target *target = bank->target;
        uint32_t buffer_size = 2048;            /* 8192 for MB9Bxx6! */
+       struct working_area *write_algorithm;
        struct working_area *source;
        uint32_t address = bank->base + offset;
        struct reg_param reg_params[6];
@@ -341,18 +292,13 @@ static int fm3_write_block(struct flash_bank *bank, uint8_t *buffer,
 
        u32FlashType = (uint32_t) fm3_info->flashtype;
 
-       if (u32FlashType == fm3_flash_type1)
-       {
+       if (u32FlashType == fm3_flash_type1) {
                u32FlashSeqAddress1 = 0x00001550;
                u32FlashSeqAddress2 = 0x00000AA8;
-       }
-       else if (u32FlashType == fm3_flash_type2)
-       {
+       } else if (u32FlashType == fm3_flash_type2) {
                u32FlashSeqAddress1 = 0x00000AA8;
                u32FlashSeqAddress2 = 0x00000554;
-       }
-       else
-       {
+       } else {
                LOG_ERROR("Flash/Device type unknown!");
                return ERROR_FLASH_OPERATION_FAILED;
        }
@@ -485,8 +431,8 @@ static int fm3_write_block(struct flash_bank *bank, uint8_t *buffer,
        /* The following address pointers assume, that the code is running from   */
        /* address 0x1FFF8008. These address pointers will be patched, if a       */
        /* different start address in RAM is used (e.g. for Flash type 2)!        */
-       0x00, 0x80, 0xFF, 0x1F,         /* u32DummyRead address in RAM (0x1FFF8000)   */
-       0x04, 0x80, 0xFF, 0x1F          /* u32FlashResult address in RAM (0x1FFF8004) */
+       0x00, 0x80, 0xFF, 0x1F,     /* u32DummyRead address in RAM (0x1FFF8000)   */
+       0x04, 0x80, 0xFF, 0x1F      /* u32FlashResult address in RAM (0x1FFF8004) */
        };
 
        LOG_INFO("Fujitsu MB9B500: FLASH Write ...");
@@ -507,36 +453,31 @@ static int fm3_write_block(struct flash_bank *bank, uint8_t *buffer,
        count = count / 2;              /* number bytes -> number halfwords */
 
        /* check code alignment */
-       if (offset & 0x1)
-       {
+       if (offset & 0x1) {
                LOG_WARNING("offset 0x%" PRIx32 " breaks required 2-byte alignment", offset);
                return ERROR_FLASH_DST_BREAKS_ALIGNMENT;
        }
 
        /* allocate working area with flash programming code */
        if (target_alloc_working_area(target, sizeof(fm3_flash_write_code),
-                       &fm3_info->write_algorithm) != ERROR_OK)
-       {
+                       &write_algorithm) != ERROR_OK) {
                LOG_WARNING("no working area available, can't do block memory writes");
                return ERROR_TARGET_RESOURCE_NOT_AVAILABLE;
        }
 
-       retval = target_write_buffer(target, fm3_info->write_algorithm->address,
+       retval = target_write_buffer(target, write_algorithm->address,
                sizeof(fm3_flash_write_code), fm3_flash_write_code);
        if (retval != ERROR_OK)
                return retval;
 
+
+
        /* memory buffer */
-       while (target_alloc_working_area(target, buffer_size, &source) != ERROR_OK)
-       {
+       while (target_alloc_working_area(target, buffer_size, &source) != ERROR_OK) {
                buffer_size /= 2;
-               if (buffer_size <= 256)
-               {
-                       /* free working area, if write algorithm already allocated */
-                       if (fm3_info->write_algorithm)
-                       {
-                               target_free_working_area(target, fm3_info->write_algorithm);
-                       }
+               if (buffer_size <= 256) {
+                       /* free working area, write algorithm already allocated */
+                       target_free_working_area(target, write_algorithm);
 
                        LOG_WARNING("No large enough working area available, can't do block memory writes");
                        return ERROR_TARGET_RESOURCE_NOT_AVAILABLE;
@@ -553,36 +494,32 @@ static int fm3_write_block(struct flash_bank *bank, uint8_t *buffer,
        init_reg_param(&reg_params[4], "r4", 32, PARAM_OUT); /* Flash Sequence address 1 */
        init_reg_param(&reg_params[5], "r5", 32, PARAM_IN);  /* result */
 
-       /* write code buffer and use Flash programming code within fm3 */
-       /* Set breakpoint to 0 with time-out of 1000 ms */
-       while (count > 0)
-       {
+       /* write code buffer and use Flash programming code within fm3           */
+       /* Set breakpoint to 0 with time-out of 1000 ms                          */
+       while (count > 0) {
                uint32_t thisrun_count = (count > (buffer_size / 2)) ? (buffer_size / 2) : count;
 
-               retval = target_write_buffer(target, fm3_info->write_algorithm->address,
-                               8, fm3_flash_write_code);
+               retval = target_write_buffer(target, write_algorithm->address, 8,
+                               fm3_flash_write_code);
                if (retval != ERROR_OK)
                        break;
 
                /* Patching 'local variable address' for different RAM addresses */
-               if (fm3_info->write_algorithm->address != 0x1FFF8008)
-               {
+               if (write_algorithm->address != 0x1FFF8008) {
                        /* Algorithm: u32DummyRead: */
-                       retval = target_write_u32(target, (fm3_info->write_algorithm->address)
-                                       + sizeof(fm3_flash_write_code) - 8,
-                                       (fm3_info->write_algorithm->address) - 8);
+                       retval = target_write_u32(target, (write_algorithm->address)
+                               + sizeof(fm3_flash_write_code) - 8, (write_algorithm->address) - 8);
                        if (retval != ERROR_OK)
                                break;
 
                        /* Algorithm: u32FlashResult: */
-                       retval = target_write_u32(target, (fm3_info->write_algorithm->address)
-                                       + sizeof(fm3_flash_write_code) - 4, (fm3_info->write_algorithm->address) - 4);
+                       retval = target_write_u32(target, (write_algorithm->address)
+                               + sizeof(fm3_flash_write_code) - 4, (write_algorithm->address) - 4);
                        if (retval != ERROR_OK)
                                break;
                }
 
-               retval = target_write_buffer(target, source->address, thisrun_count * 2,
-                               buffer);
+               retval = target_write_buffer(target, source->address, thisrun_count * 2, buffer);
                if (retval != ERROR_OK)
                        break;
 
@@ -593,18 +530,16 @@ static int fm3_write_block(struct flash_bank *bank, uint8_t *buffer,
                buf_set_u32(reg_params[4].value, 0, 32, u32FlashSeqAddress2);
 
                retval = target_run_algorithm(target, 0, NULL, 6, reg_params,
-                               fm3_info->write_algorithm->address, 0, 1000, &armv7m_info);
-               if (retval != ERROR_OK)
-               {
+                               write_algorithm->address, 0, 1000, &armv7m_info);
+               if (retval != ERROR_OK) {
                        LOG_ERROR("Error executing fm3 Flash programming algorithm");
                        retval = ERROR_FLASH_OPERATION_FAILED;
                        break;
                }
 
-               if (buf_get_u32(reg_params[5].value, 0, 32) != ERROR_OK)
-               {
-                       LOG_ERROR("Fujitsu MB9[A/B]FXXX: Flash programming ERROR (Timeout) \
-                                       -> Reg R3: %x", buf_get_u32(reg_params[5].value, 0, 32));
+               if (buf_get_u32(reg_params[5].value, 0, 32) != ERROR_OK) {
+                       LOG_ERROR("Fujitsu MB9[A/B]FXXX: Flash programming ERROR (Timeout) -> Reg R3: %x",
+                               buf_get_u32(reg_params[5].value, 0, 32));
                        retval = ERROR_FLASH_OPERATION_FAILED;
                        break;
                }
@@ -615,7 +550,7 @@ static int fm3_write_block(struct flash_bank *bank, uint8_t *buffer,
        }
 
        target_free_working_area(target, source);
-       target_free_working_area(target, fm3_info->write_algorithm);
+       target_free_working_area(target, write_algorithm);
 
        destroy_reg_param(&reg_params[0]);
        destroy_reg_param(&reg_params[1]);
@@ -632,8 +567,7 @@ static int fm3_probe(struct flash_bank *bank)
        struct fm3_flash_bank *fm3_info = bank->driver_priv;
        uint16_t num_pages;
 
-       if (bank->target->state != TARGET_HALTED)
-       {
+       if (bank->target->state != TARGET_HALTED) {
                LOG_ERROR("Target not halted");
                return ERROR_TARGET_NOT_HALTED;
        }
@@ -643,7 +577,6 @@ static int fm3_probe(struct flash_bank *bank)
 
        bank->sectors = malloc(sizeof(struct flash_sector) * num_pages);
        bank->base = 0x00000000;
-       num_pages = 2;                          /* start with smallest Flash pages number */
        bank->size = 32 * 1024;         /* bytes */
 
        bank->sectors[0].offset = 0;
@@ -656,8 +589,8 @@ static int fm3_probe(struct flash_bank *bank)
        bank->sectors[1].is_erased = -1;
        bank->sectors[1].is_protected = -1;
 
-       if ((fm3_info->variant == mb9bfxx1) || (fm3_info->variant == mb9afxx1))
-       {
+       if ((fm3_info->variant == mb9bfxx1)
+           || (fm3_info->variant == mb9afxx1)) {
                num_pages = 3;
                bank->size = 64 * 1024; /* bytes */
                bank->num_sectors = num_pages;
@@ -675,8 +608,7 @@ static int fm3_probe(struct flash_bank *bank)
                || (fm3_info->variant == mb9afxx2)
                || (fm3_info->variant == mb9afxx4)
                || (fm3_info->variant == mb9afxx5)
-               || (fm3_info->variant == mb9afxx6))
-       {
+               || (fm3_info->variant == mb9afxx6)) {
                num_pages = 3;
                bank->size = 128 * 1024; /* bytes */
                bank->num_sectors = num_pages;
@@ -692,8 +624,7 @@ static int fm3_probe(struct flash_bank *bank)
                || (fm3_info->variant == mb9bfxx6)
                || (fm3_info->variant == mb9afxx4)
                || (fm3_info->variant == mb9afxx5)
-               || (fm3_info->variant == mb9afxx6))
-       {
+               || (fm3_info->variant == mb9afxx6)) {
                num_pages = 4;
                bank->size = 256 * 1024; /* bytes */
                bank->num_sectors = num_pages;
@@ -707,8 +638,7 @@ static int fm3_probe(struct flash_bank *bank)
        if ((fm3_info->variant == mb9bfxx5)
                || (fm3_info->variant == mb9bfxx6)
                || (fm3_info->variant == mb9afxx5)
-               || (fm3_info->variant == mb9afxx6))
-       {
+               || (fm3_info->variant == mb9afxx6)) {
                num_pages = 5;
                bank->size = 384 * 1024; /* bytes */
                bank->num_sectors = num_pages;
@@ -720,8 +650,7 @@ static int fm3_probe(struct flash_bank *bank)
        }
 
        if ((fm3_info->variant == mb9bfxx6)
-               || (fm3_info->variant == mb9afxx6))
-       {
+               || (fm3_info->variant == mb9afxx6)) {
                num_pages = 6;
                bank->size = 512 * 1024; /* bytes */
                bank->num_sectors = num_pages;
@@ -745,44 +674,39 @@ static int fm3_auto_probe(struct flash_bank *bank)
        return fm3_probe(bank);
 }
 
-static int fm3_info_cmd(struct flash_bank *bank, char *buf, int buf_size)
+static int fm3_info(struct flash_bank *bank, char *buf, int buf_size)
 {
        snprintf(buf, buf_size, "Fujitsu fm3 Device does not support Chip-ID (Type unknown)");
        return ERROR_OK;
 }
 
+/* Chip erase */
 static int fm3_chip_erase(struct flash_bank *bank)
 {
        struct target *target = bank->target;
-       struct fm3_flash_bank *fm3_info = bank->driver_priv;
+       struct fm3_flash_bank *fm3_info2 = bank->driver_priv;
        int retval = ERROR_OK;
        uint32_t u32DummyRead;
        uint32_t u32FlashType;
        uint32_t u32FlashSeqAddress1;
        uint32_t u32FlashSeqAddress2;
 
-       u32FlashType = (uint32_t) fm3_info->flashtype;
+       u32FlashType = (uint32_t) fm3_info2->flashtype;
 
-       if (u32FlashType == fm3_flash_type1)
-       {
+       if (u32FlashType == fm3_flash_type1) {
                LOG_INFO("*** Erasing mb9bfxxx type");
                u32FlashSeqAddress1 = 0x00001550;
                u32FlashSeqAddress2 = 0x00000AA8;
-       }
-       else if (u32FlashType == fm3_flash_type2)
-       {
+       } else if (u32FlashType == fm3_flash_type2) {
                LOG_INFO("*** Erasing mb9afxxx type");
                u32FlashSeqAddress1 = 0x00000AA8;
                u32FlashSeqAddress2 = 0x00000554;
-       }
-       else
-       {
+       } else {
                LOG_ERROR("Flash/Device type unknown!");
                return ERROR_FLASH_OPERATION_FAILED;
        }
 
-       if (target->state != TARGET_HALTED)
-       {
+       if (target->state != TARGET_HALTED) {
                LOG_ERROR("Target not halted");
                return ERROR_TARGET_NOT_HALTED;
        }
@@ -836,8 +760,7 @@ static int fm3_chip_erase(struct flash_bank *bank)
        if (retval != ERROR_OK)
                return retval;
 
-       /* dummy read of FASZR */
-       retval = target_read_u32(target, 0x40000000, &u32DummyRead);
+       retval = target_read_u32(target, 0x40000000, &u32DummyRead); /* dummy read of FASZR */
 
        return retval;
 }
@@ -847,26 +770,20 @@ COMMAND_HANDLER(fm3_handle_chip_erase_command)
        int i;
 
        if (CMD_ARGC < 1)
-       {
-               command_print(CMD_CTX, "fm3 chip_erase <bank>");
-               return ERROR_OK;
-       }
+               return ERROR_COMMAND_SYNTAX_ERROR;
 
        struct flash_bank *bank;
        int retval = CALL_COMMAND_HANDLER(flash_command_get_bank, 0, &bank);
        if (ERROR_OK != retval)
                return retval;
 
-       if (fm3_chip_erase(bank) == ERROR_OK)
-       {
+       if (fm3_chip_erase(bank) == ERROR_OK) {
                /* set all sectors as erased */
                for (i = 0; i < bank->num_sectors; i++)
                        bank->sectors[i].is_erased = 1;
 
                command_print(CMD_CTX, "fm3 chip erase complete");
-       }
-       else
-       {
+       } else {
                command_print(CMD_CTX, "fm3 chip erase failed");
        }
 
@@ -876,9 +793,9 @@ COMMAND_HANDLER(fm3_handle_chip_erase_command)
 static const struct command_registration fm3_exec_command_handlers[] = {
        {
                .name = "chip_erase",
+               .usage = "<bank>",
                .handler = fm3_handle_chip_erase_command,
                .mode = COMMAND_EXEC,
-               .usage = "bank_id",
                .help = "Erase entire Flash device.",
        },
        COMMAND_REGISTRATION_DONE
@@ -889,6 +806,7 @@ static const struct command_registration fm3_command_handlers[] = {
                .name = "fm3",
                .mode = COMMAND_ANY,
                .help = "fm3 Flash command group",
+               .usage = "",
                .chain = fm3_exec_command_handlers,
        },
        COMMAND_REGISTRATION_DONE
@@ -902,6 +820,6 @@ struct flash_driver fm3_flash = {
        .write = fm3_write_block,
        .probe = fm3_probe,
        .auto_probe = fm3_auto_probe,
-       .erase_check = default_flash_mem_blank_check,
-       .info = fm3_info_cmd,
+       .erase_check = default_flash_blank_check,
+       .info = fm3_info,
 };

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)