- 17 Apr, 2019 6 commits
-
-
Kevin Gillespie authored
Change-Id: I264977d732e9dfaff8c333634492898b0fea90bc
-
Kevin Gillespie authored
Change-Id: Ie09a08787aa9c4e368ada47c42dda3ce620aa204
-
Kevin Gillespie authored
Change-Id: I6c60a761455f545d9223bc78822ffd575c0160e1
-
Kevin Gillespie authored
Change-Id: I2533490869643d283b54437e958dc663df05132e
-
Kevin Gillespie authored
Change-Id: Ia09f0a7302e58dd2836f44c3d7d4862c61cd96a6
-
Kevin Gillespie authored
Removing unused variable and buffer manipulation in get_info function. Recompiling assembly files to get propper data into write_code. Adding support for 128-bit flash operations. Adding configuration files for new targets. Moving common configuration into shared file. Change-Id: I043e861f958c6926a46735f23f6b6b466edf2a92 Signed-off-by:
Kevin Gillespie <kgills@gmail.com>
-
- 10 Apr, 2019 13 commits
-
-
Christopher Head authored
`maskisr steponly` disables interrupts during single-stepping but enables them during normal execution. This can be used as a partial workaround for 702596 erratum in Cortex-M7 r0p1. See "Cortex-M7 (AT610) and Cortex-M7 with FPU (AT611) Software Developer Errata Notice" from ARM for further details. Change-Id: I797a14e4d43f6dcb3706528ee4ab452846ebf133 Signed-off-by:
Christopher Head <chead@zaber.com> Reviewed-on: http://openocd.zylin.com/4673 Tested-by: jenkins Reviewed-by:
Freddie Chopin <freddie.chopin@gmail.com>
-
Mete Balci authored
The modified lines cause "runtime error: left shift of <X> by <Y> places cannot be represented in type 'int'", because integer literal is cast to int by default. Change-Id: Ie38119b5eb46ee470e0d149959e523b48ac4d66d Signed-off-by:
Mete Balci <metebalci@gmail.com> Reviewed-on: http://openocd.zylin.com/5005 Tested-by: jenkins Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
Marc Schink authored
Tested with SiFive HiFive1 development board. Change-Id: I96a9a528057fcf9fc54d3da46a672d2cd54c3d5f Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4885 Tested-by: jenkins Reviewed-by:
Tim Newsome <tim@sifive.com> Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
Christopher Head authored
Bit 29 of OPTSR is IO_HSLV. It ought to be read in stm32x_read_options and written in stm32x_write_options. Adjust the bitmasks to do this. Change-Id: I785a5291c991c98b774177f960dc58f2b5e045e2 Signed-off-by:
Christopher Head <chead@zaber.com> Reviewed-on: http://openocd.zylin.com/4745 Tested-by: jenkins Reviewed-by:
Tarek BOCHKATI <tarek.bouchkati@gmail.com> Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
Antonio Borneo authored
Every command has been fixed with proper usage field. Change LOG_DEBUG with LOG_ERROR so a developer will get a visible alert for missing usage field when adds a new command. Change-Id: I077c3745fd6265aded8e54b3f9d202d4d6516966 Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/5026 Tested-by: jenkins Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
Leonard Crestez authored
Copy all SJC TAPIPs from imx6 reference manuals. Some imx6 chips are based on Cortex-A7 or have an additional Cortex-M4 and need separate scripts. Change-Id: I3b07d94058c2c5e6313cfc8bb43134a90682a62e Signed-off-by:
Leonard Crestez <leonard.crestez@nxp.com> Reviewed-on: http://openocd.zylin.com/5034 Tested-by: jenkins Reviewed-by:
Matthias Welwarsky <matthias@welwarsky.de>
-
Leonard Crestez authored
OpenOCD scripts are usually indented with 4 spaces but here there are 8. Change-Id: Iaad53e3b377d246d99119bb7bb5fd75d4422f564 Signed-off-by:
Leonard Crestez <leonard.crestez@nxp.com> Reviewed-on: http://openocd.zylin.com/5039 Tested-by: jenkins Reviewed-by:
Matthias Welwarsky <matthias@welwarsky.de>
-
Leonard Crestez authored
Looking through imx6 manuals all of them claim that "In follow-on silicon revisions the ID value is subject to change by incrementing the first nibble". Handle this by passing -ignore-version to jtag newtap command. Change-Id: I7fc4779f9757d527ea20a5174a8c90f919580013 Signed-off-by:
Leonard Crestez <leonard.crestez@nxp.com> Reviewed-on: http://openocd.zylin.com/5031 Tested-by: jenkins Reviewed-by:
Matthias Welwarsky <matthias@welwarsky.de>
-
Leonard Crestez authored
This allows bus access even when CPU is off. Change-Id: I2d5c5581cd0169aecb92ac7b610810988a8dcef4 Signed-off-by:
Leonard Crestez <leonard.crestez@nxp.com> Reviewed-on: http://openocd.zylin.com/5032 Tested-by: jenkins Reviewed-by:
Matthias Welwarsky <matthias@welwarsky.de>
-
Leonard Crestez authored
This can be used to dynamically distinguish between similar chips in tcl code. Change-Id: Ic4dfb61693e78616355173142dc7f9b9683a9f73 Signed-off-by:
Leonard Crestez <leonard.crestez@nxp.com> Reviewed-on: http://openocd.zylin.com/5033 Tested-by: jenkins Reviewed-by:
Matthias Welwarsky <matthias@welwarsky.de>
-
Christopher Head authored
In Texinfo, a comma inside an @url separates parameters to the @url function rather than being included as part of a parameter. Use @comma{} instead to resolve this. Change-Id: I8b38939462cf4452e5bc2582ee484220aaf83ae0 Signed-off-by:
Christopher Head <chead@zaber.com> Reviewed-on: http://openocd.zylin.com/5028 Tested-by: jenkins Reviewed-by:
Matthias Welwarsky <matthias@welwarsky.de>
-
Christopher Head authored
Change-Id: I56e971b38f20db8c4ad0cdee5cc42b42a25319ea Signed-off-by:
Christopher Head <chead@zaber.com> Reviewed-on: http://openocd.zylin.com/5029 Tested-by: jenkins Reviewed-by:
Matthias Welwarsky <matthias@welwarsky.de>
-
Tomas Vanek authored
Command should return target->coreid, not target->working_area_size Solution appeared as an off topic fix in http://openocd.zylin.com/4935 Change-Id: Ic9701ce2001c75060dbaa75fbd013ed475a352e5 Suggested-by:
Graham Sanderson <graham.sanderson@gmail.com> Reported-by:
Tim Newsome <tim@sifive.com> Signed-off-by:
Tomas Vanek <vanekt@fbl.cz> Reviewed-on: http://openocd.zylin.com/5035 Tested-by: jenkins Reviewed-by:
Tim Newsome <tim@sifive.com> Reviewed-by:
Matthias Welwarsky <matthias@welwarsky.de>
-
- 09 Apr, 2019 1 commit
-
-
Rocco Marco Guglielmi authored
This patch remaps the Flash over ITCM region as virtual to ensure that any breakpoint placed in this area will be automatically set as an hardware breakpoint. This patch is a fix to a regression introduced with changes #4429. Change-Id: I03d46d8537ef06b33a3d4a2328274667c6481969 Signed-off-by:
Rocco Marco Guglielmi <roccomarco.guglielmi@gmail.com> Reviewed-on: http://openocd.zylin.com/5097 Reviewed-by:
Freddie Chopin <freddie.chopin@gmail.com> Tested-by: jenkins
-
- 07 Apr, 2019 18 commits
-
-
Antonio Borneo authored
By using the command "add_help_text" to add a help text to a TCL procedure it implicitly creates a new command_registration struct that has field .usage set to NULL. This triggers a debug message BUG: command '%s' does not have the '.usage' field filled out Use an empty string if usage field is NULL. Plus, do not annoy the user with a LOG_INFO when the command "add_usage_text" replaces an empty usage. Change-Id: I4a72646e0fb704ba354f938d774055540cde3967 Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/5025 Tested-by: jenkins Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
Antonio Borneo authored
The missing field causes runtime debug message BUG: command '%s' does not have the '.usage' field filled out While there, fix some minor typo in the help messages: s/deasert/deassert/ s/Deasert/Deassert/ Change-Id: If3dd18265cda103ca0d05609f67f4ca58e7cbb27 Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/5024 Tested-by: jenkins Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
Antonio Borneo authored
The missing field causes runtime debug message BUG: command 'swd' does not have the '.usage' field filled out Change-Id: I5a36c0fc822c4d5d74ef2d1c75e369648c6f7d4d Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/4830 Tested-by: jenkins Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
Antonio Borneo authored
Add usage and fix the help text of the command Change-Id: I0e157d76f5d87bcda80f82a110912994908c8e03 Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/5023 Reviewed-by:
Tomas Vanek <vanekt@fbl.cz> Tested-by: jenkins
-
Antonio Borneo authored
Remove the numeric parameter <bank> from help and add it to usage Change-Id: I630ca5450955210299f54e16fe5c9e8e51b3b266 Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/5022 Reviewed-by:
Tomas Vanek <vanekt@fbl.cz> Tested-by: jenkins
-
Antonio Borneo authored
The only allowed parameter is the string "enable" Change-Id: Iefdad38f1b4177e4194e58caf2be654357299ded Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/5021 Tested-by: jenkins Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
Antonio Borneo authored
Change-Id: I4a8a68147011525191ca79e65fc3d7e760371ae1 Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/5020 Tested-by: jenkins Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
Antonio Borneo authored
The notation "(tck tms tdi tdo)* " is incorrect, because it means the quadruple of gpio can be repeated on the command-line. The correct syntax of the command requires instead to provide either all the four gpio numbers (in order to set the values) or to pass an empty command-line (to dump the values previously set). Change the .usage field to "[tck tms tdi tdo]". Change similarly the corresponding .usage field for SWD command. Add the .usage field for the commands that individually set each gpio or gpio property. Dump the previously set values when commands bcm2835gpio_speed_coeffs or bcm2835gpio_peripheral_base are executed with empty command-line. Change-Id: Ie45d8268c1de331aded0bb52d5e6b8f0e8766a5b Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/5019 Reviewed-by:
Tomas Vanek <vanekt@fbl.cz> Tested-by: jenkins
-
Antonio Borneo authored
The notation "(tck tms tdi tdo)* " is incorrect, because it means the quadruple of gpio can be repeated on the command-line. The correct syntax of the command requires instead to provide either all the four gpio numbers (in order to set the values) or to pass an empty command-line (to dump the values previously set). Change the .usage field to "[tck tms tdi tdo]". Change similarly the corresponding .usage field for SWD command. Add the .usage field for the commands that individually set each gpio or gpio property. Dump the previously set values when commands imx_gpio_speed_coeffs or imx_gpio_peripheral_base are executed with empty command-line. Change-Id: Id56a68622817101bd0a46fed1fead6d2cd2b6ec6 Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/5018 Tested-by: jenkins Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
Antonio Borneo authored
Chained command require a subcommand as first argument. The usage field for chained commands is not really important because the "help" command will list all the subcommands with their respective usage. Add a empty usage field on all chained command. The command "jlink config" can be either followed by a subcommand or used alone, so use a dedicated usage string. Change-Id: I43c3f8a766f96a9bdab4e709e3c90713be41fcef Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/5017 Tested-by: jenkins Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
Antonio Borneo authored
There is no need to split the string in two lines. Change-Id: Ifc574904acd06c31fab589643334e86f4dd0a122 Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/5016 Tested-by: jenkins Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
Antonio Borneo authored
Keep using the same indentation rules while initializing struct command_registration. Change-Id: I900924600753ee7a72ca45da13db4112ab4b1522 Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/5015 Reviewed-by:
Tomas Vanek <vanekt@fbl.cz> Tested-by: jenkins
-
Antonio Borneo authored
Keep using the same indentation rules while initializing struct command_registration. Change-Id: Ie69d9d6a786e1ef0abbcfd3eef89a61206238ebe Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/5014 Reviewed-by:
Tomas Vanek <vanekt@fbl.cz> Tested-by: jenkins
-
Antonio Borneo authored
While initializing struct command_registration, the field's name "name" is not specified, thus relying on the fact that it is the first field declared in the struct and it's initialization value can be listed as the first one. Be coherent in the struct initialization and always use the field's name. Change-Id: Iefaeb15cc051db9f1e0f0140fe2f231b45f5bb12 Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/5013 Reviewed-by:
Tomas Vanek <vanekt@fbl.cz> Tested-by: jenkins Reviewed-by:
Tim Newsome <tim@sifive.com>
-
Antonio Borneo authored
While initializing struct command_registration, the field's name "name" is not specified, thus relying on the fact that it is the first field declared in the struct and it's initialization value can be listed as the first one. Be coherent in the struct initialization and always use the field's name. Change-Id: I65d32870bf4d3df845db69682fe445c71642f976 Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/5012 Reviewed-by:
Tomas Vanek <vanekt@fbl.cz> Tested-by: jenkins
-
Antonio Borneo authored
While initializing struct command_registration, the field's name "name" is not specified, thus relying on the fact that it is the first field declared in the struct and it's initialization value can be listed as the first one. Be coherent in the struct initialization and always use the field's name. Change-Id: I9ef7ac08e7cd5b82a6a7896c47017943d668925f Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/5011 Reviewed-by:
Tomas Vanek <vanekt@fbl.cz> Tested-by: jenkins
-
Antonio Borneo authored
While initializing struct command_registration, the field's name "name" is not specified, thus relying on the fact that it is the first field declared in the struct and it's initialization value can be listed as the first one. Be coherent in the struct initialization and always use the field's name. Change-Id: I743457acf7757df4646c9bc4e5ea1a6f9c841516 Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/5010 Reviewed-by:
Tomas Vanek <vanekt@fbl.cz> Tested-by: jenkins
-
Antonio Borneo authored
The struct log_command_handlers[] is never modified. Add const qualifier to it. Change-Id: I5785aeb09ae9c23eaf7c34a05b88d0c7285f2d78 Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/5009 Reviewed-by:
Tomas Vanek <vanekt@fbl.cz> Tested-by: jenkins
-
- 04 Apr, 2019 1 commit
-
-
Paul Fertser authored
mem2array returns a Tcl (associative) array and so the order of elements is not guaranteed. Treat it as such. Change-Id: Ie4d1219faac1e60247ca13bc2eedf22041a9a9e9 Signed-off-by:
Paul Fertser <fercerpav@gmail.com> Reviewed-on: http://openocd.zylin.com/4487 Tested-by: jenkins Reviewed-by:
Christopher Head <chead@zaber.com>
-
- 03 Apr, 2019 1 commit
-
-
Peter Lawrence authored
ctrl-z usage is incorrectly commented as ctrl-d Change-Id: I54fe1775760ee984154bbae380baa38bcaffa04e Signed-off-by:
Peter Lawrence <majbthrd@gmail.com> Reviewed-on: http://openocd.zylin.com/4863 Tested-by: jenkins Reviewed-by:
Christopher Head <chead@zaber.com> Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-