Skip to content

Conversation

@desertsagebrush
Copy link
Contributor

Detailed description

Adds in ID codes for the STM32C0[5,7,9]x microcontroller lines. Note that this doesn't distinguish yet between the C091 and C092 lines despite them having different amounts of SRAM (36kiB vs 30kiB), as there doesn't seem to be an easy way to check that.

Tested on a STM32C071 Nucleo board.

Your checklist for this pull request

@dragonmux dragonmux added this to the v2.1 release milestone Nov 11, 2025
@dragonmux dragonmux added the New Target New debug target label Nov 11, 2025
Copy link
Member

@dragonmux dragonmux left a comment

Choose a reason for hiding this comment

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

Only got the one review note, so other than the CI failures, once that's addressed we'll be happy to approve and merge this.

Note that this doesn't distinguish yet between the C091 and C092 lines,
despite them having different amounts of SRAM.
Copy link
Member

@dragonmux dragonmux left a comment

Choose a reason for hiding this comment

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

LGTM, merging. Thank you for the contribution!

@dragonmux dragonmux merged commit e1505ef into blackmagic-debug:main Nov 11, 2025
38 of 39 checks passed
@JohnAZoidberg
Copy link

Cool, thanks!

But how can I get a working build with this?
CI native firmware builds fail now :(

@JohnAZoidberg
Copy link

JohnAZoidberg commented Nov 28, 2025

I managed to cherry-pick this changed on top of v2.0.0, resolve the conflicts and locally build it: https://github.com/JohnAZoidberg/blackmagic/tree/stm32c071
But even there CI doesn't build.

I built it locally and it can work with blackmagic hardware and Nucleo STM32C071.

@dragonmux
Copy link
Member

dragonmux commented Nov 28, 2025

We're aware of the CI failures, and part of that is down to a .yml file for the config having not yet been updated till #2167, and the other part is because of an upstream fault on the compilers - ARM reorganised their CDN and broke the checksums on all downloads, thus the GHA block we use to get the compilers is having to do updates.

For what it's worth, you didn't need to git cherry-pick at all - just grab main and build that. It has the STM32C0 additions and a pile of extra fixes which you'll benefit from. If it doesn't fit, turn off a target family like renesas to reduce the build size.

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

Labels

New Target New debug target

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants