Properly exclude unused code, initialize logging properly in pure ESP-IDF

Author: nicklasb

components/robusto/base/src/logging/logging_esp-idf.c

@@ -32,7 +32,6 @@
 
 #include <robusto_logging.h>
 #ifdef USE_ESPIDF
-#define CONFIG_LOG_MAXIMUM_LEVEL CONFIG_ROB_LOG_MAXIMUM_LEVEL
 #if ROB_LOG_LOCAL_LEVEL > ROB_LOG_NONE 
 
 #include <esp_log.h>
@@ -45,7 +44,7 @@ void compat_rob_log_writev(rob_log_level_t level, const char *tag, const char *f
 // On the ESP platform, there is no need for the sparse mode.
 void compat_rob_log_write_sparse(const char *tag, const char *format)
 {
-    va_list arg_list;
+    va_list arg_list = NULL;
     compat_rob_log_writev(ROB_LOG_LOCAL_LEVEL, tag, format, arg_list);
 }
 #endif

components/robusto/network/src/media/radio/XPowers/init_pmu.hpp

@@ -1,5 +1,7 @@
 #pragma once
 #include <robconfig.h>
+// make sure this is only compiled when needed
+#ifdef CONFIG_ROBUSTO_SUPPORTS_LORA
 #ifdef __cplusplus
 extern "C"
 {
@@ -10,3 +12,4 @@ void init_pmu(void);
 #ifdef __cplusplus
 } /* extern "C" */
 #endif
+#endif

components/robusto/network/src/media/radio/XPowers/init_pmu_espidf.cpp

@@ -1,4 +1,5 @@
 #include "init_pmu.hpp"
+#ifdef CONFIG_ROBUSTO_SUPPORTS_LORA
 #ifdef USE_ESPIDF
 #include <stdio.h>
 #include <cstring>
@@ -86,4 +87,4 @@ static void pmu_hander_task(void *args)
 
 #endif
 
-
+#endif