Comments (5)
What is missing in this API?
Possibly methods to choose the capture "edge". Should this capture the rising edge or the falling edge? This is useful in generic context to capture on "high" signal: you want to capture the rising edge and then the falling edge.
Another question: is a enum Edge { Rising, Falling }
enough, or should Edge
be an associated type of the Capture
trait? Are there microcontrollers that have a "capture both edges" setting?
from embedded-hal.
Are there microcontrollers that have a "capture both edges" setting?
Yep, I think most STM32F3/4/7 and maybe 0 have options for rising/falling/both (along with a whole heap of other options...). See CCER register.
from embedded-hal.
OK. The F103 only has two options: rising and falling. The F303 has three: rising, falling and both. I would prefer if each device crate didn't end up inventing their own Edge
enum so perhaps we can have the most common Edge
enums in this crate. At least Edge2 { Rising, Falling }
and Edge3 { Rising, Falling, Both }
. Then we have to make the edge related API generic around the Edge
enum. So maybe:
trait CaptureOn<Edge> {
fn capture_on(_: Edge);
}
then generic APIs can use a bound like C: Capture + CaptureOn<Edge2>
when they need to capture the rising and falling edges in their implementation.
from embedded-hal.
Update: this trait is available in release v0.1.0 behind the "unproven" Cargo feature.
from embedded-hal.
Hello,
This is my first time contributing here, I don't know if I'm doing it the right way.
With this API, I think it would be difficult to write drivers.
Some drivers only need a capture on the rising edge, which enum should they use? Edge2? Edge3?
Maybe a better way to handle this would be to use multiple traits :
trait CaptureRising {
fn capture();
}
trait CaptureFalling {
fn capture();
}
trait CaptureBoth: CaptureRising + CaptureFalling {
fn capture();
}
And it will be used like this:
struct MyDevice<C: CaptureRising> {
capture: C
}
impl<C: CaptureRising> MyDevice {
fn something(&self) {
self.capture.capture();
}
}
With this, the driver will be compatible with all devices which implement CaptureRising
.
from embedded-hal.
Related Issues (20)
- i2c: Merging of consecutive operations in transaction contract
- SpiDevice's interface can't be used for streaming transactions HOT 6
- SpiDevice implementations in embedded-hal-bus don't provide a way to use active-high chip select HOT 3
- Create an I3C Trait
- Read not implemented for &mut [u8] HOT 1
- README: Links to LICENSE-APACHE and LICENSE-MIT are not found
- How do I share an I2c bus between tasks? HOT 1
- Implment `Clone` for async DelayNs HOT 2
- Multipin serial spi interface trait
- embedded-hal-bus no longer builds with target thumbv6m-none-eabi HOT 5
- Split embedded-can? HOT 3
- feature request: embedded_io_async::try_read HOT 2
- Upstreaming some concepts built up in `embedded-time` HOT 1
- Why was ADC removed? HOT 2
- [Discussion] Should sensor driver developers target `embedded-hal-async` only?
- Shared SPI bus between devices with different SPI modes HOT 2
- The Crate Does not Compile when the `defmt-03` Feature is Enabled HOT 4
- honor `ErrorKind::Interrupted` in `write_all()`? HOT 4
- Revisiting the possibility of including generic sensor traits HOT 1
- Generic bus traits HOT 4
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from embedded-hal.