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.

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

ITM: configure lacks API for stimulus registers #393

Open
tmplt opened this issue Jan 14, 2022 · 0 comments
Open

ITM: configure lacks API for stimulus registers #393

tmplt opened this issue Jan 14, 2022 · 0 comments
Labels
enhancement

Comments

@tmplt
Copy link
Contributor

tmplt commented Jan 14, 2022

I propose something akin to:

itm.configure(ITMConfiguration {
    // ...
    stim: Stimulus::EnableAll
}
itm.configure(ITMConfiguration {
    // ...
    stim: Stimulus::EnableSome([0..5, 8..31]),
}

Better review iprint and iprintln and see if they make any assumptions about the registers.
@tmplt tmplt mentioned this issue Jan 15, 2022
Open
tmplt added a commit to rtic-scope/cortex-m that referenced this issue Jan 15, 2022
@tmplt
itm: make ITMConfiguration non-exhaustive
e18b52d 
At least one more error enum will be added in the future. See rust-embedded#393.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@tmplt @newAM