Skip to content

[Time] Review delay #477

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Mar 26, 2019
Merged

[Time] Review delay #477

merged 5 commits into from
Mar 26, 2019

Conversation

fpistm
Copy link
Member

@fpistm fpistm commented Mar 22, 2019 •
edited
Loading

  • Introduce DWT for cortex supporting it (cortex-m3/m4/m7).
  • Use DWT cycle counter to have delayMicroseconds not dependent to interrupt (example with OneWire).
  • Fix rollover for delayMicroseconds with cortex-m0 (no DWT)

Still to provide a delayMicroseconds not dependent to interrupt for cortex-m0 to allow for example OneWire usage with MCU based on this cortex.

Fix #176

fpistm added 5 commits March 22, 2019 09:33
@fpistm
[DWT] Add API
8650f6a 
Signed-off-by: Frederic.Pillon <[email protected]>
@fpistm
[DWT] Init at startup
fdc5780 
Signed-off-by: Frederic Pillon <[email protected]>
@fpistm
[Time] Use DWT cycle counter for delayMicroseconds
94d896d 
Signed-off-by: Frederic.Pillon <[email protected]>
@fpistm
[Time] Code format
f6afa67 
Signed-off-by: Frederic.Pillon <[email protected]>
@fpistm
[Time] Fix rollover
28bf0a5 
Signed-off-by: Frederic.Pillon <[email protected]>
@fpistm fpistm added bug 🐛 Something isn't working enhancement New feature or request fix 🩹 Bug fix labels Mar 22, 2019
@fpistm fpistm added this to the 1.5.1/1.6.0 milestone Mar 22, 2019
@fpistm fpistm self-assigned this Mar 22, 2019
@fpistm fpistm mentioned this pull request Mar 23, 2019
Closed
@fpistm fpistm merged commit 1b7da56 into stm32duino:master Mar 26, 2019
@fpistm fpistm deleted the delay branch March 26, 2019 10:10
benwaffle pushed a commit to benwaffle/Arduino_Core_STM32 that referenced this pull request Apr 10, 2019
@fpistm
Merge pull request stm32duino#477 from fpistm/delay
a08c8d7 
[Time] Review delay
@hierophect hierophect mentioned this pull request Nov 12, 2019
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@fpistm fpistm

Labels
bug 🐛 Something isn't working enhancement New feature or request fix 🩹 Bug fix
Projects
None yet
Milestone
1.6.0
Development

Successfully merging this pull request may close these issues.
1 participant
@fpistm