flash/nor: Make info function optional
[fw/openocd] / src / flash / nor / fm3.c
index 78e71d2902cbbf82bade405c871dde9992289711..86e4cb2416dfd7b4eb60d5ed7da37a2e5718bcb5 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  *
@@ -17,7 +16,7 @@
  *   You should have received a copy of the GNU General Public License     *
  *   along with this program; if not, write to the                         *
  *   Free Software Foundation, Inc.,                                       *
- *   59 Temple Place - Suite 330, Boston, MA  02111-1307, USA.             *
+ *   51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.           *
  ***************************************************************************/
 
 #ifdef HAVE_CONFIG_H
 #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,
        mb9bfxx4,
        mb9bfxx5,
        mb9bfxx6,
+       mb9bfxx7,
+       mb9bfxx8,
+
        mb9afxx1,       /* Flash Type '2' */
        mb9afxx2,
        mb9afxx3,
        mb9afxx4,
        mb9afxx5,
-       mb9afxx6
+       mb9afxx6,
+       mb9afxx7,
+       mb9afxx8,
 };
 
-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,86 +68,69 @@ FLASH_BANK_COMMAND_HANDLER(fm3_flash_bank_command)
        struct fm3_flash_bank *fm3_info;
 
        if (CMD_ARGC < 6)
-       {
                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], "mb9bfxx7.cpu") == 0) {
+               fm3_info->variant = mb9bfxx7;
+               fm3_info->flashtype = fm3_flash_type1;
+       } else if (strcmp(CMD_ARGV[5], "mb9bfxx8.cpu") == 0) {
+               fm3_info->variant = mb9bfxx8;
+               fm3_info->flashtype = fm3_flash_type1;
+       } 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;
+       } else if (strcmp(CMD_ARGV[5], "mb9afxx7.cpu") == 0) {
+               fm3_info->variant = mb9afxx7;
+               fm3_info->flashtype = fm3_flash_type2;
+       } else if (strcmp(CMD_ARGV[5], "mb9afxx8.cpu") == 0) {
+               fm3_info->variant = mb9afxx8;
+               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;
@@ -161,8 +144,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)
@@ -179,13 +161,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 */
@@ -199,10 +178,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;
@@ -211,8 +188,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;
                }
@@ -237,18 +213,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;
        }
@@ -258,7 +229,7 @@ static int fm3_erase(struct flash_bank *bank, int first, int last)
                return ERROR_TARGET_NOT_HALTED;
        }
 
-       LOG_INFO("Fujitsu MB9Bxxx: Sector Erase ... (%d to %d)", first, last);
+       LOG_INFO("Fujitsu MB9[A/B]FXXX: Sector Erase ... (%d to %d)", first, last);
 
        /* FASZR = 0x01, Enables CPU Programming Mode (16-bit Flash acccess) */
        retval = target_write_u32(target, 0x40000000, 0x0001);
@@ -270,12 +241,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;
 
@@ -317,8 +286,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;
 }
@@ -328,7 +296,8 @@ 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! */
+       uint32_t buffer_size = 2048;            /* Default minimum value */
+       struct working_area *write_algorithm;
        struct working_area *source;
        uint32_t address = bank->base + offset;
        struct reg_param reg_params[6];
@@ -338,20 +307,19 @@ static int fm3_write_block(struct flash_bank *bank, uint8_t *buffer,
        uint32_t u32FlashSeqAddress1;
        uint32_t u32FlashSeqAddress2;
 
+       /* Increase buffer_size if needed */
+       if (buffer_size < (target->working_area_size / 2))
+               buffer_size = (target->working_area_size / 2);
+
        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;
        }
@@ -482,13 +450,13 @@ static int fm3_write_block(struct flash_bank *bank, uint8_t *buffer,
        0x00, 0xBE,                                     /*        BKPT     #0                         */
 
        /* 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) */
+       /* SRAM basic-address(BASE_ADDR)+8.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 (BASE_ADDR)    */
+       0x04, 0x80, 0xFF, 0x1F      /* u32FlashResult address in RAM (BASE_ADDR+4)*/
        };
 
-       LOG_INFO("Fujitsu MB9B500: FLASH Write ...");
+       LOG_INFO("Fujitsu MB9[A/B]FXXX: FLASH Write ...");
 
        /* disable HW watchdog */
        retval = target_write_u32(target, 0x40011C00, 0x1ACCE551);
@@ -506,36 +474,29 @@ 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;
@@ -543,7 +504,7 @@ static int fm3_write_block(struct flash_bank *bank, uint8_t *buffer,
        }
 
        armv7m_info.common_magic = ARMV7M_COMMON_MAGIC;
-       armv7m_info.core_mode = ARMV7M_MODE_ANY;
+       armv7m_info.core_mode = ARM_MODE_THREAD;
 
        init_reg_param(&reg_params[0], "r0", 32, PARAM_OUT); /* source start address */
        init_reg_param(&reg_params[1], "r1", 32, PARAM_OUT); /* target start address */
@@ -552,36 +513,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 != (target->working_area_phys + 8)) {
                        /* 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;
 
@@ -592,18 +549,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;
                }
@@ -614,7 +569,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]);
@@ -631,13 +586,27 @@ 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;
        }
 
-       num_pages = 6;                          /* max number of Flash pages for malloc */
+/*
+ -- page-- start -- blocksize - mpu - totalFlash --
+       page0 0x00000   16k
+       page1 0x04000   16k
+       page2 0x08000   96k             ___ fxx3  128k Flash
+       page3 0x20000  128k             ___ fxx4  256k Flash
+       page4 0x40000  128k             ___ fxx5  384k Flash
+       page5 0x60000  128k             ___ fxx6  512k Flash
+-----------------------
+       page6 0x80000  128k
+       page7 0xa0000  128k             ___ fxx7  256k Flash
+       page8 0xc0000  128k
+       page9 0xe0000  128k             ___ fxx8  256k Flash
+ */
+
+       num_pages = 10;                         /* max number of Flash pages for malloc */
        fm3_info->probed = 0;
 
        bank->sectors = malloc(sizeof(struct flash_sector) * num_pages);
@@ -654,8 +623,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;
@@ -670,11 +639,14 @@ static int fm3_probe(struct flash_bank *bank)
                || (fm3_info->variant == mb9bfxx4)
                || (fm3_info->variant == mb9bfxx5)
                || (fm3_info->variant == mb9bfxx6)
+               || (fm3_info->variant == mb9bfxx7)
+               || (fm3_info->variant == mb9bfxx8)
                || (fm3_info->variant == mb9afxx2)
                || (fm3_info->variant == mb9afxx4)
                || (fm3_info->variant == mb9afxx5)
-               || (fm3_info->variant == mb9afxx6))
-       {
+               || (fm3_info->variant == mb9afxx6)
+               || (fm3_info->variant == mb9afxx7)
+               || (fm3_info->variant == mb9afxx8)) {
                num_pages = 3;
                bank->size = 128 * 1024; /* bytes */
                bank->num_sectors = num_pages;
@@ -688,10 +660,13 @@ static int fm3_probe(struct flash_bank *bank)
        if ((fm3_info->variant == mb9bfxx4)
                || (fm3_info->variant == mb9bfxx5)
                || (fm3_info->variant == mb9bfxx6)
+               || (fm3_info->variant == mb9bfxx7)
+               || (fm3_info->variant == mb9bfxx8)
                || (fm3_info->variant == mb9afxx4)
                || (fm3_info->variant == mb9afxx5)
-               || (fm3_info->variant == mb9afxx6))
-       {
+               || (fm3_info->variant == mb9afxx6)
+               || (fm3_info->variant == mb9afxx7)
+               || (fm3_info->variant == mb9afxx8)) {
                num_pages = 4;
                bank->size = 256 * 1024; /* bytes */
                bank->num_sectors = num_pages;
@@ -704,9 +679,12 @@ static int fm3_probe(struct flash_bank *bank)
 
        if ((fm3_info->variant == mb9bfxx5)
                || (fm3_info->variant == mb9bfxx6)
+               || (fm3_info->variant == mb9bfxx7)
+               || (fm3_info->variant == mb9bfxx8)
                || (fm3_info->variant == mb9afxx5)
-               || (fm3_info->variant == mb9afxx6))
-       {
+               || (fm3_info->variant == mb9afxx6)
+               || (fm3_info->variant == mb9afxx7)
+               || (fm3_info->variant == mb9afxx8)) {
                num_pages = 5;
                bank->size = 384 * 1024; /* bytes */
                bank->num_sectors = num_pages;
@@ -718,8 +696,11 @@ static int fm3_probe(struct flash_bank *bank)
        }
 
        if ((fm3_info->variant == mb9bfxx6)
-               || (fm3_info->variant == mb9afxx6))
-       {
+               || (fm3_info->variant == mb9bfxx7)
+               || (fm3_info->variant == mb9bfxx8)
+               || (fm3_info->variant == mb9afxx6)
+               || (fm3_info->variant == mb9afxx7)
+               || (fm3_info->variant == mb9afxx8)) {
                num_pages = 6;
                bank->size = 512 * 1024; /* bytes */
                bank->num_sectors = num_pages;
@@ -730,6 +711,42 @@ static int fm3_probe(struct flash_bank *bank)
                bank->sectors[5].is_protected = -1;
        }
 
+       if ((fm3_info->variant == mb9bfxx7)
+               || (fm3_info->variant == mb9bfxx8)
+               || (fm3_info->variant == mb9afxx7)
+               || (fm3_info->variant == mb9afxx8)) {
+               num_pages = 8;
+               bank->size = 768 * 1024; /* bytes */
+               bank->num_sectors = num_pages;
+
+               bank->sectors[6].offset = 0x80000;
+               bank->sectors[6].size = 128 * 1024;
+               bank->sectors[6].is_erased = -1;
+               bank->sectors[6].is_protected = -1;
+
+               bank->sectors[7].offset = 0xa0000;
+               bank->sectors[7].size = 128 * 1024;
+               bank->sectors[7].is_erased = -1;
+               bank->sectors[7].is_protected = -1;
+       }
+
+       if ((fm3_info->variant == mb9bfxx8)
+               || (fm3_info->variant == mb9afxx8)) {
+               num_pages = 10;
+               bank->size = 1024 * 1024; /* bytes */
+               bank->num_sectors = num_pages;
+
+               bank->sectors[8].offset = 0xc0000;
+               bank->sectors[8].size = 128 * 1024;
+               bank->sectors[8].is_erased = -1;
+               bank->sectors[8].is_protected = -1;
+
+               bank->sectors[9].offset = 0xe0000;
+               bank->sectors[9].size = 128 * 1024;
+               bank->sectors[9].is_erased = -1;
+               bank->sectors[9].is_protected = -1;
+       }
+
        fm3_info->probed = 1;
 
        return ERROR_OK;
@@ -743,44 +760,33 @@ 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)
-{
-       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;
        }
@@ -834,8 +840,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;
 }
@@ -845,25 +850,20 @@ COMMAND_HANDLER(fm3_handle_chip_erase_command)
        int i;
 
        if (CMD_ARGC < 1)
-       {
                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");
        }
 
@@ -886,6 +886,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
@@ -899,6 +900,5 @@ 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,
 };