Skip to content

Conversation

@maribu
Copy link
Member

@maribu maribu commented Nov 11, 2025

Contribution description

This provides the SVD file name for all SAM0 boards and most STM32 boards.

Testing procedure

make BOARD=$board USE_PYCORTEXMDEBUG=1 -C examples/basic/hello-world flash debug

should show something like

Loading SVD file STMicro/STM32L1xx...

Tested on:

  • same54-xpro
  • nucleo-l152re
  • nucleo-g474re
  • nucleo-l452re
  • nucleo-g070rb
  • nucleo-f303re
  • nucleo-l073rz
  • nucleo-f030r8
  • nucleo-f103rb
  • nucleo-f072rb
  • nucleo-f767zi
  • nucleo-f429zi
  • nucleo-f207zg

Issues/PRs references

Follow up of #21863

With this, running `make USE_PYCORTEXMDEBUG=1 debug` for any SAM0 based
board will directly load the correct SVD file.
With this, running `make USE_PYCORTEXMDEBUG=1 debug` for almost any STM32
based board will directly load the correct SVD file.
@github-actions github-actions bot added Platform: ARM Platform: This PR/issue effects ARM-based platforms Area: build system Area: Build system Area: cpu Area: CPU/MCU ports labels Nov 11, 2025
@crasbe crasbe added Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Nov 11, 2025
Copy link
Contributor

@crasbe crasbe left a comment

Choose a reason for hiding this comment

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

The changes look sane (well, the STM32 SVD naming seems to be a bit of a clusterfuck, but that's not your fault) and I trust your testing 👍

@crasbe crasbe enabled auto-merge November 11, 2025 15:35
@riot-ci
Copy link

riot-ci commented Nov 11, 2025

Murdock results

✔️ PASSED

9a5a8e3 cpu/stm32: Provide SVD_VENDOR and SVD_MODEL

Success Failures Total Runtime
10931 0 10931 09m:53s

Artifacts

@crasbe crasbe added this pull request to the merge queue Nov 11, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to no response for status checks Nov 12, 2025
@maribu maribu added this pull request to the merge queue Nov 12, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Nov 12, 2025
@maribu maribu added this pull request to the merge queue Nov 12, 2025
Merged via the queue into RIOT-OS:master with commit 8007a1f Nov 12, 2025
28 checks passed
@maribu maribu deleted the dist/tools/PyCortexMDebug branch November 12, 2025 12:42
@maribu
Copy link
Member Author

maribu commented Nov 12, 2025

Thx :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area: build system Area: Build system Area: cpu Area: CPU/MCU ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Platform: ARM Platform: This PR/issue effects ARM-based platforms Type: enhancement The issue suggests enhanceable parts / The PR enhances parts of the codebase / documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants