machine/circuitplay: correct pin and pin mode mapping for both i2c0 (external) and i2c1 (internal) buses

Signed-off-by: Ron Evans <ron@hybridgroup.com>

src/machine/board_circuitplay_express.go

@@ -76,8 +76,16 @@ const (
 
 // I2C on the Circuit Playground Express.
 var (
-	I2C0 = I2C{Bus: sam.SERCOM5_I2CM} // external device
-	I2C1 = I2C{Bus: sam.SERCOM1_I2CM} // internal device
+	// external device
+	I2C0 = I2C{Bus: sam.SERCOM5_I2CM,
+		SDA:     SDA_PIN,
+		SCL:     SCL_PIN,
+		PinMode: GPIO_SERCOM}
+	// internal device
+	I2C1 = I2C{Bus: sam.SERCOM1_I2CM,
+		SDA:     SDA1_PIN,
+		SCL:     SCL1_PIN,
+		PinMode: GPIO_SERCOM_ALT}
 )
 
 // SPI pins (internal flash)

src/machine/machine_atsamd21.go

@@ -572,7 +572,10 @@ func handleUART1() {
 
 // I2C on the SAMD21.
 type I2C struct {
-	Bus *sam.SERCOM_I2CM_Type
+	Bus     *sam.SERCOM_I2CM_Type
+	SCL     uint8
+	SDA     uint8
+	PinMode GPIOMode
 }
 
 // I2CConfig is used to store config info for I2C.
@@ -608,7 +611,7 @@ func (i2c I2C) Configure(config I2CConfig) {
 		config.Frequency = TWI_FREQ_100KHZ
 	}
 
-	// reset SERCOM3
+	// reset SERCOM
 	i2c.Bus.CTRLA |= sam.SERCOM_I2CM_CTRLA_SWRST
 	for (i2c.Bus.CTRLA&sam.SERCOM_I2CM_CTRLA_SWRST) > 0 ||
 		(i2c.Bus.SYNCBUSY&sam.SERCOM_I2CM_SYNCBUSY_SWRST) > 0 {
@@ -632,8 +635,8 @@ func (i2c I2C) Configure(config I2CConfig) {
 	}
 
 	// enable pins
-	GPIO{SDA_PIN}.Configure(GPIOConfig{Mode: GPIO_SERCOM})
-	GPIO{SCL_PIN}.Configure(GPIOConfig{Mode: GPIO_SERCOM})
+	GPIO{i2c.SDA}.Configure(GPIOConfig{Mode: i2c.PinMode})
+	GPIO{i2c.SCL}.Configure(GPIOConfig{Mode: i2c.PinMode})
 }
 
 // SetBaudRate sets the communication speed for the I2C.