search

maniacbug / RF24

Arduino driver for nRF24L01
http://maniacbug.github.com/RF24
949 stars 770 forks source link

Wrong value of ARD in SETUP_RETR #48

Open dbolotin opened 10 years ago

dbolotin commented 10 years ago

https://github.com/maniacbug/RF24/blob/master/RF24.cpp#L356

According to nRF24L01 documentation B0100 in ARD in SETUP_RETR register stands for 1250uS delay (see page 54 of documentation with registers description) but comment above states that this value should be 1500uS (minimal universal value for the delay).

So the correct value here should be B0101. :)

PeterCuriosity commented 10 years ago

Hi, We have received your e-mail and will get back to you within 24h, should you be having an emergency pls call me or sms your number to 082 22 11 00 5.

Peter Marynowski Cell: 082 22 11 00 5 Web: www.killarneynet.co.za Snr. Technician Killarney Community Internet KIC

ghost commented 10 years ago

I ran into the same problem today, and can confirm the issue. The value should be B0101 to work with 32B payload and 250Kbps. Best regards ;)

http://blog.surserv.at/nrf24l01_wireless_modules/

PeterCuriosity commented 10 years ago

Hi, We have received your e-mail and will get back to you within 24h, should you be having an emergency pls call me or sms your number to 082 22 11 00 5.

Peter Marynowski Cell: 082 22 11 00 5 Web: www.killarneynet.co.za Snr. Technician Killarney Community Internet KIC

dbolotin commented 10 years ago

)) It's a pity that this repository is not maintained (there are many pull-requests waiting for approval), instead I've made a pull-request with this fix.