search

nRF24 / RF24

OSI Layer 2 driver for nRF24L01 on Arduino & Raspberry Pi/Linux Devices
https://nrf24.github.io/RF24
GNU General Public License v2.0
2.2k stars 1.01k forks source link

Constant `RF_PWR` looks to be wrongly defined #482

Closed ruslanbathenway closed 3 years ago

ruslanbathenway commented 5 years ago

Constant RF_PWR looks to be wrongly defined.

https://github.com/nRF24/RF24/blob/a3ab7ddb075360deac54fe2c8e2fab286793d21c/nRF24L01.h#L79

The data sheet claims the RF_PWR bit location is 2 (page 59) image

2bndy5 commented 3 years ago

its not wrong. don't forget about the obsolete bit in position 0. binary 110 = decimal 6 making 0 dBm the default setting. Not to mention the latest release takes into account that the SI24R1 uses that bit in position 0. @TMRh20 I think this issue should be closed.