This is an automated email from Gerrit.

Andrey Smirnov ([email protected]) just uploaded a new patch set to 
Gerrit, which you can find at http://openocd.zylin.com/2087

-- gerrit

commit c8b22777d398198010905001fbf7712578f8f304
Author: Andrey Smirnov <[email protected]>
Date:   Thu Apr 3 14:44:31 2014 -0700

    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 <[email protected]>

diff --git a/src/target/adi_v5_cmsis_dap.c b/src/target/adi_v5_cmsis_dap.c
index 3ff5bfe..7df4cc7 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);
 

-- 

------------------------------------------------------------------------------
_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to