Removed comment about STM32F4 limitations
[fw/stlink] / src / stlink-common.c
index a52a3346f828eb68d34a5a030d8aaab33bbfa0ae..fcdb405ff84d65cb6478e08455a9d01c605c7ba7 100644 (file)
@@ -57,6 +57,35 @@ void DD(stlink_t *sl, char *format, ...) {
 #define FLASH_CR_STRT 6
 #define FLASH_CR_LOCK 7
 
+
+//32L = 32F1 same CoreID as 32F4!
+#define STM32L_FLASH_REGS_ADDR ((uint32_t)0x40023c00)
+#define STM32L_FLASH_ACR (STM32L_FLASH_REGS_ADDR + 0x00)
+#define STM32L_FLASH_PECR (STM32L_FLASH_REGS_ADDR + 0x04)
+#define STM32L_FLASH_PDKEYR (STM32L_FLASH_REGS_ADDR + 0x08)
+#define STM32L_FLASH_PEKEYR (STM32L_FLASH_REGS_ADDR + 0x0c)
+#define STM32L_FLASH_PRGKEYR (STM32L_FLASH_REGS_ADDR + 0x10)
+#define STM32L_FLASH_OPTKEYR (STM32L_FLASH_REGS_ADDR + 0x14)
+#define STM32L_FLASH_SR (STM32L_FLASH_REGS_ADDR + 0x18)
+#define STM32L_FLASH_OBR (STM32L_FLASH_REGS_ADDR + 0x0c)
+#define STM32L_FLASH_WRPR (STM32L_FLASH_REGS_ADDR + 0x20)
+
+
+//STM32F4
+#define FLASH_F4_REGS_ADDR ((uint32_t)0x40023c00)
+#define FLASH_F4_KEYR (FLASH_F4_REGS_ADDR + 0x04)
+#define FLASH_F4_OPT_KEYR (FLASH_F4_REGS_ADDR + 0x08)
+#define FLASH_F4_SR (FLASH_F4_REGS_ADDR + 0x0c)
+#define FLASH_F4_CR (FLASH_F4_REGS_ADDR + 0x10)
+#define FLASH_F4_OPT_CR (FLASH_F4_REGS_ADDR + 0x14)
+#define FLASH_F4_CR_STRT 16
+#define FLASH_F4_CR_LOCK 31
+#define FLASH_F4_CR_SER 1
+#define FLASH_F4_CR_SNB 3
+#define FLASH_F4_CR_SNB_MASK 0x38
+#define FLASH_F4_SR_BSY 16
+
+
 void write_uint32(unsigned char* buf, uint32_t ui) {
     if (!is_bigendian()) { // le -> le (don't swap)
         buf[0] = ((unsigned char*) &ui)[0];
@@ -115,13 +144,20 @@ static inline uint32_t read_flash_obr(stlink_t *sl) {
 }
 
 static inline uint32_t read_flash_cr(stlink_t *sl) {
-    stlink_read_mem32(sl, FLASH_CR, sizeof (uint32_t));
+       if(sl->chip_id==STM32F4_CHIP_ID)
+               stlink_read_mem32(sl, FLASH_F4_CR, sizeof (uint32_t));
+       else
+               stlink_read_mem32(sl, FLASH_CR, sizeof (uint32_t));
+       fprintf(stdout, "CR:%X\n", *(uint32_t*) sl->q_buf);
     return *(uint32_t*) sl->q_buf;
 }
 
 static inline unsigned int is_flash_locked(stlink_t *sl) {
     /* return non zero for true */
-    return read_flash_cr(sl) & (1 << FLASH_CR_LOCK);
+       if(sl->chip_id==STM32F4_CHIP_ID)
+               return read_flash_cr(sl) & (1 << FLASH_F4_CR_LOCK);
+       else
+               return read_flash_cr(sl) & (1 << FLASH_CR_LOCK);
 }
 
 static void unlock_flash(stlink_t *sl) {
@@ -130,12 +166,19 @@ static void unlock_flash(stlink_t *sl) {
        an invalid sequence results in a definitive lock of
        the FPEC block until next reset.
      */
+    if(sl->chip_id==STM32F4_CHIP_ID) {
+        write_uint32(sl->q_buf, FLASH_KEY1);
+       stlink_write_mem32(sl, FLASH_F4_KEYR, sizeof (uint32_t));
+               write_uint32(sl->q_buf, FLASH_KEY2);
+               stlink_write_mem32(sl, FLASH_F4_KEYR, sizeof (uint32_t));
+    }
+       else {
+        write_uint32(sl->q_buf, FLASH_KEY1);
+       stlink_write_mem32(sl, FLASH_KEYR, sizeof (uint32_t));
+               write_uint32(sl->q_buf, FLASH_KEY2);
+               stlink_write_mem32(sl, FLASH_KEYR, sizeof (uint32_t));
+       }
 
-    write_uint32(sl->q_buf, FLASH_KEY1);
-    stlink_write_mem32(sl, FLASH_KEYR, sizeof (uint32_t));
-
-    write_uint32(sl->q_buf, FLASH_KEY2);
-    stlink_write_mem32(sl, FLASH_KEYR, sizeof (uint32_t));
 }
 
 static int unlock_flash_if(stlink_t *sl) {
@@ -151,24 +194,41 @@ static int unlock_flash_if(stlink_t *sl) {
 }
 
 static void lock_flash(stlink_t *sl) {
-    /* write to 1 only. reset by hw at unlock sequence */
-
-    const uint32_t n = read_flash_cr(sl) | (1 << FLASH_CR_LOCK);
-
-    write_uint32(sl->q_buf, n);
-    stlink_write_mem32(sl, FLASH_CR, sizeof (uint32_t));
+    if(sl->chip_id==STM32F4_CHIP_ID) {
+       const uint32_t n = read_flash_cr(sl) | (1 << FLASH_F4_CR_LOCK);
+        write_uint32(sl->q_buf, n);
+       stlink_write_mem32(sl, FLASH_F4_CR, sizeof (uint32_t));
+    }
+    else {
+        /* write to 1 only. reset by hw at unlock sequence */
+        const uint32_t n = read_flash_cr(sl) | (1 << FLASH_CR_LOCK);
+        write_uint32(sl->q_buf, n);
+        stlink_write_mem32(sl, FLASH_CR, sizeof (uint32_t));
+    }
 }
 
+
 static void set_flash_cr_pg(stlink_t *sl) {
-    const uint32_t n = 1 << FLASH_CR_PG;
-    write_uint32(sl->q_buf, n);
-    stlink_write_mem32(sl, FLASH_CR, sizeof (uint32_t));
+    if(sl->chip_id==STM32F4_CHIP_ID) {
+               uint32_t x = read_flash_cr(sl);
+               x |= (1 << FLASH_CR_PG);
+               write_uint32(sl->q_buf, x);
+       stlink_write_mem32(sl, FLASH_F4_CR, sizeof (uint32_t));
+    }
+    else {
+        const uint32_t n = 1 << FLASH_CR_PG;
+        write_uint32(sl->q_buf, n);
+        stlink_write_mem32(sl, FLASH_CR, sizeof (uint32_t));
+    }
 }
 
 static void __attribute__((unused)) clear_flash_cr_pg(stlink_t *sl) {
     const uint32_t n = read_flash_cr(sl) & ~(1 << FLASH_CR_PG);
     write_uint32(sl->q_buf, n);
-    stlink_write_mem32(sl, FLASH_CR, sizeof (uint32_t));
+    if(sl->chip_id==STM32F4_CHIP_ID)
+       stlink_write_mem32(sl, FLASH_F4_CR, sizeof (uint32_t));
+    else
+        stlink_write_mem32(sl, FLASH_CR, sizeof (uint32_t));
 }
 
 static void set_flash_cr_per(stlink_t *sl) {
@@ -196,10 +256,19 @@ static void __attribute__((unused)) clear_flash_cr_mer(stlink_t *sl) {
 }
 
 static void set_flash_cr_strt(stlink_t *sl) {
-    /* assume come on the flash_cr_per path */
-    const uint32_t n = (1 << FLASH_CR_PER) | (1 << FLASH_CR_STRT);
-    write_uint32(sl->q_buf, n);
-    stlink_write_mem32(sl, FLASH_CR, sizeof (uint32_t));
+       if(sl->chip_id == STM32F4_CHIP_ID)
+       {
+               uint32_t x = read_flash_cr(sl);
+               x |= (1 << FLASH_F4_CR_STRT);
+               write_uint32(sl->q_buf, x);
+               stlink_write_mem32(sl, FLASH_F4_CR, sizeof (uint32_t));
+       }
+       else {
+               /* assume come on the flash_cr_per path */
+           const uint32_t n = (1 << FLASH_CR_PER) | (1 << FLASH_CR_STRT);
+           write_uint32(sl->q_buf, n);
+           stlink_write_mem32(sl, FLASH_CR, sizeof (uint32_t));
+       }
 }
 
 static inline uint32_t read_flash_acr(stlink_t *sl) {
@@ -208,12 +277,19 @@ static inline uint32_t read_flash_acr(stlink_t *sl) {
 }
 
 static inline uint32_t read_flash_sr(stlink_t *sl) {
-    stlink_read_mem32(sl, FLASH_SR, sizeof (uint32_t));
-    return *(uint32_t*) sl->q_buf;
+       if(sl->chip_id==STM32F4_CHIP_ID)
+               stlink_read_mem32(sl, FLASH_F4_SR, sizeof (uint32_t));
+       else
+               stlink_read_mem32(sl, FLASH_SR, sizeof (uint32_t));
+    //fprintf(stdout, "SR:%X\n", *(uint32_t*) sl->q_buf);
+       return *(uint32_t*) sl->q_buf;
 }
 
 static inline unsigned int is_flash_busy(stlink_t *sl) {
-    return read_flash_sr(sl) & (1 << FLASH_SR_BSY);
+       if(sl->chip_id==STM32F4_CHIP_ID)
+               return read_flash_sr(sl) & (1 << FLASH_F4_SR_BSY);
+       else
+               return read_flash_sr(sl) & (1 << FLASH_SR_BSY);
 }
 
 static void wait_flash_busy(stlink_t *sl) {
@@ -243,6 +319,26 @@ static inline void write_flash_ar(stlink_t *sl, uint32_t n) {
     stlink_write_mem32(sl, FLASH_AR, sizeof (uint32_t));
 }
 
+static inline void write_flash_cr_psiz(stlink_t *sl, uint32_t n) {
+    uint32_t x = read_flash_cr(sl);
+    x &= ~(0x03 << 8);
+    x |= (n << 8);
+    fprintf(stdout, "PSIZ:%X %X\n", x, n);
+    write_uint32(sl->q_buf, x);
+    stlink_write_mem32(sl, FLASH_F4_CR, sizeof (uint32_t));
+}
+
+
+static inline void write_flash_cr_snb(stlink_t *sl, uint32_t n) {
+    uint32_t x = read_flash_cr(sl);
+    x &= ~FLASH_F4_CR_SNB_MASK;
+    x |= (n << FLASH_F4_CR_SNB);
+    x |= (1 << FLASH_F4_CR_SER);
+    fprintf(stdout, "SNB:%X %X\n", x, n);
+    write_uint32(sl->q_buf, x);
+    stlink_write_mem32(sl, FLASH_F4_CR, sizeof (uint32_t));
+}
+
 #if 0 /* todo */
 
 static void disable_flash_read_protection(stlink_t *sl) {
@@ -287,15 +383,21 @@ uint32_t stlink_core_id(stlink_t *sl) {
     sl->backend->core_id(sl);
     if (sl->verbose > 2)
         stlink_print_data(sl);
-    DD(sl, "core_id = 0x%08x\n", sl->core_id);
     return sl->core_id;
 }
 
-uint16_t stlink_chip_id(stlink_t *sl) {
-    stlink_read_mem32(sl, 0xE0042000, 4);
+void stlink_identify_device(stlink_t *sl) {
+       uint32_t core_id=stlink_core_id(sl);
+       stlink_read_mem32(sl, 0xE0042000, 4);
     uint32_t chip_id = sl->q_buf[0] | (sl->q_buf[1] << 8) | (sl->q_buf[2] << 16) |
             (sl->q_buf[3] << 24);
-    return chip_id;
+    /* Fix chip_id for F4 */
+    if (((chip_id & 0xFFF) == 0x411) && (core_id == CORE_M4_R0)) {
+      printf("Fixing wrong chip_id for STM32F4 Rev A errata\n");
+      chip_id = 0x413;
+    }
+    sl->chip_id=chip_id;
+    sl->core_id=core_id;
 }
 
 /**
@@ -703,22 +805,65 @@ int write_buffer_to_sram(stlink_t *sl, flash_loader_t* fl, const uint8_t* buf, s
     return 0;
 }
 
+uint32_t calculate_F4_sectornum(uint32_t flashaddr){
+    flashaddr &= ~STM32_FLASH_BASE;    //Page now holding the actual flash address
+    if (flashaddr<0x4000) return (0);
+    else if(flashaddr<0x8000) return(1);
+    else if(flashaddr<0xc000) return(2);
+    else if(flashaddr<0x10000) return(3);
+    else if(flashaddr<0x20000) return(4);
+    else return(flashaddr/0x20000)+4;
+
+}
+
+uint32_t calculate_sectorsize(stlink_t *sl, uint32_t flashaddr){
+       if(sl->chip_id == STM32F4_CHIP_ID) {
+               uint32_t sector=calculate_F4_sectornum(flashaddr);
+               if (sector<4) return (0x4000);
+               else if(sector<5) return(0x10000);
+               else return(0x20000);
+       }
+       else return (sl->flash_pgsz);
+}
+
 int stlink_erase_flash_page(stlink_t *sl, stm32_addr_t page)
 {
   /* page an addr in the page to erase */
 
-  if (sl->core_id == 0x2ba01477) /* stm32l */
+  stlink_identify_device(sl);
+
+  if (sl->chip_id == STM32F4_CHIP_ID)
+  {
+    /* wait for ongoing op to finish */
+    wait_flash_busy(sl);
+
+    /* unlock if locked */
+    unlock_flash_if(sl);
+
+    /* select the page to erase */
+    //Page is passed to us as an addr, so calculate the actual page
+    uint32_t addr=page;
+
+    page=calculate_F4_sectornum(addr);
+
+    fprintf(stderr, "Erasing Sector:%u SectorSize:%u\n", page, calculate_sectorsize(sl, addr));
+    write_flash_cr_snb(sl, page);
+
+    /* start erase operation */
+    set_flash_cr_strt(sl);
+
+    /* wait for completion */
+    wait_flash_busy(sl);
+
+    /* relock the flash */
+    //todo: fails to program if this is in
+    lock_flash(sl);
+       fprintf(stdout, "Erase Final CR:%X\n", read_flash_cr(sl));
+
+  }
+
+  else if (sl->core_id == STM32L_CORE_ID)
   {
-#define STM32L_FLASH_REGS_ADDR ((uint32_t)0x40023c00)
-#define STM32L_FLASH_ACR (STM32L_FLASH_REGS_ADDR + 0x00)
-#define STM32L_FLASH_PECR (STM32L_FLASH_REGS_ADDR + 0x04)
-#define STM32L_FLASH_PDKEYR (STM32L_FLASH_REGS_ADDR + 0x08)
-#define STM32L_FLASH_PEKEYR (STM32L_FLASH_REGS_ADDR + 0x0c)
-#define STM32L_FLASH_PRGKEYR (STM32L_FLASH_REGS_ADDR + 0x10)
-#define STM32L_FLASH_OPTKEYR (STM32L_FLASH_REGS_ADDR + 0x14)
-#define STM32L_FLASH_SR (STM32L_FLASH_REGS_ADDR + 0x18)
-#define STM32L_FLASH_OBR (STM32L_FLASH_REGS_ADDR + 0x0c)
-#define STM32L_FLASH_WRPR (STM32L_FLASH_REGS_ADDR + 0x20)
 
     uint32_t val;
 
@@ -733,7 +878,7 @@ int stlink_erase_flash_page(stlink_t *sl, stm32_addr_t page)
     val = read_uint32(sl->q_buf, 0);
     if (val & (1 << 0))
     {
-      fprintf(stderr, "pecr.pelock not clear\n");
+      fprintf(stderr, "pecr.pelock not clear (0x%x)\n", val);
       return -1;
     }
 
@@ -748,7 +893,7 @@ int stlink_erase_flash_page(stlink_t *sl, stm32_addr_t page)
     val = read_uint32(sl->q_buf, 0);
     if (val & (1 << 1))
     {
-      fprintf(stderr, "pecr.prglock not clear\n");
+      fprintf(stderr, "pecr.prglock not clear (0x%x)\n", val);
       return -1;
     }
 
@@ -791,7 +936,7 @@ int stlink_erase_flash_page(stlink_t *sl, stm32_addr_t page)
     write_uint32(sl->q_buf, val);
     stlink_write_mem32(sl, STM32L_FLASH_PECR, sizeof(uint32_t));
   }
-  else /* stm32vl */
+  else if (sl->core_id == STM32VL_CORE_ID)
   {
     /* wait for ongoing op to finish */
     wait_flash_busy(sl);
@@ -815,6 +960,11 @@ int stlink_erase_flash_page(stlink_t *sl, stm32_addr_t page)
     lock_flash(sl);
   }
 
+  else {
+    fprintf(stderr, "unknown device!\n");
+    return -1;
+  }
+
   /* todo: verify the erased page */
 
   return 0;
@@ -906,16 +1056,21 @@ int write_loader_to_sram(stlink_t *sl, stm32_addr_t* addr, size_t* size) {
     const uint8_t* loader_code;
     size_t loader_size;
 
-    if (sl->core_id == 0x2ba01477) /* stm32l */
+    if (sl->core_id == STM32L_CORE_ID) /* stm32l */
     {
       loader_code = loader_code_stm32l;
       loader_size = sizeof(loader_code_stm32l);
     }
-    else /* stm32vl */
+    else if (sl->core_id == STM32VL_CORE_ID)
     {
       loader_code = loader_code_stm32vl;
       loader_size = sizeof(loader_code_stm32vl);
     }
+    else
+    {
+      fprintf(stderr, "unknown coreid: %x\n", sl->core_id);
+      return -1;
+    }
 
     memcpy(sl->q_buf, loader_code, loader_size);
     stlink_write_mem32(sl, sl->sram_base, loader_size);
@@ -943,13 +1098,17 @@ int stlink_fcheck_flash(stlink_t *sl, const char* path, stm32_addr_t addr) {
     return res;
 }
 
-
-#define WRITE_BLOCK_SIZE 0x40
-
 int stlink_write_flash(stlink_t *sl, stm32_addr_t addr, uint8_t* base, unsigned len) {
     size_t off;
     flash_loader_t fl;
 
+    stlink_identify_device(sl);
+
+#if 0 /* todo: use in debugging mode only */
+    fprintf(stdout, "WriteFlash - addr:%x len:%x\n", addr, len);
+    fprintf(stdout, "CoreID:%X ChipID:%X\n", sl->core_id, sl->chip_id);
+#endif
+
     /* check addr range is inside the flash */
     if (addr < sl->flash_base) {
         fprintf(stderr, "addr too low\n");
@@ -963,317 +1122,257 @@ int stlink_write_flash(stlink_t *sl, stm32_addr_t addr, uint8_t* base, unsigned
     } else if ((addr & 1) || (len & 1)) {
         fprintf(stderr, "unaligned addr or size\n");
         return -1;
+    } else if (addr & (sl->flash_pgsz - 1)) {
+        fprintf(stderr, "addr not a multiple of pagesize, not supported\n");
+        return -1;
     }
 
-    /* needed for specializing loader */
-    stlink_core_id(sl);
-
-    if (sl->core_id == 0x2ba01477) /* stm32l */
-    {
-      /* use fast word write. todo: half page. */
-      /* todo, factorize with stlink_fwrite_flash */
-
-      uint32_t val;
-      uint32_t off;
-
-      for (off = 0; off < len; off += sl->flash_pgsz) {
+    /* erase each page */
+    for (off = 0; off < len; off += calculate_sectorsize(sl, addr + off) ) {
         /* addr must be an addr inside the page */
         if (stlink_erase_flash_page(sl, addr + off) == -1) {
-         fprintf(stderr, "erase_flash_page(0x%zx) == -1\n", addr + off);
-         return -1;
+           fprintf(stderr, "erase_flash_page(0x%zx) == -1\n", addr + off);
+           return -1;
         }
-      }
-
-      /* disable pecr protection */
-      write_uint32(sl->q_buf, 0x89abcdef);
-      stlink_write_mem32(sl, STM32L_FLASH_PEKEYR, sizeof(uint32_t));
-      write_uint32(sl->q_buf, 0x02030405);
-      stlink_write_mem32(sl, STM32L_FLASH_PEKEYR, sizeof(uint32_t));
-
-      /* check pecr.pelock is cleared */
-      stlink_read_mem32(sl, STM32L_FLASH_PECR, sizeof(uint32_t));
-      val = read_uint32(sl->q_buf, 0);
-      if (val & (1 << 0))
-      {
-       fprintf(stderr, "pecr.pelock not clear\n");
-       return -1;
-      }
-
-      /* unlock program memory */
-      write_uint32(sl->q_buf, 0x8c9daebf);
-      stlink_write_mem32(sl, STM32L_FLASH_PRGKEYR, sizeof(uint32_t));
-      write_uint32(sl->q_buf, 0x13141516);
-      stlink_write_mem32(sl, STM32L_FLASH_PRGKEYR, sizeof(uint32_t));
-
-      /* check pecr.prglock is cleared */
-      stlink_read_mem32(sl, STM32L_FLASH_PECR, sizeof(uint32_t));
-      val = read_uint32(sl->q_buf, 0);
-      if (val & (1 << 1))
-      {
-       fprintf(stderr, "pecr.prglock not clear\n");
-       return -1;
-      }
-
-      /* write a word in program memory */
-      for (off = 0; off < len; off += sizeof(uint32_t))
-      {
-       memcpy(sl->q_buf, (const void*)(base + off), sizeof(uint32_t));
-       stlink_write_mem32(sl, addr + off, sizeof(uint32_t));
-
-       /* wait for sr.busy to be cleared */
-       while (1)
-       {
-         stlink_read_mem32(sl, STM32L_FLASH_SR, sizeof(uint32_t));
-         if ((read_uint32(sl->q_buf, 0) & (1 << 0)) == 0) break ;
-       }
-      }
-
-      /* reset lock bits */
-      stlink_read_mem32(sl, STM32L_FLASH_PECR, sizeof(uint32_t));
-      val = read_uint32(sl->q_buf, 0) | (1 << 0) | (1 << 1) | (1 << 2);
-      write_uint32(sl->q_buf, val);
-      stlink_write_mem32(sl, STM32L_FLASH_PECR, sizeof(uint32_t));
-
     }
-    else /* stm32vl */
-    {
-      /* flash loader initialization */
-      if (init_flash_loader(sl, &fl) == -1) {
-        fprintf(stderr, "init_flash_loader() == -1\n");
-        return -1;
-      }
 
-      /* write each page. above WRITE_BLOCK_SIZE fails? */
-      for (off = 0; off < len; off += WRITE_BLOCK_SIZE)
-      {
-        /* adjust last write size */
-        size_t size = WRITE_BLOCK_SIZE;
-        if ((off + WRITE_BLOCK_SIZE) > len) size = len - off;
+    if (sl->chip_id == STM32F4_CHIP_ID) {
+       /* todo: check write operation */
 
-       /* unlock and set programming mode */
-       unlock_flash_if(sl);
-       set_flash_cr_pg(sl);
+       /* First unlock the cr */
+       unlock_flash_if(sl);
 
-        if (run_flash_loader(sl, &fl, addr + off, base + off, size) == -1) {
-         fprintf(stderr, "run_flash_loader(0x%zx) == -1\n", addr + off);
-         return -1;
-        }
+       /* set parallelisim to 32 bit*/
+       write_flash_cr_psiz(sl, 2);
 
-       lock_flash(sl);
-      }
-    }
+       /* set programming mode */
+       set_flash_cr_pg(sl);
 
-    for (off = 0; off < len; off += sl->flash_pgsz) {
-        size_t aligned_size;
-
-        /* adjust last page size */
-        size_t cmp_size = sl->flash_pgsz;
-        if ((off + sl->flash_pgsz) > len)
-            cmp_size = len - off;
+#define PROGRESS_CHUNK_SIZE 0x1000
+       /* write a word in program memory */
+       for (off = 0; off < len; off += sizeof(uint32_t)) {
+               if (sl->verbose >= 1) {
+                       if ((off & (PROGRESS_CHUNK_SIZE - 1)) == 0) {
+                               /* show progress. writing procedure is slow
+                                          and previous errors are misleading */
+                               const uint32_t pgnum = (off / PROGRESS_CHUNK_SIZE)+1;
+                               const uint32_t pgcount = len / PROGRESS_CHUNK_SIZE;
+                               fprintf(stdout, "Writing %ukB chunk %u out of %u\n", PROGRESS_CHUNK_SIZE/1024, pgnum, pgcount);
+                       }
+               }
 
-        aligned_size = cmp_size;
-        if (aligned_size & (4 - 1))
-            aligned_size = (cmp_size + 4) & ~(4 - 1);
+               memcpy(sl->q_buf, (const void*)(base + off), sizeof(uint32_t));
+               stlink_write_mem32(sl, addr + off, sizeof(uint32_t));
 
-        stlink_read_mem32(sl, addr + off, aligned_size);
+               /* wait for sr.busy to be cleared */
+           wait_flash_busy(sl);
 
-        if (memcmp(sl->q_buf, base + off, cmp_size))
-            return -1;
-    }
+       }
+       /* Relock flash */
+       lock_flash(sl);
 
-    return 0;
-}
+#if 0 /* todo: debug mode */
+       fprintf(stdout, "Final CR:%X\n", read_flash_cr(sl));
+#endif
 
-int stlink_fwrite_flash(stlink_t *sl, const char* path, stm32_addr_t addr) {
-    /* write the file in flash at addr */
 
-    int error = -1;
-    size_t off;
-    mapped_file_t mf = MAPPED_FILE_INITIALIZER;
-    flash_loader_t fl;
 
-    if (map_file(&mf, path) == -1) {
-        fprintf(stderr, "map_file() == -1\n");
-        return -1;
-    }
+    }  //STM32F4END
 
-    /* check addr range is inside the flash */
-    if (addr < sl->flash_base) {
-        fprintf(stderr, "addr too low\n");
-        goto on_error;
-    } else if ((addr + mf.len) < addr) {
-        fprintf(stderr, "addr overruns\n");
-        goto on_error;
-    } else if ((addr + mf.len) > (sl->flash_base + sl->flash_size)) {
-        fprintf(stderr, "addr too high\n");
-        goto on_error;
-    } else if ((addr & (sl->flash_pgsz - 1)) || (mf.len & 1)) {
-        /* todo */
-        fprintf(stderr, "unaligned addr or size\n");
-        goto on_error;
-    }
+    else if (sl->core_id == STM32L_CORE_ID)    {
+       /* use fast word write. todo: half page. */
 
-    /* needed for specializing loader */
-    stlink_core_id(sl);
+       uint32_t val;
 
-    /* erase each page. todo: mass erase faster? */
-    for (off = 0; off < mf.len; off += sl->flash_pgsz) {
-        /* addr must be an addr inside the page */
-        if (stlink_erase_flash_page(sl, addr + off) == -1) {
-            fprintf(stderr, "erase_flash_page(0x%zx) == -1\n", addr + off);
-            goto on_error;
-        }
-    }
+#if 0 /* todo: check write operation */
 
-    /* write each page. above WRITE_BLOCK_SIZE fails? */
+       uint32_t nwrites = sl->flash_pgsz;
 
-    if (sl->core_id == 0x2ba01477) /* stm32l */
-    {
-      /* use fast word write. todo: half page. */
+       redo_write:
 
-      uint32_t val;
+#endif /* todo: check write operation */
 
-#if 0 /* todo: check write operation */
+       /* disable pecr protection */
+       write_uint32(sl->q_buf, 0x89abcdef);
+       stlink_write_mem32(sl, STM32L_FLASH_PEKEYR, sizeof(uint32_t));
+       write_uint32(sl->q_buf, 0x02030405);
+       stlink_write_mem32(sl, STM32L_FLASH_PEKEYR, sizeof(uint32_t));
+
+       /* check pecr.pelock is cleared */
+       stlink_read_mem32(sl, STM32L_FLASH_PECR, sizeof(uint32_t));
+       val = read_uint32(sl->q_buf, 0);
+       if (val & (1 << 0)) {
+               fprintf(stderr, "pecr.pelock not clear\n");
+               return -1;
+       }
+
+       /* unlock program memory */
+       write_uint32(sl->q_buf, 0x8c9daebf);
+       stlink_write_mem32(sl, STM32L_FLASH_PRGKEYR, sizeof(uint32_t));
+       write_uint32(sl->q_buf, 0x13141516);
+       stlink_write_mem32(sl, STM32L_FLASH_PRGKEYR, sizeof(uint32_t));
+
+       /* check pecr.prglock is cleared */
+       stlink_read_mem32(sl, STM32L_FLASH_PECR, sizeof(uint32_t));
+       val = read_uint32(sl->q_buf, 0);
+       if (val & (1 << 1)) {
+               fprintf(stderr, "pecr.prglock not clear\n");
+               return -1;
+       }
+
+       /* write a word in program memory */
+       for (off = 0; off < len; off += sizeof(uint32_t)) {
+               if (sl->verbose >= 1) {
+                       if ((off & (sl->flash_pgsz - 1)) == 0) {
+                               /* show progress. writing procedure is slow
+                                  and previous errors are misleading */
+                               const uint32_t pgnum = off / sl->flash_pgsz;
+                               const uint32_t pgcount = len / sl->flash_pgsz;
+                               fprintf(stdout, "%u pages written out of %u\n", pgnum, pgcount);
+                       }
+               }
+
+               memcpy(sl->q_buf, (const void*)(base + off), sizeof(uint32_t));
+               stlink_write_mem32(sl, addr + off, sizeof(uint32_t));
+
+               /* wait for sr.busy to be cleared */
+               while (1) {
+                       stlink_read_mem32(sl, STM32L_FLASH_SR, sizeof(uint32_t));
+                       if ((read_uint32(sl->q_buf, 0) & (1 << 0)) == 0) break ;
+               }
+
+#if 0 /* todo: check redo write operation */
+
+               /* check written bytes. todo: should be on a per page basis. */
+               stlink_read_mem32(sl, addr + off, sizeof(uint32_t));
+               if (memcmp(sl->q_buf, base + off, sizeof(uint32_t))) {
+                       /* re erase the page and redo the write operation */
+                       uint32_t page;
+                       uint32_t val;
+
+                       /* fail if successive write count too low */
+                       if (nwrites < sl->flash_pgsz) {
+                               fprintf(stderr, "writes operation failure count too high, aborting\n");
+                               return -1;
+                       }
+
+                       nwrites = 0;
+
+                       /* assume addr aligned */
+                       if (off % sl->flash_pgsz) off &= ~(sl->flash_pgsz - 1);
+                       page = addr + off;
+
+                       fprintf(stderr, "invalid write @%x(%x): %x != %x. retrying.\n",
+                                       page, addr + off, read_uint32(base + off, 0), read_uint32(sl->q_buf, 0));
+
+                       /* reset lock bits */
+                       stlink_read_mem32(sl, STM32L_FLASH_PECR, sizeof(uint32_t));
+                       val = read_uint32(sl->q_buf, 0) | (1 << 0) | (1 << 1) | (1 << 2);
+                       write_uint32(sl->q_buf, val);
+                       stlink_write_mem32(sl, STM32L_FLASH_PECR, sizeof(uint32_t));
+
+                       stlink_erase_flash_page(sl, page);
+
+                       goto redo_write;
+               }
+
+               /* increment successive writes counter */
+               ++nwrites;
+
+#endif /* todo: check redo write operation */
+       }
+       /* reset lock bits */
+       stlink_read_mem32(sl, STM32L_FLASH_PECR, sizeof(uint32_t));
+       val = read_uint32(sl->q_buf, 0) | (1 << 0) | (1 << 1) | (1 << 2);
+       write_uint32(sl->q_buf, val);
+       stlink_write_mem32(sl, STM32L_FLASH_PECR, sizeof(uint32_t));
+       }
 
-      uint32_t nwrites = sl->flash_pgsz;
 
-    redo_write:
 
-#endif /* todo: check write operation */
 
-      /* disable pecr protection */
-      write_uint32(sl->q_buf, 0x89abcdef);
-      stlink_write_mem32(sl, STM32L_FLASH_PEKEYR, sizeof(uint32_t));
-      write_uint32(sl->q_buf, 0x02030405);
-      stlink_write_mem32(sl, STM32L_FLASH_PEKEYR, sizeof(uint32_t));
-
-      /* check pecr.pelock is cleared */
-      stlink_read_mem32(sl, STM32L_FLASH_PECR, sizeof(uint32_t));
-      val = read_uint32(sl->q_buf, 0);
-      if (val & (1 << 0))
-      {
-       fprintf(stderr, "pecr.pelock not clear\n");
-       goto on_error;
+    else if (sl->core_id == STM32VL_CORE_ID) {
+       /* flash loader initialization */
+       if (init_flash_loader(sl, &fl) == -1) {
+               fprintf(stderr, "init_flash_loader() == -1\n");
+               return -1;
+       }
+       /* write each page. above WRITE_BLOCK_SIZE fails? */
+#define WRITE_BLOCK_SIZE 0x40
+      for (off = 0; off < len; off += WRITE_BLOCK_SIZE)      {
+         /* adjust last write size */
+         size_t size = WRITE_BLOCK_SIZE;
+         if ((off + WRITE_BLOCK_SIZE) > len) size = len - off;
+
+         /* unlock and set programming mode */
+         unlock_flash_if(sl);
+         set_flash_cr_pg(sl);
+
+         if (run_flash_loader(sl, &fl, addr + off, base + off, size) == -1) {
+                 fprintf(stderr, "run_flash_loader(0x%zx) == -1\n", addr + off);
+                 return -1;
+         }
+         lock_flash(sl);
       }
+    }
 
-      /* unlock program memory */
-      write_uint32(sl->q_buf, 0x8c9daebf);
-      stlink_write_mem32(sl, STM32L_FLASH_PRGKEYR, sizeof(uint32_t));
-      write_uint32(sl->q_buf, 0x13141516);
-      stlink_write_mem32(sl, STM32L_FLASH_PRGKEYR, sizeof(uint32_t));
-
-      /* check pecr.prglock is cleared */
-      stlink_read_mem32(sl, STM32L_FLASH_PECR, sizeof(uint32_t));
-      val = read_uint32(sl->q_buf, 0);
-      if (val & (1 << 1))
-      {
-       fprintf(stderr, "pecr.prglock not clear\n");
-       goto on_error;
-      }
 
-      /* write a word in program memory */
-      for (off = 0; off < mf.len; off += sizeof(uint32_t))
-      {
-       memcpy(sl->q_buf, (const void*)(mf.base + off), sizeof(uint32_t));
-       stlink_write_mem32(sl, addr + off, sizeof(uint32_t));
 
-       /* wait for sr.busy to be cleared */
-       while (1)
-       {
-         stlink_read_mem32(sl, STM32L_FLASH_SR, sizeof(uint32_t));
-         if ((read_uint32(sl->q_buf, 0) & (1 << 0)) == 0) break ;
-       }
 
-#if 0 /* todo: check write operation */
+    else
+    {
+      fprintf(stderr, "unknown device!\n");
+      return -1;
+    }
 
-       /* check written bytes. todo: should be on a per page basis. */
-       stlink_read_mem32(sl, addr + off, sizeof(uint32_t));
-       if (memcmp(sl->q_buf, mf.base + off, sizeof(uint32_t)))
-       {
-         /* re erase the page and redo the write operation */
-         uint32_t page;
-         uint32_t val;
 
-         /* fail if successive write count too low */
-         if (nwrites < sl->flash_pgsz) {
-           fprintf(stderr, "writes operation failure count too high, aborting\n");
-           goto on_error;
-         }
 
-         nwrites = 0;
 
-         /* assume addr aligned */
-         if (off % sl->flash_pgsz) off &= ~(sl->flash_pgsz - 1);
-         page = addr + off;
 
-         fprintf(stderr, "invalid write @%x(%x): %x != %x. retrying.\n",
-                 page, addr + off, read_uint32(mf.base + off, 0), read_uint32(sl->q_buf, 0));
+#if(0)
+    //todo: F4 Can't stlink_read_mem32 an entire sector, not enough ram!
+    for (off = 0; off < len; off += sl->flash_pgsz) {
+        size_t aligned_size;
 
-         /* reset lock bits */
-         stlink_read_mem32(sl, STM32L_FLASH_PECR, sizeof(uint32_t));
-         val = read_uint32(sl->q_buf, 0) | (1 << 0) | (1 << 1) | (1 << 2);
-         write_uint32(sl->q_buf, val);
-         stlink_write_mem32(sl, STM32L_FLASH_PECR, sizeof(uint32_t));
+        /* adjust last page size */
+        size_t cmp_size = sl->flash_pgsz;
+        if ((off + sl->flash_pgsz) > len)
+            cmp_size = len - off;
 
-         stlink_erase_flash_page(sl, page);
+        aligned_size = cmp_size;
+        if (aligned_size & (4 - 1))
+            aligned_size = (cmp_size + 4) & ~(4 - 1);
 
-         goto redo_write;
-       }
+               fprintf(stdout, "AlignedSize:%x\n", aligned_size);
+        stlink_read_mem32(sl, addr + off, aligned_size);
 
-       /* increment successive writes counter */
-       ++nwrites;
+        if (memcmp(sl->q_buf, base + off, cmp_size))
+            return -1;
+    }
+#endif
 
-#endif /* todo: check write operation */
+    return 0;
+}
 
-      }
 
-      /* reset lock bits */
-      stlink_read_mem32(sl, STM32L_FLASH_PECR, sizeof(uint32_t));
-      val = read_uint32(sl->q_buf, 0) | (1 << 0) | (1 << 1) | (1 << 2);
-      write_uint32(sl->q_buf, val);
-      stlink_write_mem32(sl, STM32L_FLASH_PECR, sizeof(uint32_t));
-    }
-    else /* stm32vl */
-    {
-#define WRITE_BLOCK_SIZE 0x40
-      for (off = 0; off < mf.len; off += WRITE_BLOCK_SIZE)
-      {
-        /* adjust last write size */
-        size_t size = WRITE_BLOCK_SIZE;
-        if ((off + WRITE_BLOCK_SIZE) > mf.len) size = mf.len - off;
-
-       /* unlock and set programming mode */
-       unlock_flash_if(sl);
-       set_flash_cr_pg(sl);
-
-       if (init_flash_loader(sl, &fl) == -1) {
-         fprintf(stderr, "init_flash_loader() == -1\n");
-         goto on_error;
-       }
 
-        if (run_flash_loader(sl, &fl, addr + off, mf.base + off, size) == -1)
-       {
-         fprintf(stderr, "run_flash_loader(0x%zx) == -1\n", addr + off);
-         goto on_error;
-        }
 
-       lock_flash(sl);
-      }
+int stlink_fwrite_flash(stlink_t *sl, const char* path, stm32_addr_t addr) {
+    /* write the file in flash at addr */
 
-    } /* stm32vl */
+    int err;
+    mapped_file_t mf = MAPPED_FILE_INITIALIZER;
 
-    /* check the file ha been written */
-    if (check_file(sl, &mf, addr) == -1) {
-        fprintf(stderr, "check_file() == -1\n");
-        goto on_error;
+    if (map_file(&mf, path) == -1) {
+        fprintf(stderr, "map_file() == -1\n");
+        return -1;
     }
 
-    /* success */
-    error = 0;
+    err = stlink_write_flash(sl, addr, mf.base, mf.len);
 
-on_error:
     unmap_file(&mf);
-    return error;
+
+    return err;
 }
 
 int run_flash_loader(stlink_t *sl, flash_loader_t* fl, stm32_addr_t target, const uint8_t* buf, size_t size) {
@@ -1285,7 +1384,7 @@ int run_flash_loader(stlink_t *sl, flash_loader_t* fl, stm32_addr_t target, cons
         return -1;
     }
 
-    if (sl->core_id == 0x2ba01477) /* stm32l */ {
+    if (sl->core_id == STM32L_CORE_ID) {
 
       size_t count = size / sizeof(uint32_t);
       if (size % sizeof(uint32_t)) ++count;
@@ -1297,7 +1396,7 @@ int run_flash_loader(stlink_t *sl, flash_loader_t* fl, stm32_addr_t target, cons
       stlink_write_reg(sl, 0, 3); /* output count */
       stlink_write_reg(sl, fl->loader_addr, 15); /* pc register */
 
-    } else /* stm32vl */ {
+    } else if (sl->core_id == STM32VL_CORE_ID) {
 
       size_t count = size / sizeof(uint16_t);
       if (size % sizeof(uint16_t)) ++count;
@@ -1309,6 +1408,9 @@ int run_flash_loader(stlink_t *sl, flash_loader_t* fl, stm32_addr_t target, cons
       stlink_write_reg(sl, 0, 3); /* flash bank 0 (input) */
       stlink_write_reg(sl, fl->loader_addr, 15); /* pc register */
 
+    } else {
+      fprintf(stderr, "unknown coreid: %x\n", sl->core_id);
+      return -1;
     }
 
     /* run loader */
@@ -1318,7 +1420,7 @@ int run_flash_loader(stlink_t *sl, flash_loader_t* fl, stm32_addr_t target, cons
     while (is_core_halted(sl) == 0) ;
 
     /* check written byte count */
-    if (sl->core_id == 0x2ba01477) /* stm32l */ {
+    if (sl->core_id == STM32L_CORE_ID) {
 
       size_t count = size / sizeof(uint32_t);
       if (size % sizeof(uint32_t)) ++count;
@@ -1329,7 +1431,7 @@ int run_flash_loader(stlink_t *sl, flash_loader_t* fl, stm32_addr_t target, cons
         return -1;
       }
 
-    } else /* stm32vl */ {
+    } else if (sl->core_id == STM32VL_CORE_ID) {
 
       stlink_read_reg(sl, 2, &rr);
       if (rr.r[2] != 0) {
@@ -1337,6 +1439,11 @@ int run_flash_loader(stlink_t *sl, flash_loader_t* fl, stm32_addr_t target, cons
         return -1;
       }
 
+    } else {
+
+      fprintf(stderr, "unknown coreid: %x\n", sl->core_id);
+      return -1;
+
     }
 
     return 0;