- 14 Mar, 2019 6 commits
-
-
Marc Schink authored
Change-Id: Ifb114bfc062ed7c0ff6e4f37a1a6d546f5415344 Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4970 Tested-by: jenkins Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Marc Schink authored
Change-Id: I4e117f4e6c8c0850f565587f68f41d88da0d6b0b Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4968 Tested-by: jenkins Reviewed-by:
Tomas Vanek <vanekt@fbl.cz> Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Marc Schink authored
Change-Id: Ie7ee82bf04c76ef899ae8b19105158e71b69ae2c Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4962 Tested-by: jenkins Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Christopher Head authored
Bit 4 in OPTSR is IWDG1_SW (the code originally called it IWDG1_HW, but the reference manual refers to it as IWDG1_SW). This is broken out into a separate variable, independent_watchdog_selection, in stm32x_options. However, this is not necessary: bit 4 is included in the user_options field, which includes all of bits 2 through 7, and independent_watchdog_selection is not referenced anywhere else. Delete the field and just rely on user_options to transport that bit, along with all the other bits it contains, between stm32x_read_options and stm32x_write_options. Change-Id: I4da63df9272cf091267b956c412b95671ea1d3c9 Signed-off-by:
Christopher Head <chead@zaber.com> Reviewed-on: http://openocd.zylin.com/4744 Tested-by: jenkins Reviewed-by:
Tarek BOCHKATI <tarek.bouchkati@gmail.com> Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
Antonio Borneo authored
Write to register DP_SELECT can fail, but both functions swd_queue_dp_bankselect() and swd_queue_ap_bankselect() return void and do not propagate the error. Change the return type of the two functions to int and check the returned value. Invalidate the cached value of DP_SELECT if the write fails. Change-Id: Iba6ef8b0c2332e7f7efb66337d558fb7a4a0d39c Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/4980 Tested-by: jenkins Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
Antonio Borneo authored
When the register DP_SELECT is written directly, e.g. with command <dap> dpreg 8 <value> the cached value in OpenOCD is not completely updated with the new value, thus creating issues in the following AP and DP read/write that rely on the cached value. Update the cached value while writing to DP_SELECT. Change-Id: I8221b10cd6fc1fbe73e6b834b68820b43480e1a2 Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/4979 Tested-by: jenkins Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
- 12 Mar, 2019 2 commits
-
-
Tomas Vanek authored
While on it add two missing new lines. Change-Id: I0d54740479bc462b734f91686f0931824796b598 Signed-off-by:
Tomas Vanek <vanekt@fbl.cz> Reviewed-on: http://openocd.zylin.com/4888 Tested-by: jenkins Reviewed-by:
Marc Schink <openocd-dev@marcschink.de>
-
Tomas Vanek authored
instead of conditional compilation #if BUILD_TARGET64 Change-Id: I005ae0b3a4ed127c0f59219b1c6b185a2c76c76e Signed-off-by:
Tomas Vanek <vanekt@fbl.cz> Reviewed-on: http://openocd.zylin.com/4977 Tested-by: jenkins Reviewed-by:
Tim Newsome <tim@sifive.com>
-
- 08 Mar, 2019 6 commits
-
-
Tim Newsome authored
This should allow users to configure flash at >32-bit addresses. Change-Id: I7c9d3c5762579011a2d9708e5317e5765349845c Signed-off-by:
Tim Newsome <tim@sifive.com> Reviewed-on: http://openocd.zylin.com/4919 Tested-by: jenkins Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
Tim Newsome authored
Targets can use this to expose how many address bits there are. gdb_server uses this to send gdb the appropriate upper limit in the memory-map. (Before this change the upper limit would only be correct for 32-bit targets.) Change-Id: Idb0933255ed53951fcfb05e040674bcdf19441e1 Signed-off-by:
Tim Newsome <tim@sifive.com> Reviewed-on: http://openocd.zylin.com/4947 Tested-by: jenkins Reviewed-by:
Peter Mamonov <pmamonov@gmail.com> Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
Matthias Welwarsky authored
This patch adds "hwthread", a pseudo rtos that represents cpu cores in an SMP system as threads to gdb. This allows to debug SMP system kernels in a more sensible manner and removes the current atrocities of switching gdb manually between CPU cores to update the context. Change-Id: Ib781c6c34097689d21d9e02011e4d74a4a742379 Signed-off-by:
Matthias Welwarsky <matthias.welwarsky@sysgo.com> Reviewed-on: http://openocd.zylin.com/3999 Tested-by: jenkins Reviewed-by:
Tim Newsome <tim@sifive.com> Reviewed-by:
Matthias Welwarsky <matthias@welwarsky.de> Reviewed-by:
Graham Sanderson <graham.sanderson@gmail.com>
-
Antonio Borneo authored
API v2 deprecates using command STLINK_DEBUG_READCOREID to read the core ID. Switch to STLINK_DEBUG_APIV2_READ_IDCODES on new stlink firmware version. Change-Id: Iabadfc116c57f2c31f08f2e77baefea0cf90bdc3 Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/4826 Tested-by: jenkins Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
Antonio Borneo authored
Most of ST-Link commands return an error code, but this is almost never checked. Check for the error code for every command that returns it. Change-Id: Ifc4e32f8c73196de23700ba0989bfdfe0f5b46e3 Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/4825 Tested-by: jenkins Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
Antonio Borneo authored
API v2 implementation for command READALLREGS returns the status in the first two bytes, followed by two bytes of padding. This makes the reply 4 bytes longer and changes the offset of the first register value to 4. Fix it for the case API v2 and clean-up the management of the return value. Change-Id: I448c82bcc0baa72d66fdfe7f0c525b94f8a4468b Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/4824 Tested-by: jenkins Reviewed-by:
Tarek BOCHKATI <tarek.bouchkati@gmail.com> Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
- 07 Mar, 2019 2 commits
-
-
Antonio Borneo authored
When the connection with target is lost on the physical JTAG/SWD link (either wire disconnected or DAP/TAP in reset), STLINK fail to connect and returns error STLINK_JTAG_GET_IDCODE_ERROR. Add it in the list of known errors. Change-Id: Iabd6e6892111726f614345497d9286554bfe4ac0 Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/4982 Reviewed-by:
Tomas Vanek <vanekt@fbl.cz> Tested-by: jenkins
-
Antonio Borneo authored
The new firmwares for ST-Link support handling multiple AP, but are (actually) limited to the AP number in range [0..8]. Any tentative to operate on AP number higher than 8 returns an error "bad AP". Add the error code and the related message. Change-Id: I78a98b2612d10e580d03fc43ed9f2b09619d39d8 Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/4981 Reviewed-by:
Tomas Vanek <vanekt@fbl.cz> Tested-by: jenkins
-
- 06 Mar, 2019 24 commits
-
-
Marc Schink authored
Change-Id: I395a36ddd7ac7e85bc54f6bb073a93b0c6f24db0 Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4972 Tested-by: jenkins Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Marc Schink authored
Change-Id: I6393bb8503d64947a1f2349e1d14c1552cabf927 Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4971 Tested-by: jenkins Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Marc Schink authored
Change-Id: I9baaf617c2bfabd07ac3ca195e486893d7950398 Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4969 Tested-by: jenkins Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Marc Schink authored
Change-Id: I30b8d07ee198e8f67cd98f83c4bec15bcfe8be08 Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4967 Tested-by: jenkins Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Marc Schink authored
Change-Id: I536036b15af10bb1879dbbec84576134fd3dce35 Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4966 Tested-by: jenkins Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Marc Schink authored
Change-Id: I05245b7dc9c37ea8e0e40672070fb3e87cb7683f Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4965 Tested-by: jenkins Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Marc Schink authored
Change-Id: I0f237e0769786aa560d1a472a71499d3856ab25c Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4964 Tested-by: jenkins Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Marc Schink authored
Change-Id: I055767f1c20af539159ee59e35de8dd20b399fa4 Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4963 Tested-by: jenkins Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Marc Schink authored
Change-Id: I49f1a6c6ee6d175935eb40b49abf8ebdf56a0b6e Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4961 Tested-by: jenkins Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Marc Schink authored
Change-Id: I3016a8ba37942d674017c09e4540d9813c832872 Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4960 Tested-by: jenkins Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Marc Schink authored
Change-Id: I5f2c95d9a4bdb14cc31d72eb026f58710fb07db5 Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4959 Tested-by: jenkins Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Marc Schink authored
Change-Id: I9838d0d0ba472f029169ab37597ef367a6084fbf Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4958 Tested-by: jenkins Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Marc Schink authored
Change-Id: Ibfe4413b88ed0759a556a8777dac40f0c2c64922 Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4957 Tested-by: jenkins Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Marc Schink authored
Change-Id: I65def33f24187ab336db5732ce35200ca5cd8f5f Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4956 Tested-by: jenkins Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Marc Schink authored
Change-Id: I9cfbba9d81601cc72e2b54ec410e21c7edc4f1c4 Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4955 Tested-by: jenkins Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Marc Schink authored
Change-Id: Ia09adeab88eaf66c009f94be25ab82b0b8f88f61 Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4954 Tested-by: jenkins Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Marc Schink authored
Change-Id: I5af27247f39cf47c925260784e21292f34665471 Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4953 Tested-by: jenkins Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Marc Schink authored
Change-Id: I82e3963ea662844bb96943aee849dab35ea96bb3 Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4952 Tested-by: jenkins Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Marc Schink authored
Change-Id: I3633ced9cb2bfe6afb0ef49aa42fd1b54f155a66 Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4951 Tested-by: jenkins Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Marc Schink authored
Change-Id: I273d24997e2c844015e144a15981f8f0af522261 Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4950 Tested-by: jenkins Reviewed-by:
Antonio Borneo <borneo.antonio@gmail.com>
-
Marc Schink authored
Change-Id: Ib080fc33e8a0305990ee334019547e7643c446ac Signed-off-by:
Marc Schink <openocd-dev@marcschink.de> Reviewed-on: http://openocd.zylin.com/4949 Tested-by: jenkins Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
Antonio Borneo authored
The function command_unknown() is expected to return a value recognized as JIM error code, as it is correctly done in the other cases it returns. Fix the only case in which command_unknown() does not return a JIM error code, by s/ERROR_FAIL/JIM_ERR/ Change-Id: Ib98b75755ae36870bd68c17f8839ddbfa06c6312 Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/4973 Tested-by: jenkins Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
Antonio Borneo authored
Fix a warning identified by clang 7.0.1: stlink_usb.c:1092:3: warning: Value stored to 'p' is never read p += sprintf(p, "S%d", swim); ^ ~~~~~~~~~~~~~~~~~~~~~~~ Change-Id: Ic9da57e19ceecb0e3d65d3ea3412a05e1c2c779a Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/4945 Tested-by: jenkins Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-
Antonio Borneo authored
The null pointer used as second parameter to gdb_put_packet() is passed as second parameter to the memcpy() in line 408 of gdb_put_packet_inner(). In this case memcpy() does not segfault because also the parameter length is zero. Nevertheless, the prototype of memcpy() requires a nonnull pointer. Fixed by passing an empty string in place of the null pointer. Issue highlighted by clang 7.0.0 with warning message: "Null pointer passed as an argument to a 'nonnull' parameter" Change-Id: Ib3dde95d76fcc5fb245ee2d6669e5535d0e0b127 Signed-off-by:
Antonio Borneo <borneo.antonio@gmail.com> Reviewed-on: http://openocd.zylin.com/4946 Tested-by: jenkins Reviewed-by:
Tomas Vanek <vanekt@fbl.cz>
-