summaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
...
* nand: nand probe also outputs manufacturerSpencer Oliver2010-07-191-1/+2
| | | | | | The nand probe now outputs the manufacturer if found. Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* cortex a8: lots of error propagation fixesØyvind Harboe2010-07-191-18/+53
| | | | | | found by code inspection Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* cortex a8: mem_ap_read_buf_u32() error handlingØyvind Harboe2010-07-191-1/+3
| | | | Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* cortex a8: add missing error handling for ↵Øyvind Harboe2010-07-191-3/+9
| | | | | | cortex_a8_dap_write/read_coreregister_u32() Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* cortex a8: add missing error handling from cortex_a8_exec_opcode()Øyvind Harboe2010-07-191-13/+52
| | | | Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* cortex a8: add missing error handling for mem_ap_atomic_write_u32()Øyvind Harboe2010-07-191-1/+5
| | | | Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* cortex a8: add missing error handling for mem_ap_read_atomic_u32()Øyvind Harboe2010-07-191-0/+11
| | | | Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* comments for Luminary ICDI layoutDavid Brownell2010-07-181-3/+27
| | | | | | | Provide $defines for more of the signals involved in the Luminary ICDI hardware, and comment some of what's going on. Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
* spelling fixes in commentsØyvind Harboe2010-07-181-5/+5
| | | | Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* debug-feature: jtagtcpip, improve jtag performanceØyvind Harboe2010-07-182-4/+76
| | | | | | postpone callbacks until jtag execute queue time. Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* ARM ADI-V5: cleanup CID/PID addressingDavid Brownell2010-07-161-21/+23
| | | | | | | | | | | Use addition for offsetting, not masking. Shorten some lines. Make "component_start" print-only (unused otherwise; don't save). Still doesn't resolve the issue where multiple components are wrongly displaying as NVICs on some Cortex-M3 parts because many PIDs appear to be zeroes ... maybe adapter related?? Signed-off-by: David Brownell <db@helium.(none)>
* flash: use bkpt for str9 flash algo algorithmSpencer Oliver2010-07-161-3/+2
| | | | | | | STR9 is ARMv5 arch so use bkpt instruction rather than waste a breakpoint. Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* fix warningsØyvind Harboe2010-07-161-2/+1
| | | | Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* debug-feature: jtagtcpip, reduce performance impact of ping timesØyvind Harboe2010-07-162-46/+103
| | | | | | queue read back of data Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* debug feature: jtagtcpip, improve performanceØyvind Harboe2010-07-162-0/+26
| | | | | | | | | waiting for ZY1000 fifo to idle is now queued as an asynchronous command. This radically improves performance when waitIdle() is interspersed with writes as no readback is required over TCP/IP. Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* debug feature: jtagtcpip, improved performanceØyvind Harboe2010-07-161-9/+22
| | | | | | only check error flag when rclk is actually enabled. Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
* ARM ADI-V5: PIDs and CIDs are 8 bitsDavid Brownell2010-07-131-21/+41
| | | | | | | | | Mask the upper bits after 32-bit reads. Alsoo revert the ugly changes to use PRIx32; just cast to unsized integers when printing (two chars not eight). Signed-off-by: David Brownell <db@helium.(none)>
* jtag: fix shadow issues in adapter_initSpencer Oliver2010-07-121-1/+0
| | | | | | Use global jtag_only rather than local static. Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* arm9: revert change arm9tdmi cmd group name to arm9Spencer Oliver2010-07-121-2/+2
| | | | | | | | | This reverts the incorrect change made to the arm9 cmd group in commit d1eca9a74c7c57ba6a3210c51b2a10cc5adb22e1. The code now matches the docs and the release notes. Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* adapter.c: missing space in handle_interface_commandSpencer Oliver2010-07-121-1/+1
| | | | Signed-off-by: Spencer Oliver <ntfreak@users.sourceforge.net>
* 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>
* initial "transport" frameworkDavid Brownell2010-07-028-29/+587
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-251-15/+328
| | | | 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>
* 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>
* 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>