Fix i2c pin Glitch when Assigning Pins to I2C hardware

This pr fixes the signal glitches that occur when the SDA and SCL pins are attached to the I2C peripheral.  The I2C bus is an open drain with external pullups.  The initial device driver software set the pin drivers to OUTPUT_OPEN_DRAIN. This caused a low going transient that existed until the I2C peripheral gained ownership of the pin.  When this pulse occurred on the SDA pin it caused an illegal bus transaction that intermittently hung the bus on init. (PAINFULL).

Author: stickbreaker

cores/esp32/esp32-hal-i2c.c

@@ -175,7 +175,8 @@ i2c_err_t i2cAttachSCL(i2c_t * i2c, int8_t scl)
     if(i2c == NULL){
         return I2C_ERROR_DEV;
     }
-    pinMode(scl, OUTPUT_OPEN_DRAIN | PULLUP);
+    digitalWrite(scl, HIGH); 
+    pinMode(scl, OPEN_DRAIN | PULLUP);
     pinMatrixOutAttach(scl, I2C_SCL_IDX(i2c->num), false, false);
     pinMatrixInAttach(scl, I2C_SCL_IDX(i2c->num), false);
     return I2C_ERROR_OK;
@@ -197,7 +198,8 @@ i2c_err_t i2cAttachSDA(i2c_t * i2c, int8_t sda)
     if(i2c == NULL){
         return I2C_ERROR_DEV;
     }
-    pinMode(sda, OUTPUT_OPEN_DRAIN | PULLUP);
+    digitalWrite(sda, HIGH); 
+    pinMode(sda, OPEN_DRAIN | PULLUP);
     pinMatrixOutAttach(sda, I2C_SDA_IDX(i2c->num), false, false);
     pinMatrixInAttach(sda, I2C_SDA_IDX(i2c->num), false);
     return I2C_ERROR_OK;