Metro RP2350 I2C pins swapped

[todbot]

CircuitPython version

n/a, don't have the board yet

Code/REPL

import board; i2c = board.I2C()

Behavior

What I expect one would see:

>>> import board; i2c = board.I2C()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ValueError: Invalid pins

Description

In https://github.com/adafruit/circuitpython/blob/main/ports/raspberrypi/boards/adafruit_metro_rp2350/mpconfigboard.h, it has:

#define DEFAULT_I2C_BUS_SCL (&pin_GPIO24)
#define DEFAULT_I2C_BUS_SDA (&pin_GPIO25)

yet on Section 1.2.3, page 19 of the RP2350 datasheet PDF, it lists GPIO24 as I2C0 SDA and GPIO25 as I2C0 SCL:

Additional information

No response

[tannewt]

We're revising the metro anyway for the 80-pin chip so we'll fix this then.

[dhalbert]

Board was revised to use Rp2350B, so this is moot now.