altos: Include struct ao_data declaration only when used

[fw/altos] / src / core / ao_data.h

diff --git a/src/core/ao_data.h b/src/core/ao_data.h
index 3b66ef5d1a598e65e9369d6d7adcb33f10a08eee..1153d433669facdfbd58790367d526d504c6a4a6 100644 (file)
--- a/src/core/ao_data.h
+++ b/src/core/ao_data.h
@@ -28,7 +28,7 @@
 #include <ao_ms5607.h>
 #define AO_DATA_MS5607 (1 << 1)
 #else
-#define AO_DATA_MS5607 (1 << 1)
+#define AO_DATA_MS5607 0
 #endif
 
 #if HAS_MPU6000
@@ -52,6 +52,8 @@
 #define AO_DATA_MMA655X 0
 #endif
 
+#ifdef AO_DATA_RING
+
 #define AO_DATA_ALL    (AO_DATA_ADC|AO_DATA_MS5607|AO_DATA_MPU6000|AO_DATA_HMC5883|AO_DATA_MMA655X)
 
 struct ao_data {
@@ -95,18 +97,29 @@ extern volatile __data uint8_t              ao_data_count;
        } while (0);
 
 /*
- * Wait for data to be completed by looking at the
- * indicated bit
+ * Wait until it is time to write a sensor sample; this is
+ * signaled by the timer tick
  */
 #define AO_DATA_WAIT() do {                            \
                ao_sleep((void *) &ao_data_count);      \
        } while (0)
 
+#endif /* AO_DATA_RING */
 
-#if HAS_MS5607
+#if !HAS_BARO && HAS_MS5607
+
+/* Either an MS5607 or an MS5611 hooked to a SPI port
+ */
+
+#define HAS_BARO       1
 
 typedef int32_t        pres_t;
-typedef int32_t alt_t;
+
+#ifndef AO_ALT_TYPE
+#define AO_ALT_TYPE    int32_t
+#endif
+
+typedef AO_ALT_TYPE    alt_t;
 
 #define ao_data_pres_cook(packet)      ao_ms5607_convert(&packet->ms5607_raw, &packet->ms5607_cooked)
 
@@ -115,7 +128,11 @@ typedef int32_t alt_t;
 
 #define pres_to_altitude(p)    ao_pa_to_altitude(p)
 
-#else /* HAS_MS5607 */
+#endif
+
+#if !HAS_BARO && HAS_ADC
+
+#define HAS_BARO       1
 
 typedef int16_t pres_t;
 typedef int16_t alt_t;
@@ -125,7 +142,11 @@ typedef int16_t alt_t;
 #define pres_to_altitude(p)    ao_pres_to_altitude(p)
 #define ao_data_pres_cook(p)
 
-#endif /* else HAS_MS5607 */
+#endif
+
+#if !HAS_BARO
+typedef int16_t alt_t;
+#endif
 
 /*
  * Need a few macros to pull data from the sensors:
@@ -136,17 +157,12 @@ typedef int16_t alt_t;
  * ao_data_accel_invert        - flip rocket ends for positive acceleration
  */
 
-#if HAS_MPU6000 && !HAS_HIGHG_ACCEL
-
-typedef int16_t accel_t;
+#if HAS_ACCEL
 
-/* MPU6000 is hooked up so that positive y is positive acceleration */
-#define ao_data_accel(packet)                  ((packet)->mpu6000.accel_y)
-#define ao_data_accel_cook(packet)             (-(packet)->mpu6000.accel_y)
-#define ao_data_set_accel(packet, accel)       ((packet)->mpu6000.accel_y = (accel))
-#define ao_data_accel_invert(a)                        (-(a))
-
-#else /* HAS_MPU6000 && !HAS_HIGHG_ACCEL */
+/* This section is for an analog accelerometer hooked to one of the ADC pins. As
+ * those are 5V parts, this also requires that the 5V supply be hooked to to anothe ADC
+ * pin so that the both can be measured to correct for changes between the 3.3V and 5V rails
+ */
 
 typedef int16_t accel_t;
 #define ao_data_accel(packet)                  ((packet)->adc.accel)
@@ -245,6 +261,35 @@ typedef int16_t accel_t;
 
 #endif /* HAS_ACCEL_REF */
 
-#endif /* else some other accel sensor */
+#endif /* HAS_ACCEL */
+
+#if !HAS_ACCEL && HAS_MMA655X
+
+#define HAS_ACCEL      1
+
+typedef int16_t accel_t;
+
+/* MMA655X is hooked up so that positive values represent negative acceleration */
+
+#define ao_data_accel(packet)                  ((packet)->mma655x)
+#define ao_data_accel_cook(packet)             ((packet)->mma655x)
+#define ao_data_set_accel(packet, accel)       ((packet)->mma655x = (accel))
+#define ao_data_accel_invert(accel)            (4095 - (accel))
+
+#endif
+
+#if !HAS_ACCEL && HAS_MPU6000
+
+#define HAS_ACCEL      1
+
+typedef int16_t accel_t;
+
+/* MPU6000 is hooked up so that positive y is positive acceleration */
+#define ao_data_accel(packet)                  ((packet)->mpu6000.accel_y)
+#define ao_data_accel_cook(packet)             (-(packet)->mpu6000.accel_y)
+#define ao_data_set_accel(packet, accel)       ((packet)->mpu6000.accel_y = (accel))
+#define ao_data_accel_invert(a)                        (-(a))
+
+#endif
 
 #endif /* _AO_DATA_H_ */