Fix I2C pullups on Nano33BLE sensors

Author: facchinm

libraries/SPI/SPI.cpp

@@ -59,6 +59,7 @@ void arduino::MbedSPI::end() {
 
 #if DEVICE_SPI > 0
 arduino::MbedSPI SPI(SPI_MISO, SPI_MOSI, SPI_SCK);
-#elif DEVICE_SPI > 1
+#endif
+#if DEVICE_SPI > 1
 arduino::MbedSPI SPI1(SPI_MISO1, SPI_MOSI1, SPI_SCK1);
 #endif

libraries/SPI/SPI.h

@@ -38,6 +38,7 @@ class MbedSPI : public SPIClass
 
 #if DEVICE_SPI > 0
 extern arduino::MbedSPI SPI;
-#elif DEVICE_SPI > 1
+#endif
+#if DEVICE_SPI > 1
 extern arduino::MbedSPI SPI1;
 #endif

libraries/Wire/Wire.cpp

@@ -104,6 +104,7 @@ void arduino::MbedI2C::onRequest(void(*)(void)) {}
 
 #if DEVICE_I2C > 0
 arduino::MbedI2C Wire(I2C_SDA, I2C_SCL);
-#elif DEVICE_I2C > 1
+#endif
+#if DEVICE_I2C > 1
 arduino::MbedI2C Wire1(I2C_SDA1, I2C_SCL1);;
 #endif

libraries/Wire/Wire.h

@@ -68,9 +68,9 @@ class MbedI2C : public HardwareI2C
 
 #if DEVICE_I2C > 0
 extern arduino::MbedI2C Wire;
+#endif
 #if DEVICE_I2C > 1
 extern arduino::MbedI2C Wire1;
 #endif
-#endif
 
 typedef arduino::MbedI2C TwoWire;

variants/ARDUINO_NANO33BLE/defines.txt

@@ -11,7 +11,7 @@
 -D__CORTEX_M4
 -DDEVICE_ANALOGIN=1
 -DDEVICE_FLASH=1
--DDEVICE_I2C=1
+-DDEVICE_I2C=2
 -DDEVICE_I2C_ASYNCH=1
 -DDEVICE_INTERRUPTIN=1
 -DDEVICE_ITM=1

variants/ARDUINO_NANO33BLE/pins_arduino.h

@@ -78,6 +78,9 @@ static const uint8_t SCK  = PIN_SPI_SCK;
 #define PIN_WIRE_SDA1       (30u)
 #define PIN_WIRE_SCL1       (31u)
 
+#define PIN_ENABLE_SENSORS_3V3     (32u)
+#define PIN_ENABLE_I2C_PULLUP      (33u)
+
 #define PIN_INT_APDS (26u)
 
 // PDM Interfaces

variants/ARDUINO_NANO33BLE/variant.cpp

@@ -66,6 +66,14 @@ void initVariant() {
   // was being enabled by nrfx_clock_anomaly_132
   CoreDebug->DEMCR = 0;
   NRF_CLOCK->TRACECONFIG = 0;
+
+  // FIXME: always enable I2C pullup and power @startup
+  // Change for maximum powersave
+  pinMode(PIN_ENABLE_SENSORS_3V3, OUTPUT);
+  pinMode(PIN_ENABLE_I2C_PULLUP, OUTPUT);
+
+  digitalWrite(PIN_ENABLE_SENSORS_3V3, HIGH);
+  digitalWrite(PIN_ENABLE_I2C_PULLUP, HIGH);
 }
 
 #ifdef SERIAL_CDC