Merge pull request #6340 from gebart/pr/warn-fixes

sys+cpu: Some warning fixes

cpu/cortexm_common/cortexm_init.c

@@ -46,7 +46,7 @@ void cortexm_init(void)
     /* set SVC interrupt to same priority as the rest */
     NVIC_SetPriority(SVCall_IRQn, CPU_DEFAULT_IRQ_PRIO);
     /* initialize all vendor specific interrupts with the same value */
-    for (int i = 0; i < CPU_IRQ_NUMOF; i++) {
+    for (int i = 0; i < (int)CPU_IRQ_NUMOF; i++) {
         NVIC_SetPriority((IRQn_Type) i, CPU_DEFAULT_IRQ_PRIO);
     }
 

cpu/cortexm_common/mpu.c

@@ -67,6 +67,9 @@ int mpu_configure(uint_fast8_t region, uintptr_t base, uint_fast32_t attr) {
 
     return 0;
 #else
+    (void)region;
+    (void)base;
+    (void)attr;
     return -1;
 #endif
 }

cpu/kinetis_common/periph/gpio.c

@@ -145,7 +145,7 @@ static inline int get_ctx(int port, int pin)
  */
 static int get_free_ctx(void)
 {
-    for (int i = 0; i < CTX_NUMOF; i++) {
+    for (unsigned int i = 0; i < CTX_NUMOF; i++) {
         if (isr_ctx[i].cb == NULL) {
             return i;
         }

drivers/at86rf2xx/at86rf2xx.c

@@ -77,7 +77,7 @@ void at86rf2xx_reset(at86rf2xx_t *dev)
     cpuid_get(cpuid);
 
 #if CPUID_LEN > IEEE802154_LONG_ADDRESS_LEN
-    for (int i = IEEE802154_LONG_ADDRESS_LEN; i < CPUID_LEN; i++) {
+    for (unsigned int i = IEEE802154_LONG_ADDRESS_LEN; i < CPUID_LEN; i++) {
         cpuid[i & 0x07] ^= cpuid[i];
     }
 #endif

drivers/at86rf2xx/at86rf2xx_netdev.c

@@ -434,8 +434,8 @@ static int _set(netdev2_t *netdev, netopt_t opt, void *val, size_t len)
             }
             else {
                 uint8_t chan = ((uint8_t *)val)[0];
-                if (chan < AT86RF2XX_MIN_CHANNEL ||
-                    chan > AT86RF2XX_MAX_CHANNEL) {
+                if ((chan < AT86RF2XX_MIN_CHANNEL) ||
+                    (chan > AT86RF2XX_MAX_CHANNEL)) {
                     res = -EINVAL;
                     break;
                 }

sys/auto_init/saul/auto_init_adc.c

@@ -55,7 +55,7 @@ extern saul_driver_t adc_saul_driver;
 void auto_init_adc(void)
 {
     DEBUG("auto init SAUL ADC\n");
-    for (int i = 0; i < SAUL_ADC_NUMOF; i++) {
+    for (unsigned int i = 0; i < SAUL_ADC_NUMOF; i++) {
         const saul_adc_params_t *p = &saul_adc_params[i];
         saul_adcs[i] = p;
 

sys/auto_init/saul/auto_init_gpio.c

@@ -53,7 +53,7 @@ extern saul_driver_t gpio_saul_driver;
 void auto_init_gpio(void)
 {
     DEBUG("auto init gpio SAUL\n");
-    for (int i = 0; i < SAUL_GPIO_NUMOF; i++) {
+    for (unsigned int i = 0; i < SAUL_GPIO_NUMOF; i++) {
         const saul_gpio_params_t *p = &saul_gpio_params[i];
 
         DEBUG("[auto_init_saul] initializing direct GPIO\n");

sys/auto_init/saul/auto_init_isl29020.c

@@ -51,7 +51,7 @@ extern saul_driver_t isl29020_saul_driver;
 
 void auto_init_isl29020(void)
 {
-    for (int i = 0; i < ISL29020_NUM; i++) {
+    for (unsigned int i = 0; i < ISL29020_NUM; i++) {
         const isl29020_params_t *p = &isl29020_params[i];
 
         DEBUG("[auto_init_saul] initializing isl29020 light sensor\n");

sys/auto_init/saul/auto_init_l3g4200d.c

@@ -51,7 +51,7 @@ extern saul_driver_t l3g4200d_saul_driver;
 
 void auto_init_l3g4200d(void)
 {
-    for (int i = 0; i < L3G4200D_NUM; i++) {
+    for (unsigned int i = 0; i < L3G4200D_NUM; i++) {
         const l3g4200d_params_t *p = &l3g4200d_params[i];
 
         DEBUG("[auto_init_saul] initializing l3g4200d gyroscope\n");

sys/auto_init/saul/auto_init_lis3dh.c

@@ -51,7 +51,7 @@ extern saul_driver_t lis3dh_saul_driver;
 
 void auto_init_lis3dh(void)
 {
-    for (int i = 0; i < LIS3DH_NUM; i++) {
+    for (unsigned int i = 0; i < LIS3DH_NUM; i++) {
         const lis3dh_params_t *p = &lis3dh_params[i];
         int res;
 

sys/auto_init/saul/auto_init_lps331ap.c

@@ -51,7 +51,7 @@ extern saul_driver_t lps331ap_saul_driver;
 
 void auto_init_lps331ap(void)
 {
-    for (int i = 0; i < LPS331AP_NUM; i++) {
+    for (unsigned int i = 0; i < LPS331AP_NUM; i++) {
         const lps331ap_params_t *p = &lps331ap_params[i];
 
         DEBUG("[auto_init_saul] initializing lps331ap pressure sensor\n");

sys/auto_init/saul/auto_init_lsm303dlhc.c

@@ -53,7 +53,7 @@ extern saul_driver_t lsm303dlhc_saul_mag_driver;
 
 void auto_init_lsm303dlhc(void)
 {
-    for (int i = 0; i < LSM303DLHC_NUM; i++) {
+    for (unsigned int i = 0; i < LSM303DLHC_NUM; i++) {
         const lsm303dlhc_params_t *p = &lsm303dlhc_params[i];
 
         DEBUG("[auto_init_saul] initializing lsm303dlhc acc/mag sensor\n");

sys/auto_init/saul/auto_init_mma8652.c

@@ -52,7 +52,7 @@ extern saul_driver_t mma8652_saul_driver;
 
 void auto_init_mma8652(void)
 {
-    for (int i = 0; i < MMA8652_NUM; i++) {
+    for (unsigned int i = 0; i < MMA8652_NUM; i++) {
         const mma8652_params_t *p = &mma8652_params[i];
 
         DEBUG("[auto_init_saul] initializing mma8652 acc sensor\n");