openocd.git
12 years agofinish removing deprecated/obsolete commands
David Brownell [Mon, 9 Nov 2009 21:16:32 +0000 (13:16 -0800)]
finish removing deprecated/obsolete commands

It's been about a year since these were deprecated and, in most
cases, removed.  There's no point in carrying that documentation,
or backwards compatibility for "jtag_device" and "jtag_speed",
around forever.  (Or a few remnants of obsolete code...)

Removed a few obsolete uses of "jtag_speed":

 - The Calao stuff hasn't worked since July 2008.  (Those Atmel
   targets need to work with a 32KHz core clock after reset until
   board-specific init-reset code sets up the PLL and enables a
   faster JTAg clock.)
 - Parport speed controls don't actually work (tops out at about
   1 MHz on typical HW).
 - In general, speed controls need to live in board.cfg files (or
   sometimes target.cfg files), not interface.cfg ...

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoUser's Guide: Flash/NAND doc tweaks
David Brownell [Mon, 9 Nov 2009 20:02:23 +0000 (12:02 -0800)]
User's Guide: Flash/NAND doc tweaks

Rename the "Drivers, Options, and Commands" sections to be
just "Driver List" matching the earlier reference.  Add an
example of parallel CFI flash.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agosrc/target: remove 'extern' and wrap headers
Zachary T Welch [Mon, 9 Nov 2009 12:22:23 +0000 (04:22 -0800)]
src/target: remove 'extern' and wrap headers

Remove extern keywords from function prototypes and wrap long lines.

12 years agotarget.h: remove extern keyword and wrap
Zachary T Welch [Mon, 9 Nov 2009 12:21:36 +0000 (04:21 -0800)]
target.h: remove extern keyword and wrap

Removes 'extern' keyword from function prototypes and wraps long lines.

12 years agosrc/{server,pld,svf,xsvf}: remove 'extern' keyword
Zachary T Welch [Mon, 9 Nov 2009 11:38:34 +0000 (03:38 -0800)]
src/{server,pld,svf,xsvf}: remove 'extern' keyword

Removes 'extern' keyword from function declarations in header filess.

12 years agosrc/flash: remove 'extern' and wrap headers
Zachary T Welch [Mon, 9 Nov 2009 11:10:11 +0000 (03:10 -0800)]
src/flash: remove 'extern' and wrap headers

Removes 'extern' keywords from function prototypes in the flash headers.
Wraps long lines to fit into 80 columns.

Adds multiple inclusion protection for s3c2xx_nand.h.

12 years agosrc/jtag: remove 'extern' and wrap headers.
Zachary T Welch [Mon, 9 Nov 2009 10:53:22 +0000 (02:53 -0800)]
src/jtag: remove 'extern' and wrap headers.

Removes the 'extern' keyword from function declarations.
Wraps long prototypes to fit into 80 columns.

Fixes documentation for jtag_tap_s::{,has}idcode fields.

12 years agosrc/helper: wrap and clean headers.
Zachary T Welch [Mon, 9 Nov 2009 10:47:00 +0000 (02:47 -0800)]
src/helper: wrap and clean headers.

Remove all useless 'extern' keywords from function prototypes.
Wraps long lines for readability.

12 years agoUser's Guide: bugfix global state info
David Brownell [Mon, 9 Nov 2009 17:40:55 +0000 (09:40 -0800)]
User's Guide: bugfix global state info

The "$ocd_HOSTOS" variable was wrongly documented.  Fix its
documentation, and its value on Linux.

Shrink a few of the too-long lines.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoEmbeddedICE: minor cleanups
David Brownell [Mon, 9 Nov 2009 17:34:51 +0000 (09:34 -0800)]
EmbeddedICE: minor cleanups

Add comments (Doxygen and normal), remove unused code,
shrink some overlong lines.  Get rid of a forward decl.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoOverhaul time support API
Zachary T Welch [Sun, 8 Nov 2009 07:20:33 +0000 (23:20 -0800)]
Overhaul time support API

This patch changes the duration_* API in several ways.  First, it
updates the API to use better names.  Second, string formatting has
been removed from the API (with its associated malloc).  Finally, a
new function added to convert the time into seconds, which can be
used (or formatted) by the caller.  This eliminates hidden calls to
malloc that require associated calls to free().

This patch also removes the useless extern keyword from prototypes,
and it eliminates the duration_t typedef (use 'struct duration').
These API also allows proper error checking, as it is possible for
gettimeofday to fail in certain circumstances.

The consumers have all been chased to use this new API as well, as
there were relatively few cases doing this type of measurement.
In most cases, the code performs additional checks for errors, but
the calling code looks much cleaner in every case.

12 years agoAdd private header for ARM11 internals.
Zachary T Welch [Fri, 6 Nov 2009 11:50:26 +0000 (03:50 -0800)]
Add private header for ARM11 internals.

Reduces confusion about location of associated routines and
reduces clutter in the arm11 header.

Removes extra whitespace around the lines touched by these changes.

12 years agoARM11: remove exports and forward decls
Zachary T Welch [Fri, 6 Nov 2009 11:36:46 +0000 (03:36 -0800)]
ARM11: remove exports and forward decls

Unneeded exports cause confusion about the module interfaces.
Make almost everything static in the arm11.c module.

12 years agoARM: minor simulator cleanup
David Brownell [Sun, 8 Nov 2009 20:44:28 +0000 (12:44 -0800)]
ARM: minor simulator cleanup

Make several functions be static.  Shrink some of the overlong
lines.  Use pure tab indents in some places that mixed in spaces.
This gives a minor object code shrink (about 2% on amd64).

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoCortex-M3: fix (debug) message priority
David Brownell [Sun, 8 Nov 2009 20:23:17 +0000 (12:23 -0800)]
Cortex-M3: fix (debug) message priority

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agotarget.cfg: remove "-work-area-virt 0"
David Brownell [Sun, 8 Nov 2009 16:52:40 +0000 (08:52 -0800)]
target.cfg: remove "-work-area-virt 0"

The semantics of "-work-area-virt 0" (or phys) changed with
the patch to require specifying physical or virtrual work
area addresses.  Specifying zero was previously a NOP.  Now
it means that address zero is valid.

This patch addresses three related issues:

 - MMU-less processors should never specify work-area-virt;
   remove those specifications.  Such processors include
   ARM7TDMI, Cortex-M3, and ARM966.

 - MMU-equipped processors *can* specify work-area-virt...
   but zero won't be appropriate, except in mischievous
   contexts (which hide null pointer exceptions).

   Remove those specs from those processors too.  If any of
   those mappings is valid, someone will need to submit a
   patch adding it ... along with a comment saying what OS
   provides the mapping, and in which context.  Example,
   say "works with Linux 2.6.30+, in kernel mode".  (Note
   that ARM Linux doesn't map kernel memory to zero ...)

 - Clarify docs on that "-virt" and other work area stuff.

Seems to me work-area-virt is quite problematic; not every
operating system provides such static mappings; if they do,
they're not in every MMU context...

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoFix vsllink bulk out endpoint parsing.
Zachary T Welch [Sat, 7 Nov 2009 21:36:47 +0000 (13:36 -0800)]
Fix vsllink bulk out endpoint parsing.

12 years agoMerge branch 'master' of ssh://dbrownell@openocd.git.sourceforge.net/gitroot/openocd...
David Brownell [Fri, 6 Nov 2009 23:01:46 +0000 (15:01 -0800)]
Merge branch 'master' of ssh://dbrownell@openocd.git.sourceforge.net/gitroot/openocd/openocd

12 years agotarget: don't swap MMU/no-MMU work areas
David Brownell [Fri, 6 Nov 2009 22:57:21 +0000 (14:57 -0800)]
target: don't swap MMU/no-MMU work areas

Resolve serious bug inserted by the "target: require working
area for physical/virtual addresses to be specified" patch.
It forced use of (invalid) virtual addresses when the MMU
was disabled, and vice versa.

Observed to break at least Cortex-M3, ARM926, ARM7TDMI whenever
work areas are used, such as during bulk writes to flash, DDR2,
SRAM, and so on.

Also, fix overlong lines and whitespace goofs.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoSimplify nand indentation.
Zachary T Welch [Fri, 6 Nov 2009 02:40:52 +0000 (18:40 -0800)]
Simplify nand indentation.

Removes check covered by new nand_command_get_device_by_num helper.
Reverses logic of probe check to further reduce indentation.

12 years agoImprove flash indentation.
Zachary T Welch [Fri, 6 Nov 2009 07:52:03 +0000 (23:52 -0800)]
Improve flash indentation.

Removes redundant tests and reverses backwards logic to reduce the
indentation level in flash.c.

12 years agoFix arm11 vcr command parsing.
Zachary T Welch [Fri, 6 Nov 2009 08:16:39 +0000 (00:16 -0800)]
Fix arm11 vcr command parsing.

12 years agoARM: shrink offsets
David Brownell [Fri, 6 Nov 2009 06:04:25 +0000 (22:04 -0800)]
ARM: shrink offsets

Move various embedded target structs to the beginnings of
their containers ... pretty much the way C++ or Obj-C
would for single inheritance.

This shrinks code that accesses those embedded structs by
letting common offsets use smaller instructions.  Sample
before/after sizes (on amd64):

  17181     312       0   17493    4455 arm920t.o
  16810     312       0   17122    42e2 arm920t.o

Where the "after" is the smaller number, with this patch
over the ones leveraging that embedding knowledge.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoARM: other code uses the new inheritance/nesting scheme
David Brownell [Fri, 6 Nov 2009 06:04:22 +0000 (22:04 -0800)]
ARM: other code uses the new inheritance/nesting scheme

Remove most remaining uses of target->arch_info from ARM
infrastructure, where it hasn't already been updated.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoCortex-A8: use the new inheritance/nesting scheme
David Brownell [Fri, 6 Nov 2009 06:04:13 +0000 (22:04 -0800)]
Cortex-A8: use the new inheritance/nesting scheme

Use target_to_armv7a() etc, replacing needless pointer traversals.
Stop using X->arch_info scheme in most ARMv7-A and Cortex-A8 code.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoARM7TDMI uses the new inheritance/nesting scheme
David Brownell [Fri, 6 Nov 2009 06:04:04 +0000 (22:04 -0800)]
ARM7TDMI uses the new inheritance/nesting scheme

Use target_to_arm7_9(), replacing needless pointer traversals.

Also:  remove now-useless contents of arm7tdmi struct; it's
almost ready to be removed.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoARM720 uses the new inheritance/nesting scheme
David Brownell [Fri, 6 Nov 2009 06:03:56 +0000 (22:03 -0800)]
ARM720 uses the new inheritance/nesting scheme

Use target_to_arm720(), replacing needless pointer traversals
and simplifying a bunch of nasty code.  Stop setting arch_info
for arm720 type parts, it's not used any longer.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoXScale uses the new inheritance/nesting scheme
David Brownell [Fri, 6 Nov 2009 06:03:45 +0000 (22:03 -0800)]
XScale uses the new inheritance/nesting scheme

Use target_to_xscale(), replacing needless pointer traversals
and simplifying a bunch of code.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoARM9TDMI uses the new inheritance/nesting scheme
David Brownell [Fri, 6 Nov 2009 06:03:40 +0000 (22:03 -0800)]
ARM9TDMI uses the new inheritance/nesting scheme

Replace needless pointer traversals and simplify.  Also remove most
remaining contents from arm9tdmi struct; it's almost removable.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoARM966 uses the new inheritance/nesting scheme
David Brownell [Fri, 6 Nov 2009 06:03:33 +0000 (22:03 -0800)]
ARM966 uses the new inheritance/nesting scheme

Use target_to_arm966(), replacing needless pointer traversals.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoARM926 uses the new inheritance/nesting scheme
David Brownell [Fri, 6 Nov 2009 06:03:30 +0000 (22:03 -0800)]
ARM926 uses the new inheritance/nesting scheme

Use target_to_arm926(), replacing needless pointer traversals
and simplifying a bunch of code.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoFA526 uses the new inheritance/nesting scheme
David Brownell [Fri, 6 Nov 2009 06:03:24 +0000 (22:03 -0800)]
FA526 uses the new inheritance/nesting scheme

Replace needless pointer traversals, simplify.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoARM920 uses the new inheritance/nesting scheme
David Brownell [Fri, 6 Nov 2009 06:03:13 +0000 (22:03 -0800)]
ARM920 uses the new inheritance/nesting scheme

Use target_to_arm920(), replacing needless pointer traversals
and simplifying.  Stop setting arm9tdmi->arch_info for arm920
type parts, it's not used any longer.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoCortex-M3: use the new inheritance/nesting scheme
David Brownell [Fri, 6 Nov 2009 05:59:45 +0000 (21:59 -0800)]
Cortex-M3: use the new inheritance/nesting scheme

Use new target_to_cm3() and target_to_armv7m() inlines,
instead of a series of x->arch_info conversions.  Remove
arch_info, since nothing uses it.

Also fix an omission:  the Cortex-M3 commands didn't verify
that they were operating on that kind of target.  Add comment
about the ARMv7M version of that omission.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agotarget: provide container_of()
David Brownell [Fri, 6 Nov 2009 05:59:39 +0000 (21:59 -0800)]
target: provide container_of()

Provide a cleaner way to handle single inheritance of targets
in C, using the same model Linux does:  structs containing other
structs, un-nested via calls to a "container_of()" macro that
are packaged in typesafe inline functions.

Targets already use this containment idiom, but make it much
more complicated because they un-nest using embedded "void *"
pointers ... in chains of up to five per target, which is all
pure needless complication.  (Example: arm92x core, arm9tdmi,
arm7_9, armv4_5 ... on top of the base "target" class.)

Applying this scheme consistently simplifies things, and gets
rid of many error-prone untyped pointers.  It won't change any
part of the type model though -- it just simplifies things.
(And facilitates more cleanup later on.)

Rule of thumb:  where there's an X->arch_info void* pointer,
access to that pointer can and should be removed.  It may be
convenient to set up pointers to some of the embedded structs;
and shrink their current "*_common" names (annoyingly long).

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoARM7TDMI: remove forward decls
David Brownell [Fri, 6 Nov 2009 04:36:42 +0000 (20:36 -0800)]
ARM7TDMI: remove forward decls

The forward decls are just code clutter; remove them, by moving
their references after definitions.  This is another file which
never needed even one internal forward declaration.

Also shrink a few overly-long lines with function declarations
or definitions;  get rid of arm7tdmi_register_commands(), it's
not needed (just delegated); minor whitespace declutter.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoARM720: remove exports and forward decls
David Brownell [Fri, 6 Nov 2009 04:36:27 +0000 (20:36 -0800)]
ARM720: remove exports and forward decls

Unneeded exports cause confusion about the module interfaces.
Make all functions static.  Add a short header comment.

The forward decls are just code clutter; remove them, by moving
their references after definitions.  This is another file which
never needed even one internal forward declaration.

Remove unneeded indirection for the write_memory() method.  Make
a table static, remove a can't-happen case with nasty exit().

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoFA526: remove exports and forward decls
David Brownell [Fri, 6 Nov 2009 04:36:18 +0000 (20:36 -0800)]
FA526: remove exports and forward decls

Unneeded exports cause confusion about the module interfaces.
Make most functions static.

The forward decls are just code clutter; remove them, by moving
their references after definitions.  This is another file which
never needed even one internal forward declaration.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoARM9TDMI: remove forward decls
David Brownell [Fri, 6 Nov 2009 04:36:09 +0000 (20:36 -0800)]
ARM9TDMI: remove forward decls

The forward decls are just code clutter; remove them, by moving
their references after definitions.  This is another file which
never needed even one internal forward declaration.

Also shrink a few overly-long lines with function declarations
or definitions.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoARM966: remove exports and forward decls
David Brownell [Fri, 6 Nov 2009 04:35:53 +0000 (20:35 -0800)]
ARM966: remove exports and forward decls

Unneeded exports cause confusion about the module interfaces.
Make most functions static.

The forward decls are just code clutter; remove them, by moving
their references after definitions.  This is another file which
never needed even one internal forward declaration.

Also remove needless arm966e_init_target(), in favor of the
arm9tdmi routine to which it delegates its work.

This saved over 100 bytes of code on x86_32.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoARM926: more cleanup
David Brownell [Fri, 6 Nov 2009 04:35:47 +0000 (20:35 -0800)]
ARM926: more cleanup

An init_target() wrapper isn't needed, and target_create()
can shrink a bit.  Add a header comment and some doxygen.

Remove arm926ejs_catch_broken_irscan() which has been a NOP
for quite a few months now, and in any case duplicates logic
in the JTAG core to validate IR capture data.  But force the
capture mask to 0x0f, so those tests are most effective.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoARM920: remove exports and forward decls
David Brownell [Fri, 6 Nov 2009 04:35:37 +0000 (20:35 -0800)]
ARM920: remove exports and forward decls

Unneeded exports cause confusion about the module interfaces.
Make most functions static.  Add a short header comment.

The forward decls are just code clutter; remove them, by moving
their references after definitions.  This is another file which
never needed even one internal forward declaration.

This saved almost 900 bytes of code on x86_32; it seems the
compiler can leverage its knowledge that these functions are
not called from the outside world...

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoImprove str9xpec command argument parsing.
Zachary T Welch [Fri, 23 Oct 2009 08:18:25 +0000 (01:18 -0700)]
Improve str9xpec command argument parsing.

12 years agoImprove str9x config command argument parsing.
Zachary T Welch [Fri, 23 Oct 2009 08:25:22 +0000 (01:25 -0700)]
Improve str9x config command argument parsing.

12 years agoImprove str7x config command argument parsing.
Zachary T Welch [Fri, 23 Oct 2009 08:32:20 +0000 (01:32 -0700)]
Improve str7x config command argument parsing.

12 years agoImprove stm32x.c command argument parsing.
Zachary T Welch [Fri, 23 Oct 2009 05:33:12 +0000 (22:33 -0700)]
Improve stm32x.c command argument parsing.

12 years agoImprove stellaris.c erase argument parsing.
Zachary T Welch [Fri, 23 Oct 2009 08:40:47 +0000 (01:40 -0700)]
Improve stellaris.c erase argument parsing.

12 years agoImprove pic32mx.c command argument parsing.
Zachary T Welch [Fri, 23 Oct 2009 05:33:12 +0000 (22:33 -0700)]
Improve pic32mx.c command argument parsing.

12 years agoImprove mflash.c command argument parsing.
Zachary T Welch [Fri, 23 Oct 2009 09:17:17 +0000 (02:17 -0700)]
Improve mflash.c command argument parsing.

12 years agoImprove lpc3180_nand_controller.c parsing.
Zachary T Welch [Fri, 23 Oct 2009 09:17:17 +0000 (02:17 -0700)]
Improve lpc3180_nand_controller.c parsing.

This fixes a memory leak in lpc3180_nand_device_command by
reordering the malloc to occur after all parsing has completed.

12 years agoImprove lpc2900.c command argument parsing.
Zachary T Welch [Fri, 23 Oct 2009 09:17:17 +0000 (02:17 -0700)]
Improve lpc2900.c command argument parsing.

12 years agoImprove lpc288x.c command argument parsing.
Zachary T Welch [Fri, 23 Oct 2009 09:17:17 +0000 (02:17 -0700)]
Improve lpc288x.c command argument parsing.

12 years agoImprove lpc2000.c command argument parsing.
Zachary T Welch [Fri, 23 Oct 2009 09:17:17 +0000 (02:17 -0700)]
Improve lpc2000.c command argument parsing.

12 years agoImprove cfi.c command argument parsing.
Zachary T Welch [Fri, 23 Oct 2009 09:17:17 +0000 (02:17 -0700)]
Improve cfi.c command argument parsing.

12 years agoImprove avrf.c command argument parsing.
Zachary T Welch [Fri, 23 Oct 2009 09:17:17 +0000 (02:17 -0700)]
Improve avrf.c command argument parsing.

12 years agoImprove orion_nand.c command argument parsing.
Zachary T Welch [Fri, 23 Oct 2009 05:33:12 +0000 (22:33 -0700)]
Improve orion_nand.c command argument parsing.

12 years agoImprove davinci_nand.c command argument parsing.
Zachary T Welch [Fri, 23 Oct 2009 05:33:12 +0000 (22:33 -0700)]
Improve davinci_nand.c command argument parsing.

12 years agoImprove at91sam7.c command argument parsing.
Zachary T Welch [Fri, 23 Oct 2009 05:33:12 +0000 (22:33 -0700)]
Improve at91sam7.c command argument parsing.

12 years agoImprove at91sam3.c command argument parsing.
Zachary T Welch [Fri, 23 Oct 2009 05:33:12 +0000 (22:33 -0700)]
Improve at91sam3.c command argument parsing.

12 years agoImprove nand.c command argument parsing.
Zachary T Welch [Fri, 23 Oct 2009 05:33:12 +0000 (22:33 -0700)]
Improve nand.c command argument parsing.

12 years agoImprove flash.c command argument parsing.
Zachary T Welch [Fri, 23 Oct 2009 05:33:12 +0000 (22:33 -0700)]
Improve flash.c command argument parsing.

12 years agoAdd Flash/NAND bank command argument helpers.
Zachary T Welch [Fri, 23 Oct 2009 08:13:19 +0000 (01:13 -0700)]
Add Flash/NAND bank command argument helpers.

This eliminates redundant code for parsing and retreiving the bank
specified from a script command argument.  This patch was written to
replace existing functionality; however, the parsing logic can be
updated later to allow flash commands to accept bank names as well as
their numbers.

12 years agoImprove ETM tracemode update command.
Zachary T Welch [Thu, 22 Oct 2009 15:11:55 +0000 (08:11 -0700)]
Improve ETM tracemode update command.

12 years agoImprove etm command argument parsing.
Zachary T Welch [Thu, 22 Oct 2009 15:15:37 +0000 (08:15 -0700)]
Improve etm command argument parsing.

12 years agoImprove trace command argument parsing.
Zachary T Welch [Sat, 24 Oct 2009 13:36:06 +0000 (06:36 -0700)]
Improve trace command argument parsing.

12 years agoImprove arm_adi_v5 command argument parsing.
Zachary T Welch [Sat, 24 Oct 2009 13:36:05 +0000 (06:36 -0700)]
Improve arm_adi_v5 command argument parsing.

12 years agoImprove cortex_m3 command argument parsing.
Zachary T Welch [Sat, 24 Oct 2009 13:36:06 +0000 (06:36 -0700)]
Improve cortex_m3 command argument parsing.

12 years agoImprove xscale command argument parsing.
Zachary T Welch [Sat, 24 Oct 2009 13:36:06 +0000 (06:36 -0700)]
Improve xscale command argument parsing.

12 years agoImprove arm11 command argument parsing.
Zachary T Welch [Sat, 24 Oct 2009 13:36:05 +0000 (06:36 -0700)]
Improve arm11 command argument parsing.

12 years agoImprove arm966e command argument parsing.
Zachary T Welch [Sat, 24 Oct 2009 13:36:05 +0000 (06:36 -0700)]
Improve arm966e command argument parsing.

12 years agoImprove arm926ejs command argument parsing.
Zachary T Welch [Sat, 24 Oct 2009 13:36:05 +0000 (06:36 -0700)]
Improve arm926ejs command argument parsing.

12 years agoImprove arm920t command argument parsing.
Zachary T Welch [Sat, 24 Oct 2009 13:36:05 +0000 (06:36 -0700)]
Improve arm920t command argument parsing.

12 years agoImprove arm720t command argument parsing.
Zachary T Welch [Sat, 24 Oct 2009 13:36:05 +0000 (06:36 -0700)]
Improve arm720t command argument parsing.

12 years agoImprove arm7_9_common command argument parsing.
Zachary T Welch [Thu, 22 Oct 2009 14:03:20 +0000 (07:03 -0700)]
Improve arm7_9_common command argument parsing.

12 years agoImprove armv7m command argument parsing.
Zachary T Welch [Sat, 24 Oct 2009 13:36:06 +0000 (06:36 -0700)]
Improve armv7m command argument parsing.

12 years agoImprove armv7a command argument parsing.
Zachary T Welch [Sat, 24 Oct 2009 13:36:06 +0000 (06:36 -0700)]
Improve armv7a command argument parsing.

12 years agoImprove armv4_5 command argument parsing.
Zachary T Welch [Sat, 24 Oct 2009 13:36:05 +0000 (06:36 -0700)]
Improve armv4_5 command argument parsing.

12 years agoImprove xscale command argument parsing.
Zachary T Welch [Sat, 24 Oct 2009 12:55:23 +0000 (05:55 -0700)]
Improve xscale command argument parsing.

12 years agoImprove pld command argument parsing.
Zachary T Welch [Sat, 24 Oct 2009 08:22:58 +0000 (01:22 -0700)]
Improve pld command argument parsing.

12 years agoUpdate all server port command to use new helper.
Zachary T Welch [Sat, 24 Oct 2009 08:12:56 +0000 (01:12 -0700)]
Update all server port command to use new helper.

12 years agoAdd server port command helper function.
Zachary T Welch [Sat, 24 Oct 2009 08:11:13 +0000 (01:11 -0700)]
Add server port command helper function.

12 years agoImprove target.c command argument parsing.
Zachary T Welch [Fri, 23 Oct 2009 04:04:23 +0000 (21:04 -0700)]
Improve target.c command argument parsing.

Passes cmd_ctx into parse_load_image_command_args for reporting the
parsing errors therein.

12 years agoImprove jtag command argument parsing.
Zachary T Welch [Fri, 23 Oct 2009 04:05:50 +0000 (21:05 -0700)]
Improve jtag command argument parsing.

12 years agoImprove debug_level command argument parsing.
Zachary T Welch [Fri, 23 Oct 2009 05:34:19 +0000 (22:34 -0700)]
Improve debug_level command argument parsing.

12 years agoAdd macro for parsing numeric command arguments.
Zachary T Welch [Thu, 22 Oct 2009 14:44:54 +0000 (07:44 -0700)]
Add macro for parsing numeric command arguments.

This helper eliminates significant amount of redundant code in command
handler functions throughout the system.  It wraps the lower-level
parse_* macros to implement a policy for reporting parse errors to the
active command context (cmd_ctx).  If errors do occur, this macro causes
the calling function to abort with the proper return code.

12 years agoAdd stringify macros in src/helper/types.h.
Zachary T Welch [Thu, 22 Oct 2009 14:39:23 +0000 (07:39 -0700)]
Add stringify macros in src/helper/types.h.

12 years agoarm920t: add mrcmcr interface fn's.
Øyvind Harboe [Tue, 3 Nov 2009 11:54:26 +0000 (12:54 +0100)]
arm920t: add mrcmcr interface fn's.

The arm920t has a concept of read modify write cycles
that may have to be represented in the mrcmcr interface
eventually.

Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
12 years agoarm11: check if target is halted before executing mrc/mcr commands.
Øyvind Harboe [Tue, 27 Oct 2009 12:43:42 +0000 (13:43 +0100)]
arm11: check if target is halted before executing mrc/mcr commands.

Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
12 years agotarget: fix ordering of arguments to mcr and mrc commands
Øyvind Harboe [Mon, 26 Oct 2009 17:57:52 +0000 (18:57 +0100)]
target: fix ordering of arguments to mcr and mrc commands

Now matches machine code syntax and old arm11 syntax.

Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
12 years agotarget: check args to mrc/mcr.
Øyvind Harboe [Mon, 26 Oct 2009 17:53:19 +0000 (18:53 +0100)]
target: check args to mrc/mcr.

Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
12 years agoARM11: added mrc/mcr support to arm11 code.
Øyvind Harboe [Mon, 26 Oct 2009 13:39:32 +0000 (14:39 +0100)]
ARM11: added mrc/mcr support to arm11 code.

Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
12 years agoTODO: Wrote up list of remaining tasks for target->type->mrc/mcr
Øyvind Harboe [Sat, 24 Oct 2009 12:50:57 +0000 (14:50 +0200)]
TODO: Wrote up list of remaining tasks for target->type->mrc/mcr

Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
12 years agotarget: Only register mrc mcr commands when one of the targets support them.
Øyvind Harboe [Sat, 24 Oct 2009 11:24:35 +0000 (13:24 +0200)]
target: Only register mrc mcr commands when one of the targets support them.

This avoids polluting help for targets that can never support mrc/mcr

Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
12 years agocortex_a8: add mrc mcr interface.
Øyvind Harboe [Sat, 24 Oct 2009 11:17:04 +0000 (13:17 +0200)]
cortex_a8: add mrc mcr interface.

Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
12 years agoUser's Guide: TAP setup tweakage
David Brownell [Thu, 5 Nov 2009 19:31:32 +0000 (11:31 -0800)]
User's Guide: TAP setup tweakage

Highlight that the "-expected-id" probably comes from vendor
documentation, and that it *should* be used where possible.

Don't use ircapture/irmask in examples, to help discourage
use of those params when they're not required.  Explain a
bit better about why/when those params get used.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agowatchpoint_add() cleanup
David Brownell [Thu, 5 Nov 2009 09:47:44 +0000 (01:47 -0800)]
watchpoint_add() cleanup

Fail watchpoint_add() if it's the same address but the
parameters are different ... don't just assume having
the same address means the same watchpoint!  (Note that
overlapping watchpoints aren't detected...)

Handle unrecognized return codes more sanely; don't exit()!
And describe command params right.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoCortex-M3: expose most DWT registers
David Brownell [Thu, 5 Nov 2009 09:04:08 +0000 (01:04 -0800)]
Cortex-M3: expose most DWT registers

Expose most DWT registers via Tcl; there are a few more, but
those are mostly for profiling along with the ITM.  Having
this set available enables operations which aren't possible
with just the standard watchpoint operations.

The cycle counter may be interesting.  Turn it on after reset
by setting the LSB of the dwt_ctrl register, and it counts
CPU clocks.  You can program the comparator 0 watchpoint to
trigger on a given cycle count, rather than a data address.

Likewise, comparator 1 may be able to match data values given
address matches from one or two other comparators.  (Not all
hardware supports this capability though; try it.  That is
something the standard watchpoint methods should eventually
handle, for the single address case.)

Minor cleanup:  remove needless functional indirection for
exposing the v7m architctural registers.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoCortex-M3: minor cleanup
David Brownell [Thu, 5 Nov 2009 09:03:54 +0000 (01:03 -0800)]
Cortex-M3: minor cleanup

There's no reason to read which interrupts are enabled from
the NVIC; that state isn't used.  Plus, it's highly dynamic
since firmware can change it at any time; remove the support
for those state records.

Remove duplicate definition of DWT_CTRL address; shrink a line.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoCortex-M3: DWT cleanup/fixes
David Brownell [Thu, 5 Nov 2009 09:03:17 +0000 (01:03 -0800)]
Cortex-M3: DWT cleanup/fixes

Fix the watchpoint error checks, and do them in add(), not later
in set() when it's mostly too late.  Support the full range of
watchpoint sizes (1 to 32K bytes each), and check alignments.

Minor cleanup of DWT access:  shrink lines, use "+" for address
calculations, comment a few issues.  Add debug message reporting
DWT capabilities, matching the message for FBP, and some minor
code and spec review comments.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
12 years agoARMv7M: add docs, remove exports
David Brownell [Thu, 5 Nov 2009 09:02:52 +0000 (01:02 -0800)]
ARMv7M: add docs, remove exports

Add Doxygen for the exported ARMv7-M interfaces.

Make the non-exported stuff static.  Remove functions and
data which are now observably unused.

Add comment about a small speedup that the run_algorithm()
logic could use.  Shrink a few too-long lines.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>

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)