-
Notifications
You must be signed in to change notification settings - Fork 12
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.
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
Adds I2C implementation and basic accelerometer crate. #30
Conversation
|
@Dr-Emann Agreed; I'll swap that for this release. |
@@ -69,7 +71,7 @@ extern crate alloc_system; | |||
|
|||
* Build the project (in release mode to optimize the size of the binary) | |||
``` | |||
cargo build --release | |||
cargo build --release --target=mipsel-unknown-linux-gnu |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you add this just for clarity? I don't think I needed this but maybe my setup was just different.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, mine required it. Guess it doesn't hurt to be specific when cross-compiling.
I'm not experienced enough to comment on that API design as-is but I'm curious about why the |
@johnnyman727 I think we did something like this when we started with T1; basically we had all the code in one repo to help evolve the API all at once, and broke out the modules once the API settled down. I expect that to happen here. Also, whoops, did not notice the Rust version of the MMA84 code. Probably past the point of using it now though :( |
This commits the I2C interface and an accelerometer crate that can read acceleration values.
Note that API design questions should be surfaced on #29. Our goal is to get our interfaces working and tested asap, but design the ergonomics more carefully before 1.0.