-
Notifications
You must be signed in to change notification settings - Fork 11
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
Many new features #134
Many new features #134
Conversation
… a message or config file.
Added function to pull the CS pin low for 500us. Added new sleeping state. Added function to put a ready radio to sleep. Added function to wake up a sleeping radio.
# Conflicts: # dw1000/Cargo.toml # dwm1001/Cargo.toml # dwm1001/src/lib.rs
Added better cs delay
…now been clamped.
# Conflicts: # dw1000/src/hl.rs
More stable setup for double buffering.
# Conflicts: # dw1000/src/configs.rs
…igs (auto CS). Removed all function calls that were put in to slow down SPI communication. Added function to read otp. LDOTUNE and LDELOAD now work without checking a bit. This could cause infinite loops if something went wrong with the chip.
Thank you, @diondokter! I'll try to take a look next week, but it might be a bit later (coming back from vacation next week, and there's a backlog to catch up on). |
Don't worry! This stuff is more than half a year old, so one or more weeks won't make a difference :) |
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.
Thank you again, @diondokter, these are awesome additions! I took a look and everything looks reasonable, although to be honest, I didn't look to closely at a lot of it. It's just too much to properly review, given the time I have available.
The only problem I found is that (at least some of) the examples aren't building (looks like CI didn't run for this PR, see #128). I'm looking into that right now.
|
Turns out the example were pretty trivial to fix. I pushed a commit that does it. I didn't test that they actually work, for lack of time. I'll do that later (need to do it anyway for #135). I'm going to merge this now. Having such a large pull request sit open for too long won't help anyone. Thanks again, @diondokter! |
Thank you as well! Learned a lot by working on this repo. This is a good driver that helped me quickly develop my system. Thanks! |
Closes #133
Note: Throughout the commits you may see some stuff about SPI delays and stuff. All of that was taken out in the end.
My SPI just wasn't set up properly, so I had some problems I could mask with setting some delays.