This is an automated email from Gerrit.

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

-- gerrit

commit 29cd14fa4f74a91dbfeb26c4a8afdb8d7ed7a2de
Author: Robert Jarzmik <[email protected]>
Date:   Mon Jun 24 04:52:51 2013 +0200

    target: xscale make reset init work properly
    
    On XScale architecture, to write debug control register DCSR
    and activate JTAG debug (ie. to choose Halt Mode), the
    enabling can only be done while the board is held in reset
    state (ie. PXAxx #RST line held low).
    
    The current implementation writes to the register before
    asserting the SRST line. Swap the order to activate the SRST
    line before writing to DCSR.
    
    Change-Id: I914b9d53d39bdeb5fe4ee5e11068cafafe0da4d2
    Signed-off-by: Robert Jarzmik <[email protected]>

diff --git a/src/target/xscale.c b/src/target/xscale.c
index 92c4ede..ed2afa4 100644
--- a/src/target/xscale.c
+++ b/src/target/xscale.c
@@ -1447,6 +1447,13 @@ static int xscale_assert_reset(struct target *target)
        LOG_DEBUG("target->state: %s",
                target_state_name(target));
 
+       /* assert reset */
+       jtag_add_reset(0, 1);
+
+       /* sleep 1ms, to be sure we fulfill any requirements */
+       jtag_add_sleep(1000);
+       jtag_execute_queue();
+
        /* select DCSR instruction (set endstate to R-T-I to ensure we don't
         * end up in T-L-R, which would reset JTAG
         */
@@ -1463,13 +1470,6 @@ static int xscale_assert_reset(struct target *target)
        xscale_jtag_set_instr(target->tap, ~0, TAP_IDLE);
        jtag_execute_queue();
 
-       /* assert reset */
-       jtag_add_reset(0, 1);
-
-       /* sleep 1ms, to be sure we fulfill any requirements */
-       jtag_add_sleep(1000);
-       jtag_execute_queue();
-
        target->state = TARGET_RESET;
 
        if (target->reset_halt) {

-- 

------------------------------------------------------------------------------
This SF.net email is sponsored by Windows:

Build for Windows Store.

http://p.sf.net/sfu/windows-dev2dev
_______________________________________________
OpenOCD-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openocd-devel

Reply via email to