Add SparkFun ThingPlus RP2350 #2038
Conversation
|
LGTM, but I guess this might need a line to satisfy the upcoming #2034 ? ping @will-v-pi |
Yeah, that’d be good, so I don’t have to rebase #2034 on top of this to add it |
|
Done! |
|
|
||
| // gpio pin for spi data out to the cyw43 chip | ||
| #ifndef CYW43_DEFAULT_PIN_WL_DATA_OUT | ||
| #define CYW43_DEFAULT_PIN_WL_DATA_OUT 24u |
There was a problem hiding this comment.
I'm just looking at this as part of the update for CYW43 support in arduino-pico on the RP2350 boards, so maybe I'm missing something, but are data_in, data_out, and host_wake all on pin 24?
There was a problem hiding this comment.
Yep! Theoretically any GPIO pins can be used (see here: #1938 (comment)), but Raspberry Pi managed to cleverly connect all 3 pins of the radio to a single GPIO pin on the host processor. For reference, see the Pico W datasheet (schematic on page 23). This board uses the same connections with the same resistors to avoid contention.
There was a problem hiding this comment.
https://github.com/raspberrypi/pico-sdk/blob/develop/src/rp2_common/pico_cyw43_driver/cyw43_bus_pio_spi.c is where the software-side magic works.
There was a problem hiding this comment.
Nice, thanks for the explanation!
Upcoming board from us!