openocd.git
14 years agoVersion 0.4.0-dev
David Brownell [Thu, 5 Nov 2009 03:44:36 +0000 (19:44 -0800)]
Version 0.4.0-dev

Add "-dev" tag.  Update minor version number.
Archive old NEWS file, start a new one.

14 years agoVersion 0.3.0 v0.3.0
David Brownell [Thu, 5 Nov 2009 03:39:59 +0000 (19:39 -0800)]
Version 0.3.0

Remove -dev tag, remove -rc tag.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoRelease docs: fix notes
David Brownell [Thu, 5 Nov 2009 01:49:06 +0000 (17:49 -0800)]
Release docs: fix notes

We currently do something unusual:  version codes in config.in get
updated after the release, which means that "git describe" won't
match up to development version labels.  Comment that trouble spot.

We can fix this by switching away from the major/minor/micro type
release numbering, as various other projects have done.  The major
numbers basically don't tend to change, and doing a good job with
micro versions is so annoying that they rarely change either.

14 years agoDoc: fix broken link
David Brownell [Thu, 5 Nov 2009 01:12:53 +0000 (17:12 -0800)]
Doc: fix broken link

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoNEWS: mention switch to git!
David Brownell [Thu, 5 Nov 2009 01:03:20 +0000 (17:03 -0800)]
NEWS: mention switch to git!

14 years agoOther files: stop referring to ChangeLog too
David Brownell [Thu, 5 Nov 2009 00:46:27 +0000 (16:46 -0800)]
Other files: stop referring to ChangeLog too

The ChangeLog idiom is redundant given any decent SCM.
Time to phase it out here.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoNEWS refs repository history, not ChangeLog
David Brownell [Wed, 4 Nov 2009 23:54:33 +0000 (15:54 -0800)]
NEWS refs repository history, not ChangeLog

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoTweak release docs
David Brownell [Wed, 4 Nov 2009 23:38:06 +0000 (15:38 -0800)]
Tweak release docs

Contrast releases to git snapshot tarballs.  Mention that
releases have some quality-improvement focus, with special
non-"dev" version IDs.  Explain more about version IDs,
using "openocd -v" to see them, etc;

Make release milestone info be less specific about timing,
and presume we have both a merge window and an RC stage.

Rework the release process information to match reality a
bit more closely.  Reference the version.sh script (in one
place the wrong script was referenced).  Bugfix branches
get special treatment, while non-bugfix releases are more
or less what *defines* being the mainline branch.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoRelease scripts: comments, run on Ubuntu
David Brownell [Wed, 4 Nov 2009 23:20:18 +0000 (15:20 -0800)]
Release scripts: comments, run on Ubuntu

The "source" command isn't accepted by ASH; easy to fix.
Failures with "-e" are harder to fix.  Remove the "-e"
(for now) and force bash, for safety.

Un-obfuscate the release steps, by using names instead
of numbers.  Comment the version-number manipulation.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoconfigure: fix build problems with eCos
Øyvind Harboe [Wed, 4 Nov 2009 12:16:38 +0000 (13:16 +0100)]
configure: fix build problems with eCos

Various include files require some other include files
to be included first. Copied solution from net/if.h.

Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
14 years agodocs: add reference to git bisect docs on BUGS page
Øyvind Harboe [Wed, 4 Nov 2009 18:25:20 +0000 (19:25 +0100)]
docs: add reference to git bisect docs on BUGS page

Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
14 years agotarget: 20 second timeout/megabyte for CRC check
Øyvind Harboe [Tue, 3 Nov 2009 14:38:09 +0000 (15:38 +0100)]
target: 20 second timeout/megabyte for CRC check

There was a fixed 20 second timeout which is too little
for large, slow timeout checks.

Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
14 years agoarm920t: memory writes were broken when MMU was disabled
Øyvind Harboe [Tue, 3 Nov 2009 11:28:00 +0000 (12:28 +0100)]
arm920t: memory writes were broken when MMU was disabled

To support breakpoints, flush data cache line and invalidate
instruction cache when 4 and 2 byte words are written.

The previous code was trying to write directly to the physical
memory, which was buggy and had a number of other situations
that were not handled.

Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
14 years agotarget: require working area for physical/virtual addresses to be specified
Øyvind Harboe [Mon, 2 Nov 2009 10:10:09 +0000 (11:10 +0100)]
target: require working area for physical/virtual addresses to be specified

Fixed bug: if virtual address for working memory was not specified
and MMU was enabled, then address 0 would be used.

Require working address to be specified for both MMU enabled
and disabled case.

For some completely inexplicable reason this fixes the regression
in svn 2646 for flash write in arm926ejs target. The logs showed
that MMU was disabled in the case below:

https://lists.berlios.de/pipermail/openocd-development/2009-November/011882.html

Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
14 years agoFT2232: increase read retry counts
Dimitar Dimitrov [Fri, 30 Oct 2009 18:54:07 +0000 (20:54 +0200)]
FT2232: increase read retry counts

This change is necessary to debug AT91SAM9260 on my PC with a
FT2232H dongle.

Signed-off-by: Dimitar Dimitrov <dinuxbg@gmail.com>
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoUser's Guide: more init info, autoprobing, etc
David Brownell [Mon, 2 Nov 2009 01:54:47 +0000 (17:54 -0800)]
User's Guide: more init info, autoprobing, etc

Mention the autoprobing as a tool that may be useful when
figuring out how to set up; and add a section showing how
to use that mechanism (with an example).

Strengthen the differences between config and run stage
descriptions; add a section for the latter.

Mention Dragonite.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agodoxygen: avoid most internals
David Brownell [Mon, 2 Nov 2009 01:34:52 +0000 (17:34 -0800)]
doxygen: avoid most internals

For some reason, all the interals are documented by default.
This is wrong for two basic reasons:

 - We need to focus on public interfaces, since those are
   the architectural interfaces and relationships.

 - Since virtually nothing has doxygen support yet, this
   maximizes the noise, and minimizes the usefulness of
   doxygen output.

So don't expose so much by default.

14 years agoremove "-ircapture 0x1 -irmask 0x1" from stm32.cfg
Freddie Chopin [Sun, 1 Nov 2009 10:58:48 +0000 (11:58 +0100)]
remove "-ircapture 0x1 -irmask 0x1" from stm32.cfg

Gets rid of the runtime warning "stm32.bs: nonstandard IR mask"

[dbrownell@users.sourceforge.net: line lengths, note issue, section ref]

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoarm9tdmi: more correct fix for vector_catch
David Brownell [Sun, 1 Nov 2009 01:03:54 +0000 (18:03 -0700)]
arm9tdmi: more correct fix for vector_catch

Just use the array of names we're given, ignoring indices.
The "reserved means don't use" patch missed that change.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agotarget.cfg: use $_TARGETNAME for flash
Freddie Chopin [Thu, 29 Oct 2009 18:23:05 +0000 (19:23 +0100)]
target.cfg: use $_TARGETNAME for flash

This gets rid of runtime warnings from the use of numbers.
STM32 and LPC2103 were tested.  Other LPC updates are the
same, and so are safe.  The CFI updates match other tested
changes now in the tree.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoNEWS: more info
David Brownell [Sat, 31 Oct 2009 03:21:31 +0000 (20:21 -0700)]
NEWS: more info

There were a few more changes worth mentioning, including support
for more JTAG adapters, boundary scan improvements, another NAND
driver, and the Win64 stuff.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoARM926: fix arm926ejs_mmu() reading from bad pointer
David Brownell [Sat, 31 Oct 2009 00:29:38 +0000 (17:29 -0700)]
ARM926: fix arm926ejs_mmu() reading from bad pointer

I'm suspecting this code can never have worked, since the
original commit (svn #335) in early 2008.

Fix is just copy/paste from another (working) function.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agobin2char: for win32 set stdin/stdout to binary mode
Spencer Oliver [Fri, 30 Oct 2009 11:59:57 +0000 (11:59 +0000)]
bin2char: for win32 set stdin/stdout to binary mode

Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
14 years agoSVF: fix checking bit pattern against length
Michael Roth [Thu, 29 Oct 2009 13:24:28 +0000 (14:24 +0100)]
SVF: fix checking bit pattern against length

The code works like follow (N = bit_len):

N -1 %4 2<< -1 ~ (binary)
--------------------------------------------------
1 0 0 2 1 1111 1110
2 1 1 4 3 1111 1100
3 2 2 8 7 1111 1000
4 3 3 16 15 1111 0000
5 4 0 2 1 1111 1110
6 5 1 4 3 1111 1100
7 6 2 8 7 1111 1000
8 7 3 16 15 1111 0000
... ... ... ... ... ...

Addresses a bug reported by FangfangLi <ffli@syntest.com.cn>.

[dbrownell@users.sourceforge.net: fix spelling bug too]

Signed-off-by: Michael Roth <mroth@nessie.de>
Cc: FangfangLi <ffli@syntest.com.cn>
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoOlimex FT2232H JTAG adapters
Dimitar Dimitrov [Thu, 29 Oct 2009 22:39:03 +0000 (15:39 -0700)]
Olimex FT2232H JTAG adapters

Add interface configs for two new high speed JTAG
adapters from Olimex.  They need some other speed
related tweaks to work well at high speed.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoXSVF: bugfix handling state paths
David Brownell [Thu, 29 Oct 2009 19:42:41 +0000 (12:42 -0700)]
XSVF: bugfix handling state paths

Implement XSVF support for detailed state path transitions,
by collecting sequences of XSTATE transitions into paths
and then calling pathmove().

It seems that the Xilinx tools want to force state-by-state
transitions instead of relying on the standardized SVF paths.
Like maybe there are XSVF tools not implementing SVF paths,
which are all that we support using svf_statemove().

So from IRPAUSE, instead of just issuing "XSTATE DRPAUSE"
they will issue XSTATES for each intermediate state: first
IREXIT2, then IRUPDATE, DRSELECT, DRCAPTURE, DREXIT1, and
finally DRPAUSE.  This works now.

Handling of paths that go *through* reset is a trifle dodgey,
but it should be safe.

Tested-by: Wookey <wookey@wookware.org>
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoBump rc version and add -dev tag.
Zachary T Welch [Thu, 29 Oct 2009 04:23:17 +0000 (21:23 -0700)]
Bump rc version and add -dev tag.

Bump rc package version number: 0.3.0-rc0 -> 0.3.0-rc1
Add '-dev' version tag: 0.3.0-rc1 -> 0.3.0-rc1-dev

14 years agoThe openocd 0.3.0-rc0 release. v0.3.0-rc0
Zachary T Welch [Thu, 29 Oct 2009 04:23:17 +0000 (21:23 -0700)]
The openocd 0.3.0-rc0 release.

Remove '-dev' version tag: 0.3.0-rc0-dev -> 0.3.0-rc0

14 years agoCortex-M3: remove exports and forward decls
David Brownell [Wed, 28 Oct 2009 17:53:11 +0000 (10:53 -0700)]
Cortex-M3: remove exports and forward decls

Unneeded exports cause confusion about the module interfaces.
Make most functions static, and fix some line-too-long issues.
Delete some now-obviously-unused code.

The forward decls are just code clutter; move their references
later, after the normal declarations.  (Or vice versa.)

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoARM926: remove exports and forward decls
David Brownell [Wed, 28 Oct 2009 17:42:23 +0000 (10:42 -0700)]
ARM926: remove exports and forward decls

Unneeded exports cause confusion about the module interfaces.
Only the Feroceon code builds on this, so only routines it
reuses should be public..  Make most remaining functions
static, and fix some of the line-too-long issues.

The forward decls are just code clutter; move their references
later, after the normal declarations.  Turns out we don't need
even one forward declaration in this file.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agobugfix: stack corruption loading IHex images
Franck HÉRÉSON [Wed, 28 Oct 2009 17:24:55 +0000 (10:24 -0700)]
bugfix: stack corruption loading IHex images

The Hex parser uses a fixed number of sections.  When the
number of sections in the file is greater than that, the
stack get corrupted and a CHECKSUM ERROR is detected
which is very confusing.

This checks the number of sections read, and increases
IMAGE_MAX_SECTIONS so it works on my file.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoAdd script to test the release process.
Zachary T Welch [Thu, 22 Oct 2009 10:04:08 +0000 (03:04 -0700)]
Add script to test the release process.

Runs the release.sh script in a freshly cloned repository, charting
one hypothetical future of OpenOCD's lineage.

14 years agoRewrite release script to use GIT.
Zachary T Welch [Wed, 21 Oct 2009 00:11:33 +0000 (17:11 -0700)]
Rewrite release script to use GIT.

Update documentation to reflect GIT methodology.  Rewrite release.sh
script to use appropriate process.  With this update, tools/release.sh
can be used for producing private release tags on local branches.
The documentation still needs work, but their use for v0.3.x should
help rectify the deficiences.

14 years agoFactor version munging capabilities out of release.sh.
Zachary T Welch [Wed, 21 Oct 2009 00:12:17 +0000 (17:12 -0700)]
Factor version munging capabilities out of release.sh.

14 years agoFactor release version functions into new script.
Zachary T Welch [Thu, 22 Oct 2009 08:38:22 +0000 (01:38 -0700)]
Factor release version functions into new script.

14 years agoAdd git2cl from repo.or.cz as a submodule in tools/git2cl.
Zachary T Welch [Sat, 24 Oct 2009 10:04:28 +0000 (03:04 -0700)]
Add git2cl from repo.or.cz as a submodule in tools/git2cl.

14 years agoImprove .gitignore rules.
Zachary T Welch [Sat, 24 Oct 2009 10:05:41 +0000 (03:05 -0700)]
Improve .gitignore rules.

A '.*' rule prevents the 'git submodule add' from correctly adding the
first submodule, because it creates the .gitmodule file.  This file will
not be added (without -f) result in incomplete submodule commits.
The new rules mask the specific files present in my own build tree, but
additional rules may be needed to hide other types of temporary files.

14 years agoARM: fix single-step of Thumb unconditional branch
Nicolas Pitre [Wed, 28 Oct 2009 01:19:43 +0000 (21:19 -0400)]
ARM: fix single-step of Thumb unconditional branch

Only type 1 branch instruction has a condition code, not type 2.
Currently they're both tagged with ARM_B which doesn't allow for the
distinction.

Signed-off-by: Nicolas Pitre <nico@marvell.com>
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoARM: fix target address when disassembling Thumb BLX
Nicolas Pitre [Wed, 28 Oct 2009 01:19:42 +0000 (21:19 -0400)]
ARM: fix target address when disassembling Thumb BLX

A Thumb BLX instruction is branching to ARM code, and therefore the
first 2 bits of the target address must be cleared.

Signed-off-by: Nicolas Pitre <nico@marvell.com>
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoSignalyzer: H2 and H4 support
Oleg Seiljus [Tue, 27 Oct 2009 19:40:24 +0000 (12:40 -0700)]
Signalyzer: H2 and H4 support

This patch includes partial support for these new JTAG adapters.
More complete support will require updates to the libftdi code,
for EEPROM access.

[dbrownell@users.sourceforge.net: fix whitespace, linelen, etc ]

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoSignalyzer: new config files
Oleg Seiljus [Tue, 27 Oct 2009 18:56:04 +0000 (11:56 -0700)]
Signalyzer: new config files

Add configs for H2, H4, LITE.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoARM: fix Thumb mode handling when single-stepping register based branch insns
Nicolas Pitre [Tue, 27 Oct 2009 05:14:34 +0000 (01:14 -0400)]
ARM: fix Thumb mode handling when single-stepping register based branch insns

Currently, OpenOCD is always caching the PC value without the T bit.
This means that assignment to the PC register must clear that bit and set
the processor state to Thumb when it is set.  And when the PC register
value is transferred to another register or stored into memory then
the T bit must be restored.

Discussion: It is arguable if OpenOCd should have preserved the original
PC value which would have greatly simplified this code.  The processor
state could then be obtained simply by getting at bit 0 of the PC.  This
however would require special handling elsewhere instead since the T bit
is not always relevant (like when PC is used with ALU insns or as an index
with some addressing modes).  It is unclear which way would be simpler in
the end.

Signed-off-by: Nicolas Pitre <nico@marvell.com>
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoARM: allow proper single stepping of Thumb BL and BLX instructions
Nicolas Pitre [Tue, 27 Oct 2009 05:14:33 +0000 (01:14 -0400)]
ARM: allow proper single stepping of Thumb BL and BLX instructions

Whenever an unconditional branch with the H bits set to 0b10 is met, the
offset must be combined with the offset from the following opcode and not
ignored like it is now.

A comment in evaluate_b_bl_blx_thumb() suggests that the Thumb2 decoder
would be a simpler solution.  That might be true when single-stepping of
Thumb2 code is implemented.  But for now this appears to be the simplest
solution to fix Thumb1 support.

Signed-off-by: Nicolas Pitre <nico@marvell.com>
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoARM: call thumb_pass_branch_condition() only for actual branch opcodes
Nicolas Pitre [Tue, 27 Oct 2009 05:14:32 +0000 (01:14 -0400)]
ARM: call thumb_pass_branch_condition() only for actual branch opcodes

Calling it first with every opcodes and then testing if the opcode
was indeed a branch instruction is wasteful and rather strange.
If ever thumb_pass_branch_condition() has side effects (say, like
printing a debugging traces) then the result would be garbage for most
Thumb instructions which have no condition code.

While at it, let's make the nearby code more readable by reducing some of
the redundant brace noise and reworking the error handling construct.

Signed-off-by: Nicolas Pitre <nico@marvell.com>
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoft2232: less noise with _DEBUG_JTAG_IO_
David Brownell [Tue, 27 Oct 2009 06:10:40 +0000 (23:10 -0700)]
ft2232: less noise with _DEBUG_JTAG_IO_

Don't log "Yes, I'm *still* in TAP_IDLE" every seven runtest clocks.

14 years agoJTAG: "jtag newtap ..." cleanup
David Brownell [Tue, 27 Oct 2009 06:06:22 +0000 (23:06 -0700)]
JTAG: "jtag newtap ..." cleanup

Get rid of needless variable, improve and shrink diagnostic.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoPXA255: force reset config
David Brownell [Tue, 27 Oct 2009 05:59:46 +0000 (22:59 -0700)]
PXA255: force reset config

These chips need both SRST and TRST when debugging,
and SRST doesn't gate JTAG.

14 years agoomap3530: target reset/init improvements
David Brownell [Tue, 27 Oct 2009 05:53:18 +0000 (22:53 -0700)]
omap3530: target reset/init improvements

Now I can issue "reset halt" and have everything act smoothly;
the vector_catch hardware is obviously not kicking in, but the
rest of the reset sequence acts sanely.

 - TAP "setup" event enables the DAP, not omap3_dbginit
   (resolving a chicken/egg bug I noted a while back)
 - Remove stuff from omap3_dbginit which should never be
   used in event handlers
 - Cope better with slow clocking during reset

Also, stop hard-wiring the target name: use the input params in
the standard way, and set up $_TARGETNAME as an output param.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoARM ADIv5: "dap info" gets more readable
David Brownell [Mon, 26 Oct 2009 23:02:45 +0000 (16:02 -0700)]
ARM ADIv5: "dap info" gets more readable

Make the "dap info" output more comprehensible:

 - Don't show CIDs unless they're incorrect (only four bits matter)
 - For CoreSight parts, interpret the part type
 - Interpret the part number
 - Show all five PID bytes together
 - Other minor cleanups

Also some whitespace fixes, and shrink a few overlong source lines.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoFix incorrect line endings
Spencer Oliver [Mon, 26 Oct 2009 22:39:24 +0000 (22:39 +0000)]
Fix incorrect line endings

Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
14 years agoMerge branch 'master' of ssh://dbrownell@openocd.git.sourceforge.net/gitroot/openocd...
David Brownell [Mon, 26 Oct 2009 18:14:28 +0000 (11:14 -0700)]
Merge branch 'master' of ssh://dbrownell@openocd.git.sourceforge.net/gitroot/openocd/openocd

14 years agoballoon3 board base config
Wookey [Mon, 26 Oct 2009 17:06:05 +0000 (17:06 +0000)]
balloon3 board base config

This is the very basic board config for the balloon3 board cpu JTAG
channel.

The rest of the config comprises another 14 .cfg files which I suspect
openocd doesn't really want all of. I'm still not sure how to deal
with this. I'll post another mail/patch to discuss.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoSVF: fix parsing hex strings containing leading '0' characters
Michael Roth [Mon, 26 Oct 2009 13:01:42 +0000 (14:01 +0100)]
SVF: fix parsing hex strings containing leading '0' characters

Ignore leading '0' characters on hex strings.  For example a bit
pattern consisting of 6 bits could be written as 3f, 03f or 003f and
so on.

Signed-off-by: Michael Roth <mroth@nessie.de>
Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoIdea for adding watchpoint masks.
Øyvind Harboe [Fri, 23 Oct 2009 14:13:10 +0000 (16:13 +0200)]
Idea for adding watchpoint masks.

14 years agoJTAG: simple autoprobing
David Brownell [Mon, 26 Oct 2009 07:36:03 +0000 (00:36 -0700)]
JTAG: simple autoprobing

This patch adds basic autoprobing support for the JTAG scan chains
which cooperate.  To use, you can invoke OpenOCD with just:

 - interface spec: "-f interface/...cfg"
 - possibly with "-c 'reset_config ...'" for SRST/TRST
 - possibly with "-c 'jtag_khz ...'" for the JTAG clock

Then set up config files matching the reported TAPs.  It doesn't
declare targets ... just TAPs.  So facilities above the JTAG and
SVF/XSVF levels won't be available without a real config; this is
almost purely a way to generate diagnostics.

Autoprobe was successful with most boards I tested, except ones
incorporating C55x DSPs (which don't cooperate with this scheme
for IR length autodetection).  Here's what one multi-TAP chip
reported, with the "Warn:" prefixes removed:

 clock speed 500 kHz
 There are no enabled taps.  AUTO PROBING MIGHT NOT WORK!!
 AUTO auto0.tap - use "jtag newtap auto0 tap -expected-id 0x2b900f0f ..."
 AUTO auto1.tap - use "jtag newtap auto1 tap -expected-id 0x07926001 ..."
 AUTO auto2.tap - use "jtag newtap auto2 tap -expected-id 0x0b73b02f ..."
 AUTO auto0.tap - use "... -irlen 4"
 AUTO auto1.tap - use "... -irlen 4"
 AUTO auto2.tap - use "... -irlen 6"
 no gdb ports allocated as no target has been specified

The patch tweaks IR setup a bit, so we can represent TAPs with
undeclared IR length.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agominor fixes to TODO list
David Brownell [Sun, 25 Oct 2009 23:30:30 +0000 (16:30 -0700)]
minor fixes to TODO list

14 years agofix syntax of mww phys.
Øyvind Harboe [Sun, 25 Oct 2009 17:29:17 +0000 (18:29 +0100)]
fix syntax of mww phys.

14 years agocheck if mmu is enabled before using mmu code path
Øyvind Harboe [Sun, 25 Oct 2009 17:24:18 +0000 (18:24 +0100)]
check if mmu is enabled before using mmu code path

14 years agoARM: rename "arm9tdmi vector_catch" to "arm9 ..."
David Brownell [Sun, 25 Oct 2009 21:03:14 +0000 (14:03 -0700)]
ARM: rename "arm9tdmi vector_catch" to "arm9 ..."

And update doc accordingly.  That EmbeddedICE register was
introduced for ARM9TDMI and then carried forward into most
new chips that use EmbeddedICE.

14 years agoJTAG: jtag_tap_init() bugfixes
David Brownell [Sun, 25 Oct 2009 20:07:57 +0000 (13:07 -0700)]
JTAG: jtag_tap_init() bugfixes

Stop allocating three bytes per IR bit, and cope somewhat better
with IR lengths over 32 bits.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoxscale: always reload handler after reset
David Brownell [Sun, 25 Oct 2009 20:06:47 +0000 (13:06 -0700)]
xscale: always reload handler after reset

Remove needless debug handler state.

 - "handler_installed" became wrong as soon as the second TRST+SRST
   reset was issued ... so the handler was never reloaded after the
   reset removed it from the mini-icache.

   This fixes the bug where subsequent resets fail on PXA255 (if the
   first one even worked, which is uncommon).  Other XScale chips
   would have problems too; PXA270 seems to have, IXP425 maybe not.

 - "handler_running" was never tested; it's pointless.

Plus a related bugfix: invalidate OpenOCD's ARM register cache on reset.
It was no more valid than the XScale's mini-icache.  (Though ... such
invalidations might be better done in "SRST asserted" callbacks.)

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agovector_catch and watchpoint TODO items.
Øyvind Harboe [Sat, 24 Oct 2009 11:06:13 +0000 (13:06 +0200)]
vector_catch and watchpoint TODO items.

14 years agoarm9tdmi vector_catch: reserved means "don't use"
David Brownell [Fri, 23 Oct 2009 19:28:03 +0000 (12:28 -0700)]
arm9tdmi vector_catch:  reserved means "don't use"

Bit 5 shouldn't be used.  Remove all support for modifying it.
Matches the exception vector table, of course ... more than one
bootloader uses that non-vector to help distinguish valid boot
images from random garbage in flash.

14 years agoImprove help for arm9 vector_catch.
Øyvind Harboe [Fri, 23 Oct 2009 14:02:42 +0000 (16:02 +0200)]
Improve help for arm9 vector_catch.

14 years agoRemove debug output that could cause compile warnings.
Øyvind Harboe [Fri, 23 Oct 2009 11:09:16 +0000 (13:09 +0200)]
Remove debug output that could cause compile warnings.

14 years agomcr/mrc interface work. Implemented for arm926ejs and arm720t. mcr/mrc commands added.
Øyvind Harboe [Fri, 23 Oct 2009 10:38:19 +0000 (12:38 +0200)]
mcr/mrc interface work. Implemented for arm926ejs and arm720t. mcr/mrc commands added.

14 years agoEmbedded ICE version is now dumped with debug_level 1
Øyvind Harboe [Fri, 23 Oct 2009 07:54:43 +0000 (09:54 +0200)]
Embedded ICE version is now dumped with debug_level 1

14 years agojtag: clean up TAP state name handling
David Brownell [Fri, 23 Oct 2009 08:02:22 +0000 (01:02 -0700)]
jtag: clean up TAP state name handling

Some cosmetic cleanup, and switch to a single table mapping
between state names and symbols (vs two routines which only
share that state with difficulty).

Get rid of TAP_NUM_STATES, and some related knowledge about
how TAP numbers are assigned.  Later on, this will help us
get rid of more such hardwired knowlege.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoSVF: clean up, mostly for TAP state name handling
David Brownell [Fri, 23 Oct 2009 08:00:32 +0000 (01:00 -0700)]
SVF: clean up, mostly for TAP state name handling

 - Use the name mappings all the other code uses:
    + name-to-state ... needed to add one special case
    + state-to-name
 - Improve various diagnostics:
    + don't complain about a "valid" state when the issue
      is actually that it must be "stable"
    + say which command was affected
 - Misc:
    + make more private data and code be static
    + use public DIM() not private dimof()
    + shorten the affected lines

Re the mappings, this means we're more generous in inputs we
accept, since case won't matter.  Also our output diagnostics
will be a smidgeon more informative, saying "RUN/IDLE" not
just "IDLE" (emphasizing that there can be side effects).

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoFerocion: fix corruption of r0 when resuming Thumb mode
Nicolas Pitre [Fri, 23 Oct 2009 03:23:44 +0000 (23:23 -0400)]
Ferocion: fix corruption of r0 when resuming Thumb mode

The wrong variable (pc instead of r0) was used.  Furthermore, someone
did cover this error by stupidly silencing the compiler warning that
occurred before a dummy void reference to r0 was added to the code.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agohave "reg" command print cache names too
David Brownell [Thu, 22 Oct 2009 19:05:04 +0000 (12:05 -0700)]
have "reg" command print cache names too

When dumping over 100 registers (as on most ARM9 + ETM cores),
aid readability by splitting them into logical groups.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoETM: rename registers, doc tweaks
David Brownell [Thu, 22 Oct 2009 19:01:27 +0000 (12:01 -0700)]
ETM: rename registers, doc tweaks

The register names are perversely not documented as zero-indexed,
so rename them to match that convention.  Also switch to lowercase
suffixes and infix numbering, matching ETB and EmbeddedICE usage.

Update docs to be a bit more accurate, especially regarding what
the "trigger" event can cause; and to split the issues into a few
more paragraphs, for clarity.

Make "configure" helptext point out that "oocd_trace" is prototype
hardware, not anything "real".

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agodisable ZY1000's UART forwarding test code.
Øyvind Harboe [Thu, 22 Oct 2009 08:22:34 +0000 (10:22 +0200)]
disable ZY1000's UART forwarding test code.

14 years agomww_phys retired. Replaced by generic mww phys in target.c
Øyvind Harboe [Wed, 21 Oct 2009 18:19:47 +0000 (20:19 +0200)]
mww_phys retired. Replaced by generic mww phys in target.c

14 years agovirt2phys is now implemented by target.c globally, retire target specific documentation.
Øyvind Harboe [Wed, 21 Oct 2009 18:16:31 +0000 (20:16 +0200)]
virt2phys is now implemented by target.c globally, retire target specific documentation.

14 years agoretire obsolete mXY_phys commands. Handled by generic memory read/modify commands...
Øyvind Harboe [Wed, 21 Oct 2009 13:36:31 +0000 (15:36 +0200)]
retire obsolete mXY_phys commands. Handled by generic memory read/modify commands and target read/write physical memory callbacks.

14 years agoread/write physical target fn's
Øyvind Harboe [Wed, 21 Oct 2009 13:32:29 +0000 (15:32 +0200)]
read/write physical target fn's

14 years agoadd support for target_read/write_phys_memory callbacks.
Øyvind Harboe [Wed, 21 Oct 2009 12:46:12 +0000 (14:46 +0200)]
add support for target_read/write_phys_memory callbacks.

14 years agoAdded target_read/write_phys_memory() fn's. mdX/mwX commands updated to support phys...
Øyvind Harboe [Wed, 21 Oct 2009 12:45:39 +0000 (14:45 +0200)]
Added target_read/write_phys_memory() fn's. mdX/mwX commands updated to support phys flag to specify bypassing of MMU.

14 years agoMerge branch 'master' of ssh://gowinex@openocd.git.sourceforge.net/gitroot/openocd...
Øyvind Harboe [Wed, 21 Oct 2009 11:20:29 +0000 (13:20 +0200)]
Merge branch 'master' of ssh://gowinex@openocd.git.sourceforge.net/gitroot/openocd/openocd into HEAD

14 years agoRetire obsolete and superfluous implementations of virt2phys in each target. This...
Øyvind Harboe [Wed, 21 Oct 2009 11:10:32 +0000 (13:10 +0200)]
Retire obsolete and superfluous implementations of virt2phys in each target. This is done in a polymorphic implementation in target.c

14 years agoRetire obsolete and superfluous implementations of virt2phys in each target. This...
Øyvind Harboe [Wed, 21 Oct 2009 11:10:32 +0000 (13:10 +0200)]
Retire obsolete and superfluous implementations of virt2phys in each target. This is done in a polymorphic implementation in target.c

14 years agoFirst cut at implementing software breakpoints for mmu read only memory
Øyvind Harboe [Wed, 21 Oct 2009 11:07:44 +0000 (13:07 +0200)]
First cut at implementing software breakpoints for mmu read only memory

14 years agoDefined target_write_memory() to be able to handle implementing breakpoints for read...
Øyvind Harboe [Wed, 21 Oct 2009 10:54:41 +0000 (12:54 +0200)]
Defined target_write_memory() to be able to handle implementing breakpoints for read only ram(e.g. MMU write protected.

14 years agoeCos synthetic target updates.
Øyvind Harboe [Wed, 21 Oct 2009 10:02:04 +0000 (12:02 +0200)]
eCos synthetic target updates.

14 years agoXSVF: use svf_add_statemove()
David Brownell [Wed, 21 Oct 2009 03:04:36 +0000 (20:04 -0700)]
XSVF: use svf_add_statemove()

XSVF improvements:

 - Layer parts of XSVF directly over SVF, calling svf_add_statemove()
   instead of expecting jtag_add_statemove() to conform to the SVF/XSVF
   requirements (which it doesn't).

   This should improve XSTATE handling a lot; it removes most users of
   jtag_add_statemove(), and the comments about how it should really do
   what svf_add_statemove() does.

 - Update XSTATE logic to be a closer match to the XSVF spec.  The main
   open issue here is (still) that this implementation doesn't know how
   to build and submit paths from single-state transitions ... but now
   it will report that error case.

 - Update the User's Guide to mention the two utility scripts for
   working with XSVF, and to mention the five extension opcodes.

Handling of state transition paths is, overall, still a mess.  I think
they should all be specified as paths not unlike SVF uses, and compiled
to the bitstrings later ... so that we can actually make sense of the
paths.  (And see the extra clocks, detours through RUN, etc.)

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoRemoved unused interface_jtag_set_end_state and wrote down some notes on TCP/IP clien...
Øyvind Harboe [Tue, 20 Oct 2009 13:54:53 +0000 (15:54 +0200)]
Removed unused interface_jtag_set_end_state and wrote down some notes on TCP/IP client/server scheme.

14 years agoAdded the faux flash driver and target. Used for testing.
Øyvind Harboe [Tue, 20 Oct 2009 10:22:55 +0000 (12:22 +0200)]
Added the faux flash driver and target. Used for testing.

14 years agoAdded 'unlock' option to flash write_image
Øyvind Harboe [Tue, 20 Oct 2009 10:03:36 +0000 (12:03 +0200)]
Added 'unlock' option to flash write_image

14 years agoMore svn to git version string fixes.
Øyvind Harboe [Tue, 20 Oct 2009 07:46:23 +0000 (09:46 +0200)]
More svn to git version string fixes.

14 years agoSVF: better spec conformance for STATE switch
David Brownell [Tue, 20 Oct 2009 06:33:08 +0000 (23:33 -0700)]
SVF: better spec conformance for STATE switch

Don't add extra TCK in current state; exit from RESET had four extras.
Only IDLE --> IDLE needs such an extra clock.  (At least one TCK must
be issued.)

Allow entry to RESET; SVF allows it, so must we (despite those entries
being commented out of the statemove table).

When entering RESET, always use TLR ... we might end up with extra clocks
in reset that way, which is harmless, but we'll never end up in any other
state than RESET, which is useful paranoia.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agodavinci: add watchdog reset method
David Brownell [Tue, 20 Oct 2009 05:50:51 +0000 (22:50 -0700)]
davinci: add watchdog reset method

Lightly tested on dm365.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agojtag_add_statemove() always uses TLR to get to RESET
David Brownell [Tue, 20 Oct 2009 05:50:51 +0000 (22:50 -0700)]
jtag_add_statemove() always uses TLR to get to RESET

As decided a while back, this isn't a transition we want to chance.
Whenever someone wants to got to RESET, force it.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoSVF/XSVF: comment and whitespace fixes
David Brownell [Tue, 20 Oct 2009 05:50:51 +0000 (22:50 -0700)]
SVF/XSVF: comment and whitespace fixes

SVF: comment the predefined/default paths; make them static const

SVF, XSVF: whitespace fixes, mostly so copyrights display sanely

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
14 years agoImprove Makefile rules for XScale debug handler; fixes 'make distcheck'.
Zachary T Welch [Tue, 20 Oct 2009 02:59:02 +0000 (19:59 -0700)]
Improve Makefile rules for XScale debug handler; fixes 'make distcheck'.

14 years agoMinGW: always use "-D__USE_MINGW_ANSI_STDIO"
Redirect 'Slash' NIL [Tue, 20 Oct 2009 00:55:40 +0000 (17:55 -0700)]
MinGW: always use "-D__USE_MINGW_ANSI_STDIO"

This is unfortunately needed to make stdio work like OpenOCD expects -- matching
the ANSI-C standard, instead of MS-Windows.

I tested it in both MinGW-W64 on Vista 64 and MinGW-W32 on XP, and I don't
see any adverse effects to enabling it for all MinGW versions.

14 years agoMinGW: use WinSock2
Redirect 'Slash' NIL [Tue, 20 Oct 2009 00:48:19 +0000 (17:48 -0700)]
MinGW: use WinSock2

After reading a bit further, it appears that ws2_32 (Windows Sockets 2)
is included in all versions of Windows and backwards compatible with
wsock32, at least according to

 http://msdn.microsoft.com/en-us/library/ms740673%28VS.85%29.aspx.

Only Win95 seems to require a manual installation; is not a big deal.

So I think we can drop this whole business of detecting 64 bit MinGW and
just use -lws2_32 for all MinGW platforms.

14 years agoDoc: jtag_init must validate scan chain too
David Brownell [Mon, 19 Oct 2009 21:45:43 +0000 (14:45 -0700)]
Doc: jtag_init must validate scan chain too

Same requirement as like init_reset, and for the same reason:
we need to start with a known and working state.

14 years agofix single step of bx instruction going into Thumb mode
Nicolas Pitre [Mon, 19 Oct 2009 19:49:46 +0000 (15:49 -0400)]
fix single step of bx instruction going into Thumb mode

Without this fix, the following code cannot be single stepped:

add ip, pc, #1
bx ip
[thumb code here]

14 years agoIgnore openocd.exe for "git status"
David Brownell [Mon, 19 Oct 2009 16:53:45 +0000 (09:53 -0700)]
Ignore openocd.exe for "git status"

Linking to existing account procedure

If you already have an account and want to add another login method you MUST first sign in with your existing account and then change URL to read https://review.openocd.org/login/?link to get to this page again but this time it'll work for linking. Thank you.

SSH host keys fingerprints

1024 SHA256:YKx8b7u5ZWdcbp7/4AeXNaqElP49m6QrwfXaqQGJAOk gerrit-code-review@openocd.zylin.com (DSA)
384 SHA256:jHIbSQa4REvwCFG4cq5LBlBLxmxSqelQPem/EXIrxjk gerrit-code-review@openocd.org (ECDSA)
521 SHA256:UAOPYkU9Fjtcao0Ul/Rrlnj/OsQvt+pgdYSZ4jOYdgs gerrit-code-review@openocd.org (ECDSA)
256 SHA256:A13M5QlnozFOvTllybRZH6vm7iSt0XLxbA48yfc2yfY gerrit-code-review@openocd.org (ECDSA)
256 SHA256:spYMBqEYoAOtK7yZBrcwE8ZpYt6b68Cfh9yEVetvbXg gerrit-code-review@openocd.org (ED25519)
+--[ED25519 256]--+
|=..              |
|+o..   .         |
|*.o   . .        |
|+B . . .         |
|Bo. = o S        |
|Oo.+ + =         |
|oB=.* = . o      |
| =+=.+   + E     |
|. .=o   . o      |
+----[SHA256]-----+
2048 SHA256:0Onrb7/PHjpo6iVZ7xQX2riKN83FJ3KGU0TvI0TaFG4 gerrit-code-review@openocd.zylin.com (RSA)