TeleMetrum v4.0 work in progress

[fw/altos] / src / samd21 / ao_serial_samd21.c

diff --git a/src/samd21/ao_serial_samd21.c b/src/samd21/ao_serial_samd21.c
index a52966c1249ba39dc1b29c98b4f6fcfb2bdb3d2f..504dd49055f57869136279c2e4a186e592130a7a 100644 (file)
--- a/src/samd21/ao_serial_samd21.c
+++ b/src/samd21/ao_serial_samd21.c
@@ -17,7 +17,7 @@
 static int
 _ao_usart_tx_start(struct ao_samd21_usart *usart)
 {
-       if (!ao_fifo_empty(&usart->tx_fifo)) {
+       if (!ao_fifo_empty(usart->tx_fifo)) {
 #if HAS_SERIAL_SW_FLOW
                if (usart->gpio_cts && ao_gpio_get(usart->gpio_cts, usart->pin_cts, foo) == 1) {
                        ao_exti_enable(usart->gpio_cts, usart->pin_cts);
@@ -28,7 +28,7 @@ _ao_usart_tx_start(struct ao_samd21_usart *usart)
                {
                        usart->tx_running = 1;
                        usart->reg->intenset = (1 << SAMD21_SERCOM_INTFLAG_DRE) | (1 << SAMD21_SERCOM_INTFLAG_TXC);
-                       usart->reg->data = ao_fifo_remove(&usart->tx_fifo);
+                       ao_fifo_remove(usart->tx_fifo, usart->reg->data);
                        ao_wakeup(&usart->tx_fifo);
                        return 1;
                }
@@ -40,8 +40,8 @@ static void
 _ao_usart_rx(struct ao_samd21_usart *usart, int is_stdin)
 {
        if (usart->reg->intflag & (1 << SAMD21_SERCOM_INTFLAG_RXC)) {
-               if (!ao_fifo_full(&usart->rx_fifo)) {
-                       ao_fifo_insert(&usart->rx_fifo, usart->reg->data);
+               if (!ao_fifo_full(usart->rx_fifo)) {
+                       ao_fifo_insert(usart->rx_fifo, usart->reg->data);
                        ao_wakeup(&usart->rx_fifo);
                        if (is_stdin)
                                ao_wakeup(&ao_stdin_ready);
@@ -89,14 +89,26 @@ static const uint32_t ao_usart_speeds[] = {
 static void
 ao_usart_set_speed(struct ao_samd21_usart *usart, uint8_t speed)
 {
+       struct samd21_sercom *reg = usart->reg;
        uint64_t        top = (uint64_t) ao_usart_speeds[speed] << (4 + 16);
-       uint16_t        baud = 65536 - (top + AO_SYSCLK/2) / AO_SYSCLK;
+       uint16_t        baud = (uint16_t) (65536 - (top + AO_SYSCLK/2) / AO_SYSCLK);
+       uint32_t        ctrla = reg->ctrla;
 
+       if (ctrla & (1UL << SAMD21_SERCOM_CTRLA_ENABLE)) {
+               usart->reg->ctrla = ctrla & ~(1UL << SAMD21_SERCOM_CTRLA_ENABLE);
+               while (reg->syncbusy & (1 << SAMD21_SERCOM_SYNCBUSY_ENABLE))
+                       ;
+       }
        usart->reg->baud = baud;
+       if (ctrla & (1UL << SAMD21_SERCOM_CTRLA_ENABLE)) {
+               usart->reg->ctrla = ctrla;
+               while (reg->syncbusy & (1 << SAMD21_SERCOM_SYNCBUSY_ENABLE))
+                       ;
+       }
 }
 
 static void
-ao_usart_init(struct ao_samd21_usart *usart, int hw_flow, int id)
+ao_usart_init(struct ao_samd21_usart *usart, bool hw_flow, uint8_t id, uint8_t txpo, uint8_t rxpo)
 {
        struct samd21_sercom *reg = usart->reg;
 
@@ -137,14 +149,17 @@ ao_usart_init(struct ao_samd21_usart *usart, int hw_flow, int id)
                      (1 << SAMD21_SERCOM_CTRLA_RUNSTDBY) |
                      (0 << SAMD21_SERCOM_CTRLA_IBON) |
                      (0 << SAMD21_SERCOM_CTRLA_SAMPR) |
-                     (1 << SAMD21_SERCOM_CTRLA_TXPO) | /* pad[2] */
-                     (3 << SAMD21_SERCOM_CTRLA_RXPO) | /* pad[3] */
+                     (txpo << SAMD21_SERCOM_CTRLA_TXPO) |      /* pad[2] */
+                     (rxpo << SAMD21_SERCOM_CTRLA_RXPO) |      /* pad[3] */
                      (0 << SAMD21_SERCOM_CTRLA_SAMPA) |
                      (0 << SAMD21_SERCOM_CTRLA_FORM) | /* no parity */
                      (0 << SAMD21_SERCOM_CTRLA_CMODE) | /* async */
                      (0 << SAMD21_SERCOM_CTRLA_CPOL) |
                      (1 << SAMD21_SERCOM_CTRLA_DORD)); /* LSB first */
 
+       while (reg->syncbusy & (1 << SAMD21_SERCOM_SYNCBUSY_ENABLE))
+               ;
+
        /* Enable receive interrupt */
        reg->intenset = (1 << SAMD21_SERCOM_INTFLAG_RXC);
 }
@@ -154,13 +169,13 @@ _ao_usart_pollchar(struct ao_samd21_usart *usart)
 {
        int     c;
 
-       if (ao_fifo_empty(&usart->rx_fifo))
+       if (ao_fifo_empty(usart->rx_fifo))
                c = AO_READ_AGAIN;
        else {
                uint8_t u;
-               u = ao_fifo_remove(&usart->rx_fifo);
+               ao_fifo_remove(usart->rx_fifo, u);
                if ((usart->reg->intenset & (1 << SAMD21_SERCOM_INTFLAG_RXC)) == 0) {
-                       if (ao_fifo_barely(&usart->rx_fifo))
+                       if (ao_fifo_barely(usart->rx_fifo))
                                usart->reg->intenset = (1 << SAMD21_SERCOM_INTFLAG_RXC);
                }
 #if HAS_SERIAL_SW_FLOW
@@ -190,9 +205,9 @@ static void
 ao_usart_putchar(struct ao_samd21_usart *usart, char c)
 {
        ao_arch_block_interrupts();
-       while (ao_fifo_full(&usart->tx_fifo))
+       while (ao_fifo_full(usart->tx_fifo))
                ao_sleep(&usart->tx_fifo);
-       ao_fifo_insert(&usart->tx_fifo, c);
+       ao_fifo_insert(usart->tx_fifo, c);
        _ao_usart_tx_start(usart);
        ao_arch_release_interrupts();
 }
@@ -201,7 +216,7 @@ static void
 ao_usart_drain(struct ao_samd21_usart *usart)
 {
        ao_arch_block_interrupts();
-       while (!ao_fifo_empty(&usart->tx_fifo) || usart->tx_running) {
+       while (!ao_fifo_empty(usart->tx_fifo) || usart->tx_running) {
                usart->draining = 1;
                ao_sleep(&usart->tx_fifo);
        }
@@ -246,9 +261,48 @@ ao_serial0_set_speed(uint8_t speed)
 }
 #endif /* HAS_SERIAL_0 */
 
+#if HAS_SERIAL_1
+
+struct ao_samd21_usart ao_samd21_usart1;
+
+void samd21_sercom1_isr(void) { ao_usart_isr(&ao_samd21_usart1, USE_SERIAL_1_STDIN); }
+
+char
+ao_serial1_getchar(void)
+{
+       return ao_usart_getchar(&ao_samd21_usart1);
+}
+
+void
+ao_serial1_putchar(char c)
+{
+       ao_usart_putchar(&ao_samd21_usart1, c);
+}
+
+int
+_ao_serial1_pollchar(void)
+{
+       return _ao_usart_pollchar(&ao_samd21_usart1);
+}
+
+void
+ao_serial1_drain(void)
+{
+       ao_usart_drain(&ao_samd21_usart1);
+}
+
+void
+ao_serial1_set_speed(uint8_t speed)
+{
+       ao_usart_drain(&ao_samd21_usart1);
+       ao_usart_set_speed(&ao_samd21_usart1, speed);
+}
+#endif /* HAS_SERIAL_1 */
+
 void
 ao_serial_init(void)
 {
+       uint8_t txpo, rxpo;
 #if HAS_SERIAL_0
 
 #if SERIAL_0_PA10_PA11
@@ -256,12 +310,14 @@ ao_serial_init(void)
        ao_enable_port(&samd21_port_a);
        samd21_port_pmux_set(&samd21_port_a, 10, SAMD21_PORT_PMUX_FUNC_C);
        samd21_port_pmux_set(&samd21_port_a, 11, SAMD21_PORT_PMUX_FUNC_C);
+       txpo = SAMD21_SERCOM_CTRLA_TXPO_TX_2; /* pad 2 */
+       rxpo = SAMD21_SERCOM_CTRLA_RXPO_RX_3; /* pad 3 */
 #else
 #error "No SERIAL_0 port configuration specified"
 #endif
 
        ao_samd21_usart0.reg = &samd21_sercom0;
-       ao_usart_init(&ao_samd21_usart0, 0, 0);
+       ao_usart_init(&ao_samd21_usart0, 0, 0, txpo, rxpo);
 
 #if USE_SERIAL_0_STDIN
        ao_add_stdio(_ao_serial0_pollchar,
@@ -269,4 +325,26 @@ ao_serial_init(void)
                     NULL);
 #endif
 #endif
+#if HAS_SERIAL_1
+
+#if SERIAL_1_PA00_PA01
+       /* Pin settings */
+       ao_enable_port(&samd21_port_a);
+       samd21_port_pmux_set(&samd21_port_a, 0, SAMD21_PORT_PMUX_FUNC_D);
+       samd21_port_pmux_set(&samd21_port_a, 1, SAMD21_PORT_PMUX_FUNC_D);
+       txpo = SAMD21_SERCOM_CTRLA_TXPO_TX_0;
+       rxpo = SAMD21_SERCOM_CTRLA_RXPO_RX_1;
+#else
+#error "No SERIAL_1 port configuration specified"
+#endif
+
+       ao_samd21_usart1.reg = &samd21_sercom1;
+       ao_usart_init(&ao_samd21_usart1, 0, 1, txpo, rxpo);
+
+#if USE_SERIAL_1_STDIN
+       ao_add_stdio(_ao_serial1_pollchar,
+                    ao_serial1_putchar,
+                    NULL);
+#endif
+#endif
 }