Compile warnings

Author: pschatzmann

src/Driver/es7210/es7210.c

@@ -149,7 +149,7 @@ static error_t es7210_update_reg_bit(uint8_t reg_addr, uint8_t update_bits, uint
 
 static int get_coeff(uint32_t mclk, uint32_t lrck)
 {
-    for (int i = 0; i < (sizeof(coeff_div) / sizeof(coeff_div[0])); i++) {
+    for (unsigned i = 0; i < (sizeof(coeff_div) / sizeof(coeff_div[0])); i++) {
         if (coeff_div[i].lrck == lrck && coeff_div[i].mclk == mclk)
             return i;
     }

src/Driver/es7243/es7243.c

@@ -69,8 +69,9 @@ error_t es7243_adc_set_addr(int addr)
 
 static error_t es7243_mclk_active(uint8_t mclk_gpio)
 {
-    arduino_pin_mode(mclk_gpio, OUTPUT);
-    /*
+#ifndef ARDUINO_ARCH_NRF52840
+    pinMode(mclk_gpio, OUTPUT);
+#endif    /*
         Before initializing es7243, it is necessary to output
         mclk to es7243 to activate the I2C configuration.
         So give some clocks to active es7243.

src/Driver/es8311/es8311.c

@@ -218,7 +218,7 @@ static int es8311_read_reg(uint8_t reg_addr)
 */
 static int get_coeff(uint32_t mclk, uint32_t rate)
 {
-    for (int i = 0; i < (sizeof(coeff_div) / sizeof(coeff_div[0])); i++) {
+    for (unsigned i = 0; i < (sizeof(coeff_div) / sizeof(coeff_div[0])); i++) {
         if (coeff_div[i].rate == rate && coeff_div[i].mclk == mclk)
             return i;
     }

src/Driver/tas5805m/tas5805m.c

@@ -151,7 +151,7 @@ error_t tas5805m_get_volume(int *value)
     uint8_t cmd[2] = {MASTER_VOL_REG_ADDR, 0x00};
     error_t ret = i2c_bus_read_bytes(i2c_handler, TAS5805M_ADDR, &cmd[0], 1, &cmd[1], 1);
     TAS5805M_ASSERT(ret, "Fail to get volume", RESULT_FAIL);
-    int i;
+    unsigned i;
     for (i = 0; i < sizeof(tas5805m_volume); i++) {
         if (cmd[1] >= tas5805m_volume[i])
             break;

src/DriverPins.h

@@ -311,6 +311,9 @@ class DriverPins {
           pinMode(tmp.pin, OUTPUT);
           digitalWrite(tmp.pin, HIGH);
           break;
+        default:
+          // do nothing
+          break;
         }
       }
     }

src/Utils/etc.cpp

@@ -3,26 +3,33 @@
 #ifdef ARDUINO
 #  include "Arduino.h"
 #else
-
-#define HIGH 0x1
-#define LOW  0x0
-
-#define INPUT 0x0
-#define OUTPUT 0x1
-#define INPUT_PULLUP 0x2
-void pinMode(int, int);
-void digitalWrite(int, int);
-void delay(uint32_t);
+#ifndef HIGH
+#  define HIGH 0x1
+#endif
+#ifndef LOW
+#  define LOW  0x0
+#endif
+#ifndef INPUT
+#  define INPUT 0x0
+#endif
+#ifndef OUTPUT
+#  define OUTPUT 0x1
+#endif
+#ifndef INPUT_PULLUP
+#  define INPUT_PULLUP 0x2
 #endif
+#endif
+
 
 #ifdef __cplusplus
 extern "C" {
 #endif
 
-
-
-// pinMode gives linker error if called from c for Nano BLE Sense
-void arduino_pin_mode(int pin, int mode);
+#ifndef ARDUINO
+extern void pinMode(int, int);
+extern void digitalWrite(int, int);
+extern void delay(uint32_t);
+#endif
 
 
 #ifdef __cplusplus