Add pico-de-gallo battery example #676
Conversation
Cargo Vet Audit Passed
|
tullom
force-pushed
the
pico-de-gallo-examples
branch
from
24a1cf5 to
4a630c5
Compare
tullom
force-pushed
the
pico-de-gallo-examples
branch
from
08cc2a3 to
b4db5a7
Compare
tullom
force-pushed
the
pico-de-gallo-examples
branch
from
7f0f011 to
c925d15
Compare
|
|
||
| /// Platform specific battery controller. | ||
| struct Battery { | ||
| pub driver: Bq40z50R5<pico_de_gallo_hal::I2c, pico_de_gallo_hal::Delay>, |
There was a problem hiding this comment.
Can't you use embedded_hal::i2c::I2c and embedded_hal::delay::DelayNs instead?
There was a problem hiding this comment.
I could make this struct generic on those traits but I consider this app level code, so the types should be resolved at this level. We don't want to construct multiple instances of Battery with differing types in this example.
tullom
force-pushed
the
pico-de-gallo-examples
branch
from
c925d15 to
6ef312a
Compare
tullom
requested review from
RobertZ2011,
asasine,
felipebalbi,
jeffglaum,
jerrysxie,
kurtjd and
williampMSFT
Yeah i tried without the unsafe impl but it is still needed. Not sure why because all its fields are Send but ¯\_(ツ)_/¯ |
Not to beat a dead horse but I think actually the reason is because you take a reference. So for |
tullom
force-pushed
the
pico-de-gallo-examples
branch
from
daba71f to
e2e9b0a
Compare
Nah no dead horse, the horse is still alive and well. Reading more into |
| #[cfg_attr(feature = "defmt", derive(defmt::Format))] | ||
| pub struct Data<'a> { | ||
| contents: &'a dyn Any, | ||
| contents: &'a (dyn Any + Send + Sync), |
There was a problem hiding this comment.
This is a breaking change, could you make sure to announce it?
There was a problem hiding this comment.
Why is this necessary now?
| } | ||
| } | ||
|
|
||
| #[tokio::main] |
There was a problem hiding this comment.
We should add comment or a README to describe the setup required to run this example.
5 participants
Add support for examples with the pico-de-gallo development board.
This PR sets up a new workspace for pico-de-gallo examples and adds an example that runs the battery service with the bq40z50-r5 fuel gauge connected to the pico-de-gallo's I2C bus.