From 4675e34f7e7c052bd021c3df163a5f332ee5e3e8 Mon Sep 17 00:00:00 2001 From: Rahul Tank Date: Tue, 10 Oct 2023 16:25:10 +0530 Subject: [PATCH] fix(espcoredump): fixed compilation warning Added conditional compilation for logging stack usage only when CONFIG_ESP_COREDUMP_LOGS is enabled. Closes: https://github.com/espressif/esp-idf/issues/12318 --- components/espcoredump/src/core_dump_common.c | 2 ++ 1 file changed, 2 insertions(+) diff --git a/components/espcoredump/src/core_dump_common.c b/components/espcoredump/src/core_dump_common.c index c8d905917d2..9d437d2d699 100644 --- a/components/espcoredump/src/core_dump_common.c +++ b/components/espcoredump/src/core_dump_common.c @@ -106,9 +106,11 @@ FORCE_INLINE_ATTR uint32_t esp_core_dump_free_stack_space(const uint8_t *pucStac */ FORCE_INLINE_ATTR void esp_core_dump_report_stack_usage(void) { +#if CONFIG_ESP_COREDUMP_LOGS uint32_t bytes_free = esp_core_dump_free_stack_space(s_coredump_stack); ESP_COREDUMP_LOGI("Core dump used %u bytes on stack. %u bytes left free.", s_core_dump_sp - s_coredump_stack - bytes_free, bytes_free); +#endif /* Restore the stack pointer. */ ESP_COREDUMP_LOGI("Restoring stack @ %p", s_stack_context.sp);