]> granicus.if.org Git - esp-idf/commitdiff
Fix remaining -Wstrict-prototypes warnings
authorAnton Maklakov <anton@espressif.com>
Tue, 30 Jul 2019 04:53:48 +0000 (11:53 +0700)
committerAnton Maklakov <anton@espressif.com>
Thu, 1 Aug 2019 09:28:56 +0000 (16:28 +0700)
22 files changed:
components/app_trace/app_trace.c
components/app_update/test/test_switch_ota.c
components/driver/test/param_test/include/param_test.h
components/fatfs/vfs/vfs_fat_sdmmc.c
components/freertos/include/freertos/deprecated_definitions.h
components/freertos/tasks.c
components/freertos/test/test_freertos_backported_functions.c
components/freertos/test/test_freertos_eventgroups.c
components/freertos/test/test_freertos_task_notify.c
components/unity/include/unity_test_runner.h
components/wpa_supplicant/src/rsn_supp/wpa_i.h
examples/bluetooth/esp_ble_mesh/ble_mesh_console/ble_mesh_node/main/ble_mesh_register_node_cmd.c
examples/bluetooth/esp_ble_mesh/ble_mesh_console/ble_mesh_node/main/register_bluetooth.c
examples/bluetooth/esp_ble_mesh/ble_mesh_console/ble_mesh_provisioner/main/ble_mesh_register_node_cmd.c
examples/bluetooth/esp_ble_mesh/ble_mesh_console/ble_mesh_provisioner/main/ble_mesh_register_provisioner_cmd.c
examples/peripherals/mcpwm/mcpwm_bldc_control/main/mcpwm_bldc_control_hall_sensor_example.c
examples/peripherals/rmt_nec_tx_rx/main/infrared_nec_main.c
examples/peripherals/uart/uart_async_rxtxtasks/main/uart_async_rxtxtasks_main.c
examples/peripherals/uart/uart_echo/main/uart_echo_example_main.c
examples/peripherals/uart/uart_echo_rs485/main/rs485_example.c
examples/peripherals/uart/uart_select/main/uart_select_example_main.c
examples/system/select/main/select_example.c

index c85ad6a2ac7eac2f1db75f16ab6fa38f3a9d989f..bc0d0c20b56647fff708dda2181727d165863b61 100644 (file)
@@ -379,10 +379,10 @@ static inline void esp_apptrace_log_unlock(void)
 #endif
 }
 
-static inline esp_err_t esp_apptrace_lock_initialize(void)
+static inline esp_err_t esp_apptrace_lock_initialize(esp_apptrace_lock_t *lock)
 {
 #if CONFIG_ESP32_APPTRACE_LOCK_ENABLE
-    esp_apptrace_lock_init(&s_trace_buf.lock);
+    esp_apptrace_lock_init(lock);
 #endif
     return ESP_OK;
 }
index 232bfbc88e0bb36fb34af106b18f48dfe3d3a5ef..f0d056d9bdbdacff0b582be7501f26b44afbd992 100644 (file)
@@ -264,19 +264,19 @@ static void test_flow1(void)
         case 2:
             ESP_LOGI(TAG, "Factory");
             TEST_ASSERT_EQUAL(ESP_PARTITION_SUBTYPE_APP_FACTORY, cur_app->subtype);
-            copy_current_app_to_next_part_and_reboot(cur_app);
+            copy_current_app_to_next_part_and_reboot();
             break;
         case 3:
             ESP_LOGI(TAG, "OTA0");
             TEST_ASSERT_EQUAL(ESP_PARTITION_SUBTYPE_APP_OTA_0, cur_app->subtype);
             mark_app_valid();
-            copy_current_app_to_next_part_and_reboot(cur_app);
+            copy_current_app_to_next_part_and_reboot();
             break;
         case 4:
             ESP_LOGI(TAG, "OTA1");
             TEST_ASSERT_EQUAL(ESP_PARTITION_SUBTYPE_APP_OTA_1, cur_app->subtype);
             mark_app_valid();
-            copy_current_app_to_next_part_and_reboot(cur_app);
+            copy_current_app_to_next_part_and_reboot();
             break;
         case 5:
             ESP_LOGI(TAG, "OTA0");
@@ -307,7 +307,7 @@ static void test_flow2(void)
         case 2:
             ESP_LOGI(TAG, "Factory");
             TEST_ASSERT_EQUAL(ESP_PARTITION_SUBTYPE_APP_FACTORY, cur_app->subtype);
-            copy_current_app_to_next_part_and_reboot(cur_app);
+            copy_current_app_to_next_part_and_reboot();
             break;
         case 3:
             ESP_LOGI(TAG, "OTA0");
@@ -344,13 +344,13 @@ static void test_flow3(void)
         case 2:
             ESP_LOGI(TAG, "Factory");
             TEST_ASSERT_EQUAL(ESP_PARTITION_SUBTYPE_APP_FACTORY, cur_app->subtype);
-            copy_current_app_to_next_part_and_reboot(cur_app);
+            copy_current_app_to_next_part_and_reboot();
             break;
         case 3:
             ESP_LOGI(TAG, "OTA0");
             TEST_ASSERT_EQUAL(ESP_PARTITION_SUBTYPE_APP_OTA_0, cur_app->subtype);
             mark_app_valid();
-            copy_current_app_to_next_part_and_reboot(cur_app);
+            copy_current_app_to_next_part_and_reboot();
             break;
         case 4:
             ESP_LOGI(TAG, "OTA1");
@@ -402,7 +402,7 @@ static void test_flow4(void)
             nvs_close(handle);
             nvs_flash_deinit();
 
-            copy_current_app_to_next_part_and_reboot(cur_app);
+            copy_current_app_to_next_part_and_reboot();
             break;
         case 3:
             ESP_LOGI(TAG, "OTA0");
index 879e2e5e92996856ac6af07d2ae03e3585aa3eac..e04a2bee6ffbf0c09423a45fe9d7d02818daa783 100644 (file)
@@ -159,6 +159,6 @@ void test_serializer(const param_group_t *param_group, const ptest_func_t* test_
  * @param slave_func ``ptest_func_t`` to be executed by slave.
  */
 #define TEST_MASTER_SLAVE(name, param_group, tag, master_func, slave_func) \
-    static void PTEST_MASTER_NAME(name) () { test_serializer(&PGROUP_NAME(param_group), master_func); } \
-    static void PTEST_SLAVE_NAME(name) () { test_serializer(&PGROUP_NAME(param_group), slave_func); } \
+    static void PTEST_MASTER_NAME(name) (void) { test_serializer(&PGROUP_NAME(param_group), master_func); } \
+    static void PTEST_SLAVE_NAME(name) (void) { test_serializer(&PGROUP_NAME(param_group), slave_func); } \
     TEST_CASE_MULTIPLE_DEVICES("master slave test: "#name, tag, PTEST_MASTER_NAME(name), PTEST_SLAVE_NAME(name))
index ac1a37a7fac8630a0025897b7da8e278c25b3025..83cc0aee5b31daf8361ca3f12fc21876ae091ed3 100644 (file)
@@ -171,7 +171,7 @@ esp_err_t esp_vfs_fat_sdmmc_unmount(void)
     char drv[3] = {(char)('0' + s_pdrv), ':', 0};
     f_mount(0, drv, 0);
     // release SD driver
-    esp_err_t (*host_deinit)() = s_card->host.deinit;
+    esp_err_t (*host_deinit)(void) = s_card->host.deinit;
     ff_diskio_unregister(s_pdrv);
     free(s_card);
     s_card = NULL;
index fb031cdde63e2a2f791c58a64bba97cd7a65e4b6..dc061f33f3cec486a1d91c285c0d892747c96f0b 100644 (file)
@@ -82,12 +82,12 @@ projects should not use them. */
 
 #ifdef OPEN_WATCOM_INDUSTRIAL_PC_PORT
        #include "..\..\Source\portable\owatcom\16bitdos\pc\portmacro.h"
-       typedef void ( __interrupt __far *pxISR )();
+       typedef void ( __interrupt __far *pxISR )(void);
 #endif
 
 #ifdef OPEN_WATCOM_FLASH_LITE_186_PORT
        #include "..\..\Source\portable\owatcom\16bitdos\flsh186\portmacro.h"
-       typedef void ( __interrupt __far *pxISR )();
+       typedef void ( __interrupt __far *pxISR )(void);
 #endif
 
 #ifdef GCC_MEGA_AVR
@@ -255,7 +255,7 @@ projects should not use them. */
        FreeRTOSConfig.h when using the Borland compiler. */
        #include "frconfig.h"
        #include "..\portable\BCC\16BitDOS\PC\prtmacro.h"
-    typedef void ( __interrupt __far *pxISR )();
+    typedef void ( __interrupt __far *pxISR )(void);
 #endif
 
 #ifdef BCC_FLASH_LITE_186_PORT
@@ -263,7 +263,7 @@ projects should not use them. */
        FreeRTOSConfig.h when using the Borland compiler. */
        #include "frconfig.h"
        #include "..\portable\BCC\16BitDOS\flsh186\prtmacro.h"
-    typedef void ( __interrupt __far *pxISR )();
+    typedef void ( __interrupt __far *pxISR )(void);
 #endif
 
 #ifdef __GNUC__
index 8c4fd61a3677cc3445bbfe5515fe5e88e68d2c8a..634e64e486e0542bf0a35a3b15c4746f173ee38c 100644 (file)
@@ -2149,7 +2149,7 @@ void vTaskSuspendAll( void )
 
 #if ( portNUM_PROCESSORS > 1 )
 
-       static BaseType_t xHaveReadyTasks()
+       static BaseType_t xHaveReadyTasks( void )
        {
                for (int i = tskIDLE_PRIORITY + 1; i < configMAX_PRIORITIES; ++i)
                {
index 5071ce1fe905eba95348c0a09da292010db22ba5..f3fab9a8e6233268a48ba8b9c3db8fee01a7cac8 100644 (file)
@@ -221,7 +221,7 @@ static void del_cb(int index, void *ptr)
     *((uint32_t *)ptr) = (TLSP_DEL_BASE << index);   //Indicate deletion by setting task storage element to a unique value
 }
 
-static void task_cb(void)
+static void task_cb(void *arg)
 {
     int core = xPortGetCoreID();
     for(int i = 0; i < NO_OF_TLSP; i++){
index c2672fe27cab8138a44f984805a226f3d12e42e6..9b86f759fbc776229294b214b1e1cb708ee4214f 100644 (file)
@@ -135,7 +135,7 @@ static timer_isr_handle_t isr_handle;
 static bool test_set_bits;
 static bool test_clear_bits;
 
-static void IRAM_ATTR event_group_isr(void)
+static void IRAM_ATTR event_group_isr(void *arg)
 {
     portBASE_TYPE task_woken = pdFALSE;
     TIMERG0.int_clr_timers.t0 = 1;
index fd230d16234e3eb7db010bbc84aa87643329297b..8179cef9b4337d30cf041dc54bb353e738eedc2f 100644 (file)
@@ -94,7 +94,7 @@ static void receiver_task (void* arg){
     vTaskDelete(NULL);
 }
 
-static void IRAM_ATTR sender_ISR (void)
+static void IRAM_ATTR sender_ISR (void *arg)
 {
     int curcore = xPortGetCoreID();
     if(curcore == 0){      //Clear timer interrupt
index a9a1fae49d1e65d42519118a264c1b0f8c92260c..e9e46ebd439823809ec4fc4379290ba5e7d91163 100644 (file)
@@ -84,7 +84,7 @@ void unity_testcase_register(test_desc_t* desc);
 
 #define TEST_CASE(name_, desc_) \
     static void UNITY_TEST_UID(test_func_) (void); \
-    static void __attribute__((constructor)) UNITY_TEST_UID(test_reg_helper_) () \
+    static void __attribute__((constructor)) UNITY_TEST_UID(test_reg_helper_) (void) \
     { \
         static test_func test_fn_[] = {&UNITY_TEST_UID(test_func_)}; \
         static test_desc_t UNITY_TEST_UID(test_desc_) = { \
@@ -115,7 +115,7 @@ void unity_testcase_register(test_desc_t* desc);
 
 #define TEST_CASE_MULTIPLE_STAGES(name_, desc_, ...) \
     UNITY_TEST_FN_SET(__VA_ARGS__); \
-    static void __attribute__((constructor)) UNITY_TEST_UID(test_reg_helper_) () \
+    static void __attribute__((constructor)) UNITY_TEST_UID(test_reg_helper_) (void) \
     { \
         static test_desc_t UNITY_TEST_UID(test_desc_) = { \
             .name = name_, \
@@ -140,7 +140,7 @@ void unity_testcase_register(test_desc_t* desc);
 
 #define TEST_CASE_MULTIPLE_DEVICES(name_, desc_, ...) \
     UNITY_TEST_FN_SET(__VA_ARGS__); \
-    static void __attribute__((constructor)) UNITY_TEST_UID(test_reg_helper_) () \
+    static void __attribute__((constructor)) UNITY_TEST_UID(test_reg_helper_) (void) \
     { \
         static test_desc_t UNITY_TEST_UID(test_desc_) = { \
             .name = name_, \
index 3a09eff1d23ba86b93e6e2e973cb3dd1e4d0589d..69fc1a5e3d8d548e6875ff15c914ffc4519d2d91 100644 (file)
@@ -81,7 +81,7 @@ struct wpa_sm {
     int (*get_ppkey) (uint8_t *ifx, int *alg, uint8_t *addr, int *key_idx,
                uint8_t *key, size_t key_len, int key_entry_valid);
     void (*wpa_deauthenticate)(u8 reason_code);
-    void (*wpa_neg_complete)();
+    void (*wpa_neg_complete)(void);
     struct wpa_gtk_data gd; //used for calllback save param
     u16 key_info;       //used for txcallback param
     u16 txcb_flags;
@@ -145,7 +145,7 @@ typedef int (*WPA_GET_KEY) (u8 *ifx, int *alg, u8 *addt, int *keyidx, u8 *key, s
 
 typedef void (*WPA_DEAUTH_FUNC)(u8 reason_code);
 
-typedef void (*WPA_NEG_COMPLETE)();
+typedef void (*WPA_NEG_COMPLETE)(void);
 
 void wpa_register(char * payload, WPA_SEND_FUNC snd_func, \
                                                       WPA_SET_ASSOC_IE set_assoc_ie_func, \
index 6dc356cdab442da8b87edb462468b5a2b097ff2c..375c224c6c3349ca238e7715f181451570c84e20 100644 (file)
@@ -97,7 +97,7 @@ void ble_mesh_register_mesh_node(void)
     ble_mesh_register_node_cmd();
 }
 
-int ble_mesh_register_node_cb(void)
+int ble_mesh_register_node_cb(int argc, char** argv)
 {
     ESP_LOGD(TAG, "enter %s\n", __func__);
     ble_mesh_node_init();
@@ -364,7 +364,7 @@ int ble_mesh_node_enable_bearer(int argc, char **argv)
     return err;
 }
 
-int ble_mesh_node_reset(void)
+int ble_mesh_node_reset(int argc, char** argv)
 {
     esp_err_t err;
     ESP_LOGD(TAG, "enter %s\n", __func__);
index 0489388ec1adf32824ca34a7f0f4272a28effa5e..64358ba60670554b527c302ef86f0bec6a201281 100644 (file)
@@ -25,7 +25,7 @@ void register_bluetooth(void)
     register_ble_address();
 }
 
-int bt_mac(void)
+int bt_mac(int argc, char** argv)
 {
     const uint8_t *mac = esp_bt_dev_get_address();
     printf("+BTMAC:"MACSTR"\n", MAC2STR(mac));
@@ -38,7 +38,7 @@ void register_ble_address(void)
         .command = "btmac",
         .help = "get BT mac address",
         .hint = NULL,
-        .func = (esp_console_cmd_func_t)&bt_mac,
+        .func = &bt_mac,
     };
     ESP_ERROR_CHECK(esp_console_cmd_register(&cmd));
 }
index dc96f7907920f08fd8534287dfc7208ac63e065e..b9bd565c40c2e513a380848b6fbcebecac4c5915 100644 (file)
@@ -76,7 +76,7 @@ void ble_mesh_register_mesh_node(void)
     ble_mesh_register_node_cmd();
 }
 
-int ble_mesh_register_node_cb(void)
+int ble_mesh_register_node_cb(int argc, char** argv)
 {
     ESP_LOGD(TAG, "enter %s\n", __func__);
     ble_mesh_node_init();
index 4c95d45b28ee2f9745150a39f08bc71d7fe8146c..6b74255d37af749eeb6cb34739b5979dc5621f88 100644 (file)
@@ -101,7 +101,7 @@ void ble_mesh_prov_adv_cb(const esp_bd_addr_t addr, const esp_ble_addr_type_t ad
     ESP_LOGD(TAG, "exit %s\n", __func__);
 }
 
-int ble_mesh_provisioner_register(void)
+int ble_mesh_provisioner_register(int argc, char** argv)
 {
     ESP_LOGD(TAG, "enter %s \n", __func__);
     // esp_ble_mesh_register_unprov_adv_pkt_callback(ble_mesh_prov_adv_cb);
index c8bfe00010dec89ee3b319085f69c81c90929b66..6784b63991a70ba93896a2836eeca03e5bd6f3d7 100644 (file)
@@ -162,7 +162,7 @@ static void disp_captured_signal(void *arg)
 /**
  * @brief this is ISR handler function, here we check for interrupt that triggers rising edge on CAP0 signal and according take action
  */
-static void IRAM_ATTR isr_handler(void)
+static void IRAM_ATTR isr_handler(void *arg)
 {
     uint32_t mcpwm_intr_status;
     capture evt;
index 1dc28e5accd3b6744bbe5307226bc2bdcd2fc96f..df5a6b08d9857abb03a5eeee6740db5c5604073d 100644 (file)
@@ -276,7 +276,7 @@ static void nec_rx_init(void)
  * @brief RMT receiver demo, this task will print each received NEC data.
  *
  */
-static void rmt_example_nec_rx_task(void)
+static void rmt_example_nec_rx_task(void *arg)
 {
     int channel = RMT_RX_CHANNEL;
     nec_rx_init();
@@ -317,7 +317,7 @@ static void rmt_example_nec_rx_task(void)
  * @brief RMT transmitter demo, this task will periodically send NEC data. (100 * 32 bits each time.)
  *
  */
-static void rmt_example_nec_tx_task(void)
+static void rmt_example_nec_tx_task(void *arg)
 {
     vTaskDelay(10);
     nec_tx_init();
index f4f4eb18da63d53265dde7f02359e180d76b6fb6..3b4a5872ea61a88cf5be143c6a456af9cba0ea8d 100644 (file)
@@ -41,7 +41,7 @@ int sendData(const char* logName, const char* data)
     return txBytes;
 }
 
-static void tx_task(void)
+static void tx_task(void *arg)
 {
     static const char *TX_TASK_TAG = "TX_TASK";
     esp_log_level_set(TX_TASK_TAG, ESP_LOG_INFO);
@@ -51,7 +51,7 @@ static void tx_task(void)
     }
 }
 
-static void rx_task(void)
+static void rx_task(void *arg)
 {
     static const char *RX_TASK_TAG = "RX_TASK";
     esp_log_level_set(RX_TASK_TAG, ESP_LOG_INFO);
index 6e6a77a74986c43a117e8f5f0173765d49a5d0d8..c9b5dbc7b3138fda170bcc56bf615c9f05067ce3 100644 (file)
@@ -31,7 +31,7 @@
 
 #define BUF_SIZE (1024)
 
-static void echo_task(void)
+static void echo_task(void *arg)
 {
     /* Configure parameters of an UART driver,
      * communication pins and install the driver */
index c68c22edf0cce46ed3f82733ce83d8d13502a736..e2d53dac4d74813f28e4e149599d52ec87b23944 100644 (file)
@@ -52,7 +52,7 @@
 static const char *TAG = "RS485_ECHO_APP";
 
 // An example of echo test with hardware flow control on UART
-static void echo_task(void)
+static void echo_task(void *arg)
 {
     const int uart_num = ECHO_UART_PORT;
     uart_config_t uart_config = {
index 2c46a3d114cad3c5e94e8bfbb634f1abd9cdc5b3..d7b3c88491248f8efd72dcaf6d981b75cc3ccabe 100644 (file)
@@ -20,7 +20,7 @@
 
 static const char* TAG = "uart_select_example";
 
-static void uart_select_task(void)
+static void uart_select_task(void *arg)
 {
     uart_config_t uart_config = {
         .baud_rate = 115200,
index 96290242959af69c18276b66bc0b863ce7741d62..5da28c7dde05ce09813051c531fc66a09654f210 100644 (file)
@@ -128,7 +128,7 @@ static void uart1_write_task(void *param)
         }
     }
 
-    uart1_deinit(uart_fd);
+    uart1_deinit();
     vTaskDelete(NULL);
 }