Skip to content

Switch back to MonoTimer from HAL #357

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

Closed
wants to merge 1 commit into from

Conversation

sirhcel
Copy link
Member

@sirhcel sirhcel commented Jun 22, 2021

According to stm32-rs/stm32f3xx-hal#247 it got removed accidentially and has been resurrected. There is no need for the local copy from here.

@rust-highfive
Copy link

r? @therealprof

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-resources labels Jun 22, 2021
This switches back from the local replacement for MonoTimer from 814c051
to the one re-added with HAL release 0.8.0. MonoTimers interface
changed in the meantime so this commit takes this into account as well.
@sirhcel sirhcel force-pushed the monotimer-from-hal branch from 7767eb7 to dc1c8a5 Compare October 2, 2021 22:00
@sirhcel
Copy link
Member Author

sirhcel commented Oct 2, 2021

A fixed HAL got released with 0.8.0 meanwhile. So rubberduck203/stm32f3-discovery#44 is on the way for updating the BSP.

@BartMassey
Copy link
Member

Not sure after all this time of the relevance? Would probably need to be updated for HAL 1.x

@sirhcel
Copy link
Member Author

sirhcel commented May 24, 2024

You are absolutely right. This draft PR has been dry-aging and with rust-embedded/wg#759 on the horizon this looks like too much effort for restoring nostalgia.

@sirhcel sirhcel closed this May 24, 2024
@BartMassey
Copy link
Member

Sorry we never got to this. We'll set up a maintenance plan to prioritize fixes on the current Discovery book going forward.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-resources
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants