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

fix cyccnt extension based on cortex-m change #8

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

TDHolmes
Copy link
Owner

Closes #3. Need to wait on rust-embedded/cortex-m#367

@TDHolmes TDHolmes force-pushed the tholmes/fix-extended-cyccnt branch 2 times, most recently from 9636a91 to 26c2fed Compare December 29, 2021 21:12
@TDHolmes TDHolmes marked this pull request as ready for review December 29, 2021 21:14
@TDHolmes
Copy link
Owner Author

Blocked by rust-embedded/cortex-m#368

@TDHolmes TDHolmes force-pushed the tholmes/fix-extended-cyccnt branch 3 times, most recently from b2adadb to b1cbeed Compare January 5, 2022 23:46
@TDHolmes TDHolmes force-pushed the tholmes/fix-extended-cyccnt branch from b1cbeed to 2aedaf2 Compare July 2, 2022 02:59
TDHolmes added 2 commits July 2, 2022 10:44
fire just before overflow instead and don't reset cyccnt

latest cortex-m changes

make `new` falible

remove debug code
@TDHolmes TDHolmes force-pushed the tholmes/fix-extended-cyccnt branch from e4d2c56 to 60d49f7 Compare July 2, 2022 15:45
@TDHolmes
Copy link
Owner Author

TDHolmes commented Oct 8, 2022

Also needs rust-embedded/cortex-m#377

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.

ep-dwt's extended feature currently doesn't work
1 participant