Michael Fischer pisze:
only a test. Take a look at the LPC2148 here you will find the
following lines:

jtag_nsrst_delay 200
jtag_ntrst_delay 200

Please add this two lines to your lpc2103 cfg too.

With those delays the patched r1889 works. The delays can by very low - I've tried with 10ms and it still works. But here is the funny thing - the _unpatched_ r1889 works as well when I add those delays...

Anyway - I attach a patch that adds those delays to lpc2103.cfg, lpc2124.cfg and lpc2129.cfg (all LPC2000 cfgs that don't have those delays specified).

If those delays are mandatory now (it doesn't work without them) shouldn't those be always enabled to some low value (like 10ms?) so that user wouldn't need to specify that? When used does specify a value, that would override the default hidden values in the code. 0 could also be allowed.

regards
Index: src/target/target/lpc2103.cfg
===================================================================
--- src/target/target/lpc2103.cfg       (revision 1889)
+++ src/target/target/lpc2103.cfg       (working copy)
@@ -21,6 +21,10 @@
 # LPC2000 -> SRST causes TRST
 reset_config trst_and_srst srst_pulls_trst
 
+# reset delays
+jtag_nsrst_delay 100
+jtag_ntrst_delay 100
+
 jtag newtap $_CHIPNAME cpu -irlen 4 -ircapture 0x1 -irmask 0xf -expected-id 
$_CPUTAPID
 
 set _TARGETNAME [format "%s.cpu" $_CHIPNAME]
Index: src/target/target/lpc2124.cfg
===================================================================
--- src/target/target/lpc2124.cfg       (revision 1889)
+++ src/target/target/lpc2124.cfg       (working copy)
@@ -22,7 +22,11 @@
 
 #use combined on interfaces or targets that can't set TRST/SRST separately
 reset_config trst_and_srst srst_pulls_trst
-jtag_nsrst_delay 10
+
+# reset delays
+jtag_nsrst_delay 100
+jtag_ntrst_delay 100
+
 jtag_khz 1000
 
 #jtag scan chain
Index: src/target/target/lpc2129.cfg
===================================================================
--- src/target/target/lpc2129.cfg       (revision 1889)
+++ src/target/target/lpc2129.cfg       (working copy)
@@ -23,6 +23,11 @@
 
 #use combined on interfaces or targets that can't set TRST/SRST separately
 reset_config trst_and_srst srst_pulls_trst
+
+# reset delays
+jtag_nsrst_delay 100
+jtag_ntrst_delay 100
+
 #jtag scan chain
 jtag newtap $_CHIPNAME cpu -irlen 4 -ircapture 0x1 -irmask 0xf -expected-id 
$_CPUTAPID
 
_______________________________________________
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development

Reply via email to