Skip to content
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

Update RF24registers.h #1180

Closed
wants to merge 1 commit into from

Conversation

Projects
None yet
4 participants
@mfalkvidd

This comment has been minimized.

Copy link
Contributor

commented Aug 24, 2018

Could the problem be realted to https://forum.mysensors.org/topic/9642/nrf24-transmission-of-data-works-fine-but-constant-nack-s-produced/ so the fix suggested there is a better solution?

@Berkseo

This comment has been minimized.

Copy link
Author

commented Aug 24, 2018

Increasing the number of attempts is aimed at correcting this line - 2912 !TSF:MSG:SEND,100-100-0-0,s=255,c=3,t=26,pt=1,l=1,sg=0,ft=0,st=NACK:2.

This does not apply to NOACK in the first message after waking up

As I understand in this the frost is offered to exclude delay in 10ms(RF24.cpp). I think it's the wrong decision. The specification says it explicitly - at least 10 MS (>=10). For the original radio modules, a 10ms delay is probably sufficient to output the signal to the output. For most radio modules, this is clearly not enough.

@@ -41,7 +41,7 @@
#define RF24_CRC_16 (3)

// ARD, auto retry delay
#define RF24_SET_ARD (5) //=1500us
#define RF24_SET_ARD (10) //=1500us

This comment has been minimized.

Copy link
@mfalkvidd

mfalkvidd Aug 24, 2018

Contributor

Is the comment (1500us) still valid after this change?

@Berkseo

This comment has been minimized.

Copy link
Author

commented Aug 24, 2018

no, I'm sorry, I just did not remove the comment, of course there will be more time.

@Yveaux

This comment has been minimized.

Copy link
Member

commented Aug 26, 2018

This solution is likely to conflict with #1134.

@tekka007

This comment has been minimized.

Copy link
Contributor

commented Aug 30, 2018

see #1193

@tekka007 tekka007 closed this Aug 30, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.