search

maniacbug / RF24

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

Problems on setting Power level #7

Open questuk opened 12 years ago

questuk commented 12 years ago

Hi maniacbug,

I have set the nRF24L01+ module power output using this code: radio.setPALevel(RF24_PA_HIGH);

When I then check this using radio.printDetails(); I get Power = LA_MED

Also I believe we are using these mnemonics? "According to 6.5 of the nRF24L01+ specification sheet, they translate to: RF24_PA_MIN=-18dBm, RF24_PA_LOW=-12dBm, RF24_PA_MED=-6dBM, and RF24_PA_HIGH=0dBm."

RF24_PA_MED will not compile as get this message = error: 'RF24_PA_MED' was not declared in this scope. All other mnemonics work.

If I use RF24_PA_MIN ,RF24_PA_LOW, RF24_PA_HIGH, RF24_PA_MAX these all work.

Regards

Gary

hayden-t commented 10 years ago

if you look at line 24 of RF24.h you will see that 'RF24_PA_MED' is not in the list of option, but I imagine you could add it... but also i think the setPALevel function will need changing too

hayden-t commented 10 years ago

can you fix this maniac ?

gcopeland commented 10 years ago

This is fixed/corrected in my fork. Has been for a long time. On Dec 4, 2013 3:39 PM, "hayden-t" notifications@github.com wrote:

can you fix this maniac ?

— Reply to this email directly or view it on GitHubhttps://github.com/maniacbug/RF24/issues/7#issuecomment-29848304 .

hayden-t commented 10 years ago

thanks for pointing this out, i dont think i would have found out otherwise...

hayden-t commented 10 years ago

hey gcoepland, i just lookeh at your fork, and i see talk of multicast which is something i was interested in adding too. where can i find out more on this ? you mention in commit comments documentation but your doc link just go back maniacs ? it would be good if you had more detail say a list of what you added somewhere and description of some bits. that one reason i wouldnt have found out you had done this, as there are a few forks of RF24 and its hard to get an overview of what each one is about or has added/changed...

gcopeland commented 10 years ago

For now, comments are the docs. It's what the online documentation is built from.

Also check out my three blog posts on using these radios.

Http:\Maniacalbits.blogspot.com

hey gcoepland, i just lookeh at your fork, and i see talk of multicast which is something i was interested in adding too. where can i find out more on this ? you mention in commit comments documentation but your doc link just go back maniacs ? it would be good if you had more detail say a list of what you added somewhere and description of some bits. that one reason i wouldnt have found out you had done this, as there are a few forks of RF24 and its hard to get an overview of what each one is about or has added/changed...

— Reply to this email directly or view it on GitHubhttps://github.com/maniacbug/RF24/issues/7#issuecomment-30047169 .

gcopeland commented 10 years ago

Some info is also here: http://maniacalbits.blogspot.com/2013/04/new-rf24-driver-release-fork.html On Dec 6, 2013 10:06 PM, "hayden-t" notifications@github.com wrote:

hey gcoepland, i just lookeh at your fork, and i see talk of multicast which is something i was interested in adding too. where can i find out more on this ? you mention in commit comments documentation but your doc link just go back maniacs ? it would be good if you had more detail say a list of what you added somewhere and description of some bits. that one reason i wouldnt have found out you had done this, as there are a few forks of RF24 and its hard to get an overview of what each one is about or has added/changed...

— Reply to this email directly or view it on GitHubhttps://github.com/maniacbug/RF24/issues/7#issuecomment-30047169 .

hayden-t commented 10 years ago

thanks for this, it is what is was looking for