-
Notifications
You must be signed in to change notification settings - Fork 2.1k
drivers/sx127x: add functions required for loramac #8261
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
drivers/sx127x: add functions required for loramac #8261
Conversation
a16ce81
to
e9374e5
Compare
Maybe @dylad you can have a look at this one ? |
Sure ! I already assign myself :) |
Thanks ! I missed that. |
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.
Few issues within comments otherwise looks good.
I'll ACK once fix.
drivers/include/sx127x.h
Outdated
/** | ||
* @brief Checks that channel is free with specified RSSI threshold. | ||
* | ||
* @param[in] dev The sx1276 device structure pointer |
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.
sx127x would be better (same for the function beneath)
drivers/include/sx127x.h
Outdated
* | ||
* @param[in] dev The sx1276 device structure pointer | ||
* @param[in] freq channel RF frequency | ||
* @param[in] rssi_thresh RSSI treshold |
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.
threshold ("missing h")
e9374e5
to
b543d0b
Compare
Done and directly squashed |
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.
ACK I'm fine with the changes !
Contribution description
This PR adds a missing function in the SX127x driver that is required for LoRaMAC. Plus it adds a missing definition for a function already present in the driver.
There's also a small cleanup in one debug output.
Issues/PRs references
refs #7331