From: Andrey Smirnov Date: Thu, 3 Apr 2014 21:44:31 +0000 (-0700) Subject: adi_v5_cmsis_dap: Fix logging order of appearance X-Git-Tag: v0.9.0-rc1~398 X-Git-Url: https://review.openocd.org/gitweb?p=openocd.git;a=commitdiff_plain;h=8f9cea457d85fd1fb6e060bc1a7fe402d47e2c15 adi_v5_cmsis_dap: Fix logging order of appearance Move logging for cmsis_dap_queue_ap_read/write to happen after a call to cmsis_dap_ap_q_bankselect so that that SWD operation would appear in the log in the same sequence they happen on the bus. Change-Id: Ic046bc753e661da7924b019c9100d6932fb686bf Signed-off-by: Andrey Smirnov Reviewed-on: http://openocd.zylin.com/2087 Reviewed-by: Andreas Fritiofson Tested-by: jenkins --- diff --git a/src/target/adi_v5_cmsis_dap.c b/src/target/adi_v5_cmsis_dap.c index 3ff5bfe03f..7df4cc7836 100644 --- a/src/target/adi_v5_cmsis_dap.c +++ b/src/target/adi_v5_cmsis_dap.c @@ -132,12 +132,12 @@ static int cmsis_dap_ap_q_bankselect(struct adiv5_dap *dap, unsigned reg) static int (cmsis_dap_queue_ap_read)(struct adiv5_dap *dap, unsigned reg, uint32_t *data) { - LOG_DEBUG("CMSIS-ADI: cmsis_dap_queue_ap_read %d", reg); - int retval = cmsis_dap_ap_q_bankselect(dap, reg); if (retval != ERROR_OK) return retval; + LOG_DEBUG("CMSIS-ADI: cmsis_dap_queue_ap_read %d", reg); + retval = jtag_interface->swd->read_reg( (CMSIS_CMD_AP | CMSIS_CMD_READ | CMSIS_CMD_A32(reg)), data); @@ -152,7 +152,7 @@ static int (cmsis_dap_queue_ap_read)(struct adiv5_dap *dap, unsigned reg, uint32 static int (cmsis_dap_queue_ap_write)(struct adiv5_dap *dap, unsigned reg, uint32_t data) { - LOG_DEBUG("CMSIS-ADI: cmsis_dap_queue_ap_write %d 0x%08" PRIx32, reg, data); + /* TODO: CSW_DBGSWENABLE (bit31) causes issues for some targets * disable until we find out why */ @@ -163,6 +163,8 @@ static int (cmsis_dap_queue_ap_write)(struct adiv5_dap *dap, unsigned reg, uint3 if (retval != ERROR_OK) return retval; + LOG_DEBUG("CMSIS-ADI: cmsis_dap_queue_ap_write %d 0x%08" PRIx32, reg, data); + retval = jtag_interface->swd->write_reg( (CMSIS_CMD_AP | CMSIS_CMD_WRITE | CMSIS_CMD_A32(reg)), data);