makefiles/tools/openocd-adapters: let OpenOCD auto-select transport#21871
Merged
crasbe merged 1 commit intoRIOT-OS:masterfrom Nov 11, 2025
Merged
makefiles/tools/openocd-adapters: let OpenOCD auto-select transport#21871crasbe merged 1 commit intoRIOT-OS:masterfrom
crasbe merged 1 commit intoRIOT-OS:masterfrom
Conversation
maribu
approved these changes
Nov 11, 2025
Member
|
This looks exactly like a downstream fix I need to use on Alpine when using the |
Contributor
Author
|
Test Traces: OpenOCD v0.12.0 upstream with Nucleo-C071RB: OpenOCD v0.11.0 Release with Nucleo-F302R8: OpenOCD v0.12.0 Release with Nucleo-F302R8: OpenOCD v0.12.0 Upstream with Nucleo-F302R8: OpenOCD v0.11.0 with Nucleo-WL55JC (this has an ST-Link V3 on board) WITHOUT fix from #21615 applied: OpenOCD v0.12.0 release with Nucleo-WL55JC WITH fix from #21615 applied: OpenOCD v0.12.0 upstream with Nucleo-WL55JC (with fix from #21615 applied): |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Contribution description
As described in #21781, different OpenOCD versions require different transport protocols for the ST-Link and there is no common ground except for "let OpenOCD decide".
Therefore we do exactly that and let OpenOCD decide.
This PR also removes the forced prefixing, because perhaps we want to select
swdand nothla_swdand the current code does not allow it.Testing procedure
Flash an STM32 based board with OpenOCD 0.11, 0.12 Release and 0.12 Upstream, preferrably one based on an ST-Link V2 and ST-Link V3, also one with a new(ish) microcontroller such as the STM32C0.
Insert Test Traces here
Issues/PRs references
Fixes #21781.
Partly reverts #14480.