Half-duplex UART (single wire) with and two nRF52840-DK

RE: Half-duplex UART (single wire) with and two nRF52840-DK

TSonono — Thu, 10 Sep 2020 13:52:12 GMT

Thanks Jorgen, that fixed my problem, and the communication between the DK:s now works in half-duplex.

For anyone trying to do accomplish something similar, keep in mind that the flag might have to be volatile if you wait for the flag in your main context and modify it in the handler.

RE: Half-duplex UART (single wire) with and two nRF52840-DK

Jørgen Holmefjord — Thu, 10 Sep 2020 11:30:01 GMT

You need to first un-initialize the uart library using app_uart_close(). The APP_UART_FIFO_INIT() macro will initialize the UART driver. If you call this macro again without first un-initializing the driver, it will report that it is already initialized and give you an error.

I'm not 100% sure if the close-function will wait for any events that has lower or equal IRQ priority to the one in uart_error_handle(). It is safer to set a flag in the handler and do the re-initialization in the main context.

RE: Half-duplex UART (single wire) with and two nRF52840-DK

TSonono — Thu, 10 Sep 2020 11:03:24 GMT

Hi Jörgen,

Thanks for the swift reply.

I just tried your suggestion, where it currently looks like this:

Master:

static void init_uart(uint32_t rx_pin, uint32_t tx_pin);

void uart_error_handle(app_uart_evt_t * p_event)
{
    if (p_event->evt_type == APP_UART_COMMUNICATION_ERROR)
    {
        APP_ERROR_HANDLER(p_event->data.error_communication);
    }
    else if (p_event->evt_type == APP_UART_FIFO_ERROR)
    {
        APP_ERROR_HANDLER(p_event->data.error_code);
    }
    else if (p_event->evt_type == APP_UART_TX_EMPTY)
    {
        init_uart(27, UART_PIN_DISCONNECTED);
        NRF_LOG_DEBUG("TX Empty");
    }
}

static void init_uart(uint32_t rx_pin, uint32_t tx_pin)
{
    uint32_t err_code;

    const app_uart_comm_params_t comm_params =
    {
        rx_pin,
        tx_pin,
        RTS_PIN_NUMBER,
        CTS_PIN_NUMBER,
        APP_UART_FLOW_CONTROL_DISABLED,
        false,
#if defined(UART_PRESENT)
        NRF_UART_BAUDRATE_115200
#else
        NRF_UARTE_BAUDRATE_115200
#endif
    };

    APP_UART_FIFO_INIT(&comm_params,
                       UART_RX_BUF_SIZE,
                       UART_TX_BUF_SIZE,
                       uart_error_handle,
                       APP_IRQ_PRIORITY_LOWEST,
                       err_code);
    APP_ERROR_CHECK(err_code);
}

int main(void)
{
    .
    .
    .
    init_uart(UART_PIN_DISCONNECTED, 27);
    
    uint8_t cr = 5;
    NRF_LOG_INFO("Sent char %d", cr);
    while (app_uart_put(cr) != NRF_SUCCESS);
    nrf_delay_ms(2500);
    
    while (app_uart_get(&cr) != NRF_SUCCESS);
    nrf_delay_ms(2500);
    NRF_LOG_INFO("Received char %d\r\n", cr);
    .
    .
    .
}

This results in a fatal error from APP_ERROR_CHECK in init_uart(). Should I not be re-initializing app_uart in the UART event handler?

RE: Half-duplex UART (single wire) with and two nRF52840-DK

Jørgen Holmefjord — Thu, 10 Sep 2020 09:19:43 GMT

Hi,

I do not think that the UART peripheral will work correctly if you set both TX and RX pin to the same GPIO. You should rather set the RX pin to UART_PIN_DISCONNECTED initially and switch the pins after transmitting the data, by re-initializing the UART library. If you have a 2500 ms delay, there should be plenty of time to do this. You can use the APP_UART_TX_EMPTY event to determine when it is safe to reinit the library.

Best regards,
Jørgen