From 2f61ef9a5046c39f4eeaaf741031bb6a9607b8a6 Mon Sep 17 00:00:00 2001 From: Keith Packard Date: Wed, 15 Aug 2018 15:31:57 -0700 Subject: [PATCH] altos: Switch global 'log' to 'ao_log_data' Avoid conflicts with 'log' math function. Signed-off-by: Keith Packard --- src/kernel/ao_log.c | 12 ++--- src/kernel/ao_log.h | 2 +- src/kernel/ao_log_big.c | 56 ++++++++++---------- src/kernel/ao_log_gps.c | 64 +++++++++++----------- src/kernel/ao_log_mega.c | 106 ++++++++++++++++++------------------- src/kernel/ao_log_metrum.c | 54 +++++++++---------- src/kernel/ao_log_mini.c | 42 +++++++-------- 7 files changed, 168 insertions(+), 168 deletions(-) diff --git a/src/kernel/ao_log.c b/src/kernel/ao_log.c index f70c7232..5fb086ce 100644 --- a/src/kernel/ao_log.c +++ b/src/kernel/ao_log.c @@ -117,7 +117,7 @@ ao_log_erase_mark(void) * structure. */ -__xdata ao_log_type log; +__xdata ao_log_type ao_log_data; static uint8_t ao_log_csum(__xdata uint8_t *b) __reentrant @@ -154,7 +154,7 @@ ao_log_write(__xdata ao_log_type *log) __reentrant uint8_t ao_log_check_data(void) { - if (ao_log_csum((uint8_t *) &log) != 0) + if (ao_log_csum((uint8_t *) &ao_log_data) != 0) return 0; return 1; } @@ -162,7 +162,7 @@ ao_log_check_data(void) uint8_t ao_log_check_clear(void) { - uint8_t *b = (uint8_t *) &log; + uint8_t *b = (uint8_t *) &ao_log_data; uint8_t i; for (i = 0; i < sizeof (ao_log_type); i++) { @@ -176,17 +176,17 @@ int16_t ao_log_flight(uint8_t slot) { if (!ao_storage_read(ao_log_pos(slot), - &log, + &ao_log_data, sizeof (ao_log_type))) return -(int16_t) (slot + 1); if (ao_log_check_clear()) return 0; - if (!ao_log_check_data() || log.type != AO_LOG_FLIGHT) + if (!ao_log_check_data() || ao_log_data.type != AO_LOG_FLIGHT) return -(int16_t) (slot + 1); - return log.u.flight.flight; + return ao_log_data.u.flight.flight; } #endif diff --git a/src/kernel/ao_log.h b/src/kernel/ao_log.h index 5f04ef9a..e2f1e0e4 100644 --- a/src/kernel/ao_log.h +++ b/src/kernel/ao_log.h @@ -507,7 +507,7 @@ typedef struct ao_log_record ao_log_type; #endif #ifndef AO_LOG_UNCOMMON -extern __xdata ao_log_type log; +extern __xdata ao_log_type ao_log_data; #define AO_LOG_SIZE sizeof(ao_log_type) diff --git a/src/kernel/ao_log_big.c b/src/kernel/ao_log_big.c index 28a893c7..7d7332e9 100644 --- a/src/kernel/ao_log_big.c +++ b/src/kernel/ao_log_big.c @@ -41,13 +41,13 @@ ao_log(void) while (!ao_log_running) ao_sleep(&ao_log_running); - log.type = AO_LOG_FLIGHT; - log.tick = ao_sample_tick; + ao_log_data.type = AO_LOG_FLIGHT; + ao_log_data.tick = ao_sample_tick; #if HAS_ACCEL - log.u.flight.ground_accel = ao_ground_accel; + ao_log_data.u.flight.ground_accel = ao_ground_accel; #endif - log.u.flight.flight = ao_flight_number; - ao_log_write(&log); + ao_log_data.u.flight.flight = ao_flight_number; + ao_log_write(&ao_log_data); /* Write the whole contents of the ring to the log * when starting up. @@ -58,38 +58,38 @@ ao_log(void) for (;;) { /* Write samples to EEPROM */ while (ao_log_data_pos != ao_sample_data) { - log.tick = ao_data_ring[ao_log_data_pos].tick; - if ((int16_t) (log.tick - next_sensor) >= 0) { - log.type = AO_LOG_SENSOR; - log.u.sensor.accel = ao_data_ring[ao_log_data_pos].adc.accel; - log.u.sensor.pres = ao_data_ring[ao_log_data_pos].adc.pres; - ao_log_write(&log); + ao_log_data.tick = ao_data_ring[ao_log_data_pos].tick; + if ((int16_t) (ao_log_data.tick - next_sensor) >= 0) { + ao_log_data.type = AO_LOG_SENSOR; + ao_log_data.u.sensor.accel = ao_data_ring[ao_log_data_pos].adc.accel; + ao_log_data.u.sensor.pres = ao_data_ring[ao_log_data_pos].adc.pres; + ao_log_write(&ao_log_data); if (ao_log_state <= ao_flight_coast) - next_sensor = log.tick + AO_SENSOR_INTERVAL_ASCENT; + next_sensor = ao_log_data.tick + AO_SENSOR_INTERVAL_ASCENT; else - next_sensor = log.tick + AO_SENSOR_INTERVAL_DESCENT; + next_sensor = ao_log_data.tick + AO_SENSOR_INTERVAL_DESCENT; } - if ((int16_t) (log.tick - next_other) >= 0) { - log.type = AO_LOG_TEMP_VOLT; - log.u.temp_volt.temp = ao_data_ring[ao_log_data_pos].adc.temp; - log.u.temp_volt.v_batt = ao_data_ring[ao_log_data_pos].adc.v_batt; - ao_log_write(&log); - log.type = AO_LOG_DEPLOY; - log.u.deploy.drogue = ao_data_ring[ao_log_data_pos].adc.sense_d; - log.u.deploy.main = ao_data_ring[ao_log_data_pos].adc.sense_m; - ao_log_write(&log); - next_other = log.tick + AO_OTHER_INTERVAL; + if ((int16_t) (ao_log_data.tick - next_other) >= 0) { + ao_log_data.type = AO_LOG_TEMP_VOLT; + ao_log_data.u.temp_volt.temp = ao_data_ring[ao_log_data_pos].adc.temp; + ao_log_data.u.temp_volt.v_batt = ao_data_ring[ao_log_data_pos].adc.v_batt; + ao_log_write(&ao_log_data); + ao_log_data.type = AO_LOG_DEPLOY; + ao_log_data.u.deploy.drogue = ao_data_ring[ao_log_data_pos].adc.sense_d; + ao_log_data.u.deploy.main = ao_data_ring[ao_log_data_pos].adc.sense_m; + ao_log_write(&ao_log_data); + next_other = ao_log_data.tick + AO_OTHER_INTERVAL; } ao_log_data_pos = ao_data_ring_next(ao_log_data_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_sample_tick; - log.u.state.state = ao_log_state; - log.u.state.reason = 0; - ao_log_write(&log); + ao_log_data.type = AO_LOG_STATE; + ao_log_data.tick = ao_sample_tick; + ao_log_data.u.state.state = ao_log_state; + ao_log_data.u.state.reason = 0; + ao_log_write(&ao_log_data); if (ao_log_state == ao_flight_landed) ao_log_stop(); diff --git a/src/kernel/ao_log_gps.c b/src/kernel/ao_log_gps.c index a55d93f1..7284932c 100644 --- a/src/kernel/ao_log_gps.c +++ b/src/kernel/ao_log_gps.c @@ -27,37 +27,37 @@ void ao_log_gps_flight(void) { - log.type = AO_LOG_FLIGHT; - log.tick = ao_time(); - log.u.flight.flight = ao_flight_number; - ao_log_write(&log); + ao_log_data.type = AO_LOG_FLIGHT; + ao_log_data.tick = ao_time(); + ao_log_data.u.flight.flight = ao_flight_number; + ao_log_write(&ao_log_data); } void ao_log_gps_data(uint16_t tick, struct ao_telemetry_location *gps_data) { - log.tick = tick; - log.type = AO_LOG_GPS_TIME; - log.u.gps.latitude = gps_data->latitude; - log.u.gps.longitude = gps_data->longitude; - log.u.gps.altitude_low = gps_data->altitude_low; - log.u.gps.altitude_high = gps_data->altitude_high; + ao_log_data.tick = tick; + ao_log_data.type = AO_LOG_GPS_TIME; + ao_log_data.u.gps.latitude = gps_data->latitude; + ao_log_data.u.gps.longitude = gps_data->longitude; + ao_log_data.u.gps.altitude_low = gps_data->altitude_low; + ao_log_data.u.gps.altitude_high = gps_data->altitude_high; - log.u.gps.hour = gps_data->hour; - log.u.gps.minute = gps_data->minute; - log.u.gps.second = gps_data->second; - log.u.gps.flags = gps_data->flags; - log.u.gps.year = gps_data->year; - log.u.gps.month = gps_data->month; - log.u.gps.day = gps_data->day; - log.u.gps.course = gps_data->course; - log.u.gps.ground_speed = gps_data->ground_speed; - log.u.gps.climb_rate = gps_data->climb_rate; - log.u.gps.pdop = gps_data->pdop; - log.u.gps.hdop = gps_data->hdop; - log.u.gps.vdop = gps_data->vdop; - log.u.gps.mode = gps_data->mode; - ao_log_write(&log); + ao_log_data.u.gps.hour = gps_data->hour; + ao_log_data.u.gps.minute = gps_data->minute; + ao_log_data.u.gps.second = gps_data->second; + ao_log_data.u.gps.flags = gps_data->flags; + ao_log_data.u.gps.year = gps_data->year; + ao_log_data.u.gps.month = gps_data->month; + ao_log_data.u.gps.day = gps_data->day; + ao_log_data.u.gps.course = gps_data->course; + ao_log_data.u.gps.ground_speed = gps_data->ground_speed; + ao_log_data.u.gps.climb_rate = gps_data->climb_rate; + ao_log_data.u.gps.pdop = gps_data->pdop; + ao_log_data.u.gps.hdop = gps_data->hdop; + ao_log_data.u.gps.vdop = gps_data->vdop; + ao_log_data.u.gps.mode = gps_data->mode; + ao_log_write(&ao_log_data); } void @@ -65,27 +65,27 @@ ao_log_gps_tracking(uint16_t tick, struct ao_telemetry_satellite *gps_tracking_d { uint8_t c, n, i; - log.tick = tick; - log.type = AO_LOG_GPS_SAT; + ao_log_data.tick = tick; + ao_log_data.type = AO_LOG_GPS_SAT; i = 0; n = gps_tracking_data->channels; for (c = 0; c < n; c++) - if ((log.u.gps_sat.sats[i].svid = gps_tracking_data->sats[c].svid)) + if ((ao_log_data.u.gps_sat.sats[i].svid = gps_tracking_data->sats[c].svid)) { - log.u.gps_sat.sats[i].c_n = gps_tracking_data->sats[c].c_n_1; + ao_log_data.u.gps_sat.sats[i].c_n = gps_tracking_data->sats[c].c_n_1; i++; if (i >= 12) break; } - log.u.gps_sat.channels = i; - ao_log_write(&log); + ao_log_data.u.gps_sat.channels = i; + ao_log_write(&ao_log_data); } int8_t ao_log_check(uint32_t pos) { if (!ao_storage_read(pos, - &log, + &ao_log_data, sizeof (struct ao_log_gps))) return AO_LOG_INVALID; diff --git a/src/kernel/ao_log_mega.c b/src/kernel/ao_log_mega.c index c6bdf1e2..0ae4e536 100644 --- a/src/kernel/ao_log_mega.c +++ b/src/kernel/ao_log_mega.c @@ -47,22 +47,22 @@ ao_log(void) ao_sleep(&ao_log_running); #if HAS_FLIGHT - log.type = AO_LOG_FLIGHT; - log.tick = ao_sample_tick; + ao_log_data.type = AO_LOG_FLIGHT; + ao_log_data.tick = ao_sample_tick; #if HAS_ACCEL - log.u.flight.ground_accel = ao_ground_accel; + ao_log_data.u.flight.ground_accel = ao_ground_accel; #endif #if HAS_GYRO - log.u.flight.ground_accel_along = ao_ground_accel_along; - log.u.flight.ground_accel_across = ao_ground_accel_across; - log.u.flight.ground_accel_through = ao_ground_accel_through; - log.u.flight.ground_roll = ao_ground_roll; - log.u.flight.ground_pitch = ao_ground_pitch; - log.u.flight.ground_yaw = ao_ground_yaw; + ao_log_data.u.flight.ground_accel_along = ao_ground_accel_along; + ao_log_data.u.flight.ground_accel_across = ao_ground_accel_across; + ao_log_data.u.flight.ground_accel_through = ao_ground_accel_through; + ao_log_data.u.flight.ground_roll = ao_ground_roll; + ao_log_data.u.flight.ground_pitch = ao_ground_pitch; + ao_log_data.u.flight.ground_yaw = ao_ground_yaw; #endif - log.u.flight.ground_pres = ao_ground_pres; - log.u.flight.flight = ao_flight_number; - ao_log_write(&log); + ao_log_data.u.flight.ground_pres = ao_ground_pres; + ao_log_data.u.flight.flight = ao_flight_number; + ao_log_write(&ao_log_data); #endif /* Write the whole contents of the ring to the log @@ -74,54 +74,54 @@ ao_log(void) for (;;) { /* Write samples to EEPROM */ while (ao_log_data_pos != ao_data_head) { - log.tick = ao_data_ring[ao_log_data_pos].tick; - if ((int16_t) (log.tick - next_sensor) >= 0) { - log.type = AO_LOG_SENSOR; + ao_log_data.tick = ao_data_ring[ao_log_data_pos].tick; + if ((int16_t) (ao_log_data.tick - next_sensor) >= 0) { + ao_log_data.type = AO_LOG_SENSOR; #if HAS_MS5607 - log.u.sensor.pres = ao_data_ring[ao_log_data_pos].ms5607_raw.pres; - log.u.sensor.temp = ao_data_ring[ao_log_data_pos].ms5607_raw.temp; + ao_log_data.u.sensor.pres = ao_data_ring[ao_log_data_pos].ms5607_raw.pres; + ao_log_data.u.sensor.temp = ao_data_ring[ao_log_data_pos].ms5607_raw.temp; #endif #if HAS_MPU6000 - log.u.sensor.accel_x = ao_data_ring[ao_log_data_pos].mpu6000.accel_x; - log.u.sensor.accel_y = ao_data_ring[ao_log_data_pos].mpu6000.accel_y; - log.u.sensor.accel_z = ao_data_ring[ao_log_data_pos].mpu6000.accel_z; - log.u.sensor.gyro_x = ao_data_ring[ao_log_data_pos].mpu6000.gyro_x; - log.u.sensor.gyro_y = ao_data_ring[ao_log_data_pos].mpu6000.gyro_y; - log.u.sensor.gyro_z = ao_data_ring[ao_log_data_pos].mpu6000.gyro_z; + ao_log_data.u.sensor.accel_x = ao_data_ring[ao_log_data_pos].mpu6000.accel_x; + ao_log_data.u.sensor.accel_y = ao_data_ring[ao_log_data_pos].mpu6000.accel_y; + ao_log_data.u.sensor.accel_z = ao_data_ring[ao_log_data_pos].mpu6000.accel_z; + ao_log_data.u.sensor.gyro_x = ao_data_ring[ao_log_data_pos].mpu6000.gyro_x; + ao_log_data.u.sensor.gyro_y = ao_data_ring[ao_log_data_pos].mpu6000.gyro_y; + ao_log_data.u.sensor.gyro_z = ao_data_ring[ao_log_data_pos].mpu6000.gyro_z; #endif #if HAS_HMC5883 - log.u.sensor.mag_x = ao_data_ring[ao_log_data_pos].hmc5883.x; - log.u.sensor.mag_z = ao_data_ring[ao_log_data_pos].hmc5883.z; - log.u.sensor.mag_y = ao_data_ring[ao_log_data_pos].hmc5883.y; + ao_log_data.u.sensor.mag_x = ao_data_ring[ao_log_data_pos].hmc5883.x; + ao_log_data.u.sensor.mag_z = ao_data_ring[ao_log_data_pos].hmc5883.z; + ao_log_data.u.sensor.mag_y = ao_data_ring[ao_log_data_pos].hmc5883.y; #endif #if HAS_MPU9250 - log.u.sensor.accel_x = ao_data_ring[ao_log_data_pos].mpu9250.accel_x; - log.u.sensor.accel_y = ao_data_ring[ao_log_data_pos].mpu9250.accel_y; - log.u.sensor.accel_z = ao_data_ring[ao_log_data_pos].mpu9250.accel_z; - log.u.sensor.gyro_x = ao_data_ring[ao_log_data_pos].mpu9250.gyro_x; - log.u.sensor.gyro_y = ao_data_ring[ao_log_data_pos].mpu9250.gyro_y; - log.u.sensor.gyro_z = ao_data_ring[ao_log_data_pos].mpu9250.gyro_z; - log.u.sensor.mag_x = ao_data_ring[ao_log_data_pos].mpu9250.mag_x; - log.u.sensor.mag_z = ao_data_ring[ao_log_data_pos].mpu9250.mag_z; - log.u.sensor.mag_y = ao_data_ring[ao_log_data_pos].mpu9250.mag_y; + ao_log_data.u.sensor.accel_x = ao_data_ring[ao_log_data_pos].mpu9250.accel_x; + ao_log_data.u.sensor.accel_y = ao_data_ring[ao_log_data_pos].mpu9250.accel_y; + ao_log_data.u.sensor.accel_z = ao_data_ring[ao_log_data_pos].mpu9250.accel_z; + ao_log_data.u.sensor.gyro_x = ao_data_ring[ao_log_data_pos].mpu9250.gyro_x; + ao_log_data.u.sensor.gyro_y = ao_data_ring[ao_log_data_pos].mpu9250.gyro_y; + ao_log_data.u.sensor.gyro_z = ao_data_ring[ao_log_data_pos].mpu9250.gyro_z; + ao_log_data.u.sensor.mag_x = ao_data_ring[ao_log_data_pos].mpu9250.mag_x; + ao_log_data.u.sensor.mag_z = ao_data_ring[ao_log_data_pos].mpu9250.mag_z; + ao_log_data.u.sensor.mag_y = ao_data_ring[ao_log_data_pos].mpu9250.mag_y; #endif - log.u.sensor.accel = ao_data_accel(&ao_data_ring[ao_log_data_pos]); - ao_log_write(&log); + ao_log_data.u.sensor.accel = ao_data_accel(&ao_data_ring[ao_log_data_pos]); + ao_log_write(&ao_log_data); if (ao_log_state <= ao_flight_coast) - next_sensor = log.tick + AO_SENSOR_INTERVAL_ASCENT; + next_sensor = ao_log_data.tick + AO_SENSOR_INTERVAL_ASCENT; else - next_sensor = log.tick + AO_SENSOR_INTERVAL_DESCENT; + next_sensor = ao_log_data.tick + AO_SENSOR_INTERVAL_DESCENT; } - if ((int16_t) (log.tick - next_other) >= 0) { - log.type = AO_LOG_TEMP_VOLT; - log.u.volt.v_batt = ao_data_ring[ao_log_data_pos].adc.v_batt; - log.u.volt.v_pbatt = ao_data_ring[ao_log_data_pos].adc.v_pbatt; - log.u.volt.n_sense = AO_ADC_NUM_SENSE; + if ((int16_t) (ao_log_data.tick - next_other) >= 0) { + ao_log_data.type = AO_LOG_TEMP_VOLT; + ao_log_data.u.volt.v_batt = ao_data_ring[ao_log_data_pos].adc.v_batt; + ao_log_data.u.volt.v_pbatt = ao_data_ring[ao_log_data_pos].adc.v_pbatt; + ao_log_data.u.volt.n_sense = AO_ADC_NUM_SENSE; for (i = 0; i < AO_ADC_NUM_SENSE; i++) - log.u.volt.sense[i] = ao_data_ring[ao_log_data_pos].adc.sense[i]; - log.u.volt.pyro = ao_pyro_fired; - ao_log_write(&log); - next_other = log.tick + AO_OTHER_INTERVAL; + ao_log_data.u.volt.sense[i] = ao_data_ring[ao_log_data_pos].adc.sense[i]; + ao_log_data.u.volt.pyro = ao_pyro_fired; + ao_log_write(&ao_log_data); + next_other = ao_log_data.tick + AO_OTHER_INTERVAL; } ao_log_data_pos = ao_data_ring_next(ao_log_data_pos); } @@ -129,11 +129,11 @@ ao_log(void) /* 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_time(); - log.u.state.state = ao_log_state; - log.u.state.reason = 0; - ao_log_write(&log); + ao_log_data.type = AO_LOG_STATE; + ao_log_data.tick = ao_time(); + ao_log_data.u.state.state = ao_log_state; + ao_log_data.u.state.reason = 0; + ao_log_write(&ao_log_data); if (ao_log_state == ao_flight_landed) ao_log_stop(); diff --git a/src/kernel/ao_log_metrum.c b/src/kernel/ao_log_metrum.c index afb8f637..34729070 100644 --- a/src/kernel/ao_log_metrum.c +++ b/src/kernel/ao_log_metrum.c @@ -46,14 +46,14 @@ ao_log(void) ao_sleep(&ao_log_running); #if HAS_FLIGHT - log.type = AO_LOG_FLIGHT; - log.tick = ao_sample_tick; + ao_log_data.type = AO_LOG_FLIGHT; + ao_log_data.tick = ao_sample_tick; #if HAS_ACCEL - log.u.flight.ground_accel = ao_ground_accel; + ao_log_data.u.flight.ground_accel = ao_ground_accel; #endif - log.u.flight.ground_pres = ao_ground_pres; - log.u.flight.flight = ao_flight_number; - ao_log_write(&log); + ao_log_data.u.flight.ground_pres = ao_ground_pres; + ao_log_data.u.flight.flight = ao_flight_number; + ao_log_write(&ao_log_data); #endif /* Write the whole contents of the ring to the log @@ -65,29 +65,29 @@ ao_log(void) for (;;) { /* Write samples to EEPROM */ while (ao_log_data_pos != ao_data_head) { - log.tick = ao_data_ring[ao_log_data_pos].tick; - if ((int16_t) (log.tick - next_sensor) >= 0) { - log.type = AO_LOG_SENSOR; + ao_log_data.tick = ao_data_ring[ao_log_data_pos].tick; + if ((int16_t) (ao_log_data.tick - next_sensor) >= 0) { + ao_log_data.type = AO_LOG_SENSOR; #if HAS_MS5607 - log.u.sensor.pres = ao_data_ring[ao_log_data_pos].ms5607_raw.pres; - log.u.sensor.temp = ao_data_ring[ao_log_data_pos].ms5607_raw.temp; + ao_log_data.u.sensor.pres = ao_data_ring[ao_log_data_pos].ms5607_raw.pres; + ao_log_data.u.sensor.temp = ao_data_ring[ao_log_data_pos].ms5607_raw.temp; #endif #if HAS_ACCEL - log.u.sensor.accel = ao_data_accel(&ao_data_ring[ao_log_data_pos]); + ao_log_data.u.sensor.accel = ao_data_accel(&ao_data_ring[ao_log_data_pos]); #endif - ao_log_write(&log); + ao_log_write(&ao_log_data); if (ao_log_state <= ao_flight_coast) - next_sensor = log.tick + AO_SENSOR_INTERVAL_ASCENT; + next_sensor = ao_log_data.tick + AO_SENSOR_INTERVAL_ASCENT; else - next_sensor = log.tick + AO_SENSOR_INTERVAL_DESCENT; + next_sensor = ao_log_data.tick + AO_SENSOR_INTERVAL_DESCENT; } - if ((int16_t) (log.tick - next_other) >= 0) { - log.type = AO_LOG_TEMP_VOLT; - log.u.volt.v_batt = ao_data_ring[ao_log_data_pos].adc.v_batt; - log.u.volt.sense_a = ao_data_ring[ao_log_data_pos].adc.sense_a; - log.u.volt.sense_m = ao_data_ring[ao_log_data_pos].adc.sense_m; - ao_log_write(&log); - next_other = log.tick + AO_OTHER_INTERVAL; + if ((int16_t) (ao_log_data.tick - next_other) >= 0) { + ao_log_data.type = AO_LOG_TEMP_VOLT; + ao_log_data.u.volt.v_batt = ao_data_ring[ao_log_data_pos].adc.v_batt; + ao_log_data.u.volt.sense_a = ao_data_ring[ao_log_data_pos].adc.sense_a; + ao_log_data.u.volt.sense_m = ao_data_ring[ao_log_data_pos].adc.sense_m; + ao_log_write(&ao_log_data); + next_other = ao_log_data.tick + AO_OTHER_INTERVAL; } ao_log_data_pos = ao_data_ring_next(ao_log_data_pos); } @@ -95,11 +95,11 @@ ao_log(void) /* 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_time(); - log.u.state.state = ao_log_state; - log.u.state.reason = 0; - ao_log_write(&log); + ao_log_data.type = AO_LOG_STATE; + ao_log_data.tick = ao_time(); + ao_log_data.u.state.state = ao_log_state; + ao_log_data.u.state.reason = 0; + ao_log_write(&ao_log_data); if (ao_log_state == ao_flight_landed) ao_log_stop(); diff --git a/src/kernel/ao_log_mini.c b/src/kernel/ao_log_mini.c index af2fa605..12c5d6bb 100644 --- a/src/kernel/ao_log_mini.c +++ b/src/kernel/ao_log_mini.c @@ -44,11 +44,11 @@ ao_log(void) ao_sleep(&ao_log_running); #if HAS_FLIGHT - log.type = AO_LOG_FLIGHT; - log.tick = ao_sample_tick; - log.u.flight.flight = ao_flight_number; - log.u.flight.ground_pres = ao_ground_pres; - ao_log_write(&log); + ao_log_data.type = AO_LOG_FLIGHT; + ao_log_data.tick = ao_sample_tick; + ao_log_data.u.flight.flight = ao_flight_number; + ao_log_data.u.flight.ground_pres = ao_ground_pres; + ao_log_write(&ao_log_data); #endif /* Write the whole contents of the ring to the log @@ -60,23 +60,23 @@ ao_log(void) for (;;) { /* Write samples to EEPROM */ while (ao_log_data_pos != ao_data_head) { - log.tick = ao_data_ring[ao_log_data_pos].tick; - if ((int16_t) (log.tick - next_sensor) >= 0) { - log.type = AO_LOG_SENSOR; - ao_log_pack24(log.u.sensor.pres, + ao_log_data.tick = ao_data_ring[ao_log_data_pos].tick; + if ((int16_t) (ao_log_data.tick - next_sensor) >= 0) { + ao_log_data.type = AO_LOG_SENSOR; + ao_log_pack24(ao_log_data.u.sensor.pres, ao_data_ring[ao_log_data_pos].ms5607_raw.pres); - ao_log_pack24(log.u.sensor.temp, + ao_log_pack24(ao_log_data.u.sensor.temp, ao_data_ring[ao_log_data_pos].ms5607_raw.temp); #if AO_LOG_FORMAT != AO_LOG_FORMAT_DETHERM - log.u.sensor.sense_a = ao_data_ring[ao_log_data_pos].adc.sense_a; - log.u.sensor.sense_m = ao_data_ring[ao_log_data_pos].adc.sense_m; - log.u.sensor.v_batt = ao_data_ring[ao_log_data_pos].adc.v_batt; + ao_log_data.u.sensor.sense_a = ao_data_ring[ao_log_data_pos].adc.sense_a; + ao_log_data.u.sensor.sense_m = ao_data_ring[ao_log_data_pos].adc.sense_m; + ao_log_data.u.sensor.v_batt = ao_data_ring[ao_log_data_pos].adc.v_batt; #endif - ao_log_write(&log); + ao_log_write(&ao_log_data); if (ao_log_state <= ao_flight_coast) - next_sensor = log.tick + AO_SENSOR_INTERVAL_ASCENT; + next_sensor = ao_log_data.tick + AO_SENSOR_INTERVAL_ASCENT; else - next_sensor = log.tick + AO_SENSOR_INTERVAL_DESCENT; + next_sensor = ao_log_data.tick + AO_SENSOR_INTERVAL_DESCENT; } ao_log_data_pos = ao_data_ring_next(ao_log_data_pos); } @@ -84,11 +84,11 @@ ao_log(void) /* 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_time(); - log.u.state.state = ao_log_state; - log.u.state.reason = 0; - ao_log_write(&log); + ao_log_data.type = AO_LOG_STATE; + ao_log_data.tick = ao_time(); + ao_log_data.u.state.state = ao_log_state; + ao_log_data.u.state.reason = 0; + ao_log_write(&ao_log_data); if (ao_log_state == ao_flight_landed) ao_log_stop(); -- 2.30.2