summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* transport selection tweaksDavid Brownell2010-07-095-15/+39
| | | | | | | | | | | | * Bugfix and simplify legacy jtag-only defaulting * Make "dummy" declare its jtag-only nature * likewise update ft2232 * warn if selection is _required_ (multi-transport adapters), fixes the "only ft2232 works" bug for at least dummy, with other drivers going the "legacy" path (submit patches). Signed-off-by: David Brownell <db@helium.(none)>
* transport: fix bug/typo in transport_register cmdSpencer Oliver2010-07-093-5/+3
| | | | Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* transport: fix segfault in transport selectØyvind Harboe2010-07-071-7/+6
| | | | | | | String compare against addresses in range 0 or so due to not checking if there was an active session first. Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* transport: fix bug/typo in interface_transports commandØyvind Harboe2010-07-071-1/+2
| | | | | | | Classic manifestation of weakness of dangling { formatting :-) Hard to spot these sort of mistakes visually. Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* gdbserver: incorrect memory map for multiple targets (bug #24)sb-sf2010-07-051-2/+4
| | | | | | | The gdb server incorrectly reports the memory map if we have multiple targets with multiple flash banks. Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* transport: fix segfault in setup_command_handler()Stacey Sheldon2010-07-051-0/+1
| | | | | | | | Commit 93f2afa45f4c dropped the sentinel off the end of the command_registrants[] array. The loop immediately following the initialization will walk right off the end. Signed-off-by: Stacey Sheldon <stac@solidgoldbomb.org>
* Fix minor openocd.texi bugDavid Brownell2010-07-021-0/+1
| | | | | ::X Signed-off-by: David Brownell <db@helium.(none)>
* initial "transport" frameworkDavid Brownell2010-07-0210-30/+633
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds the guts of a transport framework with initialization, which should work with current JTAG-only configurations (tested with FT2232). Each debug adapter can declare the transports it supports, and exactly one transport is initialized. (with its commands) in any given OpenOCD session. * Define a new "struct transport with init hooks and a few "transport" subcommands to support it: "list" ... list the transports configured (just "jtag" for now) "select" ... makes the debug session use that transport "init" ... initializes the selected transport (internal) * "interface_transports" ... declares transports the current interface can support. (Some will do this from C code instead, when there are no hardware versioning (or other) issues to prevent it. Plus some FT2232 tweaks, including a few to streamline upcoming support for an SWD transport (initially for Luminary adapters). Eventually src/jtag should probably become src/transport, moving jtag-specific stuff to transport/jtag. Signed-off-by: David Brownell <db@helium.(none)>
* ft2232: revert ft2232_read_scan changesSpencer Oliver2010-07-021-10/+9
| | | | | | | Revert change made in commit dd88b461da1cb8642200dd5c96fb1ff384ca9f7b. Caused segfaults when using ftdi driver under win32. Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* ft2232: bugfix from previous commitSpencer Oliver2010-07-011-1/+1
| | | | | | fix build error with commit dd88b461da1cb8642200dd5c96fb1ff384ca9f7b Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* ft2232: simplify ft2232_read_scanMarc Pignat2010-07-011-9/+10
|
* ft2232: cleanup ft2232_writeMarc Pignat2010-07-011-42/+43
| | | | | * Include the size check into ft2232_write, so calling it is simpler. * Use sizeof(buffer) when possible
* at91sam3s* supportOlaf Lüke2010-06-255-51/+404
| | | | Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* openocd: setup_command_handler() must not be staticØyvind Harboe2010-06-231-2/+4
| | | | | | | when OpenOCD is linked with an app this fn can be used from the outside. Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* arm11: fix gaffe in no-ack transfersØyvind Harboe2010-06-231-1/+1
| | | | | | | | | | | The code did not transfer the last word in no-ack transfers. The strange thing is that this did not lead to any observable errors. This gaffe was introduced in commit 1f5883ea56cb058221f Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* error number: reviewAntonio Borneo2010-06-233-11/+11
| | | | | | | | | Review allocation of error numbers in openocd to avoid overlap. Put brackets around negative numbers to avoid issues during macro expansion. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* am3517 evm: use physical write to memory while target is runningØyvind Harboe2010-06-221-3/+3
| | | | Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* cortex a8: only physical read/write's are available when target is runningØyvind Harboe2010-06-221-2/+9
| | | | | | | | | Memory read/writes to virtual memory, requires that the CPU is halted. Use 'phys' option to write to memory while target is running. Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* target: $_TARGET mdw now has a phys optionØyvind Harboe2010-06-221-8/+28
| | | | | | just like the mdw command Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* target: mwX on target object now supporst phys argumentØyvind Harboe2010-06-221-8/+25
| | | | | | | | $_TARGETNAME mww phys 0x10 0xdeadbeef => write 0xdeadbeef to physical address 0x10 Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* xsvf: Fix shadow issues on MacEdgar Grimberg2010-06-221-6/+6
| | | | | | wait is declared in /usr/include/sys/wait.h Signed-off-by: Edgar Grimberg <edgar.grimberg@zylin.com>
* target: Fix shadow issues on MacEdgar Grimberg2010-06-221-3/+3
| | | | | | wait is declared in /usr/include/sys/wait.h Signed-off-by: Edgar Grimberg <edgar.grimberg@zylin.com>
* flash: fix shadow issues on MacEdgar Grimberg2010-06-221-4/+4
| | | | | | Wait is declared in /usr/include/sys/wait.h Signed-off-by: Edgar Grimberg <edgar.grimberg@zylin.com>
* board: add alpha am3517evm ti board config fileØyvind Harboe2010-06-221-0/+97
| | | | | | | Signs of life: reset(kinda), halt, resume and memory display/modify. Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* gitignore: start list of emacs temp files to ignoreØyvind Harboe2010-06-221-0/+3
| | | | Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* cortex a8: print message that locking debug access succeeded on second tryØyvind Harboe2010-06-221-0/+4
| | | | | | | when locking the debug access fails on the first try, it's a bit noisy, so print out message that it succeeded on second try. Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* cortex a8: add error propagation for poll/resumeØyvind Harboe2010-06-221-5/+15
| | | | Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* cortex a8: add timeouts waiting for restart, prepare and haltØyvind Harboe2010-06-221-6/+36
| | | | | | | It would previously sit in an infinite loop rather than reporting an error. Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* cortex a8: add error propagation for mem_ap_read/write_atomic_u32Øyvind Harboe2010-06-221-7/+31
| | | | | | | Error propagation avoids e.g. infinite loops waiting for target to halt, etc. Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* arm_adi_v5: error propagation fixesØyvind Harboe2010-06-221-18/+22
| | | | Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* nand/mx2: review scope of symbolsAntonio Borneo2010-06-211-1/+1
| | | | | | Add "static" qualifier to private variable. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* nor/str7x: review scope of symbolsAntonio Borneo2010-06-211-1/+1
| | | | | | Add "static" qualifier to private function. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* openocd.c: review scope of symbolsAntonio Borneo2010-06-211-1/+1
| | | | | | Add "static" qualifier to private data. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* nor/virtual: review scope of symbolsAntonio Borneo2010-06-211-2/+2
| | | | | | Add "static" qualifier to private functions. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* target/avrt: review unused symbolsAntonio Borneo2010-06-211-56/+0
| | | | | | | | | | Remove unused functions: - mcu_write_dr_u16 - mcu_write_dr_u8 - mcu_write_ir_u16 - mcu_write_ir_u32 Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* target/avr: review scope of symbolsAntonio Borneo2010-06-213-46/+40
| | | | | | | | | Add "static" qualifier to private functions. Move duplicated global declarations from "target/avrt.c" and "nor/avrf.c" to "target/avrt.h". Remove unused declarations form "nor/avrf.c". Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* target/feroceon: review scope of symbolsAntonio Borneo2010-06-211-23/+32
| | | | | | Add "static" qualifier to private functions. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* target/mips_m4k: review scope of symbolsAntonio Borneo2010-06-212-44/+46
| | | | | | | Add "static" qualifier to private functions. Remove private prototypes from include file. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* helper/jim-eventloop.h: review unused definitionsAntonio Borneo2010-06-211-14/+1
| | | | | | Remove unused typedef and define Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* helper/jim-eventloop: review unused symbolsAntonio Borneo2010-06-211-10/+0
| | | | | | | | Remove unused functions: - Jim_CreateSignalHandler - Jim_DeleteSignalHandler Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* helper/jim-eventloop: review scope of symbolsAntonio Borneo2010-06-212-22/+9
| | | | | | | | Add "static" qualifier to private functions. Remove private prototypes from include file. Remove empty definition of JIM_STATIC. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* jtag: retire jtag_get_error()Øyvind Harboe2010-06-212-9/+4
| | | | | | | This fn is an implementation detail of jtag_execute_queue() that is not to be exposed externally. Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* jtag: do not use jtag_get_error()Øyvind Harboe2010-06-212-6/+3
| | | | | | | normal code should not call jtag_get_error(), but rather check the return code from jtag_execute_queue(). Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* adi_v5_jtag: add missing error handlingØyvind Harboe2010-06-211-1/+5
| | | | Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* cortex a8: add missing error handlingØyvind Harboe2010-06-211-3/+11
| | | | | | cortex examine was missing error handling. Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* arm-jtag-ew: -Wshadow fixAndreas Fritiofson2010-06-191-6/+6
| | | | | | | declaration of ‘index’ shadows a global declaration in /usr/include/string.h Signed-off-by: Andreas Fritiofson <andreas.fritiofson@gmail.com>
* cortex a8: fix segfault for unexamined targetsØyvind Harboe2010-06-192-3/+13
| | | | | | | print error message instead of segfaulting for unexamined targets. Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* target/dsp563xx: review unused symbolsAntonio Borneo2010-06-182-56/+0
| | | | | | | | | | Remove unused functions: - dsp563xx_jtag_senddat - dsp563xx_write_ir_u16 - dsp563xx_write_dr_u16 - dsp563xx_write_ir_u32 Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* target/dsp563xx: review scope of symbolsAntonio Borneo2010-06-182-50/+27
| | | | | | | Add "static" qualifier to private functions. Remove private prototypes from include file. Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
* reverse order of Jim stack trace outputAndreas Fritiofson2010-06-181-1/+1
| | | | | | The stack traces makes much more sense this way. Signed-off-by: Andreas Fritiofson <andreas.fritiofson@gmail.com>