-
Notifications
You must be signed in to change notification settings - Fork 90
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
Add LR1110 library for WM1110 LoRa radio #490
Conversation
We had to modify some library files to get the radio to work.
We provide a modified version for Tock
850fbb6
to
fcdfb5b
Compare
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.
This looks good to me. Lots of high-quality, self-contained stuff
Can you clarify the relationship between this and #458? Does this PR replace that PR? (It looks to me like it does) |
Yes this replaces it with a cleaned up commit history. |
Ok the CI passes! |
This is a followup to #458 with the commits cleaned up.
We've verified that with tock/tock#4320 this is working and we can join TTN with the board and send sensor readings.
We have three patches to core files in the library that we found we needed when trying to get this working. We don't have the bandwidth to investigate those further right now, and while there has been some activity to the upstream repo, there haven't been any source changes in two years.
This isn't a perfect port, but it is self contained and we want to build on this.