openocd.git
10 years agojtag: remove useless declarations
Zachary T Welch [Tue, 10 Nov 2009 05:21:06 +0000 (21:21 -0800)]
jtag: remove useless declarations

Contrary to my previous assessment, some opportunities to remove forward
declarations were overlooked.  Remove them by moving the definitions
of the command registration and interface structure to the end of files.

10 years agotarget.c: remove useless declarations
Zachary T Welch [Tue, 10 Nov 2009 03:24:06 +0000 (19:24 -0800)]
target.c: remove useless declarations

This patch removes the last batch of forward references from the tree,
moving the target command registration routines to the end of the file.

10 years ago{pic32m,stm32}x.c: remove useless declarations
Zachary T Welch [Mon, 9 Nov 2009 17:20:34 +0000 (09:20 -0800)]
{pic32m,stm32}x.c: remove useless declarations

Remove useless forward declarations.
Moves command registrations to end of files.
Moves flash structure definitions to end of files.

Signed-off-by: Zachary T Welch <zw@superlucidity.net>
10 years agostr{7,9}x*.c: remove useless forward declarations
Zachary T Welch [Mon, 9 Nov 2009 17:20:34 +0000 (09:20 -0800)]
str{7,9}x*.c: remove useless forward declarations

Remove useless forward declarations.
Moves command registrations to end of files.
Moves flash structure definitions to end of files.

Signed-off-by: Zachary T Welch <zw@superlucidity.net>
10 years agoflash/<various>.c: remove useless declarations
Zachary T Welch [Mon, 9 Nov 2009 17:20:33 +0000 (09:20 -0800)]
flash/<various>.c: remove useless declarations

Remove useless forward declarations.
Moves flash structure definitions to end of files.

Signed-off-by: Zachary T Welch <zw@superlucidity.net>
10 years agoflash/<assorted>.c: remove useless declarations
Zachary T Welch [Mon, 9 Nov 2009 17:20:33 +0000 (09:20 -0800)]
flash/<assorted>.c: remove useless declarations

Remove useless forward declarations.
Moves command registrations to end of files.
Moves flash structure definitions to end of files.

Signed-off-by: Zachary T Welch <zw@superlucidity.net>
10 years agoflash/at91sam[37].c: remove useless declarations
Zachary T Welch [Mon, 9 Nov 2009 17:20:33 +0000 (09:20 -0800)]
flash/at91sam[37].c: remove useless declarations

Remove useless forward declarations.
Moves command registration to end of file.
Moves flash structure definitions to end of files.

Changes a few references to global flash structure to local refs.

Signed-off-by: Zachary T Welch <zw@superlucidity.net>
10 years agoflash/lpc2???.c: remove useless declarations
Zachary T Welch [Mon, 9 Nov 2009 17:20:33 +0000 (09:20 -0800)]
flash/lpc2???.c: remove useless declarations

Remove useless forward declarations.
Moves command registrations to end of files.
Moves flash structure definitions to end of files.

Signed-off-by: Zachary T Welch <zw@superlucidity.net>
10 years agoflash/*nand*.c: remove useless declarations
Zachary T Welch [Mon, 9 Nov 2009 17:20:33 +0000 (09:20 -0800)]
flash/*nand*.c: remove useless declarations

Remove useless forward declarations.
Moves command registration to end of files.
Moves flash structure definition to end of files.

Signed-off-by: Zachary T Welch <zw@superlucidity.net>
10 years agoflash/flash.c: remove forward declarations
Zachary T Welch [Mon, 9 Nov 2009 17:20:33 +0000 (09:20 -0800)]
flash/flash.c: remove forward declarations

Remove useless forward declarations.
Moves command registration to end of file.
Moves flash structure definition to end of file.

Signed-off-by: Zachary T Welch <zw@superlucidity.net>
10 years agoRevert "target: add target->type->has_mmu fn"
David Brownell [Mon, 9 Nov 2009 22:46:23 +0000 (14:46 -0800)]
Revert "target: add target->type->has_mmu fn"

This patch introduced a bug preventing flash writes from working
on Cortex-M3 targets like the STM32.  Moreover, it's the wrong
approach for handling no-MMU targets.

The right way to handle no-MMU targets is to provide accessors
for physical addresses, and use them everywhere; and any code
which tries to work with virtual-to-physical mappings should use
a identity mapping (which can be defaulted).

And ... we can tell if a target has an MMU by seeing if it's
got an mmu() method.  No such methood means no MMU.

Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
10 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>
10 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>
10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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>
10 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>
10 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.

10 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.

10 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.

10 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>
10 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>
10 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>
10 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.

10 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

10 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>
10 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.

10 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.

10 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.

10 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>
10 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>
10 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>
10 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>
10 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>
10 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>
10 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>
10 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>
10 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>
10 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>
10 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>
10 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>
10 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>
10 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>
10 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>
10 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>
10 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>
10 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>
10 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>
10 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>
10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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.

10 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>
10 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>
10 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>