-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Add W55RP20-EVB-Pico, W6300-EVB-Pico2 Board #10610
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
base: main
Are you sure you want to change the base?
Add W55RP20-EVB-Pico, W6300-EVB-Pico2 Board #10610
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for the PR!
Why do you need a native PIO_SPI object? It seems you are only using this from the Python driver. The PIOASM library has an example of implementing a busio-like UART via PIO all from Python here: https://github.com/adafruit/Adafruit_CircuitPython_PIOASM/blob/main/examples/pioasm_rxuart.py
Want to split out the board definitions from the wiznet
module addition? I'm happy to take the new board defs as we sort out the PIO stuff.
Sorry for the late reply, and thank you for the feedback.
So while I agree that a Python-only solution is possible in principle, for this use case I found that a native PIO_SPI provides the performance and stability needed. |
The W6300 will need library changes for the extra pins to take advantage of the quad SPI? |
Did you try the Python-only version first? I'm worried this is a lot of specific code for us to maintain. Having it in a library of yours using rp2pio directly means you can maintain it easier. |
Add support for W55RP20-EVB-Pico and WIZNET_PIO_SPI communication (#10440).
Additionally, add support for the W6300-EVB-Pico2 board with Quad SPI functionality.
Tested with example code and libraries below.