Stop log dumping at flight end. Print 'end' at end of log.

[fw/altos] / ao_log.c

diff --git a/ao_log.c b/ao_log.c
index c74893f87af9a61b11d4343ae6a21b0b69630e5a..7284121db33c47b61b1f85659df69e54175badfc 100644 (file)
--- a/ao_log.c
+++ b/ao_log.c
@@ -121,19 +121,15 @@ ao_log(void)
        
        log.type = AO_LOG_FLIGHT;
        log.tick = ao_flight_tick;
-       log.u.flight.serial = 0;
+       log.u.flight.ground_accel = ao_ground_accel;
        log.u.flight.flight = ao_log_dump_flight + 1;
        ao_log_data(&log);
+
+       /* Write the whole contents of the ring to the log
+        * when starting up.
+        */
+       ao_log_adc_pos = ao_adc_ring_next(ao_adc_head);
        for (;;) {
-               /* Write state change to EEPROM */
-               if (ao_flight_state != ao_log_state) {
-                       ao_log_state = ao_flight_state;
-                       log.type = AO_LOG_STATE;
-                       log.tick = ao_flight_tick;
-                       log.u.state.state = ao_log_state;
-                       log.u.state.reason = 0;
-                       ao_log_data(&log);
-               }
                /* Write samples to EEPROM */
                while (ao_log_adc_pos != ao_adc_head) {
                        log.type = AO_LOG_SENSOR;
@@ -155,9 +151,25 @@ ao_log(void)
                        }
                        ao_log_adc_pos = ao_adc_ring_next(ao_log_adc_pos);
                }
+               /* Write state change to EEPROM */
+               if (ao_flight_state != ao_log_state) {
+                       ao_log_state = ao_flight_state;
+                       log.type = AO_LOG_STATE;
+                       log.tick = ao_flight_tick;
+                       log.u.state.state = ao_log_state;
+                       log.u.state.reason = 0;
+                       ao_log_data(&log);
+
+                       if (ao_log_state == ao_flight_landed)
+                               ao_log_stop();
+               }
                
                /* Wait for a while */
                ao_delay(AO_MS_TO_TICKS(100));
+
+               /* Stop logging when told to */
+               while (!ao_log_running)
+                       ao_sleep(&ao_log_running);
        }
 }
 
@@ -179,7 +191,7 @@ ao_log_stop(void)
 static void
 dump_log(void)
 {
-       __xdata uint8_t more;
+       uint8_t more;
 
        for (more = ao_log_dump_first(); more; more = ao_log_dump_next()) {
                printf("%c %4x %4x %4x\n",
@@ -187,7 +199,11 @@ dump_log(void)
                       ao_log_dump.tick,
                       ao_log_dump.u.anon.d0,
                       ao_log_dump.u.anon.d1);
+               if (ao_log_dump.type == AO_LOG_STATE &&
+                   ao_log_dump.u.state.state == ao_flight_landed)
+                       break;
        }
+       printf("end\n");
 }
 
 __code struct ao_cmds ao_log_cmds[] = {