X-Git-Url: https://git.gag.com/?p=fw%2Faltos;a=blobdiff_plain;f=src%2Fao_packet.c;h=f627e02b4022ee8004fd5400a4eb20d0d0ed60d8;hp=da33bb4c6dfc8d4e92cf59ec4f94af46ab2f27e4;hb=2681a17500913cbaf3966f09380bb1d6b59e3863;hpb=d46797e5c08d4955d516458185e2cfb51ee2d567

diff --git a/src/ao_packet.c b/src/ao_packet.c
index da33bb4c..f627e02b 100644
--- a/src/ao_packet.c
+++ b/src/ao_packet.c
@@ -17,40 +17,32 @@
 
 #include "ao.h"
 
-static __xdata struct ao_packet_recv rx_packet;
-static __xdata struct ao_packet tx_packet;
+__xdata struct ao_packet_recv ao_rx_packet;
+__xdata struct ao_packet ao_tx_packet;
+__pdata uint8_t ao_packet_rx_len, ao_packet_rx_used, ao_packet_tx_used;
+
 static __xdata char tx_data[AO_PACKET_MAX];
 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 struct ao_task	ao_packet_task;
-static __xdata uint8_t ao_packet_enable;
-static __xdata uint8_t ao_packet_master_sleeping;
+__xdata struct ao_task	ao_packet_task;
+__xdata uint8_t ao_packet_enable;
+__xdata uint8_t ao_packet_master_sleeping;
 
 void
 ao_packet_send(void)
 {
-	ao_config_get();
-	ao_mutex_get(&ao_radio_mutex);
-	ao_radio_idle();
-	ao_radio_done = 0;
-	RF_CHANNR = ao_config.radio_channel;
-	ao_dma_set_transfer(ao_radio_dma,
-			    &tx_packet,
-			    &RFDXADDR,
-			    sizeof (struct ao_packet),
-			    DMA_CFG0_WORDSIZE_8 |
-			    DMA_CFG0_TMODE_SINGLE |
-			    DMA_CFG0_TRIGGER_RADIO,
-			    DMA_CFG1_SRCINC_1 |
-			    DMA_CFG1_DESTINC_0 |
-			    DMA_CFG1_PRIORITY_HIGH);
-	ao_dma_start(ao_radio_dma);
-	RFST = RFST_STX;
-	__critical while (!ao_radio_done)
-		ao_sleep(&ao_radio_done);
-	ao_mutex_put(&ao_radio_mutex);
+	ao_led_on(AO_LED_RED);
+	/* If any tx data is pending then copy it into the tx packet */
+	if (ao_packet_tx_used && ao_tx_packet.len == 0) {
+		memcpy(&ao_tx_packet.d, tx_data, ao_packet_tx_used);
+		ao_tx_packet.len = ao_packet_tx_used;
+		ao_tx_packet.seq++;
+		ao_packet_tx_used = 0;
+		ao_wakeup(&tx_data);
+	}
+	ao_radio_send(&ao_tx_packet, sizeof (ao_tx_packet));
+	ao_led_off(AO_LED_RED);
 }
 
 uint8_t
@@ -58,191 +50,99 @@ ao_packet_recv(void)
 {
 	uint8_t	dma_done;
 
-	ao_config_get();
-	ao_mutex_get(&ao_radio_mutex);
-	ao_radio_idle();
-	RF_CHANNR = ao_config.radio_channel;
-	ao_dma_set_transfer(ao_radio_dma,
-			    &RFDXADDR,
-			    &rx_packet,
-			    sizeof (struct ao_packet_recv),
-			    DMA_CFG0_WORDSIZE_8 |
-			    DMA_CFG0_TMODE_SINGLE |
-			    DMA_CFG0_TRIGGER_RADIO,
-			    DMA_CFG1_SRCINC_0 |
-			    DMA_CFG1_DESTINC_1 |
-			    DMA_CFG1_PRIORITY_HIGH);
-	ao_dma_start(ao_radio_dma);
-	RFST = RFST_SRX;
-	__critical while (!ao_radio_dma_done)
-			   if (ao_sleep(&ao_radio_dma_done) != 0) {
-				   printf("recv timeout\n"); flush();
-				   ao_radio_abort();
-			   }
-	dma_done = ao_radio_dma_done;
-	ao_mutex_put(&ao_radio_mutex);
-
-	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)
-			{
-				memcpy(rx_data, rx_packet.packet.d, rx_packet.packet.len);
-				rx_used = 0;
-				rx_len = rx_packet.packet.len;
-				rx_seq = rx_packet.packet.seq;
-				tx_packet.ack = rx_seq;
-				ao_wakeup(&rx_data);
-			}
-		}
-		if (rx_packet.packet.ack == tx_packet.seq) {
-			tx_packet.len = 0;
-			ao_wakeup(&tx_packet);
+#ifdef AO_LED_GREEN
+	ao_led_on(AO_LED_GREEN);
+#endif
+	dma_done = ao_radio_recv(&ao_rx_packet, sizeof (struct ao_packet_recv));
+#ifdef AO_LED_GREEN
+	ao_led_off(AO_LED_GREEN);
+#endif
+
+	/* Check to see if we got a valid packet */
+	if (!dma_done)
+		return 0;
+	if (!(ao_rx_packet.status & PKT_APPEND_STATUS_1_CRC_OK))
+		return 0;
+
+	/* SYN packets carry no data */
+	if (ao_rx_packet.packet.len == AO_PACKET_SYN) {
+		rx_seq = ao_rx_packet.packet.seq;
+		ao_tx_packet.seq = ao_rx_packet.packet.ack;
+		ao_tx_packet.ack = rx_seq;
+	} else if (ao_rx_packet.packet.len) {
+
+		/* Check for incoming data at the next sequence and
+		 * for an empty data buffer
+		 */
+		if (ao_rx_packet.packet.seq == (uint8_t) (rx_seq + (uint8_t) 1) &&
+		    ao_packet_rx_used == ao_packet_rx_len) {
+
+			/* Copy data to the receive data buffer and set up the
+			 * offsets
+			 */
+			memcpy(rx_data, ao_rx_packet.packet.d, ao_rx_packet.packet.len);
+			ao_packet_rx_used = 0;
+			ao_packet_rx_len = ao_rx_packet.packet.len;
+
+			/* Mark the sequence that we've received to
+			 * let the sender know when we return a packet
+			 */
+			rx_seq = ao_rx_packet.packet.seq;
+			ao_tx_packet.ack = rx_seq;
+
+			/* Poke anyone looking for received data */
+			ao_wakeup(&ao_stdin_ready);
 		}
 	}
-	return dma_done;
-}
 
-void
-ao_packet_slave(void)
-{
-	tx_packet.addr = ao_serial_number;
-	ao_radio_set_packet();
-	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);
+	/* If the other side has seen the latest data we queued,
+	 * wake up any task waiting to send data and let them go again
+	 */
+	if (ao_rx_packet.packet.ack == ao_tx_packet.seq) {
+		ao_tx_packet.len = 0;
+		ao_wakeup(&ao_tx_packet);
 	}
-	ao_exit();
+	return 1;
 }
 
-/* Thread for the master side of the packet link */
-
-void
-ao_packet_master(void)
-{
-	uint8_t	status;
-
-	ao_radio_set_packet();
-	tx_packet.addr = ao_serial_number;
-	tx_packet.len = AO_PACKET_SYN;
-	while (ao_packet_enable) {
-		ao_led_on(AO_LED_RED);
-		ao_delay(AO_MS_TO_TICKS(100));
-		ao_packet_send();
-		ao_led_off(AO_LED_RED);
-		ao_led_on(AO_LED_GREEN);
-		ao_alarm(AO_MS_TO_TICKS(1000));
-		status = ao_packet_recv();
-		ao_led_off(AO_LED_GREEN);
-		if (status & AO_DMA_DONE) {
-			ao_packet_master_sleeping = 1;
-			ao_delay(AO_MS_TO_TICKS(1000));
-			ao_packet_master_sleeping = 0;
-		}
-	}
-	ao_exit();
-}
+#ifndef PACKET_HAS_MASTER
+#define PACKET_HAS_MASTER 1
+#endif
 
+#if PACKET_HAS_MASTER
 void
 ao_packet_flush(void)
 {
-	if (!tx_used)
-		return;
-
-	/* Wait for previous packet to be received
+	/* If there is data to send, and this is the master,
+	 * then poke the master to send all queued data
 	 */
-	while (tx_packet.len)
-		ao_sleep(&tx_packet);
-
-	/* Prepare next packet
-	 */
-	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 (ao_packet_tx_used && ao_packet_master_sleeping)
+		ao_wakeup(&ao_packet_master_sleeping);
 }
+#endif /* PACKET_HAS_MASTER */
 
 void
-ao_packet_putchar(char c)
+ao_packet_putchar(char c) __reentrant
 {
-	while (tx_used == AO_PACKET_MAX && ao_packet_enable)
+	while (ao_packet_tx_used == AO_PACKET_MAX && ao_packet_enable) {
+#if PACKET_HAS_MASTER
 		ao_packet_flush();
+#endif
+		ao_sleep(&tx_data);
+	}
 
 	if (ao_packet_enable)
-		tx_data[tx_used++] = c;
+		tx_data[ao_packet_tx_used++] = c;
 }
 
 char
-ao_packet_getchar(void) __critical
+ao_packet_pollchar(void) __critical
 {
-	while (rx_used == rx_len && ao_packet_enable)
-		ao_sleep(&rx_data);
-
 	if (!ao_packet_enable)
-		return 0;
-
-	return rx_data[rx_used++];
-}
-
-static void
-ao_packet_echo(void) __reentrant
-{
-	uint8_t	c;
-	while (ao_packet_enable) {
-		c = ao_packet_getchar();
-		if (ao_packet_enable)
-			putchar(c);
-	}
-	ao_exit();
-}
+		return AO_READ_AGAIN;
 
-static __xdata struct ao_task	ao_packet_echo_task;
+	if (ao_packet_rx_used == ao_packet_rx_len)
+		return AO_READ_AGAIN;
 
-static void
-ao_packet_forward(void) __reentrant
-{
-	char c;
-	ao_packet_enable = 1;
-	ao_cmd_white();
-
-	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()) != '~') {
-		ao_packet_putchar(c);
-		if (c == '\n')
-			ao_packet_flush();
-	}
-	ao_packet_enable = 0;
-	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[] = {
-	{ 'p',	ao_packet_forward,	"p {m|s}                            Remote packet link. m=master, s=slave" },
-	{ 0,	ao_packet_forward,	NULL },
-};
-
-void
-ao_packet_init(void)
-{
-	ao_cmd_register(&ao_packet_cmds[0]);
+	return rx_data[ao_packet_rx_used++];
 }