[OpenOCD-commit] Main OpenOCD repository branch, master, updated. v0.11.0-722-g62058c0a3

[OpenOCD-Gerrit <ope...@us...>]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "Main OpenOCD repository".

The branch, master has been updated
       via  62058c0a32e0e98db39087f8c1e29085e611f2b8 (commit)
      from  8f8fb0fa79742c5e6357e9a2a1609d0500d91293 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 62058c0a32e0e98db39087f8c1e29085e611f2b8
Author: Kevin Burke <ke...@os...>
Date:   Sat Aug 14 18:55:28 2021 +0200

    adi_v5_jtag: extend memaccess_tck to every AP access
    
    ADIv5 reports:
            Accessing AP registers or debug resources in connected
            device through an AP can be subjected to other variable
            response delays in the system. A debugger that can adapt
            to these delays and avoid wasting WAIT scans operates more
            efficiently and provides higher maximum data throughput.
    
    The existing code in OpenOCD uses extra tck only for accessing
    resources through an AP.
    
    Extend the use of extra tck also for accessing an AP register.
    
    Split from change https://review.openocd.org/6077/
    
    Change-Id: I2082362e098d09f4ba0668e01f5196afc965c8f3
    Signed-off-by: Kevin Burke <ke...@os...>
    Signed-off-by: Daniel Goehring <dgo...@os...>
    Signed-off-by: Antonio Borneo <bor...@gm...>
    Reviewed-on: https://review.openocd.org/c/openocd/+/6460
    Tested-by: jenkins

diff --git a/src/target/adi_v5_jtag.c b/src/target/adi_v5_jtag.c
index 810a5ab76..67ad0b1e9 100644
--- a/src/target/adi_v5_jtag.c
+++ b/src/target/adi_v5_jtag.c
@@ -284,17 +284,14 @@ static int adi_jtag_dp_scan_cmd(struct adiv5_dap *dap, struct dap_cmd *cmd, uint
 
 	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.
+	/* Add specified number of tck clocks after starting AP register
+	 * access or 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/ADIv6 spec.
 	 */
-	if (cmd->instr == JTAG_DP_APACC) {
-		if ((cmd->reg_addr == MEM_AP_REG_DRW(dap) ||
-			 (cmd->reg_addr & 0xFF0) == MEM_AP_REG_BD0(dap)) &&
-			cmd->memaccess_tck != 0)
-			jtag_add_runtest(cmd->memaccess_tck, TAP_IDLE);
-	}
+	if (cmd->instr == JTAG_DP_APACC && cmd->memaccess_tck != 0)
+		jtag_add_runtest(cmd->memaccess_tck, TAP_IDLE);
 
 	return ERROR_OK;
 }

-----------------------------------------------------------------------

Summary of changes:
 src/target/adi_v5_jtag.c | 13 +++++--------
 1 file changed, 5 insertions(+), 8 deletions(-)


hooks/post-receive
-- 
Main OpenOCD repository