fix clang 19 warnings/errors

firmware/config/boards/proteus/knock_config.h

@@ -30,4 +30,4 @@
 
 	#define KNOCK_SAMPLE_TIME ADC_SAMPLE_84
 	#define KNOCK_SAMPLE_RATE (STM32_PCLK2 / (4 * (84 + 12)))
-#endif
+#endif

firmware/controllers/actuators/electronic_throttle_impl.h

@@ -10,9 +10,10 @@
 // include the "public" ETB interface
 #include "electronic_throttle.h"
 
-#include "sensor.h"
 #include "efi_pid.h"
 #include "electronic_throttle_generated.h"
+#include "sensor_type.h"
+#include "timer.h"
 
 /**
  * Hard code ETB update speed.

firmware/controllers/algo/fuel_math.cpp

@@ -30,7 +30,6 @@
 #include "fuel_math.h"
 #include "fuel_computer.h"
 #include "injector_model.h"
-#include "speed_density.h"
 #include "speed_density_base.h"
 #include "lua_hooks.h"
 

firmware/controllers/algo/rusefi_hw_enums.h

@@ -7,6 +7,8 @@
 
 #pragma once
 
+#include <stdint.h>
+
 // todo: migrate/unify with pin_output_mode_e? rename? something is messy here
 // this enum is currently only used for SPI pins
 typedef enum __attribute__ ((__packed__)) {

firmware/controllers/engine_cycle/knock_logic.h

@@ -7,6 +7,8 @@
 
 #pragma once
 
+#include "efitime.h"
+#include "engine_module.h"
 #include "peak_detect.h"
 #include "knock_controller_generated.h"
 

firmware/controllers/math/engine_math.cpp

@@ -21,8 +21,6 @@
 
 #include "pch.h"
 
-#include "event_registry.h"
-#include "fuel_math.h"
 #include "advance_map.h"
 #include "gppwm_channel.h"
 

firmware/controllers/sensors/impl/software_knock.cpp

@@ -95,10 +95,10 @@ void initSoftwareKnock() {
 
 		efiPrintf("Knock sense configuring filter with frequency %.2f khz", freqKhz);
 
-		knockFilter.configureBandpass(KNOCK_SAMPLE_RATE, 1000 * freqKhz, 3);
+		knockFilter.configureBandpass((int)KNOCK_SAMPLE_RATE, 1000 * freqKhz, 3);
 
 		efiSetPadMode("knock ch1", KNOCK_PIN_CH1, PAL_MODE_INPUT_ANALOG);
-#if KNOCK_HAS_CH2		
+#if KNOCK_HAS_CH2	
 		efiSetPadMode("knock ch2", KNOCK_PIN_CH2, PAL_MODE_INPUT_ANALOG);
 #endif
 		kt.start();

firmware/controllers/system/thread_controller.h

@@ -7,6 +7,8 @@
 
 #pragma once
 
+#include "global.h"
+
 /**
  * @brief A base class for a controller that requires its own thread.
  * 

firmware/init/sensor/init_lambda.cpp

@@ -3,7 +3,6 @@
 #include "init.h"
 #include "adc_subscription.h"
 #include "linear_func.h"
-#include "live_data.h"
 
 static LinearFunc func;
 

firmware/init/sensor/init_tps.cpp

@@ -6,7 +6,9 @@
 #include "proxy_sensor.h"
 #include "linear_func.h"
 #include "tps.h"
+#if EFI_UNIT_TEST
 #include "auto_generated_sensor.h"
+#endif
 
 #ifndef MAX_TPS_PPS_DISCREPANCY
 #define MAX_TPS_PPS_DISCREPANCY 5.0f

firmware/util/efilib.h

@@ -9,6 +9,7 @@
 
 #include <stdint.h>
 #include <rusefi/arrays.h>
+#include "rusefi_hw_enums.h"
 
 int djb2lowerCase(const char *str);