From b09f4acb2721a204a92b0b3ced64359fc0349b43 Mon Sep 17 00:00:00 2001 From: Alexey Gerenkov Date: Thu, 14 Sep 2017 19:36:10 +0300 Subject: [PATCH] Added proper handling of apptrace and pthread initialization errors --- components/esp32/cpu_start.c | 12 +++--------- 1 file changed, 3 insertions(+), 9 deletions(-) diff --git a/components/esp32/cpu_start.c b/components/esp32/cpu_start.c index 2ceb87f58..25e31de45 100644 --- a/components/esp32/cpu_start.c +++ b/components/esp32/cpu_start.c @@ -302,17 +302,13 @@ void start_cpu0_default(void) esp_set_time_from_rtc(); #if CONFIG_ESP32_APPTRACE_ENABLE err = esp_apptrace_init(); - if (err != ESP_OK) { - ESP_EARLY_LOGE(TAG, "Failed to init apptrace module on CPU0 (%d)!", err); - } + assert(err == ESP_OK && "Failed to init apptrace module on PRO CPU!"); #endif #if CONFIG_SYSVIEW_ENABLE SEGGER_SYSVIEW_Conf(); #endif err = esp_pthread_init(); - if (err != ESP_OK) { - ESP_EARLY_LOGE(TAG, "Failed to init pthread module (%d)!", err); - } + assert(err == ESP_OK && "Failed to init pthread module!"); do_global_ctors(); #if CONFIG_INT_WDT @@ -356,9 +352,7 @@ void start_cpu1_default(void) #endif #if CONFIG_ESP32_APPTRACE_ENABLE esp_err_t err = esp_apptrace_init(); - if (err != ESP_OK) { - ESP_EARLY_LOGE(TAG, "Failed to init apptrace module on CPU1 (%d)!", err); - } + assert(err == ESP_OK && "Failed to init apptrace module on APP CPU!"); #endif //Take care putting stuff here: if asked, FreeRTOS will happily tell you the scheduler //has started, but it isn't active *on this CPU* yet.