gromero commented on a change in pull request #8021:
URL: https://github.com/apache/tvm/pull/8021#discussion_r631297626
##########
File path: apps/microtvm/zephyr/demo_runtime/src/main.c
##########
@@ -248,6 +249,9 @@ void uart_irq_cb(const struct device* dev, void* user_data)
{
// Used to initialize the UART receiver.
void uart_rx_init(struct ring_buf* rbuf, const struct device* dev) {
+ const struct uart_config config = {
+ .baudrate = UART_BAUDRATE, .parity = 0, .stop_bits = 1, .data_bits = 3,
.flow_ctrl = 0};
Review comment:
I've tested your changes against the Disco board and it's alright. So
aside the magic `25` in `TVM_CRT_MAX_PACKET_SIZE_BYTES+25` for the RX buffer
which you're currently discussing with Andrew and seems to be determined
experimentally (maybe leave a comment about it?) the change LGTM.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
