Merge pull request #750 from nekomona/count-patch

Fix count overflow on readBytes/readWords

Author: jrowberg

Arduino/I2Cdev/I2Cdev.cpp

@@ -217,7 +217,7 @@ int8_t I2Cdev::readBytes(uint8_t devAddr, uint8_t regAddr, uint8_t length, uint8
         Serial.print("...");
     #endif
 
-    int8_t count = 0;
+    uint8_t count = 0;
     uint32_t t1 = millis();
 
     #if (I2CDEV_IMPLEMENTATION == I2CDEV_ARDUINO_WIRE || I2CDEV_IMPLEMENTATION == I2CDEV_BUILTIN_SBWIRE || I2CDEV_IMPLEMENTATION == I2CDEV_TEENSY_3X_WIRE)
@@ -329,7 +329,7 @@ int8_t I2Cdev::readWords(uint8_t devAddr, uint8_t regAddr, uint8_t length, uint1
         Serial.print("...");
     #endif
 
-    int8_t count = 0;
+    uint8_t count = 0;
     uint32_t t1 = millis();
 
 #if I2CDEV_IMPLEMENTATION == I2CDEV_ARDUINO_WIRE || I2CDEV_IMPLEMENTATION == I2CDEV_BUILTIN_SBWIRE || I2CDEV_IMPLEMENTATION == I2CDEV_TEENSY_3X_WIRE

Arduino/IAQ2000/IAQ2000.cpp

@@ -113,7 +113,7 @@ int8_t IAQ2000::readAllBytes(uint8_t devAddr, uint8_t length, uint8_t *data, uin
     Serial.print(" bytes...");
 #endif
 
-    int8_t count = 0;
+    uint8_t count = 0;
 
     Wire.requestFrom(devAddr, length);
 

MSP430/I2Cdev/I2Cdev.cpp

@@ -215,7 +215,7 @@ int8_t I2Cdev::readBytes(uint8_t devAddr, uint8_t regAddr, uint8_t length, uint8
         Serial.print("...");
     #endif
 
-    int8_t count = 0;
+    uint8_t count = 0;
     uint32_t t1 = millis();
 
     #if (I2CDEV_IMPLEMENTATION == I2CDEV_ARDUINO_WIRE)
@@ -330,7 +330,7 @@ int8_t I2Cdev::readWords(uint8_t devAddr, uint8_t regAddr, uint8_t length, uint1
         Serial.print("...");
     #endif
 
-    int8_t count = 0;
+    uint8_t count = 0;
     uint32_t t1 = millis();
 
     #if (I2CDEV_IMPLEMENTATION == I2CDEV_ARDUINO_WIRE)

PIC18/I2Cdev/I2Cdev.c

@@ -41,7 +41,7 @@ THE SOFTWARE.
  * @return Number of bytes read (-1 indicates failure)
  */
 int8_t I2Cdev_readBytes(uint8_t devAddr, uint8_t regAddr, uint8_t length, uint8_t *data) {
-    int8_t count = 0;
+    uint8_t count = 0;
 
     // S
     IdleI2C();
@@ -104,7 +104,7 @@ int8_t I2Cdev_readByte(uint8_t devAddr, uint8_t regAddr, uint8_t *data) {
  * @return Number of words read (-1 indicates failure)
  */
 int8_t I2Cdev_readWords(uint8_t devAddr, uint8_t regAddr, uint8_t length, uint16_t *data) {
-    int8_t count = 0;
+    uint8_t count = 0;
 
     // S
     IdleI2C();

RP2040/I2Cdev/I2Cdev.cpp

@@ -149,7 +149,7 @@ int8_t I2Cdev::readWord(uint8_t devAddr, uint8_t regAddr, uint16_t *data, uint32
  * @return Number of bytes read (-1 indicates failure)
  */
 int8_t I2Cdev::readBytes(uint8_t devAddr, uint8_t regAddr, uint8_t length, uint8_t *data, uint32_t timeout) {
-    int8_t count = 0;
+    uint8_t count = 0;
 
     i2c_write_blocking(i2c_default, devAddr, &regAddr, 1, true);
     count = i2c_read_timeout_us(i2c_default, devAddr, data, length, false, timeout * 1000);
@@ -166,7 +166,7 @@ int8_t I2Cdev::readBytes(uint8_t devAddr, uint8_t regAddr, uint8_t length, uint8
  * @return Number of words read (-1 indicates failure)
  */
 int8_t I2Cdev::readWords(uint8_t devAddr, uint8_t regAddr, uint8_t length, uint16_t *data, uint32_t timeout) {
-    int8_t count = 0;
+    uint8_t count = 0;
     uint8_t data_buf[length*2];
 
     i2c_write_blocking(i2c_default, devAddr, &regAddr, 1, true);