Wire pins for Adafruit Feather RP2350 with HSTX

[greiman]

I think there is a problem with the way Wire pins are defined in the variant file for this board.

Here is the Wire section of the variant file:

// Wire
#define PIN_WIRE0_SDA  (2u)
#define PIN_WIRE0_SCL  (3u)
#define PIN_WIRE1_SDA  (31u) // not pinned out
#define PIN_WIRE1_SCL  (31u)

Since __WIRE0_DEVICE is not defined in the variant file, it defaults to i2c0 here. Pins 2 and 3 don't seem to be valid for i2c0.

Probably Wire1 should be i2c1 like other Feather RP2040 boards. Pins 24 and 25 would be an OK choice for Wire1 but Wire1 is not pinned out in many RP2040 variants.

Here is the section for the basic Feather RP2040:

// Wire
#define __WIRE0_DEVICE i2c1
#define PIN_WIRE0_SDA  (2u)
#define PIN_WIRE0_SCL  (3u)
#define __WIRE1_DEVICE i2c0
#define PIN_WIRE1_SDA  (24u)
#define PIN_WIRE1_SCL  (25u)

[greiman]

I added the two line to the variant file like this:

#define __WIRE0_DEVICE    i2c1  // <<< add
#define PIN_WIRE0_SDA  (2u)
#define PIN_WIRE0_SCL  (3u)
#define __WIRE1_DEVICE    i2c0  // <<< add
#define PIN_WIRE1_SDA  (31u) // not pinned out
#define PIN_WIRE1_SCL  (31u) 

Now the RTC on my Feather Adalogger works and my Feather OLED works with no mods.

I can use Wire1 if I setSDA() and setSCL().

[earlephilhower]

Got it, will update. Thx!

[greiman]

Thanks again for this great board package for RP2040/RP2350. It has become my favorite. I use it for Pico, Feather, and Metro format boards.

Metro is nice since I can use some old Uno style stuff plus Metro 2040/2350 has a SDIO compatible SD socket.

[earlephilhower]

And thank you for keeping SdFat updated and running across so many boards. Just keeping my libraries working across ESP32 and Pico is hard enough. I can't imagine the grief keeping the dozens of architectures you support up and running!

[ladyada]

thanks to both of you, sorry i must have gotten this messed up and it wasnt caught! 😭