This is an automated email from Gerrit.

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

-- gerrit

commit bb25a5eeb7eece1cdc09ef4aa990ee2c8c864dec
Author: Andreas Fritiofson <[email protected]>
Date:   Sun Dec 6 16:29:36 2015 +0100

    adi_v5_jtag: Remove TAR and CSW prints from jtagdp_transaction_endcheck
    
    The AP for which the TAR/CSW is printed may not be the one that caused
    the failure. Remove the flawed output entirely. The correct info is
    printed in mem_ap_read/write anyway.
    
    Change-Id: I97580a0662dcf02e80646e45445cdbfc251122d8
    Signed-off-by: Andreas Fritiofson <[email protected]>

diff --git a/src/target/adi_v5_jtag.c b/src/target/adi_v5_jtag.c
index 3ab5399..2753713 100644
--- a/src/target/adi_v5_jtag.c
+++ b/src/target/adi_v5_jtag.c
@@ -274,21 +274,6 @@ static int jtagdp_transaction_endcheck(struct adiv5_dap 
*dap)
                        LOG_ERROR("Debug regions are unpowered, an unexpected 
reset might have happened");
                        return ERROR_JTAG_DEVICE_ERROR;
                } 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 (dap->ap[dap_ap_get_select(dap)].tar_value != 
(uint32_t) -1)
-                               LOG_DEBUG("MEM-AP Cached values: "
-                                       "ap_bank 0x%" PRIx32
-                                       ", ap_csw 0x%" PRIx32
-                                       ", ap_tar 0x%" PRIx32,
-                                       dap->ap_bank_value,
-                                       
dap->ap[dap_ap_get_select(dap)].csw_value,
-                                       
dap->ap[dap_ap_get_select(dap)].tar_value);
-
                        if (ctrlstat & SSTICKYORUN)
                                LOG_ERROR("JTAG-DP OVERRUN - check clock, "
                                        "memaccess, or reduce jtag speed");
@@ -312,23 +297,6 @@ static int jtagdp_transaction_endcheck(struct adiv5_dap 
*dap)
                                return retval;
 
                        LOG_DEBUG("jtag-dp: CTRL/STAT 0x%" PRIx32, ctrlstat);
-
-                       retval = dap_queue_ap_read(dap,
-                                       MEM_AP_REG_CSW, &mem_ap_csw);
-                       if (retval != ERROR_OK)
-                               return retval;
-
-                       retval = dap_queue_ap_read(dap,
-                                       MEM_AP_REG_TAR, &mem_ap_tar);
-                       if (retval != ERROR_OK)
-                               return retval;
-
-                       retval = jtag_execute_queue();
-                       if (retval != ERROR_OK)
-                               return retval;
-                       LOG_ERROR("MEM_AP_CSW 0x%" PRIx32 ", MEM_AP_TAR 0x%"
-                                       PRIx32, mem_ap_csw, mem_ap_tar);
-
                }
                retval = jtag_execute_queue();
                if (retval != ERROR_OK)

-- 

------------------------------------------------------------------------------
Go from Idea to Many App Stores Faster with Intel(R) XDK
Give your users amazing mobile app experiences with Intel(R) XDK.
Use one codebase in this all-in-one HTML5 development environment.
Design, debug & build mobile apps & 2D/3D high-impact games for multiple OSs.
http://pubads.g.doubleclick.net/gampad/clk?id=254741911&iu=/4140
_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to