-
-
Notifications
You must be signed in to change notification settings - Fork 892
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
Major RF24 library rework: minimal driver at full specs #392
Conversation
tekka007
commented
Feb 8, 2016
- strip C++ code
- 2-fold speed increase
- reduce code size ~2kb
- eliminate non-essential register polling/writing
- use 2 RX pipes
// Requires DEBUG to be enabled. | ||
// This will add even more to the size of the final sketch! | ||
//#define MY_DEBUG_VERBOSE | ||
|
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.
Why is this removed from MyConfig.h?
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.
Seems to be called MY_DEBUG_VERBOSE_RF24 now. A more specific define is fine by me. We should perhaps keep an updated comment in MyConfig so it's easier to find.
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.
I suggest adding a commented section, like MY_DEBUG_VERBOSE_SIGNING below. Perhaps even add doxygen tags to these so that the debug features gets documented. But that can be made in a separate commit. I encourage everyone to improve on the doxygen parts :)
d1360bc
to
5fcd3fd
Compare
0cf0052
to
6daf3d3
Compare
6daf3d3
to
ee1a27a
Compare
77ef71c
to
ab75bf9
Compare
Core tests successful. |
ab75bf9
to
b415c69
Compare
Core tests successful. |
Major RF24 library rework: minimal driver at full specs