Determine the device parameters explicit after running

[fw/stlink] / src / stlink-common.c

diff --git a/src/stlink-common.c b/src/stlink-common.c
index 3b84cc689119be4f11664f741057a71c080e2184..8bc2a8bf02b0a84d0f409f7bf52487a8aa7af2c2 100644 (file)
--- a/src/stlink-common.c
+++ b/src/stlink-common.c
@@ -122,17 +122,17 @@ uint32_t read_uint32(const unsigned char *c, const int pt) {
 
 static uint32_t __attribute__((unused)) read_flash_rdp(stlink_t *sl) {
     stlink_read_mem32(sl, FLASH_WRPR, sizeof (uint32_t));
-    return (*(uint32_t*) sl->q_buf) & 0xff;
+    return read_uint32(sl->q_buf, 0) & 0xff;
 }
 
 static inline uint32_t read_flash_wrpr(stlink_t *sl) {
     stlink_read_mem32(sl, FLASH_WRPR, sizeof (uint32_t));
-    return *(uint32_t*) sl->q_buf;
+    return read_uint32(sl->q_buf, 0);
 }
 
 static inline uint32_t read_flash_obr(stlink_t *sl) {
     stlink_read_mem32(sl, FLASH_OBR, sizeof (uint32_t));
-    return *(uint32_t*) sl->q_buf;
+    return read_uint32(sl->q_buf, 0);
 }
 
 static inline uint32_t read_flash_cr(stlink_t *sl) {
@@ -143,7 +143,7 @@ static inline uint32_t read_flash_cr(stlink_t *sl) {
 #if DEBUG_FLASH
        fprintf(stdout, "CR:0x%x\n", *(uint32_t*) sl->q_buf);
 #endif
-       return *(uint32_t*) sl->q_buf;
+       return read_uint32(sl->q_buf, 0);
 }
 
 static inline unsigned int is_flash_locked(stlink_t *sl) {
@@ -269,7 +269,7 @@ static void set_flash_cr_strt(stlink_t *sl) {
 
 static inline uint32_t read_flash_acr(stlink_t *sl) {
     stlink_read_mem32(sl, FLASH_ACR, sizeof (uint32_t));
-    return *(uint32_t*) sl->q_buf;
+    return read_uint32(sl->q_buf, 0);
 }
 
 static inline uint32_t read_flash_sr(stlink_t *sl) {
@@ -278,7 +278,7 @@ static inline uint32_t read_flash_sr(stlink_t *sl) {
        else
                stlink_read_mem32(sl, FLASH_SR, sizeof (uint32_t));
     //fprintf(stdout, "SR:0x%x\n", *(uint32_t*) sl->q_buf);
-       return *(uint32_t*) sl->q_buf;
+    return read_uint32(sl->q_buf, 0);
 }
 
 static inline unsigned int is_flash_busy(stlink_t *sl) {
@@ -457,7 +457,7 @@ int stlink_load_device_params(stlink_t *sl) {
     sl->sys_base = params->bootrom_base;
     sl->sys_size = params->bootrom_size;
     
-    ILOG("Device connected is: %s\n", params->description);
+    ILOG("Device connected is: %s, id %#x\n", params->description, chip_id);
     // TODO make note of variable page size here.....
     ILOG("SRAM size: %#x bytes (%d KiB), Flash: %#x bytes (%d KiB) in pages of %zd bytes\n",
         sl->sram_size, sl->sram_size / 1024, sl->flash_size, sl->flash_size / 1024, 
@@ -470,6 +470,11 @@ void stlink_reset(stlink_t *sl) {
     sl->backend->reset(sl);
 }
 
+void stlink_jtag_reset(stlink_t *sl, int value) {
+    DLOG("*** stlink_jtag_reset ***\n");
+    sl->backend->jtag_reset(sl, value);
+}
+
 void stlink_run(stlink_t *sl) {
     DLOG("*** stlink_run ***\n");
     sl->backend->run(sl);
@@ -524,6 +529,17 @@ void stlink_version(stlink_t *sl) {
     }
 }
 
+uint32_t stlink_read_debug32(stlink_t *sl, uint32_t addr) {
+    uint32_t data = sl->backend->read_debug32(sl, addr);
+    DLOG("*** stlink_read_debug32 %x is %#x\n", data, addr);
+    return data;
+}
+
+void stlink_write_debug32(stlink_t *sl, uint32_t addr, uint32_t data) {
+    DLOG("*** stlink_write_debug32 %x to %#x\n", data, addr);
+    sl->backend->write_debug32(sl, addr, data);
+}
+
 void stlink_write_mem32(stlink_t *sl, uint32_t addr, uint16_t len) {
     DLOG("*** stlink_write_mem32 %u bytes to %#x\n", len, addr);
     if (len % 4 != 0) {
@@ -969,17 +985,36 @@ int stlink_erase_flash_page(stlink_t *sl, stm32_addr_t flashaddr)
     write_uint32(sl->q_buf, val);
     stlink_write_mem32(sl, STM32L_FLASH_PECR, sizeof(uint32_t));
 
-    /* wait for sr.busy to be cleared */
+#if 0 /* fix_to_be_confirmed */
+
+    /* wait for sr.busy to be cleared
+       MP: Test shows that busy bit is not set here. Perhaps, PM0062 is
+       wrong and we do not need to wait here for clearing the busy bit.
+       TEXANE: ok, if experience says so and it works for you, we comment
+       it. If someone has a problem, please drop an email.
+     */
     while (1)
     {
       stlink_read_mem32(sl, STM32L_FLASH_SR, sizeof(uint32_t));
       if ((read_uint32(sl->q_buf, 0) & (1 << 0)) == 0) break ;
     }
 
+#endif /* fix_to_be_confirmed */
+
     /* write 0 to the first word of the page to be erased */
     memset(sl->q_buf, 0, sizeof(uint32_t));
     stlink_write_mem32(sl, flashaddr, sizeof(uint32_t));
 
+    /* MP: It is better to wait for clearing the busy bit after issuing
+    page erase command, even though PM0062 recommends to wait before it.
+    Test shows that a few iterations is performed in the following loop
+    before busy bit is 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);
@@ -1175,7 +1210,6 @@ int stlink_verify_write_flash(stlink_t *sl, stm32_addr_t address, uint8_t *data,
         if (aligned_size & (4 - 1))
             aligned_size = (cmp_size + 4) & ~(4 - 1);
 
-               fprintf(stdout, "AlignedSize:%#zx\n", aligned_size);
         stlink_read_mem32(sl, address + off, aligned_size);
 
         if (memcmp(sl->q_buf, data + off, cmp_size)) {
@@ -1272,7 +1306,6 @@ int stlink_write_flash(stlink_t *sl, stm32_addr_t addr, uint8_t* base, unsigned
 
     else if (sl->core_id == STM32L_CORE_ID)    {
        /* use fast word write. todo: half page. */
-
        uint32_t val;
 
 #if 0 /* todo: check write operation */