Commits · 6812993483005ace4ec954e47ac864d3dbe52bad · Sam Calisch / openocd_nrf52_patch

Apr 15, 2014

The openocd-0.8.0-rc2 release candidate · 68129934
Paul Fertser authored 11 years ago
```
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
```
View commits for tag v0.8.0-rc2 v0.8.0-rc2

68129934

Paul Fertser authored 11 years ago


Change-Id: Iba04baae4f3278db8b36283574a82c077d28bb4a
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/2113
Tested-by: jenkins

186c442f

Apr 14, 2014

svf: try to reallocate buffers if they're not enough · 11a1080c

Paul Fertser authored 11 years ago


On larger FPGAs the default buffer size is not nearly enough;
automatically reallocate it as needed.

Change-Id: Ic8071e8cceddeef897ce76996c029250fd22662b
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/2099


Tested-by: jenkins
Reviewed-by: Kenneth Lorthioir <ibelimb@gmail.com>
Reviewed-by: Freddie Chopin <freddie.chopin@gmail.com>
Reviewed-by: Andreas Fritiofson <andreas.fritiofson@gmail.com>

11a1080c

cortex_a: fix endiannes issues on TI TMS570 · 3427cf2b

Seth LaForge authored 11 years ago


The TI TMS470 and TMS570 series of processors are BE-32 processors,
despite BE-32 not being supported by ARM in the Cortex-R4 core. TI
hacked in BE-32 support, which requires odd swizzling in OpenOCD to
make memory reads and writes function correctly. In particular,
without this change, OpenOCD word reads and writes had the bytes
reversed, and halfword and byte packed reads were reading garbage.
In my testing, this change fixes these problems.

Change-Id: I21dd30f4b9003f20fcc85f674ab833407bb61f74
Signed-off-by: Seth LaForge <sethml@google.com>
Reviewed-on: http://openocd.zylin.com/2064


Tested-by: jenkins
Reviewed-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-by: Andreas Fritiofson <andreas.fritiofson@gmail.com>

3427cf2b

usb_blaster: fix warning: array subscript has type ‘char’ · 31496c2b

Spencer Oliver authored 11 years ago


caught by clang 3.4 running on cygwin.

Change-Id: I59f652337334e557fb50374f5270ba9c30392b6e
Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk>
Reviewed-on: http://openocd.zylin.com/2100


Reviewed-by: Jörg Wunsch <openocd@uriah.heep.sax.de>
Reviewed-by: Andreas Fritiofson <andreas.fritiofson@gmail.com>
Tested-by: jenkins

31496c2b

mips32, dsp563xx: fix segfault on Gdb attach · 151c3178

Paul Fertser authored 11 years ago


Since c6216201 gdb target description
generation support is enabled by default and it counts on checking
"feature" pointer in reg_list. Both mips32 and dsp563xx neither used
calloc nor explicitly set feature (as it was a newly introduced struct
field).

This patch changes all targets to use calloc for consistency.

Change-Id: I9eceadef8b04aacb108e24ae23cb51ca3009586f
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/2102


Tested-by: jenkins
Reviewed-by: Salvador Arroyo <sarroyofdez@yahoo.es>
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>

151c3178

configure: presto ftdi driver requires libftdi · cb115642

Paul Fertser authored 11 years ago


Change-Id: I4b04eec5084d4b8129e2aff8f3411c6d92e84431
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/2101


Tested-by: jenkins
Reviewed-by: Freddie Chopin <freddie.chopin@gmail.com>
Reviewed-by: Xiaofan <xiaofanc@gmail.com>

cb115642

jtag/drivers/remote_bitbang: use sizeof to determine maximum unix socket path length · 3a590658

Paul Fertser authored 11 years ago


Different *nix systems use different constants, so sizeof should be
used instead. This fixes the build on OS X (as sun_path length is
hardcoded to 104 on Darwin).

Change-Id: I1df611383dedcfc6d153ec59b453a895c7d84e94
Reported-by: Xiaofan Chen <xiaofanc@gmail.com>
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/2095


Tested-by: jenkins
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>
Reviewed-by: Xiaofan <xiaofanc@gmail.com>

3a590658

target: fix incorrect arm cpu monitor mode encoding · 4835a21d

Tim Sander authored 11 years ago


According to the "Arm Arch Ref Manual ARMv7-a and ARMv7-R edition" the
CPSR encoding for Monitor mode is 0b10110 (22) not 0b11010 (26) as is
currently used.

Change-Id: I73373a0029a81abc92febf518b88bf0dd4dec1fa
Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk>
Reviewed-on: http://openocd.zylin.com/2081


Reviewed-by: Jörg Wunsch <openocd@uriah.heep.sax.de>
Tested-by: jenkins
Reviewed-by: Younes REGAIEG <y.regaieg@gmail.com>
Reviewed-by: Tim Sander <tim@krieglstein.org>
Reviewed-by: Freddie Chopin <freddie.chopin@gmail.com>

4835a21d

tcl: bcm6348 target config, BT HomeHub v1 board config · 47830f0e

Lee Bowyer authored 11 years ago


Also add the board to the firmware recovery script.

Change-Id: I4f9c895dae171df7249e3b1c0563b288518b9fe0
Signed-off-by: Lee Bowyer <lee@sodnpoo.com>
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/2097


Tested-by: jenkins
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>

47830f0e

contrib/rpc_examples: add example for python3 · e2b35204

Andreas Ortmann authored 11 years ago


Halt, resume, memory read/write are used in various ways.

Change-Id: Ia6727678bfc19cc764f822b739bddaae56e9dc70
Signed-off-by: Andreas Ortmann <ortmann@finf.uni-hannover.de>
Reviewed-on: http://openocd.zylin.com/2000


Tested-by: jenkins
Reviewed-by: Paul Fertser <fercerpav@gmail.com>

e2b35204

openocd.rules: match CMSIS DAP devices by product string · b182f934

Paul Fertser authored 11 years ago


Since the standard requires to have "CMSIS-DAP" somewhere in product
string, use that to automatically match all the compliant adapters.

Change-Id: I1e2ac088333a7d69a136af825248914339debdd8
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/2082


Tested-by: jenkins
Reviewed-by: Jörg Wunsch <openocd@uriah.heep.sax.de>
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>

b182f934

parport: add "aspo" hotpluggable adapter config · 7256d6ac

Karl Hammar authored 11 years ago

This adds a new adapter config.

Design files are available from:

http://turkos.aspodata.se/git/openhw/isp/pp_jtag_arm20.sch
http://turkos.aspodata.se/git/openhw/isp/pp_jtag_arm20.pcb



No need to document it anywhere as only a single device was ever
built.

Change-Id: If031fdb9dedb1e544401747bfb7fbc5afc4d5407
Signed-off-by: Karl Hammar <karl@aspodata.se>
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/2094


Tested-by: jenkins
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>

7256d6ac

target/cortex_a: check gdb_service before dereferencing in update_halt_gdb · ee54f7e9

Paul Fertser authored 11 years ago


If gdb was disconnected by the moment the target entered halted state,
update_halt_gdb would segfault.

Change-Id: I67477e9199c1df097be83a49e38602f975c083f5
Reported-by: Younes REGAIEG <younes.regaieg@imag.fr>
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/2098


Tested-by: jenkins
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>

ee54f7e9

README.OSX: add 10.9 specific hints · 85c1790b

Paul Fertser authored 11 years ago


Change-Id: I8c684f2964262fb0670ed74fded26676833aee10
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/2080


Tested-by: jenkins
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>
Reviewed-by: Xiaofan <xiaofanc@gmail.com>

85c1790b

Makefile.am: fix build for libftdi1, pass CFLAGS where appropriate · 45f71f10

Paul Fertser authored 11 years ago


Change-Id: I9d8afa6ae32fc01e69ec434b5bc9d71524d386a2
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/2079


Tested-by: jenkins
Reviewed-by: Freddie Chopin <freddie.chopin@gmail.com>
Reviewed-by: Xiaofan <xiaofanc@gmail.com>

45f71f10

Apr 08, 2014

jtag/aice/aice_usb: remove unused constants · 9f2bc3b8

Paul Fertser authored 11 years ago


Those are breaking the build with clang 3.4 (current default compiler
on OS X 10.9.2).

Change-Id: I9f2fbfbb6d544498cb8d6a62565e4ebe7223e784
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/2077


Tested-by: jenkins
Reviewed-by: Hsiangkai Wang <hsiangkai@gmail.com>

9f2bc3b8

flash/nor/at91sam3: remove sam3_page_write_opcodes · 61905a16

Paul Fertser authored 11 years ago


Apparently this helper was never actually used, and current clang
produces an unused const variable warning, so it's breaking the build.

Change-Id: Ib088bef5e9c0a48942c1d417d46f738469ee7e28
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/2076


Tested-by: jenkins
Reviewed-by: Xiaofan <xiaofanc@gmail.com>

61905a16

Mar 30, 2014

Restore -dev suffix · d7cbdee3
Paul Fertser authored 11 years ago
```
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
```
d7cbdee3
The openocd-0.8.0-rc1 release candidate · af5cd853
Paul Fertser authored 11 years ago
```
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
```
View commits for tag v0.8.0-rc1 v0.8.0-rc1

af5cd853

Don't cast return value of [cm]alloc · e6907e6d

Andreas Fritiofson authored 11 years ago


Change-Id: I0028a5b6757b1ba00031893d9a2a1725f915a0d5
Signed-off-by: Andreas Fritiofson <andreas.fritiofson@gmail.com>
Reviewed-on: http://openocd.zylin.com/2069


Tested-by: jenkins
Reviewed-by: Jörg Wunsch <openocd@uriah.heep.sax.de>
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>

e6907e6d

Makefile.am: fix adding contrib directory for dist · 1171f078

Paul Fertser authored 11 years ago


With extra / the directory was getting added as a subdirectory of
itself.

Change-Id: I5a3d5635f3c949f39d88b34a2ddab8244643f560
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/2070


Tested-by: jenkins
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>

1171f078

Mar 29, 2014

NEWS: add release highlights · 086a523c

Paul Fertser authored 11 years ago


Change-Id: Ifa4459b3b7b58b639d6c885a2ea88cefbfac2680
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/1881
Tested-by: jenkins

086a523c

at91smad: Fixes to 'samd_protect' · dba153bb

Andrey Smirnov authored 11 years ago


Some fixes to 'samd_protect' including:
     - Fix a bug in which the value of 'set' parameter passed into the
     function was ignored so it was impossible to remove flash
     protection once it was set.
     - Check the protection status of the sector via 'is_protected'
     field of the  corresponding 'flash_sector' structure to see if
     any actual HW manipulations needs to be done.
     - Change the way the errors during protection activation are
     handled. Now even in the case of error in the middle of
     protecting a number of sectors the subroutine would still update
     the state of the sector protection in sectors array so as to
     avoid cases where openocd thinks that the sector is not protected
     while it actually is.

Change-Id: I4cc6445a98ec13bdd94c89f0711c17840738a215
Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com>
Reviewed-on: http://openocd.zylin.com/2027


Tested-by: jenkins
Reviewed-by: Andrey Yurovsky <yurovsky@gmail.com>
Reviewed-by: Paul Fertser <fercerpav@gmail.com>

dba153bb

at91samd: Bail early if trying to erase protected sector · c92a605e

Andrey Smirnov authored 11 years ago


Bail early if trying to erase protected sector and also do not double-erase already
erased sectors.

Change-Id: Ic2d39af48c3b8e10e78d52dd978b9bc01f671c6a
Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com>
Reviewed-on: http://openocd.zylin.com/2026


Tested-by: jenkins
Reviewed-by: Paul Fertser <fercerpav@gmail.com>

c92a605e

at91samd: Bail early if trying to write to protected sector · 34db6b9c

Andrey Smirnov authored 11 years ago


Bail early if trying to write to protected sector also mark the sector
we are writing to as dirty.

Change-Id: I892f83461792e1cc2dcccade7aa65717831a6805
Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com>
Reviewed-on: http://openocd.zylin.com/2025


Tested-by: jenkins
Reviewed-by: Paul Fertser <fercerpav@gmail.com>

34db6b9c

at91samd: Remove 'first' and 'last' checks in 'samd_erase' · e0f0ce96

Andrey Smirnov authored 11 years ago


Remove 'first' and 'last' checks in 'samd_protect' since those
parameters are checked in 'handle_flash_erase_command'.

Change-Id: I30e5598a9ab656d81055f26cc63e291377605300
Signed-off-by: Andrey Smirnov <andrew.smirnov@gmail.com>
Reviewed-on: http://openocd.zylin.com/2024


Tested-by: jenkins
Reviewed-by: Andrey Yurovsky <yurovsky@gmail.com>
Reviewed-by: Paul Fertser <fercerpav@gmail.com>

e0f0ce96

tcl: introduce init_target_events and use it for gdb flashing events · 207237b9

Paul Fertser authored 11 years ago


This introduces a new global Tcl procedure that is run just after
init_targets and before init_boards.

Its default behaviour is to assign gdb-flash-erase-start and
gdb-flash-write-end to reasonable defaults.

The rationale for doing "reset init" before gdb erases and flashes
memory is that all flash drivers are written in assumption that they
can safely be used only after chip reset (plus chip-specific
configuration in the init handler if any). The need to use "reset
halt" after flashing is because a user expects running firmware after
loading to be the same as running it from power-on-reset.

Change-Id: I9ddc4047611904ca4ca779b73376d2739611948a
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/2062


Tested-by: jenkins
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>

207237b9

tcl/target: make milandr configs swd-compatible · de9ebc5c

Paul Fertser authored 11 years ago


Change-Id: Ibb34f0d7829b205341bcce511ffc2624bdfe2c75
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/1962


Tested-by: jenkins
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>

de9ebc5c

target: remove memory leaks · 0f566ae1

Spencer Oliver authored 11 years ago


Found by clang.

Change-Id: Ifb25dca52f8d9e8e46a35f0947a7239f26eb3757
Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk>
Reviewed-on: http://openocd.zylin.com/2067


Tested-by: jenkins
Reviewed-by: Andreas Fritiofson <andreas.fritiofson@gmail.com>
Reviewed-by: Jörg Wunsch <openocd@uriah.heep.sax.de>

0f566ae1

target: fix handle_profile_command variable typo · 0cb97783

Spencer Oliver authored 11 years ago


Change-Id: I5d476aecb4622731890e168b1be3173718151e95
Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk>
Reviewed-on: http://openocd.zylin.com/2066


Tested-by: jenkins
Reviewed-by: Andreas Fritiofson <andreas.fritiofson@gmail.com>

0cb97783

target: remove handle_profile_command memory leak · e22bad79

Spencer Oliver authored 11 years ago


COMMAND_PARSE_NUMBER may return, thus any memory allocated may not be
freed, simple reorder fixes the issue.

Change-Id: I0ce444a5b032f5c49b6d33a03a8c0b71cad49c8c
Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk>
Reviewed-on: http://openocd.zylin.com/2065


Tested-by: jenkins
Reviewed-by: Andreas Fritiofson <andreas.fritiofson@gmail.com>

e22bad79

Add xscale debug helper sources and everything related to dist · 8b1fabd8

Paul Fertser authored 11 years ago


GPL requires providing sources for any derived work. I do not see any
reason to not include the xscale stuff into release tarballs.

Wildcard matching is used because plain directory name matches
implicit rule for executables and xscale.c built is errorneously
attempted, and directory name with a slash duplicates a directory
(xscale/xscale) in dist.

Change-Id: Ie0266470dcb97be87a09ba2dda9b3957f7cbc2fa
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/1911


Tested-by: jenkins
Reviewed-by: Robert Jarzmik <robert.jarzmik@free.fr>
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>

8b1fabd8

Add all of contrib/ to dist, not just loaders · 1a9cec02

Paul Fertser authored 11 years ago


Change-Id: I467e21de6805ea9f23ae483317d602e9d81e1148
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/1909


Tested-by: jenkins
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>

1a9cec02

Add usb_blaster cheap clone notes to dist · 7301bf1a

Paul Fertser authored 11 years ago


Change-Id: Ie9569634415ef76e88d5ca58bae6184838cf7114
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/1908


Tested-by: jenkins
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>

7301bf1a

Add rlink speed table sources and everything related to dist · 7a163403

Paul Fertser authored 11 years ago


GPL requires providing sources for any derived work. I do not see any
reason to not include the rlink stuff into release tarballs.

Change-Id: I686ab88994030312aca5bdfe731a72f10803b5c3
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/1907


Tested-by: jenkins
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>

7a163403

Move xsvf_tools and remote_bitbang to contrib/ · 6405941c

Paul Fertser authored 11 years ago


It looks like tools/ should be used only for build tools, and contrib/
is a suitable place for everything else.

Change-Id: Iddaebba0acb6d66404912ec96749b46e4be643d8
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/1906


Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>
Tested-by: Spencer Oliver <spen@spen-soft.co.uk>

6405941c

Add previous NEWS* files to dist · 6a8d39e1

Paul Fertser authored 11 years ago


Change-Id: I2f814e01c7b5aa9cd76e8e63c9c4a1ebbf3dd2ea
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/1905


Tested-by: jenkins
Reviewed-by: Luca Bruno <lucab@debian.org>
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>

6a8d39e1

Move ocl (at91sam7x flash loader) sources to contrib · 18bbffb6

Paul Fertser authored 11 years ago


Change-Id: Ib5c3c505a067d0e285a7b074cb6fddebfa6dda1e
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/1904


Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>
Tested-by: Spencer Oliver <spen@spen-soft.co.uk>

18bbffb6

doc: add "manual/" directory in its entirety to dist · 620448f9

Paul Fertser authored 11 years ago


Change-Id: I750c2938ab207012a6488bf34f55d784e5e26b6e
Signed-off-by: Paul Fertser <fercerpav@gmail.com>
Reviewed-on: http://openocd.zylin.com/1903


Tested-by: jenkins
Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>

620448f9