Skip to content

Draft: Align with mainline at fc566d74005a4aefbe125b9b2f777d9514c60c87 - #27

Open
en-sc wants to merge 628 commits into
syntacorefrom
en-sc/allign-with-mainline
Open

Draft: Align with mainline at fc566d74005a4aefbe125b9b2f777d9514c60c87#27
en-sc wants to merge 628 commits into
syntacorefrom
en-sc/allign-with-mainline

Conversation

@en-sc

@en-sc en-sc commented Jul 29, 2026

Copy link
Copy Markdown
Collaborator

Commits that revert cleanly

  • commit 2b0887c "target/target: drop the cached value when force-reading a register"
  • commit 10f1659 "target/riscv: fix bug in scratch_reserve"
  • commit 9d4c94e "target/riscv: refactor scratch_reserve() to use standardized macros for alignment and bit operations"
  • commit 0846f69 "target/riscv: always validate dmstatus.version field"
  • commit 12d8468 "implemented facilities to inject ubsan/valgrind workarounds in the codebase"
  • commit cca1212 "[to upstream] target/riscv: do not attempt to read memory if semihosting is not enabled"
  • commit fa08a3e "[cherry-pick] target/riscv: fix potential UB reported by ubsan in ac cache lookup"
  • commit 9ab6bf2 "[cherry-pick] target/riscv: fix ub during instruction decode"
  • commit af81a9b "[cherry-pick] fix error detection during trigger removal"
  • commit 7450dfc "[cherry-pick] server/gdb_server: Add more error messages in gdb_new_connection"
  • commit 98b7607 "[syntacore] tcl/interface: proper integration of digilent-hs2a.cfg"

Changes that do not revert cleanly

Re-examine

Sections vs segments

Break on shutdown

GitHub CI

Submodules

Running target memory

Verify image

jtag exectute

SCR5

Syntacore tools

Re-examine Sections vs segments Break on shutdown GitHub CI Submodules Running target memory Verify image jtag execute SCR5 Syntacore tools
configure.ac x x
doc/openocd.texi x x x
.github/workflows x
.gitmodules x
jimtcl x
Makefile.am x x
src/jtag/drivers/libjaylink x
src/jtag x
src/server/server.h x
src/target/image.c x x
src/target/riscv/riscv.c x x
src/target/riscv/riscv.h x
src/target/riscv/scr5.c x
src/target/target.c x x x
src/target/target.h x
src/target/target_type.h x
tcl/syntacore x
tcl/target/syntacore_* x
testing/tcl_commands x
tools/syntacore x

zapb-0 and others added 30 commits January 25, 2026 09:26
@zapb-0 @borneoa
tcl/target: Move GigaDevice configs into vendor directory
f36ab02 
Move the configuration files into a dedicated vendor folder as required
by the developer guidelines.

Change-Id: I9ed39e32b6281a9cb8510914690f3f7751b795c8
Signed-off-by: Marc Schink <dev@zapb.de>
Reviewed-on: https://review.openocd.org/c/openocd/+/9271
Tested-by: jenkins
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
R. Diez @borneoa
log: avoid incrementing 'count' for discarded log lines
c60bb56 
This matches the behaviour of code which calls LOG_DEBUG_IO etc. instead.

Change-Id: I7679f0fc67222f947a4d0f1d53aa627a186ebcd3
Signed-off-by: R. Diez <rdiez-2006@rd10.de>
Reviewed-on: https://review.openocd.org/c/openocd/+/9318
Tested-by: jenkins
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
@ahmed-haoues @borneoa
flash: nor: keep Windows backslashes in flash list output
eea1bb4 
Wrap the name field in braces when formatting the flash list Tcl structure
so that Tcl does not interpret backslash escape sequences.
This ensures Windows paths are preserved verbatim in the output and
can be reliably matched in Tcl scripts.

Fixes: 80fc9fa ("flash: nor: rewrite command 'flash list' as COMMAND_HANDLER")
Change-Id: Ibd0a4b292828f084122e4a02732dc69ac0885f3c
Signed-off-by: HAOUES Ahmed <ahmed.haoues@st.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9360
Tested-by: jenkins
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
@borneoa
doc: add STM32WBA as supported by flash driver 'stm32l4x'
b36de4f 
The flash driver 'stm32l4x' supports the device STM32WBA too.

Add it in the list of supported devices.

Change-Id: I5e2ba0c9ea198eb22d1bd329dd932bc0b1b3e886
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9392
Reviewed-by: Marc Schink <dev@zapb.de>
Tested-by: jenkins
Reviewed-by: Ahmed Haoues <ahmed.haoues@st.com>
@ahmed-haoues @borneoa
flash/stm32l4x: Fix flash size address for STM32WBA5
b9e4016 
Update fsize_addr parameter to reference non-secure memory address

Change-Id: I5098f478169bfaac726b96b071b99ca63d25d531
Signed-off-by: HAOUES Ahmed <ahmed.haoues@st.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9357
Tested-by: jenkins
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
@borneoa
server: fix double free() on add_service() early return
f275432 
The function add_service() can return on error when cannot create
the new service. In this case the caller cannot assume that the
parameter 'priv' has been taken in consideration and it should
take care of free() it.
To avoid a double free(), add_service() should not free() the
parameter 'priv' if it exits with error.

Replace the call to free_service() with a dedicated exit path on
error.

Change-Id: I340ec3ee46f471f31170c6717ed74fb632f0da20
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Reported-by: Karl Palsson <karlp@tweak.au>
Fixes: 171454f ("server: fix a new double free()")
Fixes: 5ff384b ("semihosting: fix memory leak and double free")
Reviewed-on: https://review.openocd.org/c/openocd/+/9372
Tested-by: jenkins
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
@borneoa
target: arm_tpiu_swo: fix memory leak on add_service() failure
70d4ac0 
When add_service() fails, the caller has to free() the private
data.

Add the missing free().

Change-Id: I998782ad3a2612dd289e269e889a3c6912fd8461
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9373
Tested-by: jenkins
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
@borneoa
server: gdb_server: check for out of memory and fix a memory leak
b10372f 
During GDB service start, check that memory is properly allocated
and if add_service() fails release the allocated memory.

While there, modify the code following the coding style.

Change-Id: Iebd1481a82f7391c110c5f6ad9878ba4abf052b3
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9374
Tested-by: jenkins
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
@borneoa
target: openrisc: jsp_server: check for out of memory, fix memory leak
00e4fb6 
During JSP service start, check that memory is properly allocated
and if add_service() fails release the allocated memory.

Change-Id: I7eb2ea8a10dbeccd27fd39485080552c5d0736d3
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9375
Tested-by: jenkins
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
@borneoa
server: close the TCP socket while stopping a service
5e71823 
The function free_service() does not close the TCL socket when the
service is shutdown.

Add the necessarily code.

Change-Id: I58c5c4fee3111949fcc143c6226522c29d976493
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9376
Tested-by: jenkins
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
@zapb-0 @borneoa
README: Use Markdown extension
bcfa2bc 
Ensures proper rendering on GitHub [1] / GitLab, clarifies that Markdown
syntax is expected from contributors, and enables editor syntax
highlighting.

[1] https://github.com/openocd-org/openocd

Change-Id: Icaff52ed2bf7d6f32b5812b5aff1c081e8b7507a
Signed-off-by: Marc Schink <dev@zapb.de>
Reviewed-on: https://review.openocd.org/c/openocd/+/9388
Tested-by: jenkins
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
@zapb-0 @borneoa
README.Windows: Use Markdown syntax
3ba83c7 
Refurbish the document and use only Markdown syntax according to the
specification in [1].

[1] https://www.markdownguide.org/

Change-Id: I7baa9a25621155ff4acabcfde724f233578c9ee1
Signed-off-by: Marc Schink <dev@zapb.de>
Reviewed-on: https://review.openocd.org/c/openocd/+/9390
Tested-by: jenkins
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
@zapb-0 @borneoa
README.macOS: Use Markdown syntax
c348b9f 
Refurbish the document and use only Markdown syntax according to the
specification in [1].

[1] https://www.markdownguide.org/

Change-Id: I7ef0d4b7359ba12fd6f2fbf2c4ea1d092dcd71bd
Signed-off-by: Marc Schink <dev@zapb.de>
Reviewed-on: https://review.openocd.org/c/openocd/+/9391
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
Tested-by: jenkins
@zapb-0 @borneoa
README: Rework installation section
7d6f3ed 
Document the OpenOCD installation with specific examples for package
managers and move the installation section to the beginning of the
document to make it more prominent.

Change-Id: I86e175fdd4060b49070a5c8d2b92ec1c6cfe6acf
Signed-off-by: Marc Schink <dev@zapb.de>
Reviewed-on: https://review.openocd.org/c/openocd/+/9395
Tested-by: jenkins
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
@zapb-0 @borneoa
README.Windows: Avoid multiple top-level headings
91799d0 
Refactor the document structure to avoid multiple top-level headings in
the same file.

The document is now free of Markdown linter [1] errors.

[1] https://github.com/jackdewinter/pymarkdown

Change-Id: I70122453c23c6230617c72d45986c88bb646edf5
Signed-off-by: Marc Schink <dev@zapb.de>
Reviewed-on: https://review.openocd.org/c/openocd/+/9402
Tested-by: jenkins
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
@zapb-0 @borneoa
README.macOS: Align structure and fix linter errors
8bc12d9 
Refactor the document structure to align with README.Windows and fix
two line-length issues so that the file is free of Markdown linter [1]
errors.

[1] https://github.com/jackdewinter/pymarkdown

Change-Id: Iab5ca09aeed2225e43934c8c2bd952a3a4b32cd0
Signed-off-by: Marc Schink <dev@zapb.de>
Reviewed-on: https://review.openocd.org/c/openocd/+/9403
Tested-by: jenkins
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
@zapb-0 @borneoa
README.md: Fix Markdown linter errors
62f49b7 
Adjust Markdown formatting to satisfy the Markdown linter [1], no content
changes are included.

[1] https://github.com/jackdewinter/pymarkdown

Change-Id: I28a2e8d12bfab08017de330ae3e81239082110ff
Signed-off-by: Marc Schink <dev@zapb.de>
Reviewed-on: https://review.openocd.org/c/openocd/+/9404
Tested-by: jenkins
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
R. Diez @borneoa
configure.ac: Detect mallinfo automatically
294538f 
Previously, configure.ac only checked whether glibc was available,
but other C runtime libraries like Newlib have mallinfo too.

This is a first step to remove configuration option --enable-malloc-logging
and replace it with a debug level configurable at runtime.

Change-Id: If30fc98a84158459e222fddf08043f46d6fa4112
Signed-off-by: R. Diez <rdiez-2006@rd10.de>
Reviewed-on: https://review.openocd.org/c/openocd/+/9394
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
Tested-by: jenkins
@borneoa
target: cortex-m: defer cache identification on Cortex-M85 under reset
95bd909 
Like for Cortex-M7, also Cortex-M85 prevents detecting the cache
properties when the CPU is kept under reset.

Extend to Cortex-M85 the same deferred mechanism already in place
for Cortex-M7.

Change-Id: Id274bb6c0b46c568554eed9671ef690c34cf7cfa
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9397
Tested-by: jenkins
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
Reviewed-by: Marc Schink <dev@zapb.de>
@borneoa
log: change line counter to unsigned
92b482a 
The incremental counter of logged lines is a signed int.

Change it to unsigned, since negative values has no sense in this
context.

Change-Id: Ia75f9ca038bba385c5f88fb9fa368faaf055f9a3
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9401
Reviewed-by: R. Diez <rdiez-2006@rd10.de>
Tested-by: jenkins
@grantramsay @borneoa
target: arm_dap: Fix crash in 'dap info' command
db34f6f 
The 'dap info' command was not checking that the target was
an ARM before dereferencing the `arm` pointer. This would
cause a crash if the current target was (say) a mem_ap.

Add 'target_to_dap' function to safely get the dap

Change-Id: I0c765f915f2ef7b9a7d20c934e144559ca4e5f1c
Signed-off-by: Grant Ramsay <grant.ramsay@hotmail.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/8415
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
Tested-by: jenkins
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
@gtortone @tom-van
flash/nor/spi: Add Boya-Micro BY25Q128AS and Byte-Semi BY25Q16ES flash
b4518ab 
- Boya-Micro BY25Q128AS: 16 MB
- Byte-Semi BY25Q16ES: 2MB

Change-Id: Ib3caee62abc34f92321425d9427054b6a9b6ec66
Signed-off-by: Gennaro Tortone <gtortone@gmail.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9412
Tested-by: jenkins
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
@jerome-pouiller @tom-van
meta: Add editor configuration
718eb22 
.editorconfig is supported out-of-the box by many editors (vim, vscode,
etc...). This file prevent new comers to do mistake with their editor
configuration.

Change-Id: I60c3dedc20161883f388af6f1cb9b95b90beff2a
Signed-off-by: Jérôme Pouiller <jerome.pouiller@silabs.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9454
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
Tested-by: jenkins
Mikhail Iakhiaev @tom-van
flash/nor/mspm0.c: organize parts
eaa0483 
1. Added comments like /* MSPM0G310x-Q1 */ to the parts table
based on the datasheet the parts came from.

2. Sorted the MSPM0G310x-Q1 device lines alphabetically: this is
how all other MSPM0G parts are organized.

Change-Id: I90d0f44289a538667c4ad0f23d44581f3c65d0c8
Signed-off-by: Mikhail Iakhiaev <iakhiaev@google.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9419
Tested-by: jenkins
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
Reviewed-by: Nishanth Menon <nm@ti.com>
Mikhail Iakhiaev @tom-van
flash/nor/mspm0.c: remove MSPM0G3505TDGS28R
0f67341 
Neither "MSPM0G3505TDGS28R" nor its variant number 0xdf is
listed in the MSPM0G3505 datasheet (likely removed):
Link: https://www.ti.com/lit/ds/symlink/mspm0g3505.pdf#page=75

Change-Id: I3828744a91477b3e3fd164055aa0fe068483f87d
Signed-off-by: Mikhail Iakhiaev <iakhiaev@google.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9420
Tested-by: jenkins
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
Reviewed-by: Nishanth Menon <nm@ti.com>
Mikhail Iakhiaev @tom-van
flash/nor/mspm0.c: add parts.
1967f85 
1. Added some missing variants for MSPM0G110x and MSPM0G150x parts.

2. Corrected the variant number for "MSPM0G1106TRHBR", according to
   datasheet:
Link: https://www.ti.com/lit/ds/symlink/mspm0g1106.pdf#page=68

3. Added MSPM0G350x-Q1 devices. Datasheet:
Link: https://www.ti.com/lit/ds/symlink/mspm0g3507-q1.pdf#page=77

Change-Id: Id4a14b284a5d032d140d695674d1912cbe87d7f6
Signed-off-by: Mikhail Iakhiaev <iakhiaev@google.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9417
Tested-by: jenkins
Reviewed-by: Nishanth Menon <nm@ti.com>
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
Mikhail Iakhiaev @tom-van
flash/nor/mspm0.c: support new parts
6eaf951 
Added support for the following parts:
* MSPM0Gx51x, datasheet:
Link: https://www.ti.com/lit/ds/symlink/mspm0g3518.pdf#page=101

* MSPM0G351x-Q1, datasheet:
Link: https://www.ti.com/lit/ds/symlink/mspm0g3518-q1.pdf#page=99

* MSPM0G3529-Q1, datasheet:
Link: https://www.ti.com/lit/ds/symlink/mspm0g3529-q1.pdf#page=92

* MSPM0G5187, datasheet:
Link: https://www.ti.com/lit/ds/symlink/mspm0g5187.pdf#page=87

Note, these parts have different "part_num" values than
the pre-existing MSPM0G parts listed in this file.

Tested by successfully programming (and verifying) an image
onto LP-MSPM0G3519 development board.

Change-Id: I15f3b368f5307661a32953074768ba810412cf0a
Signed-off-by: Mikhail Iakhiaev <iakhiaev@google.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9418
Tested-by: jenkins
Reviewed-by: Nishanth Menon <nm@ti.com>
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
Mikhail Iakhiaev @tom-van
tcl/target/ti/mspm0.cfg: drop swj on swd only target
078a6b1 
The MSPM0 devices support only SWD.
Drop swj support, add swd support.

This also gets rid of the following warnings:

Warn : DEPRECATED: auto-selecting transport "swd". Use 'transport ...
Warn : Transport "swd" was already selected

Tested by programming/verifying firmware on LP-MSPM0G3519 dev board.

Change-Id: Ieafd9c4691343124b2dfb2daa1c0d3a96b13e485
Signed-off-by: Mikhail Iakhiaev <iakhiaev@google.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9413
Tested-by: jenkins
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
@borneoa
flash: virtual: simplify error checking
5151c98 
Return directly the error code returned by last called function
instead of handling it.

Change-Id: I967deb348db535de1972056021484f0e7f32df1c
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9385
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
Tested-by: jenkins
Reviewed-by: Marc Schink <dev@zapb.de>
@borneoa
target: clean up return value of target_type::blank_check_memory()
3c1bd50 
The functions in struct target_type::blank_check_memory() return
either an OpenOCD error or a positive value that indicates the
number of blocks checked.

To prevent the mix of error codes and returned values, return the
value through an additional parameter 'checked' and then return
ERROR_OK.

While there:
- change to unsigned int the parameter 'num_blocks';
- in armv7m_blank_check_memory() verify that the working area can
  contain at least two 'algo_block'.

Change-Id: Ie22f5816819bc77ec611c3f251373d026ed9f784
Signed-off-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9386
Tested-by: jenkins
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
zapb-0 and others added 7 commits July 11, 2026 09:13
@zapb-0 @borneoa
contrib/firmware/angie: Add PREFIX to makebin command
cb52502 
Add the prefix to makebin to match other commands. This fixes building,
for example, on Fedora, which uses the 'sdcc' prefix for all SDCC commands.

Change-Id: I93705062c58d2b7821e37bf9fa1b46aaf8648d1c
Signed-off-by: Marc Schink <dev@zapb.de>
Reviewed-on: https://review.openocd.org/c/openocd/+/9777
Reviewed-by: Adrien Charruel <adrien.charruel@gmail.com>
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
Tested-by: jenkins
@EccoTheDolphin @tom-van
target/riscv: disable cache-on-read for VL and VSTART
da6c650 
When VTYPE is updated:

- VL is set to some appropriate value
- VSTART is set to zero

Change-Id: If67d0e4a1058d35f47b42c67abc2183836f34f22
Signed-off-by: Anatoly Parshintsev <kupokupokupopo@gmail.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9414
Tested-by: jenkins
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
@EccoTheDolphin @tom-van
target/riscv: do not cache access to vtype
7bb208c 
Writes to the vtype register are expected to have the side effect of
resetting vl and vstart. However, the current implementation caches the
vtype state on read, which prevents subsequent writes with
already-cached values from being issued.

Change-Id: I28beea6e2145bca52c5b37023399d6ae0937dfe0
Signed-off-by: Anatoly Parshintsev <kupokupokupopo@gmail.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9416
Tested-by: jenkins
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
@EccoTheDolphin @tom-van
target/riscv: do not cache access to vxrm, vxsat and vcsr
b63fc37 
vxrm and vxsat registers are mirrored in vcsr, so we can't cache any of
them.

Change-Id: I09760de5b76c330797caa99d2ef0ef151546e956
Signed-off-by: Anatoly Parshintsev <kupokupokupopo@gmail.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9422
Reviewed-by: Tomas Vanek <vanekt@fbl.cz>
Tested-by: jenkins
@fboyer-mchp @borneoa
tcl: add SAMA7G5 target and SAMA7G54-EK board configuration
af75d7f 
Add target configuration for the Microchip SAMA7G5 series
(Cortex-A7 based MPU). Supports both JTAG and SWD interfaces.

Add board configuration for the SAMA7G54-EK evaluation kit.

Link: https://www.microchip.com/en-us/product/sama7g54
Link: https://www.microchip.com/en-us/development-tool/EV21H18A

Change-Id: I94085babaf1b2f4ce92a44ed900128fc6093f397
Signed-off-by: Frederic Boyer <frederic.boyer@microchip.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9524
Tested-by: jenkins
Reviewed-by: zapb <dev@zapb.de>
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
@fboyer-mchp @borneoa
tcl: add SAMA7D6 target and SAMA7D65 Curiosity board configuration
b56339c 
Add target configuration for the Microchip SAMA7D6 series
(Cortex-A7 based MPU). Supports both JTAG and SWD interfaces.

Add board configuration for the SAMA7D65 Curiosity board.

Link: https://www.microchip.com/en-us/product/sama7d65
Link: https://www.microchip.com/en-us/development-tool/EV63J76A

Change-Id: Ieda7658167e87c6f011b4c267d2598b80754e067
Signed-off-by: Frederic Boyer <frederic.boyer@microchip.com>
Reviewed-on: https://review.openocd.org/c/openocd/+/9523
Tested-by: jenkins
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
Reviewed-by: zapb <dev@zapb.de>
@zapb-0 @borneoa
tcl/board/st: Add NUCLEO-U575ZI-Q config
fc566d7 
Tested on NUCLEO-U575ZI-Q development board.

Change-Id: I5870c44b3a7e5e48dc3d0a166734f116c32bfca3
Signed-off-by: Marc Schink <dev@zapb.de>
Reviewed-on: https://review.openocd.org/c/openocd/+/9749
Reviewed-by: Antonio Borneo <borneo.antonio@gmail.com>
Tested-by: jenkins
@en-sc en-sc changed the title Align with mainline at fc566d74005a4aefbe125b9b2f777d9514c60c87 Draft: Align with mainline at fc566d74005a4aefbe125b9b2f777d9514c60c87 Jul 29, 2026
en-sc added 17 commits July 30, 2026 12:05
@en-sc
Merge up to fc566d7 from upstream
f31e662 
Change-Id: I0b48208f937a0412bfae61f9fa3717418e730aa3
Signed-off-by: Evgeniy Naydanov <evgeniy.naydanov@syntacore.com>
@en-sc
git-hooks: remove the unnecessary files
bae58de 
If you need to setup the hooks, please consult the [OpenOCD docs] on how
to do so.

[OpenOCD docs]: https://review.openocd.org/Documentation/user-changeid.html
Change-Id: Ie7c4fbad11958a5abe40ec671393cd79041d8436

Change-Id: If5294a6a661ba1736f4fc744c181b1fee90c4547
Signed-off-by: Evgeniy Naydanov <evgeniy.naydanov@syntacore.com>
@en-sc
tools: remove filter_openocd_log.py
d407241 
The file is unused and is not tested.

Change-Id: Ia358b9c8952dad42c22409f297a4964b017271dd
Signed-off-by: Evgeniy Naydanov <evgeniy.naydanov@syntacore.com>
@en-sc
tcl: remove the Arty onboard FTDI files
9de290b 
These files can be found in [1].

1: https://review.openocd.org/c/openocd/+/9820
Change-Id: I6ce58db8d1a3ef1bf201c2b3a98339c31233233d
Signed-off-by: Evgeniy Naydanov <evgeniy.naydanov@syntacore.com>
@en-sc
tcl: board: remove sifive-e31arty-cjtag.cfg
972e340 
With the introduction of `oscan1_mode` the config is obsolite.
See [1].

1: https://review.openocd.org/c/openocd/+/6981
Change-Id: Ia1a69cb918f67227edf6a2ffcfb301fc7470d008
Signed-off-by: Evgeniy Naydanov <evgeniy.naydanov@syntacore.com>
@en-sc
tcl: board: sifive: align hifive1-rev-b.cfg with the mainline
eecd8d2 
The reverted change is on review [1].

1: https://review.openocd.org/c/openocd/+/9821
Change-Id: I9e7799cfaf3cc411675bad00b977f28949690881
Signed-off-by: Evgeniy Naydanov <evgeniy.naydanov@syntacore.com>
@en-sc
Revert "No Russian"
daf4e6b 
This reverts commit 6ba66c9.

The original changed lacked motivation.

Change-Id: Ia90edd00baec6c7f1923457482c5872500b7da0c
Signed-off-by: Evgeniy Naydanov <evgeniy.naydanov@syntacore.com>
@en-sc
Revert "Use LOG_DEBUG for debug messages: Discard unexpected char"
fcc4e7a 
This reverts commit c3a7441.

The change lacks motivation.

Change-Id: I24080ad6d46b5871ed992ced005f6ea9d3599470
Signed-off-by: Evgeniy Naydanov <evgeniy.naydanov@syntacore.com>
@en-sc
Revert "gdb_server: Keep working if gdb requests a non-existent reg"
6240a44 
This reverts commit 1c07a20.

The original commit seems like an ad-hoc fix for an issue that is not
present anymore.

Change-Id: If3ab19663a848ef40bd69aca1ef0e3c2dcf8ab9c
Signed-off-by: Evgeniy Naydanov <evgeniy.naydanov@syntacore.com>
@en-sc
Revert "Document how to get code coverage. (#339)"
3489ce9 
This reverts commit c3c76bf.

The described approach is outdated since
commit d4a64e3 ("autoconf: Add support
for code coverage")

Change-Id: If3b5b53aeb100c95b92d761125271d528bd131f7
Signed-off-by: Evgeniy Naydanov <evgeniy.naydanov@syntacore.com>
@en-sc
Revert "Fix Digilent JTAG-HS2 cJTAG configuration script"
61ef4c0 
This reverts commit 342ca4a.

The patch is a part of a mainline review [1].

Checkpatch-ignore: SPDX_LICENSE_TAG

1: https://review.openocd.org/c/openocd/+/9822
Change-Id: I3c5476ede8e7759282474f6a4381a1ee58b8775d
Signed-off-by: Evgeniy Naydanov <evgeniy.naydanov@syntacore.com>
@en-sc
Revert "Add Digilent JTAG-HS2 cJTAG configuration"
fcc0a73 
This reverts commit c96a24a.

The patch is a part of a mainline review [1].

1: https://review.openocd.org/c/openocd/+/9822
Change-Id: Id2dad341173b9aa739bc5aa7d1dbdcd80f0c07b3
Signed-off-by: Evgeniy Naydanov <evgeniy.naydanov@syntacore.com>
@en-sc
Revert "Make watchpoint.unique_id a uint32_t"
0c07bc5 
This reverts commit e8b0545.

The original patch is on mainline review [1].

1: https://review.openocd.org/c/openocd/+/9175
Change-Id: I4ec2ee8bad8662ea137684385475bad47723819c
Signed-off-by: Evgeniy Naydanov <evgeniy.naydanov@syntacore.com>
@en-sc
rtos: align with mainline OpenOCD
7544789 
The work on porting the FreeRTOS support on RISC-V from RISC-V fork can
be found at [1].

Checkpatch-ignore: SPACING UNNECESSARY_PARENTHESES BRACES

1: https://review.openocd.org/c/openocd/+/9757
Change-Id: I65fc4d083c27647adb8a357f508584d5b93da8e2
Signed-off-by: Evgeniy Naydanov <evgeniy.naydanov@syntacore.com>
@en-sc
Revert "Handler target_get_gdb_reg_list() better."
4e522fb 
This reverts commit f9af0b6.

The original change is an ad-hoc solution to an issue that is not
present anymore.

Change-Id: I1358ddec8ac6abc7586f5de080d9e8851bfee321
Signed-off-by: Evgeniy Naydanov <evgeniy.naydanov@syntacore.com>
@en-sc
server: gdb_server.c: [nfc] minor adjustments to align with mainline
45332d5 
The difference isn't worth the hustle of maintaining it.

Change-Id: I57602be7f995a469c5d9aa2e32c79a3e4bb141d3
Signed-off-by: Evgeniy Naydanov <evgeniy.naydanov@syntacore.com>
@en-sc
src: server: gdb_server: align with mainline
92e10ab 
The difference does not introduce any failures with the RISC-V tests
testsuite on Spike.

Since the changes do not seem like improvements, drop them.

Change-Id: I227b1096d1de1eb180d9d665fb3719caea1801a7
Signed-off-by: Evgeniy Naydanov <evgeniy.naydanov@syntacore.com>
@en-sc

en-sc commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator Author

Testing results:

  1. UnavailableHaltedTest fails with exceptions on all targets.
  2. FreeRtosTest fails with exceptions on all targets where applicable.
  3. DisconnectTest fails on spike32, spike32-2-hwthread, spike64-2, spike64-2-hwthread, passes on spike32-2, spike64.

en-sc added 2 commits July 30, 2026 18:24
@en-sc
"fix" the gdb_server
b22978f 
Change-Id: If0d30aaedb19d7c594a9f51dec37309138c816bd
@en-sc
Revert "src: server: gdb_server: align with mainline"
664aa26 
This reverts commit 92e10ab.
EccoTheDolphin
EccoTheDolphin reviewed Aug 5, 2026
View reviewed changes

@EccoTheDolphin EccoTheDolphin left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@en-sc could you also take a look at changes related to 41b5b54 , please? Looks like we may want to drop those.

Comment thread src/target/target.c Outdated
/**
* Reset the @c examined flag for the given target.
*/
static inline void target_reset_examined(struct target *target)

@EccoTheDolphin EccoTheDolphin Aug 5, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this duplicates definition on target.h

Comment thread src/target/target.c
&riscv_target,
&scr5_target,

@EccoTheDolphin EccoTheDolphin Aug 5, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

where is scr5 target?

Comment thread src/target/target.c
@@ -686,7 +698,6 @@ int target_examine_one(struct target *target)
return retval;
}

LOG_USER("[%s] Target successfully examined.", target_name(target));

@EccoTheDolphin EccoTheDolphin Aug 5, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you've mentioned that this breaks openocd_run.exp . Can we keep this for now or adjust openocd launcher?

Comment thread src/target/target.c
LOG_ERROR("Target not examined yet");
return ERROR_FAIL;
LOG_TARGET_ERROR(target, "not examined");
return ERROR_TARGET_NOT_EXAMINED;
}
if (!target->type->checksum_memory) {

@EccoTheDolphin EccoTheDolphin Aug 5, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this part resolved merge conflict incorrectly. this early check is not in upstream. Could you double-check, please?

Comment thread tcl/interface/ftdi/digilent-hs2-cjtag.cfg
@@ -1,17 +0,0 @@
adapter driver ftdi

@EccoTheDolphin EccoTheDolphin Aug 5, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is needed for Syntacore purposes.

@en-sc
en-sc force-pushed the en-sc/allign-with-mainline branch from 6cd28e8 to 664aa26 Compare August 5, 2026 10:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@EccoTheDolphin EccoTheDolphin EccoTheDolphin left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.