a stab at turning on rudimentary logging for telefiretwo

[fw/altos] / src / kernel / ao_report.c

diff --git a/src/kernel/ao_report.c b/src/kernel/ao_report.c
index 5314fc8fb82fa7a698dd3743939ef23fe54dbeda..af48b390daae0c0095c074dd6818aabc27e86d76 100644 (file)
--- a/src/kernel/ao_report.c
+++ b/src/kernel/ao_report.c
@@ -3,7 +3,8 @@
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
  *
  * This program is distributed in the hope that it will be useful, but
  * WITHOUT ANY WARRANTY; without even the implied warranty of
@@ -44,9 +45,16 @@ static const uint8_t flight_reports[] = {
 #define mid(time)      ao_beep_for(AO_BEEP_MID, time)
 #define high(time)     ao_beep_for(AO_BEEP_HIGH, time)
 #else
-#define low(time)      ao_led_for(AO_LED_GREEN, time)
-#define mid(time)      ao_led_for(AO_LED_RED, time)
-#define high(time)     ao_led_for(AO_LED_GREEN|AO_LED_RED, time)
+#ifndef AO_LED_LOW
+#define AO_LED_LOW     AO_LED_GREEN
+#endif
+#ifndef AO_LED_MID
+#define AO_LED_MID     AO_LED_RED
+#endif
+
+#define low(time)      ao_led_for(AO_LED_LOW, time)
+#define mid(time)      ao_led_for(AO_LED_MID, time)
+#define high(time)     ao_led_for(AO_LED_MID|AO_LED_LOW, time)
 #endif
 #define pause(time)    ao_delay(time)
 
@@ -246,15 +254,15 @@ ao_report_continuity(void) __reentrant
 void
 ao_report(void)
 {
-       ao_report_state = ao_flight_state;
        for(;;) {
+               ao_report_state = ao_flight_state;
 #if HAS_BATTERY_REPORT
-               if (ao_flight_state == ao_flight_startup)
+               if (ao_report_state == ao_flight_startup)
                        ao_report_battery();
                else
 #endif
                        ao_report_beep();
-               if (ao_flight_state == ao_flight_landed) {
+               if (ao_report_state == ao_flight_landed) {
                        ao_report_altitude();
 #if HAS_FLIGHT
                        ao_delay(AO_SEC_TO_TICKS(5));
@@ -262,7 +270,7 @@ ao_report(void)
 #endif
                }
 #if HAS_IGNITE_REPORT
-               if (ao_flight_state == ao_flight_idle)
+               if (ao_report_state == ao_flight_idle)
                        ao_report_continuity();
                while (ao_flight_state == ao_flight_pad) {
                        uint8_t c;
@@ -272,10 +280,8 @@ ao_report(void)
                                pause(AO_MS_TO_TICKS(100));
                }
 #endif
-
                while (ao_report_state == ao_flight_state)
                        ao_sleep(DATA_TO_XDATA(&ao_flight_state));
-               ao_report_state = ao_flight_state;
        }
 }