diff options
| -rw-r--r-- | docs/uart_driver.md | 38 | ||||
| -rw-r--r-- | keyboards/mschwingen/modelm/modelm.c | 2 | ||||
| -rw-r--r-- | keyboards/nullbitsco/common/remote_kb.c | 6 | ||||
| -rw-r--r-- | paths.mk | 1 | ||||
| -rw-r--r-- | platforms/avr/drivers/uart.c | 26 | ||||
| -rw-r--r-- | platforms/avr/drivers/uart.h | 8 | ||||
| -rw-r--r-- | platforms/chibios/drivers/uart.c | 8 | ||||
| -rw-r--r-- | platforms/chibios/drivers/uart.h | 8 | 
8 files changed, 72 insertions, 25 deletions
| diff --git a/docs/uart_driver.md b/docs/uart_driver.md index 4d1716975f..340b648189 100644 --- a/docs/uart_driver.md +++ b/docs/uart_driver.md @@ -60,30 +60,56 @@ Initialize the UART driver. This function must be called only once, before any o  --- -### `void uart_putchar(uint8_t c)` +### `void uart_write(uint8_t data)`  Transmit a single byte.  #### Arguments - - `uint8_t c`   -   The byte (character) to send, from 0 to 255. + - `uint8_t data`   +   The byte to write.  --- -### `uint8_t uart_getchar(void)` +### `uint8_t uart_read(void)`  Receive a single byte.  #### Return Value -The byte read from the receive buffer. +The byte read from the receive buffer. This function will block if the buffer is empty (ie. no data to read). + +--- + +### `void uart_transmit(const uint8_t *data, uint16_t length)` + +Transmit multiple bytes. + +#### Arguments + + - `const uint8_t *data`   +   A pointer to the data to write from. + - `uint16_t length`   +   The number of bytes to write. Take care not to overrun the length of `data`. + +--- + +### `void uart_receive(char *data, uint16_t length)` + +Receive multiple bytes. + +#### Arguments + + - `uint8_t *data`   +   A pointer to the buffer to read into. + - `uint16_t length`   +   The number of bytes to read. Take care not to overrun the length of `data`.  ---  ### `bool uart_available(void)` -Return whether the receive buffer contains data. Call this function to determine if `uart_getchar()` will return meaningful data. +Return whether the receive buffer contains data. Call this function to determine if `uart_read()` will return data immediately.  #### Return Value diff --git a/keyboards/mschwingen/modelm/modelm.c b/keyboards/mschwingen/modelm/modelm.c index c1180612ad..7dcd4ac024 100644 --- a/keyboards/mschwingen/modelm/modelm.c +++ b/keyboards/mschwingen/modelm/modelm.c @@ -27,7 +27,7 @@  #    undef sendchar  static int8_t capture_sendchar(uint8_t c) {      //  sendchar(c); -    uart_putchar(c); +    uart_write(c);      return 0;  }  #endif diff --git a/keyboards/nullbitsco/common/remote_kb.c b/keyboards/nullbitsco/common/remote_kb.c index 89cbf9a92a..4dcc9f4616 100644 --- a/keyboards/nullbitsco/common/remote_kb.c +++ b/keyboards/nullbitsco/common/remote_kb.c @@ -63,9 +63,7 @@ static void send_msg(uint16_t keycode, bool pressed) {    msg[IDX_PRESSED] = pressed;    msg[IDX_CHECKSUM] = chksum8(msg, UART_MSG_LEN-1); -  for (int i=0; i<UART_MSG_LEN; i++) { -    uart_putchar(msg[i]); -  } +  uart_transmit(msg, UART_MSG_LEN);  }  static void print_message_buffer(void) { @@ -103,7 +101,7 @@ static void process_uart(void) {  static void get_msg(void) {    while (uart_available()) { -    msg[msg_idx] = uart_getchar(); +    msg[msg_idx] = uart_read();      dprintf("idx: %u, recv: %u\n", msg_idx, msg[msg_idx]);      if (msg_idx == 0 && (msg[msg_idx] != UART_PREAMBLE)) {        dprintf("Byte sync error!\n"); @@ -24,6 +24,5 @@ COMMON_VPATH += $(QUANTUM_PATH)  COMMON_VPATH += $(QUANTUM_PATH)/keymap_extras  COMMON_VPATH += $(QUANTUM_PATH)/audio  COMMON_VPATH += $(QUANTUM_PATH)/process_keycode -COMMON_VPATH += $(QUANTUM_PATH)/api  COMMON_VPATH += $(QUANTUM_PATH)/sequencer  COMMON_VPATH += $(DRIVER_PATH) diff --git a/platforms/avr/drivers/uart.c b/platforms/avr/drivers/uart.c index c6abcb6fe0..01cf6b1fb8 100644 --- a/platforms/avr/drivers/uart.c +++ b/platforms/avr/drivers/uart.c @@ -100,7 +100,7 @@ void uart_init(uint32_t baud) {  }  // Transmit a byte -void uart_putchar(uint8_t c) { +void uart_write(uint8_t data) {      uint8_t i;      i = tx_buffer_head + 1; @@ -110,27 +110,39 @@ void uart_putchar(uint8_t c) {      while (tx_buffer_tail == i)          ;  // wait until space in buffer      // cli(); -    tx_buffer[i]   = c; +    tx_buffer[i]   = data;      tx_buffer_head = i;      UCSRnB         = (1 << RXENn) | (1 << TXENn) | (1 << RXCIEn) | (1 << UDRIEn);      // sei();  }  // Receive a byte -uint8_t uart_getchar(void) { -    uint8_t c, i; +uint8_t uart_read(void) { +    uint8_t data, i;      while (rx_buffer_head == rx_buffer_tail)          ;  // wait for character      i = rx_buffer_tail + 1;      if (i >= RX_BUFFER_SIZE) i = 0; -    c              = rx_buffer[i]; +    data           = rx_buffer[i];      rx_buffer_tail = i; -    return c; +    return data; +} + +void uart_transmit(const uint8_t *data, uint16_t length) { +    for (uint16_t i = 0; i < length; i++) { +        uart_write(data[i]); +    } +} + +void uart_receive(uint8_t *data, uint16_t length) { +    for (uint16_t i = 0; i < length; i++) { +        data[i] = uart_read(); +    }  }  // Return whether the number of bytes waiting in the receive buffer is nonzero. -// Call this before uart_getchar() to check if it will need +// Call this before uart_read() to check if it will need  // to wait for a byte to arrive.  bool uart_available(void) {      uint8_t head, tail; diff --git a/platforms/avr/drivers/uart.h b/platforms/avr/drivers/uart.h index 602eb3d8b0..9cb7652b08 100644 --- a/platforms/avr/drivers/uart.h +++ b/platforms/avr/drivers/uart.h @@ -28,8 +28,12 @@  void uart_init(uint32_t baud); -void uart_putchar(uint8_t c); +void uart_write(uint8_t data); -uint8_t uart_getchar(void); +uint8_t uart_read(void); + +void uart_transmit(const char *data, uint16_t length); + +void uart_receive(char *data, uint16_t length);  bool uart_available(void); diff --git a/platforms/chibios/drivers/uart.c b/platforms/chibios/drivers/uart.c index 0e8e0515af..297c1892c3 100644 --- a/platforms/chibios/drivers/uart.c +++ b/platforms/chibios/drivers/uart.c @@ -39,12 +39,16 @@ void uart_init(uint32_t baud) {      }  } -void uart_putchar(uint8_t c) { sdPut(&SERIAL_DRIVER, c); } +void uart_write(uint8_t data) { sdPut(&SERIAL_DRIVER, c); } -uint8_t uart_getchar(void) { +uint8_t uart_read(void) {      msg_t res = sdGet(&SERIAL_DRIVER);      return (uint8_t)res;  } +void uart_transmit(const uint8_t *data, uint16_t length) { sdWrite(&SERIAL_DRIVER, data, length); } + +void uart_receive(uint8_t *data, uint16_t length) { sdRead(&SERIAL_DRIVER, data, length); } +  bool uart_available(void) { return !sdGetWouldBlock(&SERIAL_DRIVER); } diff --git a/platforms/chibios/drivers/uart.h b/platforms/chibios/drivers/uart.h index b4e20e9fd3..5bc4875901 100644 --- a/platforms/chibios/drivers/uart.h +++ b/platforms/chibios/drivers/uart.h @@ -70,8 +70,12 @@  void uart_init(uint32_t baud); -void uart_putchar(uint8_t c); +void uart_write(uint8_t data); -uint8_t uart_getchar(void); +uint8_t uart_read(void); + +void uart_transmit(const uint8_t *data, uint16_t length); + +void uart_receive(uint8_t *data, uint16_t length);  bool uart_available(void); | 
