Add new example of ADC with RTIC using SysTick as monotonic #575
Conversation
boards/feather_m0/Cargo.toml
Outdated
|
|
||
| [dependencies.systick-monotonic] | ||
| version = "1.0.0" | ||
| optional = true |
There was a problem hiding this comment.
As this is a dependency only in an example, it should go in dev-dependencies
There was a problem hiding this comment.
Same with rtt stuff I think
There was a problem hiding this comment.
@TDHolmes thank you for your suggestions. I applied them to the PR.
ceigel
force-pushed
the
master
branch
2 times, most recently
from
eea2208
to
7a32ff6
Compare
| #[local] | ||
| struct Local { | ||
| adc: Adc<ADC>, | ||
| adc_pin: Pin<pin::PA07, Alternate<B>>, |
There was a problem hiding this comment.
It might be nice to add a pin alias to the bsp_pins! macro for this case.
| let max_range = 1 << RESOLUTION_BITS; | ||
| let voltage = ((data as f32) * 3.3 * 2.0) / (max_range as f32); | ||
| rprintln!("Battery level {} - {}", voltage, data); | ||
| // If the LED were a local resource, the lock would not be necessary |
There was a problem hiding this comment.
What is this comment referring to? Is it a copy/paste artifact?
|
Pinging @ceigel. Did you want to pick this back up? An RTIC example for the |
|
@bradleyharden sorry for the delay. I addressed the comments. I haven't tested it on hardware with the new changes. I will do that in the following days. |
|
@bradleyharden I tested this on hardware. The PR is ready to be merged. |
Summary
[describe your changes here]
Checklist
CHANGELOG.mdfor the BSP or HAL updatedIf Adding a new Board
crates.json