Nanostack 802.15.4 RF drivers update #10142
Conversation
Sync Nanostack RF drivers from master copies: -Atmel: v3.0.7 -MCR20A: v1.0.3 -S2LP: v0.0.4
|
@artokin, thank you for your changes. |
| This driver is used with 6LoWPAN and Wi-SUN stacks. | ||
|
|
||
| Driver is tested with X-Nucleo-S2868A1 RF expansion board. https://www.st.com/en/ecosystems/x-nucleo-s2868a1.html |
There was a problem hiding this comment.
Could be turned into a link for readability - X-NUCLEO-S2868A1
There was a problem hiding this comment.
OK, we will keep that in mind when updating the readme next time.
| Driver is tested with X-Nucleo-S2868A1 RF expansion board. https://www.st.com/en/ecosystems/x-nucleo-s2868a1.html | ||
|
|
||
| NOTE: Default SPI SCLK pin configuration is D13. On X-Nucleo-S2868A1, resistor R6 must be soldered instead of R11. For more information, see user manual UM2405. https://www.st.com/resource/en/user_manual/dm00498153.pdf |
There was a problem hiding this comment.
Could be turned into a link for readability - UM2405 User manual
There was a problem hiding this comment.
OK, we will keep that in mind when updating the readme next time.
|
@artokin Making sure, this can come into 5.12.1 instead of 5.12.0-rc3, correct? CC @ARMmbed/mbed-os-maintainers |
|
CI started |
Test run: FAILEDSummary: 1 of 13 test jobs failed Failed test jobs:
|
|
@cmonr , 5.12.1 is fine for this update. |
|
CI job restarted: Failures don't immediately appear to be related to PR. |
|
CI Has passed. |
|
Looks good to me. |
| @@ -15,7 +15,7 @@ | |||
| */ | |||
| #include "NanostackRfPhyMcr20a.h" | |||
There was a problem hiding this comment.
Hi
This include should be 3 lines below... after the #if check
There was a problem hiding this comment.
Making a note.
Fixed in #10185
Description
Sync Nanostack RF drivers from master copies:
-Atmel: v3.0.7
-MCR20A: v1.0.3
-S2LP: v0.0.4
Pull request type
Reviewers
@JarkkoPaso