Dne Ne 29. listopadu 2009 06:10:57 David Brownell napsal(a):
> On Saturday 28 November 2009, Marek Vasut wrote:
> > > > +static int xscale_version = XSCALE_IXP4XX_PXA2XX;
> > >
> > > Please put this in the instance-specific struct ... this approach
> > > obviously breaks on a board with both a PXA2xx and a PXA3xx.
> >
> > Have you ever seen such a mutant ? Come on ...
>
> Bad design is bad design. This is trivial to fix, and preventing
> such bugs from appearing in the first place is better than trying
> to troubleshoot them in the field later on.
>
> > btw. where should I look for that instance-specific struct ?
>
> Did you look at "xscale.h"? That's where I'd start. There
> turns out to be a "struct xscale_common", go figure. ;)
>
Btw. attached here is a config file for the pxa3xx, maybe also apply.
From 418e0a9ce6bfd5e388d4dc115866ac9b5111e81a Mon Sep 17 00:00:00 2001
From: Marek Vasut <marek.va...@gmail.com>
Date: Sun, 29 Nov 2009 12:54:41 +0100
Subject: [PATCH 2/3] Add initial PXA3xx configuration file
---
tcl/target/pxa3xx.cfg | 84 +++++++++++++++++++++++++++++++++++++++++++++++++
1 files changed, 84 insertions(+), 0 deletions(-)
create mode 100644 tcl/target/pxa3xx.cfg
diff --git a/tcl/target/pxa3xx.cfg b/tcl/target/pxa3xx.cfg
new file mode 100644
index 0000000..cb74c44
--- /dev/null
+++ b/tcl/target/pxa3xx.cfg
@@ -0,0 +1,84 @@
+#Marvell PXA3xx Script
+
+if { [info exists CHIPNAME] } {
+ set _CHIPNAME $CHIPNAME
+} else {
+ set _CHIPNAME pxa3xx
+}
+
+if { [info exists ENDIAN] } {
+ set _ENDIAN $ENDIAN
+} else {
+ set _ENDIAN little
+}
+
+# IDs for all of the PXA3xx
+if { [info exists CPUTAPID_PXA30X_A0 ] } {
+ set _CPUTAPID_PXA30X_A0 $CPUTAPID_PXA30X_A0
+} else {
+ set _CPUTAPID_PXA30X_A0 0x0E648013
+}
+if { [info exists CPUTAPID_PXA30X_A1 ] } {
+ set _CPUTAPID_PXA30X_A1 $CPUTAPID_PXA30X_A1
+} else {
+ set _CPUTAPID_PXA30X_A1 0x1E648013
+}
+if { [info exists CPUTAPID_PXA31X_A0 ] } {
+ set _CPUTAPID_PXA31X_A0 $CPUTAPID_PXA31X_A0
+} else {
+ set _CPUTAPID_PXA31X_A0 0x0E649013
+}
+if { [info exists CPUTAPID_PXA31X_A1 ] } {
+ set _CPUTAPID_PXA31X_A1 $CPUTAPID_PXA31X_A1
+} else {
+ set _CPUTAPID_PXA31X_A1 0x1E649013
+}
+if { [info exists CPUTAPID_PXA31X_A2 ] } {
+ set _CPUTAPID_PXA31X_A2 $CPUTAPID_PXA31X_A2
+} else {
+ set _CPUTAPID_PXA31X_A2 0x2E649013
+}
+if { [info exists CPUTAPID_PXA31X_B0 ] } {
+ set _CPUTAPID_PXA31X_B0 $CPUTAPID_PXA31X_B0
+} else {
+ set _CPUTAPID_PXA31X_B0 0x3E649013
+}
+if { [info exists CPUTAPID_PXA32X_B1 ] } {
+ set _CPUTAPID_PXA32X_B1 $CPUTAPID_PXA32X_B1
+} else {
+ set _CPUTAPID_PXA32X_B1 0x5E642013
+}
+if { [info exists CPUTAPID_PXA32X_B2 ] } {
+ set _CPUTAPID_PXA32X_B2 $CPUTAPID_PXA32X_B2
+} else {
+ set _CPUTAPID_PXA32X_B2 0x6E642013
+}
+if { [info exists CPUTAPID_PXA32X_C0 ] } {
+ set _CPUTAPID_PXA32X_C0 $CPUTAPID_PXA32X_C0
+} else {
+ set _CPUTAPID_PXA32X_C0 0x7E642013
+}
+
+# set jtag_nsrst_delay to the delay introduced by your reset circuit
+# the rest of the needed delays are built into the openocd program
+jtag_nsrst_delay 260
+# set the jtag_ntrst_delay to the delay introduced by a reset circuit
+# the rest of the needed delays are built into the openocd program
+jtag_ntrst_delay 250
+
+set _TARGETNAME $_CHIPNAME.cpu
+jtag newtap $_CHIPNAME cpu -irlen 11 -ircapture 0x1 -irmask 0x7f \
+ -expected-id $_CPUTAPID_PXA30X_A0 \
+ -expected-id $_CPUTAPID_PXA30X_A1 \
+ -expected-id $_CPUTAPID_PXA31X_A0 \
+ -expected-id $_CPUTAPID_PXA31X_A1 \
+ -expected-id $_CPUTAPID_PXA31X_A2 \
+ -expected-id $_CPUTAPID_PXA31X_B0 \
+ -expected-id $_CPUTAPID_PXA32X_B1 \
+ -expected-id $_CPUTAPID_PXA32X_B2 \
+ -expected-id $_CPUTAPID_PXA32X_C0
+
+target create $_TARGETNAME xscale -endian $_ENDIAN -chain-position $_TARGETNAME -variant pxa3xx
+# maps to PXA internal RAM. If you are using a PXA255
+# you must initialize SDRAM or leave this option off
+$_TARGETNAME configure -work-area-phys 0x5c030000 -work-area-size 0x10000 -work-area-backup 0
--
1.6.5
_______________________________________________
Openocd-development mailing list
Openocd-development@lists.berlios.de
https://lists.berlios.de/mailman/listinfo/openocd-development
