Revert "Used forward references to remove #includes from header files"

src/main/blackbox/blackbox.c

@@ -72,7 +72,6 @@
 #include "flight/altitudehold.h"
 #include "flight/failsafe.h"
 #include "flight/imu.h"
-#include "flight/pid.h"
 #include "flight/navigation.h"
 
 #include "config/runtime_config.h"

src/main/blackbox/blackbox_io.c

@@ -53,7 +53,6 @@
 #include "flight/altitudehold.h"
 #include "flight/failsafe.h"
 #include "flight/imu.h"
-#include "flight/pid.h"
 #include "flight/navigation.h"
 
 #include "config/runtime_config.h"

src/main/config/config.c

@@ -23,7 +23,6 @@
 #include "debug.h"
 
 #include "build_config.h"
-#include "debug.h"
 
 #include "blackbox/blackbox_io.h"
 

src/main/drivers/bus_i2c.h

@@ -21,8 +21,8 @@
 #define I2C_LONG_TIMEOUT             ((uint32_t)(10 * I2C_SHORT_TIMEOUT))
 #define I2C_DEFAULT_TIMEOUT          I2C_SHORT_TIMEOUT
 
-#include "io.h"
-#include "rcc.h"
+#include "drivers/io.h"
+#include "drivers/rcc.h"
 
 #ifndef I2C_DEVICE
 #define I2C_DEVICE I2CINVALID

src/main/drivers/exti.h

@@ -18,7 +18,7 @@
 
 #pragma once
 
-#include "io.h"
+#include "drivers/io.h"
 
 // old EXTI interface, to be replaced
 typedef struct extiConfig_s {

src/main/drivers/sound_beeper.h

@@ -17,7 +17,7 @@
 
 #pragma once
 
-#include "io.h"
+#include "drivers/io.h"
 
 #ifdef BEEPER
 #define BEEP_TOGGLE              systemBeepToggle()

src/main/flight/altitudehold.h

@@ -15,21 +15,19 @@
  * along with Cleanflight.  If not, see <http://www.gnu.org/licenses/>.
  */
 
-#pragma once
+#include "io/escservo.h"
+#include "io/rc_controls.h"
+#include "flight/pid.h"
+
+#include "sensors/barometer.h"
 
 extern int32_t AltHold;
 extern int32_t vario;
 
 void calculateEstimatedAltitude(uint32_t currentTime);
 
-struct pidProfile_s;
-struct barometerConfig_s;
-struct rcControlsConfig_s;
-struct escAndServoConfig_s;
-void configureAltitudeHold(struct pidProfile_s *initialPidProfile, struct barometerConfig_s *intialBarometerConfig, struct rcControlsConfig_s *initialRcControlsConfig, struct escAndServoConfig_s *initialEscAndServoConfig);
-
-struct airplaneConfig_t;
-void applyAltHold(struct airplaneConfig_s *airplaneConfig);
+void configureAltitudeHold(pidProfile_t *initialPidProfile, barometerConfig_t *intialBarometerConfig, rcControlsConfig_t *initialRcControlsConfig, escAndServoConfig_t *initialEscAndServoConfig);
+void applyAltHold(airplaneConfig_t *airplaneConfig);
 void updateAltHoldState(void);
 void updateSonarAltHoldState(void);
 

src/main/flight/imu.h

@@ -69,10 +69,9 @@ typedef struct accProcessor_s {
     accProcessorState_e state;
 } accProcessor_t;
 
-struct pidProfile_s;
 void imuConfigure(
     imuRuntimeConfig_t *initialImuRuntimeConfig,
-    struct pidProfile_s *initialPidProfile,
+    pidProfile_t *initialPidProfile,
     accDeadband_t *initialAccDeadband,
     uint16_t throttle_correction_angle
 );

src/main/flight/pid.h

@@ -14,6 +14,7 @@
  * You should have received a copy of the GNU General Public License
  * along with Cleanflight.  If not, see <http://www.gnu.org/licenses/>.
  */
+#include "rx/rx.h"
 
 #pragma once
 

src/main/io/display.c

@@ -63,8 +63,6 @@
 
 #include "io/display.h"
 
-#include "rx/rx.h"
-
 #include "scheduler/scheduler.h"
 
 extern profile_t *currentProfile;

src/main/io/rc_controls.h

@@ -17,6 +17,8 @@
 
 #pragma once
 
+#include "rx/rx.h"
+
 typedef enum {
     BOXARM = 0,
     BOXANGLE,
@@ -165,9 +167,8 @@ typedef struct rcControlsConfig_s {
 bool areUsingSticksToArm(void);
 
 bool areSticksInApModePosition(uint16_t ap_mode);
-struct rxConfig_s;
-throttleStatus_e calculateThrottleStatus(struct rxConfig_s *rxConfig, uint16_t deadband3d_throttle);
-void processRcStickPositions(struct rxConfig_s *rxConfig, throttleStatus_e throttleStatus, bool disarm_kill_switch);
+throttleStatus_e calculateThrottleStatus(rxConfig_t *rxConfig, uint16_t deadband3d_throttle);
+void processRcStickPositions(rxConfig_t *rxConfig, throttleStatus_e throttleStatus, bool disarm_kill_switch);
 
 bool isRangeActive(uint8_t auxChannelIndex, channelRange_t *range);
 void updateActivatedModes(modeActivationCondition_t *modeActivationConditions);
@@ -256,7 +257,7 @@ bool isAirmodeActive(void);
 bool isSuperExpoActive(void);
 void resetAdjustmentStates(void);
 void updateAdjustmentStates(adjustmentRange_t *adjustmentRanges);
-void processRcAdjustments(controlRateConfig_t *controlRateConfig, struct rxConfig_s *rxConfig);
+void processRcAdjustments(controlRateConfig_t *controlRateConfig, rxConfig_t *rxConfig);
 
 bool isUsingSticksForArming(void);
 

src/main/io/rc_curves.c

@@ -18,13 +18,12 @@
 #include <stdbool.h>
 #include <stdint.h>
 
-#include "config/config.h"
-
-#include "io/escservo.h"
 #include "io/rc_controls.h"
+#include "io/escservo.h"
+
 #include "io/rc_curves.h"
 
-#include "rx/rx.h"
+#include "config/config.h"
 
 #define THROTTLE_LOOKUP_LENGTH 12
 static int16_t lookupThrottleRC[THROTTLE_LOOKUP_LENGTH];    // lookup table for expo & mid THROTTLE

src/main/rx/rx.c

@@ -41,8 +41,6 @@
 #include "drivers/timer.h"
 #include "drivers/pwm_rx.h"
 #include "drivers/system.h"
-
-#include "rx/rx.h"
 #include "rx/pwm.h"
 #include "rx/sbus.h"
 #include "rx/spektrum.h"
@@ -53,6 +51,8 @@
 #include "rx/ibus.h"
 #include "rx/jetiexbus.h"
 
+#include "rx/rx.h"
+
 
 //#define DEBUG_RX_SIGNAL_LOSS
 

src/main/rx/xbus.h

@@ -17,5 +17,7 @@
 
 #pragma once
 
+#include "rx/rx.h"
+
 bool xBusInit(rxConfig_t *rxConfig, rxRuntimeConfig_t *rxRuntimeConfig, rcReadRawDataPtr *callback);
 uint8_t xBusFrameStatus(void);

src/main/sensors/battery.c

@@ -35,8 +35,6 @@
 #include "io/rc_controls.h"
 #include "io/beeper.h"
 
-#include "rx/rx.h"
-
 #define VBATT_PRESENT_THRESHOLD_MV    10
 #define VBATT_LPF_FREQ  0.4f
 

src/main/telemetry/frsky.h

@@ -17,13 +17,14 @@
 
 #pragma once
 
+#include "rx/rx.h"
+
 typedef enum {
     FRSKY_VFAS_PRECISION_LOW = 0,
     FRSKY_VFAS_PRECISION_HIGH
 } frskyVFasPrecision_e;
 
-struct rxConfig_s;
-void handleFrSkyTelemetry(struct rxConfig_s *rxConfig, uint16_t deadband3d_throttle);
+void handleFrSkyTelemetry(rxConfig_t *rxConfig, uint16_t deadband3d_throttle);
 void checkFrSkyTelemetryState(void);
 
 void initFrSkyTelemetry(telemetryConfig_t *telemetryConfig);

src/main/telemetry/hott.h

@@ -488,8 +488,7 @@ typedef struct HOTT_AIRESC_MSG_s {
 void handleHoTTTelemetry(void);
 void checkHoTTTelemetryState(void);
 
-struct telemetryConfig_s;
-void initHoTTTelemetry(struct telemetryConfig_s *telemetryConfig);
+void initHoTTTelemetry(telemetryConfig_t *telemetryConfig);
 void configureHoTTTelemetryPort(void);
 void freeHoTTTelemetryPort(void);
 

src/main/telemetry/ltm.h

@@ -19,8 +19,7 @@
 
 #pragma once
 
-struct telemetryConfig_s;
-void initLtmTelemetry(struct telemetryConfig_s *initialTelemetryConfig);
+void initLtmTelemetry(telemetryConfig_t *initialTelemetryConfig);
 void handleLtmTelemetry(void);
 void checkLtmTelemetryState(void);
 

src/main/telemetry/smartport.h

@@ -7,8 +7,7 @@
 
 #pragma once
 
-struct telemetryConfig_s;
-void initSmartPortTelemetry(struct telemetryConfig_s *);
+void initSmartPortTelemetry(telemetryConfig_t *);
 
 void handleSmartPortTelemetry(void);
 void checkSmartPortTelemetryState(void);