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

drivers/nRF24L01: Updated for ESP8266 with machine/SPI syntax #2703

Closed
wants to merge 1 commit into from

Conversation

mcauser
Copy link
Contributor

@mcauser mcauser commented Dec 19, 2016

Works between two of my WeMos D1 Minis, each running MicroPython v1.8.6-204-gde9cd00

img_8333

nrftop

WeMos D1 Mini nRF24L01+
D4 GPIO2 CE
D5 GPIO14 SCK
D6 GPIO12 MISO
D7 GPIO13 MOSI
D8 GPIO15 CSN
GND GND
3V3 VCC
N/C IRQ

Copy /drivers/nrf24l01/esp8266/*.py into /esp8266/modules/
cd esp8266
make clean && make axtls && make deploy

On slave device:

>>> import nrf24l01test
>>> nrf24l01test.slave()

On master device:

>>> import nrf24l01test
>>> nrf24l01test.master()

@mcauser mcauser changed the title drivers/nRF24L01: Updated machine/SPI syntax drivers/nRF24L01: Updated for ESP8266 with machine/SPI syntax Dec 20, 2016
@deshipu
Copy link
Contributor

deshipu commented Dec 29, 2016

See #2285 too.

@mcauser
Copy link
Contributor Author

mcauser commented Dec 29, 2016

And #2354. I should really check unmerged PRs first.

@mcauser
Copy link
Contributor Author

mcauser commented Jan 15, 2017

Closed in favour of almost identical changes #2285

@mcauser mcauser closed this Jan 15, 2017
@peterhinch
Copy link
Contributor

The current test/example program nrf24l01test.py uses the pyb module and hence is not portable. Would it not be worthwhile to replace it with this portable one?

@dpgeorge
Copy link
Member

dpgeorge commented Oct 3, 2017

Would it not be worthwhile to replace it with this portable one?

Yes that's a good idea, but the one here is not 100% portable because it relies on specific pins of the ESP8266.

In 54acd0b I made the test script use utime and machine modules to make it more portable. Feel free to submit a PR to make it run on both pyboard and esp8266.

@peterhinch
Copy link
Contributor

Please see #3366 for discussion of a portable solution.

@mcauser mcauser deleted the nrf24l01 branch July 10, 2019 02:25
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.

None yet

4 participants