Reduce compile warning

Author: EeeeBin

src/M5Display.cpp

@@ -161,7 +161,7 @@ void M5Display::drawBmpFile(fs::FS &fs, const char *path, uint16_t x, uint16_t y
         uint8_t*  bptr = lineBuffer;
         uint16_t* tptr = (uint16_t*)lineBuffer;
         // Convert 24 to 16 bit colours
-        for (uint16_t col = 0; col < w; col++) {
+        for (col = 0; col < w; col++) {
           b = *bptr++;
           g = *bptr++;
           r = *bptr++;

src/utility/CommUtil.cpp

@@ -67,12 +67,10 @@ bool CommUtil::writeBytes(uint8_t address, uint8_t subAddress, uint8_t *data, ui
     Serial.printf("result:%s\n", function_result ? "OK" : "NG");
   #endif
 
-  return (Wire.endTransmission() == 0);             // Send the Tx buffer
+  return function_result;             // Send the Tx buffer
 }
 
 bool CommUtil::readByte(uint8_t address, uint8_t *result) {
-  uint8_t data; // `data` will store the register data
-
   #ifdef I2C_DEBUG_TO_SERIAL
     Serial.printf("readByte :read from 0x%02x requestByte=1 receive=", address);
   #endif
@@ -91,8 +89,6 @@ bool CommUtil::readByte(uint8_t address, uint8_t *result) {
 }
 
 bool CommUtil::readByte(uint8_t address, uint8_t subAddress,uint8_t *result) {
-  uint8_t data; // `data` will store the register data
-
   #ifdef I2C_DEBUG_TO_SERIAL
     Serial.printf("readByte :read from 0x%02x [0x%02x] requestByte=1 receive=", address, subAddress);
   #endif

src/utility/Power.cpp

@@ -99,15 +99,6 @@ void POWER::begin() {
   }
 }
 
-
-static bool getI2CReg(uint8_t *result, uint8_t address, uint8_t *reg) {
-  return (M5.I2C.readByte(address, *reg, result));
-}
-
-static bool setI2CReg(uint8_t address, uint8_t reg, uint8_t value) {
-  return (M5.I2C.writeByte(address, reg, value));
-}
-
 bool POWER::setPowerBoostOnOff(bool en) {
   uint8_t data;
   if (M5.I2C.readByte(IP5306_ADDR, IP5306_REG_SYS_CTL1, &data) == true) {
@@ -229,6 +220,7 @@ bool POWER::setVinMaxCurrent(uint8_t cur) {
   if(M5.I2C.readByte(IP5306_ADDR, IP5306_REG_CHG_DIG, &data) == true) {
     return M5.I2C.writeByte(IP5306_ADDR, IP5306_REG_CHG_DIG, (data & 0xe0) | cur);
   }
+  return false;
 }
 
 bool POWER::setChargeVolt(uint8_t volt) {
@@ -331,7 +323,6 @@ bool POWER::isResetbyPowerSW() {
 void POWER::deepSleep(uint64_t time_in_us){
 
   // Keep power keep boost on
-  setLowPowerShutdown(false);
   setPowerBoostKeepOn(true);
 
   // power off the Lcd

src/utility/qrcode.c

@@ -850,7 +850,7 @@ int8_t qrcode_initText(QRCode *qrcode, uint8_t *modules, uint8_t version, uint8_
 }
 
 bool qrcode_getModule(QRCode *qrcode, uint8_t x, uint8_t y) {
-  if (x < 0 || x >= qrcode->size || y < 0 || y >= qrcode->size) {
+  if (x >= qrcode->size || y >= qrcode->size) {
     return false;
   }
   uint32_t offset = y * qrcode->size + x;

src/utility/quaternionFilters.cpp

@@ -18,7 +18,7 @@
 
 static float GyroMeasError = PI * (40.0f / 180.0f);
 // gyroscope measurement drift in rad/s/s (start at 0.0 deg/s/s)
-static float GyroMeasDrift = PI * (0.0f  / 180.0f);
+// static float GyroMeasDrift = PI * (0.0f  / 180.0f);
 // There is a tradeoff in the beta parameter between accuracy and response
 // speed. In the original Madgwick study, beta of 0.041 (corresponding to
 // GyroMeasError of 2.7 degrees/s) was found to give optimal accuracy.
@@ -35,7 +35,7 @@ static float GyroMeasDrift = PI * (0.0f  / 180.0f);
 static float beta = sqrt(3.0f / 4.0f) * GyroMeasError;   // Compute beta
 // Compute zeta, the other free parameter in the Madgwick scheme usually
 // set to a small or zero value
-static float zeta = sqrt(3.0f / 4.0f) * GyroMeasDrift;
+// static float zeta = sqrt(3.0f / 4.0f) * GyroMeasDrift;
 
 // Vector to hold integral error for Mahony method
 static float eInt[3] = { 0.0f, 0.0f, 0.0f };