-
Notifications
You must be signed in to change notification settings - Fork 3
Open
Description
See conversation in #4 about the return type for the Accel self-test:
Instead of returning an option we should return an error type defined by this driver:
Result<(f32, f32, f32), Driver::Error>. Having a result already present an invalid case of returning an error instead of a valid value. Adding an option on top of that just seems unnecessary
We can map any internal i2c error to a Driver::Error and just use the ? operator to pass the error up.
Please create an issue to follow up on this, thanks!
Originally posted by @jerrysxie in #4 (comment)
Metadata
Metadata
Assignees
Labels
No labels
Type
Projects
Status
Backlog