aboutsummaryrefslogtreecommitdiff
path: root/tools
AgeCommit message (Collapse)AuthorFilesLines
2022-06-23Convert filter_openocd_log.py to use python3. (#709)Tim Newsome1-5/+5
Change-Id: Ie7b42bcc3462b737cbff369ac8d3a71322699aaa Signed-off-by: Tim Newsome <tim@sifive.com>
2021-10-21Properly handle held-in-reset targets. (#654)Tim Newsome1-1/+1
* Properly handle held-in-reset targets. 1. Let OpenOCD continue into some pre-existing code that will periodically call examine() until it passes. 2. Fix crash in riscv_openocd_poll() When SMP is configured, it's not guaranteed that all targets have been examine()d when poll is called on one of them. Change-Id: Ic6c1d217dc766ea69b67bb2e9a4898e37ee94927 Signed-off-by: Tim Newsome <tim@sifive.com> * Actually poll for examine at least every 5s. That's what the comment says the code is trying to do. Change-Id: I34ff909a98f8aebb3c514e0f3ee403be7699c094 Signed-off-by: Tim Newsome <tim@sifive.com> * Compact this error message a bit. Reduces clutter when some targets haven't been properly examined yet. Change-Id: Id865f191f0fbb48abece8b8558cc9fa2041a26df Signed-off-by: Tim Newsome <tim@sifive.com>
2021-10-13Use submodules again.Tim Newsome1-0/+0
They were removed because they were causing the Travis build issues. However, this caused massive diffs, and general git repo headaches when switching between upstream and riscv forks. Now that we're using github actions, it's easy to restart an action in case there is a temporary build failure. Change-Id: I310bc62d4a0dea0411712b9dadd8ab54c6712add Signed-off-by: Tim Newsome <tim@sifive.com>
2021-10-13Remove files.Tim Newsome4-1148/+0
Change-Id: Iab0cc4c192052b79048becf1155f173fa4b17f3e
2021-10-05Merge branch 'master' into from_upstreamTim Newsome1-2/+2
Conflicts: src/jtag/drivers/remote_bitbang.c src/rtos/rtos_standard_stackings.c src/rtos/rtos_standard_stackings.h src/target/breakpoints.c src/target/riscv/riscv.c src/target/target.c Change-Id: Ia6fcba3d43be8ea31728f3bcc2be6cb7e3ccc5c5
2021-09-18gerrit url: update the gerrit server address to https://review.openocd.orgTarek BOCHKATI1-2/+2
change the gerrit server address from http://openocd.zylin.com to the new address in order to avoid re-directions. Change-Id: I76e128c277f63783d1a6f63a6a387aa838f51f80 Signed-off-by: Tarek BOCHKATI <tarek.bouchkati@gmail.com> Reviewed-on: https://review.openocd.org/c/openocd/+/6481 Tested-by: jenkins Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
2021-04-15Remove FSF address warning from checkpatch.Tim Newsome1-9/+10
This often bites me when I'm merging changes from mainline, and never catches a real problem in this fork. Change-Id: I07f4c275db2c0bccb413eeba9ec1759b216f31a9 Signed-off-by: Tim Newsome <tim@sifive.com>
2020-10-15Don't complain when a change fixes a complaint.Tim Newsome1-1/+1
The script was telling me: ``` WARNING: use relative pathname instead of absolute in changelog text \#20: FILE: contrib/cross-build.sh:25: -# /path/to/openocd/contrib/cross-build.sh <host-triplet> ``` In the change where I changed that line to not refer to an absolute path. Change-Id: I1a21af5c36d9aeb01d3e819bfe2b06eb00466467 Signed-off-by: Tim Newsome <tim@sifive.com>
2020-06-23Merge branch 'master' into from_upstreamTim Newsome4-4/+1
Conflicts: .gitmodules .travis.yml jimtcl src/jtag/core.c src/jtag/drivers/ftdi.c src/jtag/drivers/libjaylink src/jtag/drivers/mpsse.c src/jtag/drivers/stlink_usb.c src/rtos/hwthread.c src/target/riscv/riscv-013.c src/target/riscv/riscv.c tcl/board/sifive-hifive1-revb.cfg Change-Id: I2d26ebeffb4c1374730d2e20e6e2a7710403657c
2020-04-21tools/checkpatch.sh: remove flag --no-treeAntonio Borneo1-1/+1
Commit c5d89883165e02ea4f318e3cb0ba40d1fb6f04d1 ("checkpatch.pl: check for openocd tree, not for kernel tree") has already fixed the check for OpenOCD tree, thus we do not need to skip it in the shell wrapper. Remove flag --no-tree from the shell wrapper. Change-Id: I8be497258624d89bde7742fee141a8f56bf9188e Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/5110 Tested-by: jenkins
2020-02-24coding style: tools: remove empty lines at end of text filesAntonio Borneo3-3/+0
Empty lines at end of text files are useless. Remove them. Change-Id: Iea4c8425e137d6252fb2e5b62e0b124c73a01cb6 Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/5168 Tested-by: jenkins
2019-04-02More carefully ignore line numbers/time stamps. (#365)Tim Newsome1-3/+4
This helps when there are other kinds of text in the same file as the openocd log. Change-Id: I38da7d6685769a323930d6aab6cd83b7f27ea90f
2019-02-08Merge branch 'master' into from_upstreamTim Newsome1-1/+1
Conflicts: README contrib/loaders/flash/fespi/Makefile src/flash/nor/fespi.c src/flash/nor/spi.c Change-Id: I78a4e73685cc95daace95e9d16066a6fb51034fb
2019-01-08HACKING: replace refs/publish/master with refs/for/masterJerome Forissier1-1/+1
refs/publish/master is deprecated and gives a warning in newer Gerrit. Replace with refs/for/master. Change-Id: I56871cc6e80c014ba81f4458230cd67dc318ecb3 Suggested-by: Andreas Fritiofson <andreas.fritiofson@gmail.com> Signed-off-by: Jerome Forissier <jerome.forissier@linaro.org> Reviewed-on: http://openocd.zylin.com/4810 Tested-by: jenkins Reviewed-by: Andreas Fritiofson <andreas.fritiofson@gmail.com>
2018-11-12Add utility to combine runs of lines in log files. (#317)Tim Newsome1-0/+119
Hopefully satisfies #193. Change-Id: I427d763aeca2322b05ed88b42fd4a5f0446a654b
2018-08-29Flatten git2cl submoduleAndrew Waterman5-0/+1148
2018-01-13checkpatch: fix more "unescaped left brace" warningsPaul Fertser1-14/+14
Unescaped left brace in string literals is deprecated since perl v5.21.1. Not a Perl programmer here so please review! Change-Id: I724d8fa616d603e032e07afb9b8933e0ec95045b Signed-off-by: Paul Fertser <fercerpav@gmail.com> Reviewed-on: http://openocd.zylin.com/3801 Tested-by: jenkins
2018-01-13tools: release.sh: produce only SHA256 hashesPaul Fertser1-1/+1
Both MD5 and SHA-1 shouldn't be relied upon since long. Change-Id: I0dea92fbf0f68b900c0d3a314fb3956d08ba0d48 Signed-off-by: Paul Fertser <fercerpav@gmail.com> Reviewed-on: http://openocd.zylin.com/3949 Tested-by: jenkins
2017-04-23tools/scripts/checkpatch.pl: fix unescaped braceChengyu Zheng1-1/+1
Change-Id: If1d0fbe95223351ea098504cf24f076784b26a9c Signed-off-by: Chengyu Zheng <chengyu.zheng@polimi.it> Reviewed-on: http://openocd.zylin.com/4102 Tested-by: jenkins Reviewed-by: Freddie Chopin <freddie.chopin@gmail.com>
2016-05-05checkpatch.pl: fix unescaped left brace warningsPaul Fertser1-3/+3
Basically, same as upstream 4e5d56bdf892e18832a6540b63ebf709966bce2a. Unescaped left brace in string literals is deprecated since perl v5.21.1. Change-Id: I0e5f23bef821d2dca6ff4909ddbb06f4992718d4 Signed-off-by: Paul Fertser <fercerpav@gmail.com> Reviewed-on: http://openocd.zylin.com/3228 Tested-by: jenkins Reviewed-by: Daniel Thompson <daniel.thompson@linaro.org>
2015-01-08checkpatch: fix check for the FSF addressPaul Fertser1-2/+4
Commit 4525c0a4c4d0aaa199c37a6d2245617e8445f213 cherry-picked check for the FSF address presence from upstream. However, it has a typo resulting in this obscure error when triggered: Use of uninitialized value in concatenation (.) or string at /home/jenkins/.jenkins/jobs/openocd-gerrit/workspace/tools/scripts/checkpatch.pl line 1258. ERROR: This patch fixes it. Change-Id: Ia417ef4782d21c8b3f1d39de88c4ab850a5a6630 Signed-off-by: Paul Fertser <fercerpav@gmail.com> Reviewed-on: http://openocd.zylin.com/2414 Tested-by: jenkins Reviewed-by: Andreas Fritiofson <andreas.fritiofson@gmail.com>
2014-11-24checkpatch.pl: check for the FSF mailing addressAntony Pavlov1-0/+8
This check code is imported from Linux v3.17 checkpatch.pl. Change-Id: If39d834ee9b6131bccc92de38fd7c108650bd2f1 Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com> Reviewed-on: http://openocd.zylin.com/2341 Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk> Tested-by: Spencer Oliver <spen@spen-soft.co.uk>
2014-11-24checkpatch.pl: check for openocd tree, not for kernel treeAntony Pavlov1-10/+9
checkpatch.pl looks for linux kernel specific paths and files to check source tree. As openocd misses kernel files it ends with this error message: Must be run from the top-level dir. of a kernel tree This patch also renames 'kernel' -> 'openocd' in source tree-related messages. Due to checkpatch checking modifications on itself, lift the restriction on having no spaces at the start of a line for Perl scripts. This can be readded back later. Change-Id: I89b7ca976bef5e13785bd3a43e597c9feb4c2df4 Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com> Reviewed-on: http://openocd.zylin.com/2339 Tested-by: jenkins Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>
2014-03-29Move xsvf_tools and remote_bitbang to contrib/Paul Fertser3-1405/+0
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>
2013-10-29checkpatch: correct false positives reporting instructionsPaul Fertser1-1/+2
Change-Id: Ib45d3db4436a5df5215f2c2d8b8f9a6d48d5f348 Signed-off-by: Paul Fertser <fercerpav@gmail.com> Reviewed-on: http://openocd.zylin.com/1786 Tested-by: jenkins Reviewed-by: Mathias Küster <kesmtp@freenet.de> Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>
2013-10-29checkpatch: treat jenkins as valid emailPaul Fertser1-0/+2
This is needed to avoid checkpatch barking on already committed patches. Change-Id: Ic2b4cfa04c8230d45c68bbf6b370841f682b32d6 Signed-off-by: Paul Fertser <fercerpav@gmail.com> Reviewed-on: http://openocd.zylin.com/1785 Tested-by: jenkins Reviewed-by: Jens Bauer <jens@gpio.dk> Reviewed-by: Mathias Küster <kesmtp@freenet.de> Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>
2013-10-15checkpatch: add logging functionsPaul Fertser1-1/+2
It's commonly considered that user-visible strings should not be split to different lines in the sources to ease grepping for them. Hence, checkpatch traditionally makes an exception for logging functions, lines having them can be of arbitrary length. OpenOCD uses different (from Linux, the kernel) names, so they need to be added to avoid false positives. Change-Id: I1198c1ffc62b5403d97efa41e665c81a5b6bb3a3 Signed-off-by: Paul Fertser <fercerpav@gmail.com> Reviewed-on: http://openocd.zylin.com/1696 Tested-by: jenkins Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>
2013-06-05update files to correct FSF addressSpencer Oliver2-2/+2
Change-Id: I429f7fd51f77b0e7c86d7a7f110ca31afd76c173 Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk> Reviewed-on: http://openocd.zylin.com/1426 Tested-by: jenkins Reviewed-by: Paul Fertser <fercerpav@gmail.com>
2013-06-03rlink: fix speed table generationPaul Fertser1-4/+4
The speed table generation (by using explicit make -f Makefile.rlink) was broken since 865efd828a267992db0f2a92a731c5ce23a34236 Dec 2 2009 which did a bunch of renaming and included hand-editing of a generated rlink_speed_table.c file. This patch is compile-tested, i.e. the new generated rlink_speed_table.c links fine with the rlink driver. Change-Id: I1789a2f2f5bf20183b772d55c55fe68a0bd05cf5 Signed-off-by: Paul Fertser <fercerpav@gmail.com> Reviewed-on: http://openocd.zylin.com/1431 Tested-by: jenkins Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>
2013-05-29tools: add sample remote_bitbang sysfsgpio serverPaul Fertser1-0/+408
This adds a sample server that implements remote_bitbang protocol, based on sysfsgpio interface driver. Change-Id: I17633e91f819ab7b806606e1a1c38d5366ab4598 Signed-off-by: Paul Fertser <fercerpav@gmail.com> Reviewed-on: http://openocd.zylin.com/1403 Tested-by: jenkins Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>
2012-05-29tools: initial.sh fixed to accept spaces in current pathLiviu Ionescu1-1/+1
When changing the path to the hooks folder, the path needs to be surrounded by quotation marks, to avoid failures when the current path contains spaces (not only legal, but a common case on Mac OS X). Change-Id: I8f180ab5f26fab00a7d78ebfe5713d4146c27452 Signed-off-by: Liviu Ionescu <ilg@livius.net> Reviewed-on: http://openocd.zylin.com/683 Tested-by: jenkins Reviewed-by: Peter Stuge <peter@stuge.se>
2012-04-20docs: update gerrit publish refsSpencer Oliver1-1/+1
since gerrit 2.3 pushing changes to refs/for/ is deprecated in favour of using refs/publish/. Change-Id: I6244b9645da2144921583bd9778a95c563fac89f Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk> Reviewed-on: http://openocd.zylin.com/567 Tested-by: jenkins
2012-04-04tools/initial.shUlf Samuelsson1-0/+37
Small script to setup Gerrit with the local repository Usage: tools/initial.sh <username> Change-Id: I26527c35cfe040c7752efec06064d5dc9e3ec6a2 Signed-off-by: Ulf Samuelsson <ulf@emagii.com> Signed-off-by: Attila Kinali <attila@kinali.ch> Reviewed-on: http://openocd.zylin.com/290 Tested-by: jenkins Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>
2012-03-30tools: update release scripts to use configure.acSpencer Oliver3-9/+9
we have already updated autoconf to use configure.ac instead of configure.in, so update release.sh to use the new name. Change-Id: I2dc2beaf2f85058c4627183bc093052677ccba1b Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk> Reviewed-on: http://openocd.zylin.com/537 Tested-by: jenkins
2012-02-06checkpatch: remove __packed and __aligned checksSpencer Oliver1-8/+8
These checks are specific to linux kernel. Change-Id: Ia9b837b5609922a897822f1d55f96f04c0f1f838 Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk> Reviewed-on: http://openocd.zylin.com/424 Tested-by: jenkins
2012-02-06checkpatch: increase line length to 120Spencer Oliver1-3/+3
Change-Id: I963385d0a4880f2b1e55208c8dfe65c1870ac6e1 Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk> Reviewed-on: http://openocd.zylin.com/422 Tested-by: jenkins
2012-02-06checkpatch: remove volatile checkSpencer Oliver1-5/+5
We may enable this again - but at the moment is causing extra issues with reformatting the codebase. Change-Id: I5a2aaaa32ad784e011dff3079ff45501452c1819 Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk> Reviewed-on: http://openocd.zylin.com/414
2012-02-06checkpatch: remove typedef checkSpencer Oliver1-8/+8
We may enable this again - but at the moment is causing extra issues with reformatting the codebase. Change-Id: Ic64310a20605a0ef3206caa15c8e6c8ee655bfda Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk> Reviewed-on: http://openocd.zylin.com/413 Tested-by: jenkins
2011-12-16checkpatch: fix false indent triggerSpencer Oliver1-1/+1
we have changed the indent to 4 to match OpenOCD coding style. Change-Id: I4870a3410eb20fc2f6df6a3e5891d4d4e598131a Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk> Reviewed-on: http://openocd.zylin.com/285 Tested-by: jenkins
2011-12-16checkpatch: disable extern and switch indent checksSpencer Oliver1-48/+48
We allow extern's in c files so disable checkpatch checks. Change-Id: Ia649585cd70ec45289c4edaf26c1fd773a140db4 Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk> Reviewed-on: http://openocd.zylin.com/284 Tested-by: jenkins Reviewed-by: Øyvind Harboe <oyvindharboe@gmail.com>
2011-12-16Change checkpatch.pl tab expanding to 4 characters.Erik Ahlén1-1/+1
The C coding style guide says that tab width is 4 characters but checkpatch.pl expands tabs to 8 characters which produces false negatives. Change-Id: Ibdabbb55269b7cf6bcd38042cccb8bd235e42ce2 Signed-off-by: Erik Ahlén <erik.ahlen@avalonenterprise.com> Reviewed-on: http://openocd.zylin.com/275 Tested-by: jenkins Reviewed-by: Øyvind Harboe <oyvindharboe@gmail.com> Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>
2011-12-06make checkpatch.sh take an optional 'since' refspecAndreas Fritiofson1-1/+2
Change-Id: I793778037db08bd5462f61b9bcafd484708cc1b6 Signed-off-by: Andreas Fritiofson <andreas.fritiofson@gmail.com> Reviewed-on: http://openocd.zylin.com/250 Tested-by: jenkins Reviewed-by: Spencer Oliver <spen@spen-soft.co.uk>
2011-11-04tools: fix permissionsSpencer Oliver2-0/+0
Change-Id: I9419138dd2972304daf215594ca917ac8eb7fcda Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk>
2011-11-04tools: add checkpatch scriptSpencer Oliver1-0/+4
execute from openocd root dir after commiting change but before pushing to gerrit. tools/checkpatch.sh This is the same script used by the jenkins build server to validate a change. Change-Id: Ib40d44b160e1c50a5e47ab55fc48a554381ea763 Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk> Reviewed-on: http://openocd.zylin.com/167 Tested-by: jenkins
2011-11-03checkpatch: increase line length warning to 100Spencer Oliver1-3/+3
Change-Id: Ief0445ea6581929b3ffdcf8bc644ce5f27b392c4 Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk>
2011-11-03tools: add checkpatch scriptSpencer Oliver1-0/+3337
Change-Id: I3579028fc1c6ee8bea58c82e5f0eecba7794d7cb Signed-off-by: Spencer Oliver <spen@spen-soft.co.uk>
2010-01-08Coexist with quilt: rename PATCHES --> PATCHES.txtDavid Brownell1-1/+1
The issues is on Win32, which ignores case in filesystem and thus doesn't tolerate the quilt "patches" directory. Rename, and add "patches" to .gitignore so that developers can choose to use quilt for local patch management. Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
2009-11-04Other files: stop referring to ChangeLog tooDavid Brownell1-20/+12
The ChangeLog idiom is redundant given any decent SCM. Time to phase it out here. Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
2009-11-04Release scripts: comments, run on UbuntuDavid Brownell2-13/+20
The "source" command isn't accepted by ASH; easy to fix. Failures with "-e" are harder to fix. Remove the "-e" (for now) and force bash, for safety. Un-obfuscate the release steps, by using names instead of numbers. Comment the version-number manipulation. Signed-off-by: David Brownell <dbrownell@users.sourceforge.net>
2009-10-27Add script to test the release process.Zachary T Welch1-0/+121
Runs the release.sh script in a freshly cloned repository, charting one hypothetical future of OpenOCD's lineage.
generated by cgit v1.1 at 2024-08-16 02:20:48 +0000