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

Cannot read from progmem array directly. #5

Closed
wants to merge 1 commit into from

Conversation

basilfx
Copy link

@basilfx basilfx commented Feb 25, 2012

After playing with RF24Network meshping example, I noticed that I could not use addresses which involves pipes 2-5. I dug into the NRF24L01+ datasheet and the RF24 library and found out that EN_RXADDR was always set to 0x03 (pipe 0 and 1, default) after initializing, no matter which pipe you tried to open for reading.

Then I looked at openReadingPipe() in RF24.cpp and noticed that _BV(child_pipe_enable[child]) always returned 1, no matter the value of child. I fixed the problem and now the RF24Network meshping example works for me (which fixes maniacbug/RF24Network#3, I guess).

… from an array entry.


This fixes the problem that only pipe 0 and 1 could be opened (because child_pipe_enable[child] was always 0)
@maniacbug
Copy link
Owner

Aaah.. Thanks! I've been meaning to look into this problem. Will pull this as soon as I get a few free minutes to test it.

@maniacbug
Copy link
Owner

Fixed in 2c97186. Thanks again!

@maniacbug maniacbug closed this Feb 27, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Strange behaviour
2 participants