-
Notifications
You must be signed in to change notification settings - Fork 2k
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/mrf24j40: add support for IEEE 802.15.4 Radio HAL #18472
Conversation
Maybe interesting for @Carton32? |
Thanks! I will have a look. |
I rebased this one on top of #18496 and the PRR of the SubMAC seems to boost even more:
|
c7f099d
to
0fe14d2
Compare
75a4253
to
d65a4c4
Compare
rebased to latest master |
d65a4c4
to
63bb3b0
Compare
Please squash |
3db5f2b
to
fc0e59c
Compare
it passes now with the whitelist |
bors merge |
👎 Rejected by too few approved reviews |
bors merge |
bors cancel bors merge |
Canceled. |
Build succeeded: |
thanks for the review! |
I wonder why bors merged something that failed in master... |
Contribution description
Since #18383 already got merged, it's time to add the IEEE 802.15.4 Radio HAL port for MRF24J40 radios.
This one was supported by LWIP, which requires #18465 to be merged first.
This PR also adds the Arduino Pin config for nucleo64 boards and fixes some tiny issues in the CCA functions of the Radio HAL and tests.
Strictly speaking this PR changes the API of
mrf24j40_init
, but IMO adding a deprecation at this level does not make sense.It also removes the old
netdev
implementation.Testing procedure
Connect a MRF24J40 to the Arduino Pins, try
tests/ieee802154_*
tests, the LWIP tests andexamples/gnrc_networking
. Everything should work as before.Issues/PRs references
Depends on #18465