aboutsummaryrefslogtreecommitdiff
path: root/tools
AgeCommit message (Collapse)AuthorFilesLines
2023-10-25tools/scripts: iManufacturer added to camelcase.txtEasyDevKits1-0/+1
The iManufacturer is also a member of structure libusb_device_descriptor. No need to output a check message by checkpatch.sh Change-Id: Ibbb2eb9cde3482c8d4d6ea784f51a973eb36f8c5 Signed-off-by: EasyDevKits <info@easydevkits.com> Reviewed-on: https://review.openocd.org/c/openocd/+/7936 Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com> Tested-by: jenkins Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
2023-01-15tools: add disassembler helper for files .incAntonio Borneo1-0/+50
We are moving the binary helpers in files .inc in contrib/loaders/ but we have no support to disassemble them for checking their content, nor documentation to give any hint. Add a simple script that uses objdump to directly disassemble a file .inc Use Cortex-M settings as default, but provide the flexibility to reuse the script for any other target CPU. Change-Id: I12e79580f2936b1622fb7231d9a2484a763ba72a Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: https://review.openocd.org/c/openocd/+/7347 Tested-by: jenkins Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
2023-01-15nds32: drop it, together with aice adapter driverAntonio Borneo1-16/+0
The target nds32 and its companion adapter aice have not received any real improvement since 2013. It has been hard to keep them aligned during the evolution of OpenOCD code, with no way for maintainers to really check if they are still working. No real documentation is present for them in OpenOCD. The nds32 code triggers ~50 errors/warnings with scan-build. The arch nds32 has been dropped from Linux kernel v5.18-rc1. For all the reasons above, this code has been deprecated with commit 2e5df83de7f2 ("nds32: deprecate it, together with aice adapter driver") and tagged to be dropped before v0.13.0. Let it r.i.p. in OpenOCD git history. While there, drop from checkpatch list the camelcase symbols that where only used in this code. Change-Id: Ide52a217f2228e9da2f1cc5036c48f3536f26952 Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: https://review.openocd.org/c/openocd/+/7382 Tested-by: jenkins
2022-09-23checkpatch: fix for flag --no-treeAntonio Borneo1-4/+6
When checkpatch is run with command line flag --no-tree, it cannot find local 'companion' files and has to skip loading them. This has caused issues with change https://review.openocd.org/7211 on jenkins. Skip loading 'tools/scripts/camelcase.txt' with flag --no-tree. While there, rewrite the associated error message. Change-Id: I6ede7b16f9ccd77b9118fd9be7ada07a1ac96952 Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: https://review.openocd.org/c/openocd/+/7212 Tested-by: jenkins
2022-09-18checkpatch: fix path of documentationAntonio Borneo1-0/+5
In OpenOCD documentation is in folder "doc". Fix search path of 'checkpatch.rst'. This file is used to provide verbose explanation of failing checks while using command line flag '-v'. Change-Id: Id864369d371cbd5a24e76bf90c54ff03159051c3 Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: https://review.openocd.org/c/openocd/+/7022 Tested-by: jenkins
2022-09-18checkpatch: extend check for camel[0-9_]*CASEAntonio Borneo1-0/+24
Linux has some automatically generated macros that can trigger camelCASE check. This forces checkpatch to only detect the pattern [A-Z][a-z]|[a-z][A-Z] for adjacent case transition. In OpenOCD we do not have such case, so extend the check to [A-Z][0-9_]*[a-z]|[a-z][0-9_]*[A-Z] and remove the detection of Linux special cases. Change-Id: I82cb6dc668edbb093f68991337da1f4b933f1fac Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: https://review.openocd.org/c/openocd/+/7152 Tested-by: jenkins
2022-09-18checkpatch: enable CAMELCASE testAntonio Borneo2-0/+248
OpenOCD has to deal with CamelCase API, mainly from inttypes.h, jimtcl, libusb and Windows. Modify checkpatch script to load from a file the list of allowed CamelCase symbols. Populate the file 'camelcase.txt' with the symbols that OpenOCD has to get from external library, plus some of the symbols that should be fixed later. Enable CAMELCASE test in configuration script. Add generated files to .gitignore. Remove the check for 'known' CamelCase symbols from include folder as this will not work on OpenOCD Jenkins, as it run checkpatch on already patched code. Change-Id: I0415af673ed9f985394405ff8f1eeec81135410a Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: https://review.openocd.org/c/openocd/+/6170 Tested-by: jenkins
2022-09-18checkpatch: add commit-message field to ignore some checkAntonio Borneo1-0/+7
The script 'checkpatch.pl' is part of the automatic tests used by Linux Maintainers and developers to test new patches. The Linux development process is e-mail base. An error reported by checkpatch is not a blocking point; the developer can explain in the e-mail why he is submitting a patch that fails at checkpatch or that is in violation of the coding style. It's up to the maintainer to decide to accept or reject the explanation and then the patch. The OpenOCD development process relies on Gerrit and Jenkins as front-end tools. Jenkins tests every new patch with checkpatch and then builds the new code. If checkpatch fails, Jenkins adds a failure label to the patch; this often causes the patch to get ignored by maintainers that considers it as 'not ready'. Checkpatch can be instrumented to ignore some test, but this has to be specified in the command line or in the configuration file. Let checkpatch extract from the patch's commit message the new field 'Checkpatch-ignore:' that lists the additional tests that has to be ignored for that patch only. The developer that detects as error or a limitation in checkpatch can add the field and the problematic test in the commit message and re-push the patch. The maintainer should check the list of tests and decide how to proceed. Change-Id: Iafc1b2893a07c7b3fc7e3ad15bd694aba9bd8519 Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: https://review.openocd.org/c/openocd/+/6169 Tested-by: jenkins
2022-09-18checkpatch: increase the max indentation levelAntonio Borneo1-1/+3
OpenOCD uses longer lines (120 char vs 100) and smaller tab size (4 char vs 8) wrt Linux kernel coding style. Clearly deep level of indentation is bad for code readability, but let's be more permissive on the indentation level. Change-Id: I16cf0b761145ec6072509dc26bb09c693e89e608 Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: https://review.openocd.org/c/openocd/+/6167 Tested-by: jenkins
2022-09-18checkpatch: adapt shell script to the tool's new versionAntonio Borneo1-2/+2
With the new checkpatch it's not possible to send in one shot a set of patches through stdin because the Signed-off-by tag present in each patch will trigger the error of duplicated Signed-off-by. Use the new command-line flag '--git' to let checkpatch to extract the patches from git and analyse them one-by-one. While there, add the SPDX tag to the script. Change-Id: I74791b627b8cd68f2d49146d15ae35bbc610e64e Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: https://review.openocd.org/c/openocd/+/6166 Tested-by: jenkins
2022-09-18checkpatch: check for SPDX tags of licenses in useAntonio Borneo2-3/+11
Fix the patch of the external helper spdxcheck.py accordingly to OpenOCD folder structure. List only the current LICENSES subfolders in spdxcheck.py . Enable the check for SPDX headers. Extend the check for TCL and Makefile.am files. Change-Id: I0a40da0127746217ee0ac416058d5ceb922428ff Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: https://review.openocd.org/c/openocd/+/5608 Tested-by: jenkins
2022-09-18checkpatch: add list of typedef used in OpenOCDAntonio Borneo1-0/+8
The new checkpatch from Linux kernel does not recognizes the specific types used in OpenOCD, e.g. "fd_set" and "Jim_Obj". As consequence, it consider "fd_set" as the name of a variable, then misinterpret the asterisk for the pointer "fd_set *" as a multiplication, thus suggest to add a space after the asterisk and replace "fd_set *x" with "fd_set * x". Let checkpatch recognize the typedef used in OpenOCD. Change-Id: Ibc295e6a8b47ffa88b50a6d510b7970760e5199d Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: https://review.openocd.org/c/openocd/+/5607 Tested-by: jenkins
2022-09-18checkpatch: fix check for the FSF addressPaul Fertser1-1/+1
Replace s/Linux/OpenOCD/ in the message about FSF address. This is part of the old commit a9a5c17cf5e1 ("checkpatch: fix check for the FSF address"). Change-Id: I79b79769ef723f86690862277612ea8ab7855c07 Signed-off-by: Paul Fertser <fercerpav@gmail.com> Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: https://review.openocd.org/c/openocd/+/5128 Tested-by: jenkins
2022-09-18checkpatch: don't spell-check the spelling fileAntonio Borneo1-0/+3
Prevent checkpatch to complain for commits that add new entries in the dictionary of misspelled words. Remove the ignore flag TYPO_SPELLING from .checkpatch.conf, now spelling is functional. Change-Id: I911dedafc243e34f753d2be687977066719ff2eb Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: https://review.openocd.org/c/openocd/+/5123 Tested-by: jenkins
2022-09-18checkpatch: check for OpenOCD tree, not for kernel treeAntony Pavlov1-4/+14
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. This is the old commit c5d89883165e ("checkpatch.pl: check for openocd tree, not for kernel tree") re-applied. Also remove the flag "--no-tree" from .checkpatch.conf, not required anymore. Change-Id: I336a66558c75494b7ae339ea63559c31f23aad84 Signed-off-by: Antony Pavlov <antonynpavlov@gmail.com> Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: https://review.openocd.org/c/openocd/+/5122 Tested-by: jenkins
2022-09-18checkpatch: correct false positives reporting instructionsPaul Fertser1-0/+10
This is the old commit 75b4cbe35646 ("checkpatch: correct false positives reporting instructions") re-applied. Change-Id: I348ae549e9d2587093b0fb6652aadf34724f0aab Signed-off-by: Paul Fertser <fercerpav@gmail.com> Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: https://review.openocd.org/c/openocd/+/5121 Tested-by: jenkins
2022-09-18checkpatch: treat jenkins as valid emailPaul Fertser1-0/+4
This is needed to avoid checkpatch barking on already committed patches. This is the old commit cadd51971535 ("checkpatch: treat jenkins as valid email") re-applied. Remove the flag BAD_SIGN_OFF, not anymore needed. Change-Id: I6744f80de982f7934f3a5197ac2df1c29962cbd0 Signed-off-by: Paul Fertser <fercerpav@gmail.com> Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: https://review.openocd.org/c/openocd/+/5120 Tested-by: jenkins
2022-09-18checkpatch: add logging functionsPaul Fertser1-0/+7
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. This is the old commit bb3cd6ec438d ("checkpatch: add logging functions") re-applied and updated. Change-Id: Ib18e4532cf7e1f79821b74c9bb6134a8a4e8be1b Signed-off-by: Paul Fertser <fercerpav@gmail.com> Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: https://review.openocd.org/c/openocd/+/5119 Tested-by: jenkins
2022-09-18checkpatch: add variable $OpenOCDAntonio Borneo1-0/+13
The script checkpatch.pl require some adaptation for OpenOCD that cannot be achieved through the config file .checkpatch.conf; the script's code has to be modified. To merge new version of the script from Linux kernel it becomes relevant highlighting the changes, while minimizing the diff wrt the initial script. Add the perl variable '$OpenOCD' and suggest how to highlight changes. Change-Id: Ia8d26426850008f0465858a1d84cc774bc1146ed Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: https://review.openocd.org/c/openocd/+/7021 Tested-by: jenkins
2022-09-18checkpatch: import new script version from kernel v6.0-rc3Antonio Borneo5-747/+7086
Replace existing checkpatch script with the one available in the latest Linux kernel. Add also from the same kernel version the spelling database and the script spdxcheck.py, even if the script cannot be found by checkpatch in the current path. Add an empty "const_structs.checkpatch" file and an initial "spdxexclude" file. The script as is doesn't work properly in OpenOCD project. Further patches in this series are required. Gerrit will use the checkpatch in this commit to test/build the commit itself. A minimal configuration file is then required to avoid a failure in the test/build process. This commit includes the OpenOCD commits: commit 164450a01576 ("Change checkpatch.pl tab expanding to 4 characters.") commit 667d510dabd5 ("checkpatch: fix false indent trigger") already merged in upstream checkpatch in kernel v6.0-rc3. Change-Id: Ic9cdecff2df0a1e23cdb01d10f14c5988480b8d6 Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: https://review.openocd.org/c/openocd/+/5116 Tested-by: jenkins
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>
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-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-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>