Skip to content

Conversation

aabadie
Copy link
Contributor

@aabadie aabadie commented Dec 13, 2017

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

@aabadie aabadie added Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation Area: drivers Area: Device drivers Area: LoRa Area: LoRa radio support CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Dec 13, 2017
@dylad dylad self-assigned this Dec 14, 2017
@aabadie aabadie force-pushed the pr/drivers_sx17X_missing_functions branch 3 times, most recently from a16ce81 to e9374e5 Compare December 14, 2017 09:03
@aabadie
Copy link
Contributor Author

aabadie commented Dec 14, 2017

Maybe @dylad you can have a look at this one ?

@dylad
Copy link
Member

dylad commented Dec 14, 2017

Sure ! I already assign myself :)
I'm quite busy today, maybe tonight.

@aabadie
Copy link
Contributor Author

aabadie commented Dec 14, 2017

I already assign myself :)

Thanks ! I missed that.

@aabadie aabadie mentioned this pull request Dec 14, 2017
Copy link
Member

@dylad dylad left a 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.

/**
* @brief Checks that channel is free with specified RSSI threshold.
*
* @param[in] dev The sx1276 device structure pointer
Copy link
Member

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)

*
* @param[in] dev The sx1276 device structure pointer
* @param[in] freq channel RF frequency
* @param[in] rssi_thresh RSSI treshold
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

threshold ("missing h")

@aabadie aabadie force-pushed the pr/drivers_sx17X_missing_functions branch from e9374e5 to b543d0b Compare December 14, 2017 21:13
@aabadie
Copy link
Contributor Author

aabadie commented Dec 14, 2017

Done and directly squashed

Copy link
Member

@dylad dylad left a 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 !

@dylad dylad merged commit 7fe2d9b into RIOT-OS:master Dec 14, 2017
@aabadie aabadie added this to the Release 2018.01 milestone Jan 18, 2018
@aabadie aabadie deleted the pr/drivers_sx17X_missing_functions branch June 8, 2018 20:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: drivers Area: Device drivers Area: LoRa Area: LoRa radio support CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Type: cleanup The issue proposes a clean-up / The PR cleans-up parts of the codebase / documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants