Losing uart characters - activating UART in place of UARTE in nRF52840

RE: Losing uart characters - activating UART in place of UARTE in nRF52840

Malcolm — Mon, 13 Nov 2023 16:01:52 GMT

Thank you. I figured that there was a timer conflict and experimented with different values and settled on RTC2 and TIMER2. I have made some progress, although I now receive a hard fault. I will look at your links to see if there is an answer.

RE: Losing uart characters - activating UART in place of UARTE in nRF52840

Einar Thorsrud — Mon, 13 Nov 2023 15:06:39 GMT

Hi,

How did you configure libuarte? The sample does not use the SoftDevice and use instances that are not allowed in an application that use it. So you should pick another timer than TIMER0, and also avoid both RTC0 (SoftDevice) and RTC1 (app timer). See this post.

RE: Losing uart characters - activating UART in place of UARTE in nRF52840

Malcolm — Mon, 13 Nov 2023 07:39:09 GMT

I have ported LIBUARTE into my application and it is now working when I test the uart in isolation, However when I enable the BLE stack I receive an initialisation error of 0x1001, which indicates a NRF_ERROR_SDM_INCORRECT_INTERRUPT_CONFIGURATION. As I am using the default values from LIBUART and ble init, what would cause this conflict and how to correct?

RE: Losing uart characters - activating UART in place of UARTE in nRF52840

Malcolm — Wed, 08 Nov 2023 08:42:12 GMT

I appear to have encountered a second problem with missing interrupts and characters. I am sure there is not hardware FIFO for the uarte, rather it relies on direct DMA transfer to a buffer. In my case I must process every incoming character. Therefore I will need to ensure that DMA double buffering is configured. I will try libuart to determine if it is a fix.

RE: Losing uart characters - activating UART in place of UARTE in nRF52840

Einar Thorsrud — Tue, 07 Nov 2023 14:59:51 GMT

I see. I am not able to say what the issue is based on what I know, but an important limitation of the serial port library you are using is that it does not do EasyDMA double buffering. So when the buffer is full, you get a NRF_SERIAL_EVENT_RX_DATA event, and only after that has been handled the library start Rx again (using the same old Rx buffer). You can see this from the implementation of uart_event_handler() in nrf_serial.c. So if you spend too long time handling the received bytes, or something else prevents the library from starting Rx again, you can loose incoming data. (This is one of the things that has improved in libuarte).

RE: Losing uart characters - activating UART in place of UARTE in nRF52840

Malcolm — Mon, 06 Nov 2023 10:00:22 GMT

Hi Einor

Whilst there may be isues in V16 I have not encountered, and we are close to release of software for a pjoect so I do not want to do a complete migration to a new version unless absolutely essential as I have hacked some of the code.

The issue that remains appears to be that characters are not being buffered in a FIFO and are lost or corrupted by missing parts. I can send an image of a capture from a scope that best illustrates. I cannot upload the image as it pdf file and not allowed.

I think the main issue for me is that code somewhere disables interrupts for an extended period, and I must find and solve that problem.

RE: Losing uart characters - activating UART in place of UARTE in nRF52840

Einar Thorsrud — Mon, 06 Nov 2023 09:20:40 GMT

Hi Malcolm,

Ah, I see. That is why I did not find this. The Serial port library has some issues and was removed from SDK 17.x.x. I would recommend that you use libuarte instead, which is it's replacement. This is also present in SDK 16 (though I would consider migrating to 17.1.0).

Einar

RE: Losing uart characters - activating UART in place of UARTE in nRF52840

Malcolm — Fri, 03 Nov 2023 15:20:13 GMT

The code is from SDK 16

RE: Losing uart characters - activating UART in place of UARTE in nRF52840

Malcolm — Fri, 03 Nov 2023 15:18:54 GMT

I have done a scope capture of the issue to explain the problem, but I do not know how I can upload.

In the trace the first character is missed altogether. I receive 6 characters, but there are only 5 interrupts.

RE: Losing uart characters - activating UART in place of UARTE in nRF52840

Malcolm — Fri, 03 Nov 2023 13:12:07 GMT

This is library code from Nordic. I add the full initialisation. I do not recall where this comes from originally, but it remains in the Nordic library.

This code appears to work by accepting writes to the FIFO (circular buffer) and then copies into the BUFF_TX (linear buffer) for transmission. RX is similar and on an interrupt incoming characters are written to the FIFO_RX and can be taken out by a scheduled task - this keeps the interrupt routine short and allows interrupts to be reenabled.

I am happy that this works as intended, except that it misses some characters when interrupts are disabled. I was expecting characters to be buffered in the FIFO and be released when interrupts are re-enabled.

Regards

Malcolm

define RTS_PIN_DISCONNECTED NRF_UART_PSEL_DISCONNECTED
#define   CTS_PIN_DISCONNECTED NRF_UART_PSEL_DISCONNECTED

static void readGprsByteEvent(void * p_event_data, uint16_t event_size);
static void writeGprsConfirm();
static void gprsUartEventHandler(struct nrf_serial_s const * p_serial,
                                         nrf_serial_event_t event);

#define SERIAL_GPRS_FIFO_TX_SIZE 64
#define SERIAL_GPRS_FIFO_RX_SIZE 64

NRF_SERIAL_QUEUES_DEF(serial_gprs_queues, SERIAL_GPRS_FIFO_TX_SIZE, SERIAL_GPRS_FIFO_RX_SIZE);

#define SERIAL_GPRS_BUFF_TX_SIZE 16
#define SERIAL_GPRS_BUFF_RX_SIZE 1

NRF_SERIAL_BUFFERS_DEF(serial_gprs_buffs, SERIAL_GPRS_BUFF_TX_SIZE, SERIAL_GPRS_BUFF_RX_SIZE);

NRF_SERIAL_CONFIG_DEF(serial_gprs_config, NRF_SERIAL_MODE_DMA,
                      &serial_gprs_queues, &serial_gprs_buffs, gprsUartEventHandler, NULL);

#define UART_GPRS_INSTANCE_NUMBER 0

NRF_SERIAL_UART_DEF(serial_gprs_uart, UART_GPRS_INSTANCE_NUMBER);

#define GPRS_UART_CONFIG_IRQ_PRIORITY UART_DEFAULT_CONFIG_IRQ_PRIORITY

//#define GPRS_UART_CONFIG_IRQ_PRIORITY 2

static volatile bool gprsRxPending;
static volatile bool gprsRxPendingCount;
static volatile bool gprsUartError;
//static uint8_t serial_gprs_buffer2;

/***********************************************************************************
   Init GPRS, register GPRS callbacks.

   Parameters:
       none

   Return:
       none

***********************************************************************************/
PROGMEM_DECLARE_CONST(char) gprsUartMsgOpenError[]={"*** GPRS uart open error\n"};

void appStartGprsUsartManager(nrf_uart_baudrate_t baudrate)
{
   ret_code_t ret;
   nrf_drv_uart_config_t m_uarte_gprs_drv_config;

   m_uarte_gprs_drv_config.baudrate = baudrate;
   m_uarte_gprs_drv_config.hwfc = NRF_UART_HWFC_DISABLED;
   m_uarte_gprs_drv_config.interrupt_priority = GPRS_UART_CONFIG_IRQ_PRIORITY;
   m_uarte_gprs_drv_config.parity = NRF_UART_PARITY_EXCLUDED;
   m_uarte_gprs_drv_config.pselcts = CTS_PIN_DISCONNECTED;
   m_uarte_gprs_drv_config.pselrts = RTS_PIN_DISCONNECTED;
   m_uarte_gprs_drv_config.pselrxd = UART_GPRS_RX_PIN;
   m_uarte_gprs_drv_config.pseltxd = UART_GPRS_TX_PIN;
   gprsRxPending = false;
   gprsRxPendingCount = 0;
   gprsUartError = false;
   ret = nrf_serial_init(&serial_gprs_uart, &m_uarte_gprs_drv_config, &serial_gprs_config);
   APP_ERROR_CHECK(ret);
}

RE: Losing uart characters - activating UART in place of UARTE in nRF52840

Einar Thorsrud — Fri, 03 Nov 2023 07:51:14 GMT

I see. But I don't really understand your config or what you are doing, partly because the code snippets refer to code that is not provided by us so I can only assume what the configs does. Can you elaborate more about your exact configuration and what you are seeing?

RE: Losing uart characters - activating UART in place of UARTE in nRF52840

Malcolm — Thu, 02 Nov 2023 17:54:23 GMT

I have set SERIAL1_BUFF_RX_SIZE to 1, so this generates an interrupt for each character. However I do not get an interrupt for some characters, and instead I get uart error (no stop bit). I have scope trace to demonstrate the issue.

RE: Losing uart characters - activating UART in place of UARTE in nRF52840

Einar Thorsrud — Thu, 02 Nov 2023 13:45:13 GMT

Hi,

[quote user=""]This would be the behaviour of the UART, which has 6 deep hardware fifo, and would capture all the incoming characters and release them each time RX is read.[/quote]

You are rigth that the UART RX FIFO is 6 bytes. The UARTE use DMA however, so the data is written directly to a buffer that is as large as needed, offloading the CPU. This means that for most real applications (where the application is not only handlig UART), the UARTE is better, and I struggle to see why you want to use the deprecated UART peripheral instead?

[quote user=""]My question: Does the UARTE also use the hardware fifo to capture incoming characters that can be transferred to memory by DMA and interrupt generated for each character?[/quote]

The FIFO is still there, but you cannot get an interrupt for each received byte. That is a limitation in UARTE, and if you need to know the number of received bytes before the Rx buffer is full, you may want to have a look at libuarte, which use a TIEMR in counter mode and PPI to count received bytes.

[quote user=""]How do I use UART in place of UARTE, as all the code forces UARTE even if NRF_SERIAL_MODE_IRQ is selected[/quote]

I do not immediately recognize these code snippets. Where does it come from? That said, I can think of few applications where it would make sense to use the legacy UART peripheral.