Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Port test suite to STM32F3DISCOVERY #5

Merged
merged 3 commits into from
Mar 2, 2024

Conversation

helixbass
Copy link
Contributor

This PR adds an stm32f3/ version of the testsuite

Specifically I ran it on the STM32F3DISCOVERY (stm32f303) board

For context, I'm "just learning" embedded stuff, specifically this is the board used by some of the Rust Embedded "books"

The test suite looks like it is passing, here is the exact output

@@ -0,0 +1,9 @@
[target.'cfg(all(target_arch = "arm", target_os = "none"))']
Copy link
Contributor Author

Choose a reason for hiding this comment

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

All of these files are copied from testsuite/stm32f4/ and then tweaked

@@ -0,0 +1,6 @@
MEMORY
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I got this from here

Though above there they say "(There's another RAM region but for simplicity we'll ignore it)", so this may be incomplete?

@@ -0,0 +1,2 @@
source [find interface/stlink.cfg]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'd been running openocd for this board probably per the Rust Embedded books using:

$ openocd -f interface/stlink-v2-1.cfg -f target/stm32f3x.cfg

but this gives:

WARNING: interface/stlink-v2-1.cfg is deprecated, please switch to interface/stlink.cfg

so per that warning updated to interface/stlink.cfg instead


IMG=target/thumbv7m-none-eabi/debug/lilos-testsuite-stm32f3

# F3 has enough flash to build in debug mode
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Apparently this was true?

@cbiffle
Copy link
Owner

cbiffle commented Mar 2, 2024

Thanks! I've rebased this and pushed a couple of changes (in the intervening ~8 months I've renamed the .cargo/config files because the toolchain started warning about the old name, and added MSRV metadata) but otherwise this looks good. Merging shortly.

@cbiffle cbiffle merged commit 29bfcf0 into cbiffle:main Mar 2, 2024
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants