Missing UART received data with ble_app_uart

RE: Missing UART received data with ble_app_uart

matsumiya — Fri, 30 Jul 2021 10:12:17 GMT

Hi.Jared

By moving BLE transmit to MAIN instead of UART_EVENT_HANDLE
UART The problem of missing data was solved.

UART_EVENT_HANDLE counts the number of UART receive bytes, and when the BLE packet size is reached, BLE transmit request flag is set.
If Main is a BLE transmission request flag standing, BLE transmit is performed.
If the UART received data does not come for a predetermined time, it transmits the received UART data even if the BLE packet size is not satisfied.
By doing this, UART receive data could be prevented from being missing.

Thank you for your response.

main:

for (;;)
{
if (nrf_sdh_is_enabled())
{
sd_app_evt_wait();
while( 0 < uartToBleChkReq )
{
uartToBleChkReq--;
result_app_uart_get = NRF_SUCCESS;
i = 0;
while(( result_app_uart_get == NRF_SUCCESS ) && ( i < m_ble_nus_max_data_len ))
{
result_app_uart_get = app_uart_get( &uartRxBuf[ i++ ]);
}
if( result_app_uart_get != NRF_SUCCESS )
{
i--;
}
if( ( result_app_uart_get == NRF_SUCCESS ) ||
(( result_app_uart_get != NRF_SUCCESS ) && ( 0 < i ))
)
{
SEGGER_RTT_printf( 0,"[INFO] main : Do BLE Tx(%hu).\n", i );
do
{
err_code = ble_nus_data_send(&m_nus, uartRxBuf, &i, m_conn_handle);
} while (err_code == NRF_ERROR_RESOURCES);
}
}
}
}

uart event handler:

void uart_event_handle(app_uart_evt_t * p_event)
{
switch (p_event->evt_type)
{

case APP_UART_DATA_READY:
uartRxCnt++;
nrfx_rtc_cc_disable( &rtc, RTC_CH_UART_TMOUT );
if( m_ble_nus_max_data_len <= uartRxCnt )
{
uartToBleChkReq++;
uartRxCnt = 0;
}
nrfx_rtc_cc_set( &rtc, RTC_CH_UART_TMOUT, ( nrfx_rtc_counter_get( &rtc ) + RTC_CNT_UART_TMOUT ) & 0x00ffffff, true );
break;

RTC event handler:(Use RTC2)

static void rtc_handler(nrf_drv_rtc_int_type_t int_type)
{

cntr = nrfx_rtc_counter_get( &rtc );
if (int_type == NRF_DRV_RTC_INT_COMPARE0) // UART RX TIMEOUT
{

uartToBleChkReq++;
nrfx_rtc_cc_disable( &rtc, RTC_CH_UART_TMOUT );

}

regards

matsumiya

RE: Missing UART received data with ble_app_uart

matsumiya — Fri, 30 Jul 2021 04:42:10 GMT

Thank you for telling me.
The second measures have already been decided to make sure it is effective.
I will try the first measures from now on.

RE: Missing UART received data with ble_app_uart

Jared — Thu, 29 Jul 2021 13:53:58 GMT

Hi,

Most likely you are getting an error because the FIFO fills up. The application is not able to process the data in the FIFO before new data arrives. I think you have to rewrite this application for it work as you intend. You could:

1. Define a buffer that you can use to store the data from the FIFO while it's processed.

2. The ble_app_uart example sends the data as soon as it receives a newline. This is not good for maximum throughput as it causes the application to send multiple ble packets without filling them completely up. The sending of data has some overhead, so preferably you would only want to do it when the ble packet is full. Try removing this check:

RE: Missing UART received data with ble_app_uart

matsumiya — Thu, 29 Jul 2021 09:02:15 GMT

Hi,Jared

APP_UART_COMMUNICATION_ERROR

APP_UART_FIFO_ERROR

These events are currently ignored.
Do I need to add processing to clear the error?
UART flow control can not be used because the communication partner does not support.

void uart_event_handle(app_uart_evt_t * p_event)
{
uint32_t err_code;

switch (p_event->evt_type)
{

case APP_UART_DATA_READY:
if( uartEventRxBlockIf0 != 0 )
{
UNUSED_VARIABLE(app_uart_get( &uartRxBuf[ uartRxBufIndex ]));
uartRxBufIndex++;
if (( uartToBleBlockIf0 != 0 ) && ( uartRxBufIndex >= m_ble_nus_max_data_len))
{
nrfx_rtc_cc_disable( &rtc, RTC_CH_UART_TMOUT );
if ( 0 < uartRxBufIndex )
{
do
{
err_code = ble_nus_data_send(&m_nus, uartRxBuf, &uartRxBufIndex, m_conn_handle);
} while (err_code == NRF_ERROR_RESOURCES);
}
uartRxBufIndex = 0;
}
else
{
nrfx_rtc_cc_set( &rtc, RTC_CH_UART_TMOUT, ( nrfx_rtc_counter_get( &rtc ) + RTC_CNT_UART_TMOUT ) & 0x00ffffff, true );
}
}
break;

case APP_UART_COMMUNICATION_ERROR:
// modified from sample APP_ERROR_HANDLER(p_event->data.error_communication);
break;

case APP_UART_FIFO_ERROR:
// modified from sample APP_ERROR_HANDLER(p_event->data.error_code);
break;

default:
break;
}
}

regards

matsumiya

RE: Missing UART received data with ble_app_uart

Jared — Thu, 29 Jul 2021 08:49:48 GMT

Hi,

Have you disabled APP_UART_COMMUNICATION_ERROR and APP_UART_FIFO_ERROR in uart_event_handle()? The application should assert if the FIFO gets full. On a second thought, can you use uart flow control in your project?

regards

Jared

RE: Missing UART received data with ble_app_uart

matsumiya — Mon, 26 Jul 2021 02:40:27 GMT

Hi,Jared

sorry for the late reply.
Thank you for your response.
I tried to set UART_TX_BUF_SIZE and UART_RX_BUF_SIZE 4096 but did not get better.
The IRQ priority set by UART_INIT did not want to change the priority.
(APP_IRQ_PRIORITY_HIGHEST, APP_IRQ_PRIORITY_HIGH, APP_IRQ_PRIORITY_MID did not work:Hanged)

Eliminate CR / LF reception from the condition that sends to BLE, add it
By setting MIN_CONN_INTERVAL to 8 and MAX_CONN_INTERVAL
UART The missing data missing is not zero but improved.

It is likely to reply because it works on this work,
I will try if I can improve a little more.

Before correction:

void uart_event_handle(app_uart_evt_t * p_event)
{
static uint8_t data_array[BLE_NUS_MAX_DATA_LEN];
static uint8_t index = 0;
uint32_t err_code;

switch (p_event->evt_type)
{
case APP_UART_DATA_READY:
UNUSED_VARIABLE(app_uart_get(&data_array[index]));
index++;

if ((data_array[index - 1] == '\n') || // <- Remove
(data_array[index - 1] == '\r') || // <- Remove
(index >= m_ble_nus_max_data_len))
{
if (index > 1)
{
NRF_LOG_DEBUG("Ready to send data over BLE NUS");
NRF_LOG_HEXDUMP_DEBUG(data_array, index);

do
{
uint16_t length = (uint16_t)index;
err_code = ble_nus_data_send(&m_nus, data_array, &length, m_conn_handle);

Revised:

void uart_event_handle(app_uart_evt_t * p_event)
{
uint32_t err_code;

switch (p_event->evt_type)
{

case APP_UART_DATA_READY:
if( uartEventRxBlockIf0 != 0 )
{
UNUSED_VARIABLE(app_uart_get( &uartRxBuf[ uartRxBufIndex ]));
uartRxBufIndex++;

regards

Matsumiya

RE: Missing UART received data with ble_app_uart

Jared — Tue, 20 Jul 2021 12:51:08 GMT

Hi.

Have you tried increasing UART buffer size by adjusting UART_TX_BUF_SIZE and UART_RX_BUF_SIZE?

regards

Jared