-
Notifications
You must be signed in to change notification settings - Fork 64
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
Add StatefulOutputPin and ToggleableOutputPin #25
Conversation
NOTE: I shamelessly stole the implementation from https://github.com/stm32-rs/stm32f1xx-hal. |
This looks great. I'll be looking to get this in once we test out and merge #21 since it has some hefty changes to |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was able to test blinking successfully using an example here: dfrankland/proton-c@8b02d2f
Building all device features was successful as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Another small change
d5a0f86
to
136e126
Compare
Applied change requests. Ready for review again :) |
Nice work 🎉 |
I tested it on the STM32F303 / STM32DISCOVERY Board, with the method provided in the example.