SSD1306: Disambiguate Wire.begin for [email protected]

With [email protected] we can't pass the first argument to TwoWire::begin as an unsigned int; it will resolve the four-parameter overload and initialize the bus in slave mode.

We resolve the three-parameter overload for bus master mode by explicitly typing the first and third arguments, the third replacing the call to TwoWire::setClock.

Setting the frequency in TwoWire::begin has the same effect as ::setClock:

arduino-esp32@<2.0.1: i2cInit (esp32-hal-i2c.c) and setClock (Wire.cpp) both call i2cSetFrequency (esp32-hal-i2c.c)

[email protected]: esp-idf does the same clock selection in i2c_param_config and i2c_get_clk_src (driver/i2c.c) that arduino-esp32 does in i2cSetClock (esp32-hal-i2c.c).

Author: MURPHYENGINEERING

src/oled/SSD1306Wire.h

@@ -55,16 +55,16 @@ class SSD1306Wire : public OLEDDisplay {
 
 
     bool connect() {
-    		pinMode(_rst,OUTPUT);
-    		digitalWrite(_rst, LOW);
-    		delay(50);
-    		digitalWrite(_rst, HIGH);
-
-		Wire.begin(this->_sda, this->_scl);
-		// Let's use ~700khz if ESP8266 is in 160Mhz mode
-		// this will be limited to ~400khz if the ESP8266 in 80Mhz mode.
-		Wire.setClock(700000);
-		return true;
+			pinMode(_rst,OUTPUT);
+			digitalWrite(_rst, LOW);
+			delay(50);
+			digitalWrite(_rst, HIGH);
+
+			// Let's use ~700khz if ESP8266 is in 160Mhz mode
+			// this will be limited to ~400khz if the ESP8266 in 80Mhz mode.
+			Wire.begin(int(this->_sda), this->_scl, 700000U);
+		
+			return true;
     }
 
     void display(void) {