Fix usage of timeval_ms()
[openocd.git] / src / target / adi_v5_jtag.c
index 091b77ab4c57dbea85d6c0dbb782701a69ead6e5..36d5cad5135d4b8ef5bc2dcd6abec668750b86ca 100644 (file)
@@ -21,9 +21,7 @@
  *   GNU General Public License for more details.
  *
  *   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.
+ *   along with this program.  If not, see <http://www.gnu.org/licenses/>.
  ***************************************************************************/
 
 /**
@@ -39,7 +37,9 @@
 #include "arm.h"
 #include "arm_adi_v5.h"
 #include <helper/time_support.h>
+#include <helper/list.h>
 
+/*#define DEBUG_WAIT*/
 
 /* JTAG instructions/registers for JTAG-DP and SWJ-DP */
 #define JTAG_DP_ABORT          0x8
 #define JTAG_ACK_OK_FAULT      0x2
 #define JTAG_ACK_WAIT          0x1
 
+static int jtag_ap_q_abort(struct adiv5_dap *dap, uint8_t *ack);
+
+#ifdef DEBUG_WAIT
+static const char *dap_reg_name(int instr, int reg_addr)
+{
+       char *reg_name = "UNK";
+
+       if (instr == JTAG_DP_DPACC) {
+               switch (reg_addr) {
+               case DP_ABORT:
+                       reg_name =  "ABORT";
+                       break;
+               case DP_CTRL_STAT:
+                       reg_name =  "CTRL/STAT";
+                       break;
+               case DP_SELECT:
+                       reg_name = "SELECT";
+                       break;
+               case DP_RDBUFF:
+                       reg_name =  "RDBUFF";
+                       break;
+               case DP_WCR:
+                       reg_name =  "WCR";
+                       break;
+               default:
+                       reg_name = "UNK";
+                       break;
+               }
+       }
+
+       if (instr == JTAG_DP_APACC) {
+               switch (reg_addr) {
+               case MEM_AP_REG_CSW:
+                       reg_name = "CSW";
+                       break;
+               case MEM_AP_REG_TAR:
+                       reg_name = "TAR";
+                       break;
+               case MEM_AP_REG_DRW:
+                       reg_name = "DRW";
+                       break;
+               case MEM_AP_REG_BD0:
+                       reg_name = "BD0";
+                       break;
+               case MEM_AP_REG_BD1:
+                       reg_name = "BD1";
+                       break;
+               case MEM_AP_REG_BD2:
+                       reg_name = "BD2";
+                       break;
+               case MEM_AP_REG_BD3:
+                       reg_name = "BD3";
+                       break;
+               case MEM_AP_REG_CFG:
+                       reg_name = "CFG";
+                       break;
+               case MEM_AP_REG_BASE:
+                       reg_name = "BASE";
+                       break;
+               case AP_REG_IDR:
+                       reg_name = "IDR";
+                       break;
+               default:
+                       reg_name = "UNK";
+                       break;
+               }
+       }
+
+       return reg_name;
+}
+#endif
+
+struct dap_cmd {
+       struct list_head lh;
+       uint8_t instr;
+       uint8_t reg_addr;
+       uint8_t RnW;
+       uint8_t *invalue;
+       uint8_t ack;
+       uint32_t memaccess_tck;
+       uint32_t dp_select;
+
+       struct scan_field fields[2];
+       uint8_t out_addr_buf;
+       uint8_t invalue_buf[4];
+       uint8_t outvalue_buf[4];
+};
+
+static void log_dap_cmd(const char *header, struct dap_cmd *el)
+{
+#ifdef DEBUG_WAIT
+       LOG_DEBUG("%s: %2s %6s %5s 0x%08x 0x%08x %2s", header,
+               el->instr == JTAG_DP_APACC ? "AP" : "DP",
+               dap_reg_name(el->instr, el->reg_addr),
+               el->RnW == DPAP_READ ? "READ" : "WRITE",
+               buf_get_u32(el->outvalue_buf, 0, 32),
+               buf_get_u32(el->invalue, 0, 32),
+               el->ack == JTAG_ACK_OK_FAULT ? "OK" :
+                       (el->ack == JTAG_ACK_WAIT ? "WAIT" : "INVAL"));
+#endif
+}
+
+static struct dap_cmd *dap_cmd_new(uint8_t instr,
+               uint8_t reg_addr, uint8_t RnW,
+               uint8_t *outvalue, uint8_t *invalue,
+               uint32_t memaccess_tck)
+{
+       struct dap_cmd *cmd;
+
+       cmd = (struct dap_cmd *)calloc(1, sizeof(struct dap_cmd));
+       if (cmd != NULL) {
+               INIT_LIST_HEAD(&cmd->lh);
+               cmd->instr = instr;
+               cmd->reg_addr = reg_addr;
+               cmd->RnW = RnW;
+               if (outvalue != NULL)
+                       memcpy(cmd->outvalue_buf, outvalue, 4);
+               cmd->invalue = (invalue != NULL) ? invalue : cmd->invalue_buf;
+               cmd->memaccess_tck = memaccess_tck;
+       }
+
+       return cmd;
+}
+
+static void flush_journal(struct list_head *lh)
+{
+       struct dap_cmd *el, *tmp;
+
+       list_for_each_entry_safe(el, tmp, lh, lh) {
+               list_del(&el->lh);
+               free(el);
+       }
+}
+
 /***************************************************************************
  *
  * DPACC and APACC scanchain access through JTAG-DP (or SWJ-DP)
  *
 ***************************************************************************/
 
+static int adi_jtag_dp_scan_cmd(struct adiv5_dap *dap, struct dap_cmd *cmd, uint8_t *ack)
+{
+       struct jtag_tap *tap = dap->tap;
+       int retval;
+
+       retval = arm_jtag_set_instr(tap, cmd->instr, NULL, TAP_IDLE);
+       if (retval != ERROR_OK)
+               return retval;
+
+       /* Scan out a read or write operation using some DP or AP register.
+        * For APACC access with any sticky error flag set, this is discarded.
+        */
+       cmd->fields[0].num_bits = 3;
+       buf_set_u32(&cmd->out_addr_buf, 0, 3, ((cmd->reg_addr >> 1) & 0x6) | (cmd->RnW & 0x1));
+       cmd->fields[0].out_value = &cmd->out_addr_buf;
+       cmd->fields[0].in_value = (ack != NULL) ? ack : &cmd->ack;
+
+       /* NOTE: if we receive JTAG_ACK_WAIT, the previous operation did not
+        * complete; data we write is discarded, data we read is unpredictable.
+        * When overrun detect is active, STICKYORUN is set.
+        */
+
+       cmd->fields[1].num_bits = 32;
+       cmd->fields[1].out_value = cmd->outvalue_buf;
+       cmd->fields[1].in_value = cmd->invalue;
+
+       jtag_add_dr_scan(tap, 2, cmd->fields, TAP_IDLE);
+
+       /* Add specified number of tck clocks after starting memory bus
+        * access, giving the hardware time to complete the access.
+        * They provide more time for the (MEM) AP to complete the read ...
+        * See "Minimum Response Time" for JTAG-DP, in the ADIv5 spec.
+        */
+       if (cmd->instr == JTAG_DP_APACC) {
+               if (((cmd->reg_addr == MEM_AP_REG_DRW)
+                       || ((cmd->reg_addr & 0xF0) == MEM_AP_REG_BD0))
+                       && (cmd->memaccess_tck != 0))
+                       jtag_add_runtest(cmd->memaccess_tck, TAP_IDLE);
+       }
+
+       return ERROR_OK;
+}
+
+static int adi_jtag_dp_scan_cmd_sync(struct adiv5_dap *dap, struct dap_cmd *cmd, uint8_t *ack)
+{
+       int retval;
+
+       retval = adi_jtag_dp_scan_cmd(dap, cmd, ack);
+       if (retval != ERROR_OK)
+               return retval;
+
+       return jtag_execute_queue();
+}
+
 /**
  * Scan DPACC or APACC using target ordered uint8_t buffers.  No endianness
  * conversions are performed.  See section 4.4.3 of the ADIv5 spec, which
  * will be needed to collect the data which was read; the "invalue" collects
  * the posted result of a preceding operation, not the current one.
  *
- * @param swjdp the DAP
+ * @param dap the DAP
  * @param instr JTAG_DP_APACC (AP access) or JTAG_DP_DPACC (DP access)
  * @param reg_addr two significant bits; A[3:2]; for APACC access, the
  *     SELECT register has more addressing bits.
  * @param outvalue points to a 32-bit (little-endian) integer
  * @param invalue NULL, or points to a 32-bit (little-endian) integer
  * @param ack points to where the three bit JTAG_ACK_* code will be stored
+ * @param memaccess_tck number of idle cycles to add after AP access
  */
 
-/* FIXME don't export ... this is a temporary workaround for the
- * mem_ap_read_buf_u32() mess, until it's no longer JTAG-specific.
- */
-int adi_jtag_dp_scan(struct adiv5_dap *swjdp,
+static int adi_jtag_dp_scan(struct adiv5_dap *dap,
                uint8_t instr, uint8_t reg_addr, uint8_t RnW,
-               uint8_t *outvalue, uint8_t *invalue, uint8_t *ack)
+               uint8_t *outvalue, uint8_t *invalue,
+               uint32_t memaccess_tck, uint8_t *ack)
 {
-       struct arm_jtag *jtag_info = swjdp->jtag_info;
-       struct scan_field fields[2];
-       uint8_t out_addr_buf;
-
-       jtag_set_end_state(TAP_IDLE);
-       arm_jtag_set_instr(jtag_info, instr, NULL);
-
-       /* Scan out a read or write operation using some DP or AP register.
-        * For APACC access with any sticky error flag set, this is discarded.
-        */
-       fields[0].num_bits = 3;
-       buf_set_u32(&out_addr_buf, 0, 3, ((reg_addr >> 1) & 0x6) | (RnW & 0x1));
-       fields[0].out_value = &out_addr_buf;
-       fields[0].in_value = ack;
-
-       /* NOTE: if we receive JTAG_ACK_WAIT, the previous operation did not
-        * complete; data we write is discarded, data we read is unpredictable.
-        * When overrun detect is active, STICKYORUN is set.
-        */
+       struct dap_cmd *cmd;
+       int retval;
 
-       fields[1].num_bits = 32;
-       fields[1].out_value = outvalue;
-       fields[1].in_value = invalue;
+       cmd = dap_cmd_new(instr, reg_addr, RnW, outvalue, invalue, memaccess_tck);
+       if (cmd != NULL)
+               cmd->dp_select = dap->select;
+       else
+               return ERROR_JTAG_DEVICE_ERROR;
 
-       jtag_add_dr_scan(jtag_info->tap, 2, fields, TAP_IDLE);
+       retval = adi_jtag_dp_scan_cmd(dap, cmd, ack);
+       if (retval == ERROR_OK)
+               list_add_tail(&cmd->lh, &dap->cmd_journal);
 
-       /* Add specified number of tck clocks after starting memory bus
-        * access, giving the hardware time to complete the access.
-        * They provide more time for the (MEM) AP to complete the read ...
-        * See "Minimum Response Time" for JTAG-DP, in the ADIv5 spec.
-        */
-       if ((instr == JTAG_DP_APACC)
-                       && ((reg_addr == AP_REG_DRW)
-                               || ((reg_addr & 0xF0) == AP_REG_BD0))
-                       && (swjdp->memaccess_tck != 0))
-               jtag_add_runtest(swjdp->memaccess_tck,
-                               TAP_IDLE);
-
-       return jtag_get_error();
+       return retval;
 }
 
 /**
@@ -130,17 +292,18 @@ int adi_jtag_dp_scan(struct adiv5_dap *swjdp,
  * conversions are performed (so the types of invalue and outvalue
  * must be different).
  */
-static int adi_jtag_dp_scan_u32(struct adiv5_dap *swjdp,
+static int adi_jtag_dp_scan_u32(struct adiv5_dap *dap,
                uint8_t instr, uint8_t reg_addr, uint8_t RnW,
-               uint32_t outvalue, uint32_t *invalue, uint8_t *ack)
+               uint32_t outvalue, uint32_t *invalue,
+               uint32_t memaccess_tck, uint8_t *ack)
 {
        uint8_t out_value_buf[4];
        int retval;
 
        buf_set_u32(out_value_buf, 0, 32, outvalue);
 
-       retval = adi_jtag_dp_scan(swjdp, instr, reg_addr, RnW,
-                       out_value_buf, (uint8_t *)invalue, ack);
+       retval = adi_jtag_dp_scan(dap, instr, reg_addr, RnW,
+                       out_value_buf, (uint8_t *)invalue, memaccess_tck, ack);
        if (retval != ERROR_OK)
                return retval;
 
@@ -151,288 +314,389 @@ static int adi_jtag_dp_scan_u32(struct adiv5_dap *swjdp,
        return retval;
 }
 
-/**
- * Utility to write AP registers.
- */
-static inline int adi_jtag_ap_write_check(struct adiv5_dap *dap,
-               uint8_t reg_addr, uint8_t *outvalue)
+static int adi_jtag_finish_read(struct adiv5_dap *dap)
 {
-       return adi_jtag_dp_scan(dap, JTAG_DP_APACC, reg_addr, DPAP_WRITE,
-                       outvalue, NULL, NULL);
+       int retval = ERROR_OK;
+
+       if (dap->last_read != NULL) {
+               retval = adi_jtag_dp_scan_u32(dap, JTAG_DP_DPACC,
+                               DP_RDBUFF, DPAP_READ, 0, dap->last_read, 0, NULL);
+               dap->last_read = NULL;
+       }
+
+       return retval;
 }
 
-static int adi_jtag_scan_inout_check_u32(struct adiv5_dap *swjdp,
+static int adi_jtag_scan_inout_check_u32(struct adiv5_dap *dap,
                uint8_t instr, uint8_t reg_addr, uint8_t RnW,
-               uint32_t outvalue, uint32_t *invalue)
+               uint32_t outvalue, uint32_t *invalue, uint32_t memaccess_tck)
 {
        int retval;
 
        /* Issue the read or write */
-       retval = adi_jtag_dp_scan_u32(swjdp, instr, reg_addr,
-                       RnW, outvalue, NULL, NULL);
+       retval = adi_jtag_dp_scan_u32(dap, instr, reg_addr,
+                       RnW, outvalue, NULL, memaccess_tck, NULL);
        if (retval != ERROR_OK)
                return retval;
 
        /* For reads,  collect posted value; RDBUFF has no other effect.
         * Assumes read gets acked with OK/FAULT, and CTRL_STAT says "OK".
         */
-       if ((RnW == DPAP_READ) && (invalue != NULL))
-               retval = adi_jtag_dp_scan_u32(swjdp, JTAG_DP_DPACC,
-                               DP_RDBUFF, DPAP_READ, 0, invalue, &swjdp->ack);
-       return retval;
+       if ((RnW == DPAP_READ) && (invalue != NULL)) {
+               retval = adi_jtag_dp_scan_u32(dap, JTAG_DP_DPACC,
+                               DP_RDBUFF, DPAP_READ, 0, invalue, 0, NULL);
+               if (retval != ERROR_OK)
+                       return retval;
+       }
+
+       return jtag_execute_queue();
 }
 
-static int jtagdp_transaction_endcheck(struct adiv5_dap *swjdp)
+static int jtagdp_overrun_check(struct adiv5_dap *dap)
 {
        int retval;
-       uint32_t ctrlstat;
+       struct dap_cmd *el, *tmp, *prev = NULL;
+       int found_wait = 0;
+       int64_t time_now;
+       LIST_HEAD(replay_list);
 
-       /* too expensive to call keep_alive() here */
-
-#if 0
-       /* Danger!!!! BROKEN!!!! */
-       adi_jtag_scan_inout_check_u32(swjdp, JTAG_DP_DPACC,
-                       DP_CTRL_STAT, DPAP_READ, 0, &ctrlstat);
-       /* Danger!!!! BROKEN!!!! Why will jtag_execute_queue() fail here????
-       R956 introduced the check on return value here and now Michael Schwingen reports
-       that this code no longer works....
-
-       https://lists.berlios.de/pipermail/openocd-development/2008-September/003107.html
-       */
-       if ((retval = jtag_execute_queue()) != ERROR_OK)
-       {
-               LOG_ERROR("BUG: Why does this fail the first time????");
+       /* make sure all queued transactions are complete */
+       retval = jtag_execute_queue();
+       if (retval != ERROR_OK)
+               goto done;
+
+       /* skip all completed transactions up to the first WAIT */
+       list_for_each_entry(el, &dap->cmd_journal, lh) {
+               if (el->ack == JTAG_ACK_OK_FAULT) {
+                       log_dap_cmd("LOG", el);
+               } else if (el->ack == JTAG_ACK_WAIT) {
+                       found_wait = 1;
+                       break;
+               } else {
+                       LOG_ERROR("Invalid ACK (%1x) in DAP response", el->ack);
+                       log_dap_cmd("ERR", el);
+                       retval = ERROR_JTAG_DEVICE_ERROR;
+                       goto done;
+               }
        }
-       /* Why??? second time it works??? */
-#endif
 
-       /* Post CTRL/STAT read; discard any previous posted read value
-        * but collect its ACK status.
+       /*
+        * If we found a stalled transaction and a previous transaction
+        * exists, check if it's a READ access.
         */
-       adi_jtag_scan_inout_check_u32(swjdp, JTAG_DP_DPACC,
-                       DP_CTRL_STAT, DPAP_READ, 0, &ctrlstat);
-       if ((retval = jtag_execute_queue()) != ERROR_OK)
-               return retval;
-
-       swjdp->ack = swjdp->ack & 0x7;
-
-       /* common code path avoids calling timeval_ms() */
-       if (swjdp->ack != JTAG_ACK_OK_FAULT)
-       {
-               long long then = timeval_ms();
-
-               while (swjdp->ack != JTAG_ACK_OK_FAULT)
-               {
-                       if (swjdp->ack == JTAG_ACK_WAIT)
-                       {
-                               if ((timeval_ms()-then) > 1000)
-                               {
-                                       /* NOTE:  this would be a good spot
-                                        * to use JTAG_DP_ABORT.
-                                        */
-                                       LOG_WARNING("Timeout (1000ms) waiting "
-                                               "for ACK=OK/FAULT "
-                                               "in JTAG-DP transaction");
-                                       return ERROR_JTAG_DEVICE_ERROR;
+       if (found_wait && el != list_first_entry(&dap->cmd_journal, struct dap_cmd, lh)) {
+               prev = list_entry(el->lh.prev, struct dap_cmd, lh);
+               if (prev->RnW == DPAP_READ) {
+                       log_dap_cmd("PND", prev);
+                       /* search for the next OK transaction, it contains
+                        * the result of the previous READ */
+                       tmp = el;
+                       list_for_each_entry_from(tmp, &dap->cmd_journal, lh) {
+                               if (tmp->ack == JTAG_ACK_OK_FAULT) {
+                                       /* recover the read value */
+                                       log_dap_cmd("FND", tmp);
+                                       if (el->invalue != el->invalue_buf) {
+                                               uint32_t invalue = le_to_h_u32(tmp->invalue);
+                                               memcpy(el->invalue, &invalue, sizeof(uint32_t));
+                                       }
+                                       prev = NULL;
+                                       break;
                                }
                        }
-                       else
-                       {
-                               LOG_WARNING("Invalid ACK %#x "
-                                               "in JTAG-DP transaction",
-                                               swjdp->ack);
-                               return ERROR_JTAG_DEVICE_ERROR;
-                       }
 
-                       adi_jtag_scan_inout_check_u32(swjdp, JTAG_DP_DPACC,
-                                       DP_CTRL_STAT, DPAP_READ, 0, &ctrlstat);
-                       if ((retval = dap_run(swjdp)) != ERROR_OK)
-                               return retval;
-                       swjdp->ack = swjdp->ack & 0x7;
+                       if (prev != NULL) {
+                               log_dap_cmd("LST", el);
+
+                               /*
+                               * At this point we're sure that no previous
+                               * transaction completed and the DAP/AP is still
+                               * in busy state. We know that the next "OK" scan
+                               * will return the READ result we need to recover.
+                               * To complete the READ, we just keep polling RDBUFF
+                               * until the WAIT condition clears
+                               */
+                               tmp = dap_cmd_new(JTAG_DP_DPACC,
+                                               DP_RDBUFF, DPAP_READ, NULL, NULL, 0);
+                               if (tmp == NULL) {
+                                       retval = ERROR_JTAG_DEVICE_ERROR;
+                                       goto done;
+                               }
+                               /* synchronously retry the command until it succeeds */
+                               time_now = timeval_ms();
+                               do {
+                                       retval = adi_jtag_dp_scan_cmd_sync(dap, tmp, NULL);
+                                       if (retval != ERROR_OK)
+                                               break;
+                                       if (tmp->ack == JTAG_ACK_OK_FAULT) {
+                                               log_dap_cmd("FND", tmp);
+                                               if (el->invalue != el->invalue_buf) {
+                                                       uint32_t invalue = le_to_h_u32(tmp->invalue);
+                                                       memcpy(el->invalue, &invalue, sizeof(uint32_t));
+                                               }
+                                               break;
+                                       }
+                                       if (tmp->ack != JTAG_ACK_WAIT) {
+                                               LOG_ERROR("Invalid ACK (%1x) in DAP response", tmp->ack);
+                                               log_dap_cmd("ERR", tmp);
+                                               retval = ERROR_JTAG_DEVICE_ERROR;
+                                               break;
+                                       }
+
+                               } while (timeval_ms() - time_now < 1000);
+
+                               if (retval == ERROR_OK) {
+                                       /* timeout happened */
+                                       if (tmp->ack != JTAG_ACK_OK_FAULT) {
+                                               LOG_ERROR("Timeout during WAIT recovery");
+                                               jtag_ap_q_abort(dap, NULL);
+                                               retval = ERROR_JTAG_DEVICE_ERROR;
+                                       }
+                               }
+
+                               /* we're done with this command, release it */
+                               free(tmp);
+
+                               if (retval != ERROR_OK)
+                                       goto done;
+
+                       }
+                       /* make el->invalue point to the default invalue
+                       * so that we can safely retry it without clobbering
+                       * the result we just recovered */
+                       el->invalue = el->invalue_buf;
                }
        }
 
-       /* REVISIT also STICKYCMP, for pushed comparisons (nyet used) */
+       /* move all remaining transactions over to the replay list */
+       list_for_each_entry_safe_from(el, tmp, &dap->cmd_journal, lh) {
+               log_dap_cmd("REP", el);
+               list_move_tail(&el->lh, &replay_list);
+       }
 
-       /* Check for STICKYERR and STICKYORUN */
-       if (ctrlstat & (SSTICKYORUN | SSTICKYERR))
-       {
-               LOG_DEBUG("jtag-dp: CTRL/STAT error, 0x%" PRIx32, ctrlstat);
-               /* Check power to debug regions */
-               if ((ctrlstat & 0xf0000000) != 0xf0000000)
-                        ahbap_debugport_init(swjdp);
-               else
-               {
-                       uint32_t mem_ap_csw, mem_ap_tar;
-
-                       /* Maybe print information about last intended
-                        * MEM-AP access; but not if autoincrementing.
-                        * *Real* CSW and TAR values are always shown.
-                        */
-                       if (swjdp->ap_tar_value != (uint32_t) -1)
-                               LOG_DEBUG("MEM-AP Cached values: "
-                                       "ap_bank 0x%" PRIx32
-                                       ", ap_csw 0x%" PRIx32
-                                       ", ap_tar 0x%" PRIx32,
-                                       swjdp->ap_bank_value,
-                                       swjdp->ap_csw_value,
-                                       swjdp->ap_tar_value);
-
-                       if (ctrlstat & SSTICKYORUN)
-                               LOG_ERROR("JTAG-DP OVERRUN - check clock, "
-                                       "memaccess, or reduce jtag speed");
-
-                       if (ctrlstat & SSTICKYERR)
-                               LOG_ERROR("JTAG-DP STICKY ERROR");
-
-                       /* Clear Sticky Error Bits */
-                       adi_jtag_scan_inout_check_u32(swjdp, JTAG_DP_DPACC,
-                                       DP_CTRL_STAT, DPAP_WRITE,
-                                       swjdp->dp_ctrl_stat | SSTICKYORUN
-                                               | SSTICKYERR, NULL);
-                       adi_jtag_scan_inout_check_u32(swjdp, JTAG_DP_DPACC,
-                                       DP_CTRL_STAT, DPAP_READ, 0, &ctrlstat);
-                       if ((retval = dap_run(swjdp)) != ERROR_OK)
-                               return retval;
-
-                       LOG_DEBUG("jtag-dp: CTRL/STAT 0x%" PRIx32, ctrlstat);
-
-                       retval = dap_queue_ap_read(swjdp,
-                                       AP_REG_CSW, &mem_ap_csw);
-                       if (retval != ERROR_OK)
-                               return retval;
-
-                       retval = dap_queue_ap_read(swjdp,
-                                       AP_REG_TAR, &mem_ap_tar);
-                       if (retval != ERROR_OK)
-                               return retval;
-
-                       if ((retval = dap_run(swjdp)) != ERROR_OK)
-                               return retval;
-                       LOG_ERROR("MEM_AP_CSW 0x%" PRIx32 ", MEM_AP_TAR 0x%"
-                                       PRIx32, mem_ap_csw, mem_ap_tar);
+       /* we're done with the journal, flush it */
+       flush_journal(&dap->cmd_journal);
+
+       /* check for overrun condition in the last batch of transactions */
+       if (found_wait) {
+               LOG_INFO("DAP transaction stalled (WAIT) - slowing down");
+               /* clear the sticky overrun condition */
+               retval = adi_jtag_scan_inout_check_u32(dap, JTAG_DP_DPACC,
+                               DP_CTRL_STAT, DPAP_WRITE,
+                               dap->dp_ctrl_stat | SSTICKYORUN, NULL, 0);
+               if (retval != ERROR_OK)
+                       goto done;
+
+               /* restore SELECT register first */
+               if (!list_empty(&replay_list)) {
+                       el = list_first_entry(&replay_list, struct dap_cmd, lh);
+                       tmp = dap_cmd_new(JTAG_DP_DPACC,
+                                         DP_SELECT, DPAP_WRITE, (uint8_t *)&el->dp_select, NULL, 0);
+                       if (tmp == NULL) {
+                               retval = ERROR_JTAG_DEVICE_ERROR;
+                               goto done;
+                       }
+                       list_add(&tmp->lh, &replay_list);
 
+                       dap->select = DP_SELECT_INVALID;
+               }
+
+               list_for_each_entry_safe(el, tmp, &replay_list, lh) {
+                       time_now = timeval_ms();
+                       do {
+                               retval = adi_jtag_dp_scan_cmd_sync(dap, el, NULL);
+                               if (retval != ERROR_OK)
+                                       break;
+                               log_dap_cmd("REC", el);
+                               if (el->ack == JTAG_ACK_OK_FAULT) {
+                                       if (el->invalue != el->invalue_buf) {
+                                               uint32_t invalue = le_to_h_u32(el->invalue);
+                                               memcpy(el->invalue, &invalue, sizeof(uint32_t));
+                                       }
+                                       break;
+                               }
+                               if (el->ack != JTAG_ACK_WAIT) {
+                                       LOG_ERROR("Invalid ACK (%1x) in DAP response", el->ack);
+                                       log_dap_cmd("ERR", el);
+                                       retval = ERROR_JTAG_DEVICE_ERROR;
+                                       break;
+                               }
+                       } while (timeval_ms() - time_now < 1000);
+
+                       if (retval == ERROR_OK) {
+                               if (el->ack != JTAG_ACK_OK_FAULT) {
+                                       LOG_ERROR("Timeout during WAIT recovery");
+                                       jtag_ap_q_abort(dap, NULL);
+                                       retval = ERROR_JTAG_DEVICE_ERROR;
+                               }
+                       } else
+                               break;
                }
-               if ((retval = dap_run(swjdp)) != ERROR_OK)
-                       return retval;
-               return ERROR_JTAG_DEVICE_ERROR;
        }
 
-       return ERROR_OK;
+ done:
+       flush_journal(&replay_list);
+       flush_journal(&dap->cmd_journal);
+       return retval;
 }
 
-/*--------------------------------------------------------------------------*/
-
-static int jtag_idcode_q_read(struct adiv5_dap *dap,
-               uint8_t *ack, uint32_t *data)
+static int jtagdp_transaction_endcheck(struct adiv5_dap *dap)
 {
-       struct arm_jtag *jtag_info = dap->jtag_info;
        int retval;
-       struct scan_field fields[1];
+       uint32_t ctrlstat;
 
-       jtag_set_end_state(TAP_IDLE);
+       /* too expensive to call keep_alive() here */
 
-       /* This is a standard JTAG operation -- no DAP tweakage */
-       retval = arm_jtag_set_instr(jtag_info, JTAG_DP_IDCODE, NULL);
+       /* Post CTRL/STAT read; discard any previous posted read value
+        * but collect its ACK status.
+        */
+       retval = adi_jtag_scan_inout_check_u32(dap, JTAG_DP_DPACC,
+                       DP_CTRL_STAT, DPAP_READ, 0, &ctrlstat, 0);
        if (retval != ERROR_OK)
-               return retval;
+               goto done;
 
-       fields[0].num_bits = 32;
-       fields[0].out_value = NULL;
-       fields[0].in_value = (void *) data;
+       /* REVISIT also STICKYCMP, for pushed comparisons (nyet used) */
 
-       jtag_add_dr_scan(jtag_info->tap, 1, fields, TAP_IDLE);
-       retval = jtag_get_error();
-       if (retval != ERROR_OK)
-               return retval;
+       /* Check for STICKYERR */
+       if (ctrlstat & SSTICKYERR) {
+               LOG_DEBUG("jtag-dp: CTRL/STAT 0x%" PRIx32, ctrlstat);
+               /* Check power to debug regions */
+               if ((ctrlstat & (CDBGPWRUPREQ | CDBGPWRUPACK | CSYSPWRUPREQ | CSYSPWRUPACK)) !=
+                                               (CDBGPWRUPREQ | CDBGPWRUPACK | CSYSPWRUPREQ | CSYSPWRUPACK)) {
+                       LOG_ERROR("Debug regions are unpowered, an unexpected reset might have happened");
+                       retval = ERROR_JTAG_DEVICE_ERROR;
+                       goto done;
+               }
 
-       jtag_add_callback(arm_le_to_h_u32,
-                       (jtag_callback_data_t) data);
+               if (ctrlstat & SSTICKYERR)
+                       LOG_ERROR("JTAG-DP STICKY ERROR");
+               if (ctrlstat & SSTICKYORUN)
+                       LOG_DEBUG("JTAG-DP STICKY OVERRUN");
+
+               /* Clear Sticky Error Bits */
+               retval = adi_jtag_scan_inout_check_u32(dap, JTAG_DP_DPACC,
+                               DP_CTRL_STAT, DPAP_WRITE,
+                               dap->dp_ctrl_stat | SSTICKYERR, NULL, 0);
+               if (retval != ERROR_OK)
+                       goto done;
+
+               if (ctrlstat & SSTICKYERR) {
+                       retval = ERROR_JTAG_DEVICE_ERROR;
+                       goto done;
+               }
+       }
 
+ done:
+       flush_journal(&dap->cmd_journal);
        return retval;
 }
 
+/*--------------------------------------------------------------------------*/
+
 static int jtag_dp_q_read(struct adiv5_dap *dap, unsigned reg,
                uint32_t *data)
 {
-       return adi_jtag_scan_inout_check_u32(dap, JTAG_DP_DPACC,
-                       reg, DPAP_READ, 0, data);
+       int retval =  adi_jtag_dp_scan_u32(dap, JTAG_DP_DPACC, reg,
+                       DPAP_READ, 0, dap->last_read, 0, NULL);
+       dap->last_read = data;
+       return retval;
 }
 
 static int jtag_dp_q_write(struct adiv5_dap *dap, unsigned reg,
                uint32_t data)
 {
-       return adi_jtag_scan_inout_check_u32(dap, JTAG_DP_DPACC,
-                       reg, DPAP_WRITE, data, NULL);
+       int retval =  adi_jtag_dp_scan_u32(dap, JTAG_DP_DPACC,
+                       reg, DPAP_WRITE, data, dap->last_read, 0, NULL);
+       dap->last_read = NULL;
+       return retval;
 }
 
 /** Select the AP register bank matching bits 7:4 of reg. */
-static int jtag_ap_q_bankselect(struct adiv5_dap *dap, unsigned reg)
+static int jtag_ap_q_bankselect(struct adiv5_ap *ap, unsigned reg)
 {
-       uint32_t select = reg & 0x000000F0;
+       struct adiv5_dap *dap = ap->dap;
+       uint32_t sel = ((uint32_t)ap->ap_num << 24) | (reg & 0x000000F0);
 
-       if (select == dap->ap_bank_value)
+       if (sel == dap->select)
                return ERROR_OK;
-       dap->ap_bank_value = select;
 
-       select |= dap->apsel;
+       dap->select = sel;
 
-       return jtag_dp_q_write(dap, DP_SELECT, select);
+       return jtag_dp_q_write(dap, DP_SELECT, sel);
 }
 
-static int jtag_ap_q_read(struct adiv5_dap *dap, unsigned reg,
+static int jtag_ap_q_read(struct adiv5_ap *ap, unsigned reg,
                uint32_t *data)
 {
-       int retval = jtag_ap_q_bankselect(dap, reg);
-
+       int retval = jtag_ap_q_bankselect(ap, reg);
        if (retval != ERROR_OK)
                return retval;
 
-       return adi_jtag_scan_inout_check_u32(dap, JTAG_DP_APACC, reg,
-                       DPAP_READ, 0, data);
+       retval =  adi_jtag_dp_scan_u32(ap->dap, JTAG_DP_APACC, reg,
+                       DPAP_READ, 0, ap->dap->last_read, ap->memaccess_tck, NULL);
+       ap->dap->last_read = data;
+
+       return retval;
 }
 
-static int jtag_ap_q_write(struct adiv5_dap *dap, unsigned reg,
+static int jtag_ap_q_write(struct adiv5_ap *ap, unsigned reg,
                uint32_t data)
 {
-       uint8_t out_value_buf[4];
-
-       int retval = jtag_ap_q_bankselect(dap, reg);
+       int retval = jtag_ap_q_bankselect(ap, reg);
        if (retval != ERROR_OK)
                return retval;
 
-       buf_set_u32(out_value_buf, 0, 32, data);
-
-       return adi_jtag_ap_write_check(dap, reg, out_value_buf);
+       retval =  adi_jtag_dp_scan_u32(ap->dap, JTAG_DP_APACC, reg,
+                       DPAP_WRITE, data, ap->dap->last_read, ap->memaccess_tck, NULL);
+       ap->dap->last_read = NULL;
+       return retval;
 }
 
 static int jtag_ap_q_abort(struct adiv5_dap *dap, uint8_t *ack)
 {
        /* for JTAG, this is the only valid ABORT register operation */
-       return adi_jtag_dp_scan_u32(dap, JTAG_DP_ABORT,
-                       0, DPAP_WRITE, 1, NULL, ack);
+       int retval =  adi_jtag_dp_scan_u32(dap, JTAG_DP_ABORT,
+                       0, DPAP_WRITE, 1, NULL, 0, NULL);
+       if (retval != ERROR_OK)
+               return retval;
+
+       return jtag_execute_queue();
 }
 
 static int jtag_dp_run(struct adiv5_dap *dap)
 {
-       return jtagdp_transaction_endcheck(dap);
+       int retval;
+       int retval2 = ERROR_OK;
+
+       retval = adi_jtag_finish_read(dap);
+       if (retval != ERROR_OK)
+               goto done;
+       retval2 = jtagdp_overrun_check(dap);
+       retval = jtagdp_transaction_endcheck(dap);
+
+ done:
+       return (retval2 != ERROR_OK) ? retval2 : retval;
+}
+
+static int jtag_dp_sync(struct adiv5_dap *dap)
+{
+       return jtagdp_overrun_check(dap);
 }
 
 /* FIXME don't export ... just initialize as
  * part of DAP setup
 */
 const struct dap_ops jtag_dp_ops = {
-       .queue_idcode_read =    jtag_idcode_q_read,
-       .queue_dp_read =        jtag_dp_q_read,
-       .queue_dp_write =       jtag_dp_q_write,
-       .queue_ap_read =        jtag_ap_q_read,
-       .queue_ap_write =       jtag_ap_q_write,
-       .queue_ap_abort =       jtag_ap_q_abort,
-       .run =                  jtag_dp_run,
+       .queue_dp_read       = jtag_dp_q_read,
+       .queue_dp_write      = jtag_dp_q_write,
+       .queue_ap_read       = jtag_ap_q_read,
+       .queue_ap_write      = jtag_ap_q_write,
+       .queue_ap_abort      = jtag_ap_q_abort,
+       .run                 = jtag_dp_run,
+       .sync                = jtag_dp_sync,
 };
 
 
-const uint8_t swd2jtag_bitseq[] = {
+static const uint8_t swd2jtag_bitseq[] = {
        /* More than 50 TCK/SWCLK cycles with TMS/SWDIO high,
         * putting both JTAG and SWD logic into reset state.
         */

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)