Fix: I2C Scan on ESP32-S2

[brentru]

This pull request in conjunction with adafruit/ci-arduino#210 should fix the broken I2C scanning present on ESP32-S2 platforms running WipperSnapper Beta 96 or Beta 97.

This pull request also:

• Refactors the loop within scanAddresses to match Espressif's I2C Scan Test
  • A little more performant as we check the best-case first, now.
• Adds handling for I2C transmission error codes (only for ESP32 I2C Scans)

Tested on Adafruit Feather ESP32-S2 w/BME280

Requires adafruit/ci-arduino#210 to be merged in and this PR to be rebuilt before testing again

Resolves - #700

[brentru]

Noting that this needs to be re-built and re-tested after adafruit/ci-arduino#210 is merged.

We will want to cut a new release for Beta 98 after this.

[brentru]

@tyeth Tested again on ESP32-S2 Feather. Please give it a test and ✅ when done!

[brentru]

[ 55535][V][esp32-hal-periman.c:160] perimanSetPinBus(): Pin 4 successfully set to type I2C_MASTER_SCL (34) with bus 0x1
[i2c]: Scanning I2C Bus for Devices...
[i2c] Scanning Address: 0x1
[i2c] Did not find device: NACK on transmit of address!
[i2c] Scanning Address: 0x2
[i2c] Did not find device: NACK on transmit of address!
[i2c] Scanning Address: 0x3

[brentru]

Noting that we removed a call to WIRE's setClock(50000). This removes the timeout error introduced in BSP 3.1.x by putting the I2C clock's default clock speed back to the expected 100kHz.