fix some warnings with Wextra

Author: hathach

Makefile

@@ -228,7 +228,8 @@ CFLAGS += \
 	-fstack-usage \
 	-fno-strict-aliasing \
 	-Wall \
-	-Werror
+	-Werror \
+	-Wfatal-errors \
 
 CFLAGS += -Wno-error=unused-parameter
 

lib/sdk11/components/libraries/bootloader_dfu/bootloader.c

@@ -442,7 +442,8 @@ uint32_t bootloader_dfu_sd_update_continue(void)
 
 uint32_t bootloader_dfu_sd_update_finalize(void)
 {
-    dfu_update_status_t update_status = {DFU_UPDATE_SD_SWAPPED, };
+    dfu_update_status_t update_status = { 0 };
+    update_status.status_code = DFU_UPDATE_SD_SWAPPED;
 
     bootloader_dfu_update_process(update_status);
 

lib/sdk11/components/libraries/bootloader_dfu/dfu_single_bank.c

@@ -164,7 +164,8 @@ static void dfu_prepare_func_app_erase(uint32_t image_size)
  */
 static void dfu_cleared_func_app(void)
 {
-    dfu_update_status_t update_status = {DFU_BANK_0_ERASED, };
+    dfu_update_status_t update_status = { 0 };
+    update_status.status_code = DFU_BANK_0_ERASED;
     bootloader_dfu_update_process(update_status);
 }
 
@@ -498,7 +499,8 @@ uint32_t dfu_init_pkt_handle(dfu_update_packet_t * p_packet)
     {
         case DFU_STATE_RDY:
             m_dfu_state = DFU_STATE_RX_INIT_PKT;
-            // When receiving init packet in state ready just update and fall through this case.         
+            // When receiving init packet in state ready just update and fall through this case.
+            /* FALLTHRU */
 
         case DFU_STATE_RX_INIT_PKT:
             // DFU initialization has been done and a start packet has been received.

tools/build_all.py

@@ -37,9 +37,18 @@
     make_result = subprocess.run("make -j 4 BOARD={} combinehex genpkg".format(board), shell=True, stdout=subprocess.PIPE, stderr=subprocess.STDOUT)
     build_duration = time.monotonic() - start_time
 
+    flash_size = "-"
+    sram_size = "-"
+
     if make_result.returncode == 0:
         success = "\033[32msucceeded\033[0m"
         success_count += 1
+        
+        out_file = glob.glob('_build/build-{}/*.out'.format(board))[0]
+        size_output = subprocess.run('size {}'.format(out_file), shell=True, stdout=subprocess.PIPE).stdout.decode("utf-8")
+        size_list = size_output.split('\n')[1].split('\t')
+        flash_size = int(size_list[0])
+        sram_size = int(size_list[1]) + int(size_list[2])
     else:
         exit_status = make_result.returncode
         success = "\033[31mfailed\033[0m   "
@@ -50,12 +59,6 @@
             if entry.name.endswith(extension) and "nosd" not in entry.name:
                 shutil.copy(entry.path, bin_directory)
 
-    out_file = glob.glob('_build/build-{}/*.out'.format(board))[0]
-    size_output = subprocess.run('size {}'.format(out_file), shell=True, stdout=subprocess.PIPE).stdout.decode("utf-8")
-    size_list = size_output.split('\n')[1].split('\t')
-    flash_size = int(size_list[0])
-    sram_size = int(size_list[1]) + int(size_list[2])
-
     print(build_format.format(board, success, "{:.2f}s".format(build_duration), flash_size, sram_size))
 
     if make_result.returncode != 0: