Poke master to speed up packet rate when things are busy
[fw/altos] / src / ao_packet.c
index e3133b888d919a492a64b61c4af3126a42cd736d..ba5595126dd5656e25a2cb9fbd2e821a55d00873 100644 (file)
@@ -24,66 +24,25 @@ static __xdata char rx_data[AO_PACKET_MAX];
 static __pdata uint8_t rx_len, rx_used, tx_used;
 static __pdata uint8_t rx_seq;
 
-static __xdata uint16_t ao_packet_timer_delay;
-static __xdata uint8_t ao_packet_timer_cancelled;
-
 static __xdata struct ao_task  ao_packet_task;
-static __xdata struct ao_task  ao_packet_timer_task;
 static __xdata uint8_t ao_packet_enable;
 static __xdata uint8_t ao_packet_master_sleeping;
 
-void
-ao_packet_timer(void) __reentrant
-{
-       uint16_t        delay;
-
-       while (ao_packet_enable) {
-
-               /* wait until the timer task is needed
-                */
-               while (!ao_packet_timer_delay && ao_packet_enable)
-                       ao_sleep(&ao_packet_timer_delay);
-
-               delay = ao_packet_timer_delay;
-               ao_packet_timer_delay = 0;
-
-               /* pause waiting for either a timeout or
-                * a timer cancel
-                */
-               ao_delay(delay);
-
-               /* if not canceled, abort the receive
-                */
-               if (!ao_packet_timer_cancelled) {
-                       printf ("packet timeout\n"); flush();
-                       ao_radio_abort(AO_DMA_TIMEOUT);
-               }
-       }
-       ao_exit();
-}
-
-void
-ao_packet_timer_set(uint16_t delay)
-{
-       ao_packet_timer_delay = delay;
-       ao_packet_timer_cancelled = 0;
-       ao_wakeup(&ao_packet_timer_delay);
-}
-
-void
-ao_packet_timer_cancel(void)
-{
-       ao_packet_timer_cancelled = 1;
-       ao_packet_timer_delay = 0;
-       ao_wake_task(&ao_packet_timer_task);
-}
-
 void
 ao_packet_send(void)
 {
+       ao_led_on(AO_LED_RED);
        ao_config_get();
        ao_mutex_get(&ao_radio_mutex);
+       if (tx_used && tx_packet.len == 0) {
+               memcpy(&tx_packet.d, tx_data, tx_used);
+               tx_packet.len = tx_used;
+               tx_packet.seq++;
+               tx_used = 0;
+               ao_wakeup(&tx_data);
+       }
        ao_radio_idle();
+       ao_radio_done = 0;
        RF_CHANNR = ao_config.radio_channel;
        ao_dma_set_transfer(ao_radio_dma,
                            &tx_packet,
@@ -97,9 +56,10 @@ ao_packet_send(void)
                            DMA_CFG1_PRIORITY_HIGH);
        ao_dma_start(ao_radio_dma);
        RFST = RFST_STX;
-       __critical while (!ao_radio_dma_done)
-               ao_sleep(&ao_radio_dma_done);
+       __critical while (!ao_radio_done)
+               ao_sleep(&ao_radio_done);
        ao_mutex_put(&ao_radio_mutex);
+       ao_led_off(AO_LED_RED);
 }
 
 uint8_t
@@ -107,6 +67,7 @@ ao_packet_recv(void)
 {
        uint8_t dma_done;
 
+       ao_led_on(AO_LED_GREEN);
        ao_config_get();
        ao_mutex_get(&ao_radio_mutex);
        ao_radio_idle();
@@ -124,19 +85,28 @@ ao_packet_recv(void)
        ao_dma_start(ao_radio_dma);
        RFST = RFST_SRX;
        __critical while (!ao_radio_dma_done)
-               ao_sleep(&ao_radio_dma_done);
+                          if (ao_sleep(&ao_radio_dma_done) != 0)
+                                  ao_radio_abort();
        dma_done = ao_radio_dma_done;
        ao_mutex_put(&ao_radio_mutex);
+       ao_led_off(AO_LED_GREEN);
 
        if (dma_done & AO_DMA_DONE) {
-               printf ("rssi %d status %x\n", rx_packet.rssi, rx_packet.status); flush();
-               if (!(rx_packet.status & PKT_APPEND_STATUS_1_CRC_OK)) {
-                       printf ("bad crc\n"); flush();
-//                     return AO_DMA_ABORTED;
-               }
-               if (rx_packet.packet.len) {
-                       if (rx_packet.packet.seq == rx_seq + 1 && rx_used == rx_len)
-                       {
+               if (!(rx_packet.status & PKT_APPEND_STATUS_1_CRC_OK))
+                       return AO_DMA_ABORTED;
+               if (rx_packet.packet.len == AO_PACKET_SYN) {
+                       rx_seq = rx_packet.packet.seq;
+                       tx_packet.seq = rx_packet.packet.ack;
+                       tx_packet.ack = rx_seq;
+               } else if (rx_packet.packet.len) {
+                       if (rx_packet.packet.seq == (uint8_t) (rx_seq + (uint8_t) 1) && rx_used == rx_len) {
+#if 0
+                               printf ("rx len %3d seq %3d ack %3d\n",
+                                       rx_packet.packet.len,
+                                       rx_packet.packet.seq,
+                                       rx_packet.packet.ack);
+                               flush();
+#endif
                                memcpy(rx_data, rx_packet.packet.d, rx_packet.packet.len);
                                rx_used = 0;
                                rx_len = rx_packet.packet.len;
@@ -156,14 +126,12 @@ ao_packet_recv(void)
 void
 ao_packet_slave(void)
 {
-       tx_packet.addr = ao_serial_number;
        ao_radio_set_packet();
+       tx_packet.addr = ao_serial_number;
+       tx_packet.len = AO_PACKET_SYN;
        while (ao_packet_enable) {
                ao_packet_recv();
-               ao_led_toggle(AO_LED_GREEN);
-               ao_delay(AO_MS_TO_TICKS(100));
                ao_packet_send();
-               ao_led_toggle(AO_LED_RED);
        }
        ao_exit();
 }
@@ -174,19 +142,20 @@ void
 ao_packet_master(void)
 {
        uint8_t status;
-       tx_packet.addr = ao_serial_number;
+
        ao_radio_set_packet();
+       tx_packet.addr = ao_serial_number;
+       tx_packet.len = AO_PACKET_SYN;
        while (ao_packet_enable) {
-               ao_delay(AO_MS_TO_TICKS(100));
                ao_packet_send();
-               ao_led_toggle(AO_LED_RED);
-               ao_packet_timer_set(AO_MS_TO_TICKS(1000));
+               ao_alarm(AO_MS_TO_TICKS(100));
                status = ao_packet_recv();
-               ao_packet_timer_cancel();
                if (status & AO_DMA_DONE) {
-                       ao_led_toggle(AO_LED_GREEN);
+                       /* if we can transmit data, do so */
+                       if (tx_used && tx_packet.len == 0)
+                               continue;
                        ao_packet_master_sleeping = 1;
-                       ao_sleep(AO_MS_TO_TICKS(1000));
+                       ao_delay(AO_MS_TO_TICKS(1000));
                        ao_packet_master_sleeping = 0;
                }
        }
@@ -196,32 +165,20 @@ ao_packet_master(void)
 void
 ao_packet_flush(void)
 {
-       if (!tx_used)
-               return;
-
-       /* Wait for previous packet to be received
-        */
-       while (tx_packet.len)
-               ao_sleep(&tx_packet);
-
-       /* Prepare next packet
+       /* If there is data to send, and this is the master,
+        * then poke the master to send all queued data
         */
-       if (tx_used) {
-               memcpy(&tx_packet.d, tx_data, tx_used);
-               tx_packet.len = tx_used;
-               tx_packet.seq++;
-               tx_used = 0;
-
-               if (ao_packet_master_sleeping)
-                       ao_wake_task(&ao_packet_task);
-       }
+       if (tx_used && ao_packet_master_sleeping)
+               ao_wake_task(&ao_packet_task);
 }
 
 void
 ao_packet_putchar(char c)
 {
-       while (tx_used == AO_PACKET_MAX && ao_packet_enable)
+       while (tx_used == AO_PACKET_MAX && ao_packet_enable) {
                ao_packet_flush();
+               ao_sleep(&tx_data);
+       }
 
        if (ao_packet_enable)
                tx_data[tx_used++] = c;
@@ -230,8 +187,12 @@ ao_packet_putchar(char c)
 char
 ao_packet_getchar(void) __critical
 {
-       while (rx_used == rx_len && ao_packet_enable)
+       while (rx_used == rx_len && ao_packet_enable) {
+               /* poke the master to get more data */
+               if (ao_packet_master_sleeping)
+                       ao_wake_task(&ao_packet_task);
                ao_sleep(&rx_data);
+       }
 
        if (!ao_packet_enable)
                return 0;
@@ -245,8 +206,11 @@ ao_packet_echo(void) __reentrant
        uint8_t c;
        while (ao_packet_enable) {
                c = ao_packet_getchar();
-               if (ao_packet_enable)
+               if (ao_packet_enable) {
                        putchar(c);
+                       if (c == (uint8_t) '\n' || c == (uint8_t) '\r')
+                               flush();
+               }
        }
        ao_exit();
 }
@@ -260,40 +224,19 @@ ao_packet_forward(void) __reentrant
        ao_packet_enable = 1;
        ao_cmd_white();
 
-       ao_radio_set_packet();
-       if (ao_cmd_lex_c == 'm') {
-               while ((c = getchar()) != '~')
-                       ao_packet_send();
-       } else {
-               for (;;) {
-                       ao_packet_recv();
-                       ao_led_toggle(AO_LED_GREEN);
-                       if (rx_packet.packet.d[0] == (uint8_t) '@')
-                               break;
-               }
-       }
-       ao_packet_enable = 0;
-       return;
-#if 0
-       if (ao_cmd_lex_c == 'm') {
-               ao_add_task(&ao_packet_timer_task, ao_packet_timer, "timeout");
+       if (ao_cmd_lex_c == 'm')
                ao_add_task(&ao_packet_task, ao_packet_master, "master");
-       }
        else
                ao_add_task(&ao_packet_task, ao_packet_slave, "slave");
        ao_add_task(&ao_packet_echo_task, ao_packet_echo, "echo");
-       while ((c = getchar()) != '~') {
+       while ((c = getchar()) != '~')
                ao_packet_putchar(c);
-               if (c == '\n')
-                       ao_packet_flush();
-       }
        ao_packet_enable = 0;
-       ao_radio_abort(AO_DMA_ABORTED);
+       ao_radio_abort();
        while (ao_packet_echo_task.wchan || ao_packet_task.wchan) {
                ao_wake_task(&ao_packet_echo_task);
                ao_wake_task(&ao_packet_task);
        }
-#endif
 }
 
 __code struct ao_cmds ao_packet_cmds[] = {