X-Git-Url: https://git.gag.com/?a=blobdiff_plain;f=src%2Favr%2Fao_serial_avr.c;h=e0f813d5acd33fdeb62049a5c8a1814599948bbf;hb=bc3610d8cecbfed40c62d4dcb93fc9a4d2a7c9e3;hp=2fe39755487a6f18c22b27770326fafbc03c4607;hpb=f6f54d70b768dca1715ddddea64a4df00d82b09e;p=fw%2Faltos

diff --git a/src/avr/ao_serial_avr.c b/src/avr/ao_serial_avr.c
index 2fe39755..e0f813d5 100644
--- a/src/avr/ao_serial_avr.c
+++ b/src/avr/ao_serial_avr.c
@@ -17,8 +17,8 @@
 
 #include "ao.h"
 
-__xdata struct ao_fifo	ao_usart1_rx_fifo;
-__xdata struct ao_fifo	ao_usart1_tx_fifo;
+__xdata struct ao_fifo	ao_serial1_rx_fifo;
+__xdata struct ao_fifo	ao_serial1_tx_fifo;
 
 void
 ao_debug_out(char c)
@@ -31,10 +31,10 @@ ao_debug_out(char c)
 
 ISR(USART1_RX_vect)
 {
-	if (!ao_fifo_full(ao_usart1_rx_fifo))
-		ao_fifo_insert(ao_usart1_rx_fifo, UDR1);
-	ao_wakeup(&ao_usart1_rx_fifo);
-#if USE_SERIAL_STDIN
+	if (!ao_fifo_full(ao_serial1_rx_fifo))
+		ao_fifo_insert(ao_serial1_rx_fifo, UDR1);
+	ao_wakeup(&ao_serial1_rx_fifo);
+#if USE_SERIAL_1_STDIN
 	ao_wakeup(&ao_stdin_ready);
 #endif
 }
@@ -42,69 +42,68 @@ ISR(USART1_RX_vect)
 static __xdata uint8_t ao_serial_tx1_started;
 
 static void
-ao_serial_tx1_start(void)
+ao_serial1_tx_start(void)
 {
-	if (!ao_fifo_empty(ao_usart1_tx_fifo) &&
+	if (!ao_fifo_empty(ao_serial1_tx_fifo) &&
 	    !ao_serial_tx1_started)
 	{
 		ao_serial_tx1_started = 1;
-		ao_fifo_remove(ao_usart1_tx_fifo, UDR1);
+		ao_fifo_remove(ao_serial1_tx_fifo, UDR1);
 	}
 }
 
 ISR(USART1_UDRE_vect)
 {
-	ao_serial_tx1_started = 0;
-	ao_serial_tx1_start();
-	ao_wakeup(&ao_usart1_tx_fifo);
+	ao_serial1_tx_started = 0;
+	ao_serial1_tx_start();
+	ao_wakeup(&ao_serial1_tx_fifo);
 }
 
-char
-ao_serial_getchar(void) __critical
+#if USE_SERIAL_1_STDIN
+int
+_ao_serial1_pollchar(void)
 {
 	char	c;
-	cli();
-	while (ao_fifo_empty(ao_usart1_rx_fifo))
-		ao_sleep(&ao_usart1_rx_fifo);
-	ao_fifo_remove(ao_usart1_rx_fifo, c);
-	sei();
+	if (ao_fifo_empty(ao_serial1_rx_fifo)) {
+		sei();
+		return AO_READ_AGAIN;
+	}
+	ao_fifo_remove(ao_serial1_rx_fifo,c);
 	return c;
 }
+#endif
 
-#if USE_SERIAL_STDIN
 char
-ao_serial_pollchar(void) __critical
+ao_serial1_getchar(void) __critical
 {
 	char	c;
-	cli();
-	if (ao_fifo_empty(ao_usart1_rx_fifo)) {
-		sei();
-		return AO_READ_AGAIN;
-	}
-	ao_fifo_remove(ao_usart1_rx_fifo,c);
-	sei();
+
+	ao_arch_block_interrupts();
+	while (ao_fifo_empty(ao_serial1_rx_fifo))
+		ao_sleep(&ao_serial1_rx_fifo);
+	ao_fifo_remove(ao_serial1_rx_fifo, c);
+	ao_arch_release_interrupts();
 	return c;
 }
-#endif
 
 void
-ao_serial_putchar(char c) __critical
+ao_serial1_putchar(char c)
 {
-	cli();
-	while (ao_fifo_full(ao_usart1_tx_fifo))
-		ao_sleep(&ao_usart1_tx_fifo);
-	ao_fifo_insert(ao_usart1_tx_fifo, c);
+	ao_arch_block_interrupts();
+	while (ao_fifo_full(ao_serial1_tx_fifo))
+		ao_sleep(&ao_serial1_tx_fifo);
+	ao_fifo_insert(ao_serial1_tx_fifo, c);
 	ao_serial_tx1_start();
-	sei();
+	ao_arch_release_interrupts();
 }
 
 void
-ao_serial_drain(void) __critical
+ao_serial1_drain(void) __critical
 {
-	cli();
-	while (!ao_fifo_empty(ao_usart1_tx_fifo))
-		ao_sleep(&ao_usart1_tx_fifo);
-	sei();
+	ao_arch_block_interrupts();
+	while (!ao_fifo_empty(ao_serial1_tx_fifo))
+		ao_sleep(&ao_serial1_tx_fifo);
+	ao_arch_release_interrupts();
 }
 
 static const struct {
@@ -125,7 +124,7 @@ static const struct {
 };
 
 void
-ao_serial_set_speed(uint8_t speed)
+ao_serial1_set_speed(uint8_t speed)
 {
 	ao_serial_drain();
 	if (speed > AO_SERIAL_SPEED_57600)
@@ -154,13 +153,9 @@ ao_serial_init(void)
 		  (1 << TXEN1) |	/* Enable transmitter */
 		  (1 << RXCIE1) |	/* Enable receive interrupts */
 		  (1 << UDRIE1));	/* Enable transmit empty interrupts */
-#if 0
-#if USE_SERIAL_STDIN
-	int8_t	i;
-	i = ao_add_stdio(ao_serial_pollchar,
-			 ao_serial_putchar,
-			 NULL);
-	printf("Register serial stdio as %d\n", i);
-#endif
+#if USE_SERIAL_1_STDIN
+	ao_add_stdio(_ao_serial1_pollchar,
+		     ao_serial1_putchar,
+		     NULL);
 #endif
 }