Skip to content
This repository was archived by the owner on Mar 7, 2026. It is now read-only.

Added support for AT32F403A/407/415.#1151

Merged
dragonmux merged 1 commit intoblackmagic-debug:mainfrom
grumat:AT32F403A_407_415
Jul 31, 2022
Merged

Added support for AT32F403A/407/415.#1151
dragonmux merged 1 commit intoblackmagic-debug:mainfrom
grumat:AT32F403A_407_415

Conversation

@grumat
Copy link
Contributor

@grumat grumat commented Jul 30, 2022

The following PR adds support for ArteryTek ATF32F403A/407/415 family of MCUs based on Cortex M4 and peripherals compatible to STM32F103 family.
Some of the elements of this family does support an external Flash feature, which would require customized flash routines. The current solution will impose flash limit to 256KB, which are accessible using the STM32F103 compatible interface.
The PR was tested with the WeActStudio Blackpill based on AT32F403ACGU. English datasheet for these parts can be obtained on the ArteryTek website.
A simple Blink Led program was used to test using VisualGDB and Visual Studio 2022. The GDB interaction log is on the attachment: stm32_02-Debug.log

@dragonmux dragonmux added this to the v1.9 release milestone Jul 30, 2022
@dragonmux dragonmux added the New Target New debug target label Jul 30, 2022
@grumat
Copy link
Contributor Author

grumat commented Jul 30, 2022

Sorry @dragonmux for the delay. I've got real trouble with the git rebase command and found it was easier to delete my fork and restart everything from the ground. I don't want to start a rant about that tool...

@dragonmux
Copy link
Member

We see this PR replaces #1150 so you could fix the branch used and get things into a suitable state.

Regarding git rebase - if you hop on the 1bitsquared Discord we'd be happy to talk you through how it works with examples if that might help you for the future.

dragonmux
dragonmux previously approved these changes Jul 31, 2022
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. We'll update the t->cpuid component of this code as we think #1146 will have broken this. (On account of wanting to get this merged ASAP and ease the pain for you, we'll get on and adjust it to merge.)

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.

Merging. Thank you for the contribution!

@dragonmux dragonmux merged commit df513eb into blackmagic-debug:main Jul 31, 2022
@ALTracer ALTracer mentioned this pull request Jul 17, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

New Target New debug target

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants