-
Notifications
You must be signed in to change notification settings - Fork 3k
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
Nanostack 802.15.4 RF drivers update #10185
Conversation
df8b60f
to
e283db6
Compare
@artokin, thank you for your changes. |
I'm starting to wonder whether the Driver header file should produce "#error Unsupported platform" or something similar. Now this flagging just leads to very weird build errors on platforms that cannot use the driver. However, I'm not proposing changes to this PR. Just stating the fact that this if-deffing is not very elegant solution. |
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.
Link is failing...
Ex:
MCR20Drv.c is compiled and needs RF_IRQ_Disable() from a file that is not compiled any more...
Sync Nanostack RF drivers from master copy: -MCR20A: v1.0.6 -S2LP: v0.0.5
e283db6
to
0710298
Compare
@jeromecoutant , MCR20Drv.c is now flagged, would you please try building again? |
CI started |
Test run: SUCCESSSummary: 13 of 13 test jobs passed |
…_fix Nanostack 802.15.4 RF drivers update
Description
Sync Nanostack RF drivers from master copy:
-MCR20A: v1.0.5
A Fix to DEVICE_INTERRUPTIN flagging made in #10142
Pull request type
Reviewers
@JarkkoPaso