search

adafruit / circuitpython

CircuitPython - a Python implementation for teaching coding with microcontrollers
https://circuitpython.org
Other
4.04k stars 1.19k forks source link

Metro RP2350 I2C pins swapped #9519

Open todbot opened 1 month ago

todbot commented 1 month ago

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:

Screenshot 2024-08-13 at 1 00 37 PM

Additional information

No response

tannewt commented 1 month ago

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