encapsulation

firmware/pch/pch.h

@@ -19,6 +19,7 @@
 
 #include "global.h"
 #include "efifeatures.h"
+#include "efistringutil.h"
 #include "rusefi_generated.h"
 #include "loggingcentral.h"
 #include "error_handling.h"

firmware/util/cli_registry.cpp

@@ -16,6 +16,7 @@
 
 #include "pch.h"
 #include <cstring>
+#include <cstdint>
 #include "efiprintf.h"
 #include "efistringutil.h"
 #include "cli_registry.h"
@@ -23,6 +24,10 @@
 /* for isspace() */
 #include <ctype.h>
 
+#ifndef CONSOLE_MAX_ACTIONS
+#define CONSOLE_MAX_ACTIONS 256
+#endif
+
 #ifndef MAX_CMD_LINE_LENGTH
 #define MAX_CMD_LINE_LENGTH		100
 #endif

firmware/util/efilib.h

@@ -43,8 +43,6 @@ static inline uint32_t SWAP_UINT32(uint32_t x)
 // number of microseconds in one period of given frequency (per second)
 #define frequency2periodUs(freq) ((1000000.0f) / (freq))
 
-#define ATOI_ERROR_CODE 311223344
-
 #define Q(x) #x
 #define QUOTE(x) Q(x)
 
@@ -62,7 +60,6 @@ uint32_t efiStrlen(const char *param);
 int efiPow10(int param);
 bool startsWith(const char *line, const char *prefix);
 float atoff(const char *string);
-int atoi(const char *string);
 
 #ifndef UNUSED
 #define UNUSED(x) (void)(x)
@@ -82,9 +79,6 @@ char* itoa10(char *p, int num);
  */
 #define clampPercentValue(x) (clampF(0, x, 100))
 
-bool strEqualCaseInsensitive(const char *str1, const char *str2);
-bool strEqual(const char *str1, const char *str2);
-
 // Currently used by air-interp. tCharge mode (see EngineState::updateTChargeK()).
 float limitRateOfChange(float newValue, float oldValue, float incrLimitPerSec, float decrLimitPerSec, float secsPassed);
 

firmware/util/efistringutil.h

@@ -1 +1,6 @@
+#define ATOI_ERROR_CODE 311223344
+
 int indexOf(const char *string, char ch);
+bool strEqualCaseInsensitive(const char *str1, const char *str2);
+bool strEqual(const char *str1, const char *str2);
+int atoi(const char *string);

simulator/simulator/efifeatures.h

@@ -72,8 +72,6 @@
 #define EFI_TOOTH_LOGGER TRUE
 #endif
 
-#define CONSOLE_MAX_ACTIONS 256
-
 #define EFI_USE_UART_DMA FALSE
 
 #define EFI_MAP_AVERAGING TRUE

unit_tests/efifeatures.h

@@ -55,8 +55,6 @@
 
 #define EFI_HISTOGRAMS FALSE
 
-#define CONSOLE_MAX_ACTIONS 128
-
 #define EFI_CLI_SUPPORT FALSE
 
 #define EFI_SIGNAL_EXECUTOR_ONE_TIMER FALSE