fix(PeriphDrivers): Issue #1293: MAX32670 I2C Driver causes a Double-Read

[dnguye14-adi]

Description

Add more while-loop to wait until the I2C bus is not busy. They fix the double-read issue as well as the issue of hang due to quick transaction.

[Brandon-Hurst]

Hello, as I mentioned in the issue #1293, I recommend changing the first while loop to an if statement. This will prevent the possibility of infinite loops if the I2C peripheral is busy / hanging. Returning an error indicating the I2C controller is busy is sufficient in this case, and a while loop should not be needed. I tested this as well and left screenshots in the issue linked.

I would propose changing this:

while(i2c->status & MXC_F_I2C_REVA_STATUS_BUSY) {}
// Wait until last transaction finished and bus ready

to this:

// If there is an active transaction, return a BUSY error
if (i2c->status & MXC_F_I2C_REVA_STATUS_BUSY) {
    return E_BUSY;
}