altos: Blink on telem packet receive instead of toggle

[fw/altos] / src / core / ao_monitor.c

diff --git a/src/core/ao_monitor.c b/src/core/ao_monitor.c
index 69eb58e85f8ac1373b9576b66573ecedfefd8019..382af8a2a2b101426aaea54d3af82eb0a74fc3f7 100644 (file)
--- a/src/core/ao_monitor.c
+++ b/src/core/ao_monitor.c
@@ -25,15 +25,7 @@
 __xdata uint8_t ao_monitoring;
 __pdata uint8_t ao_monitor_led;
 
-#define AO_MONITOR_RING        8
-
-__xdata union ao_monitor {
-               struct ao_telemetry_raw_recv    raw;
-               struct ao_telemetry_orig_recv   orig;
-               struct ao_telemetry_tiny_recv   tiny;
-} ao_monitor_ring[AO_MONITOR_RING];
-
-#define ao_monitor_ring_next(n)        (((n) + 1) & (AO_MONITOR_RING - 1))
+__xdata union ao_monitor ao_monitor_ring[AO_MONITOR_RING];
 
 __data uint8_t ao_monitor_head;
 
@@ -63,7 +55,15 @@ ao_monitor_get(void)
                        continue;
                ao_monitor_head = ao_monitor_ring_next(ao_monitor_head);
                ao_wakeup(DATA_TO_XDATA(&ao_monitor_head));
-               ao_led_toggle(ao_monitor_led);
+       }
+}
+
+void
+ao_monitor_blink(void)
+{
+       for (;;) {
+               ao_sleep(DATA_TO_XDATA(&ao_monitor_head));
+               ao_led_for(ao_monitor_led, AO_MS_TO_TICKS(100));
        }
 }
 
@@ -159,7 +159,9 @@ ao_monitor_put(void)
                                ao_gps_print(&recv_orig.telemetry_orig.gps);
                                ao_gps_tracking_print(&recv_orig.telemetry_orig.gps_tracking);
                                putchar('\n');
+#if HAS_RSSI
                                ao_rssi_set(rssi);
+#endif
                        } else {
                                printf("CRC INVALID RSSI %3d\n", rssi);
                        }
@@ -222,7 +224,9 @@ ao_monitor_put(void)
                                       recv_tiny.telemetry_tiny.flight_vel,
                                       recv_tiny.telemetry_tiny.flight_pres);
 #endif
+#if HAS_RSSI
                                ao_rssi_set(rssi);
+#endif
                        } else {
                                printf("CRC INVALID RSSI %3d\n", rssi);
                        }
@@ -236,6 +240,12 @@ ao_monitor_put(void)
                                printf("%02x", byte);
                        }
                        printf("%02x\n", sum);
+#if HAS_RSSI
+                       if (recv_raw.packet[ao_monitoring + 1] & PKT_APPEND_STATUS_1_CRC_OK) {
+                               rssi = ((int16_t) recv_raw.packet[ao_monitoring] >> 1) - 74;
+                               ao_rssi_set(rssi);
+                       }
+#endif
                        break;
                }
                ao_usb_flush();
@@ -244,6 +254,7 @@ ao_monitor_put(void)
 
 __xdata struct ao_task ao_monitor_get_task;
 __xdata struct ao_task ao_monitor_put_task;
+__xdata struct ao_task ao_monitor_blink_task;
 
 void
 ao_set_monitor(uint8_t monitoring)
@@ -274,4 +285,6 @@ ao_monitor_init(uint8_t monitor_led, uint8_t monitoring) __reentrant
        ao_cmd_register(&ao_monitor_cmds[0]);
        ao_add_task(&ao_monitor_get_task, ao_monitor_get, "monitor_get");
        ao_add_task(&ao_monitor_put_task, ao_monitor_put, "monitor_put");
+       if (ao_monitor_led)
+               ao_add_task(&ao_monitor_blink_task, ao_monitor_blink, "monitor_blink");
 }