How to check if the NUS TX Buffer is empty?

RE: How to check if the NUS TX Buffer is empty?

Einar Thorsrud — Mon, 21 Dec 2020 13:40:44 GMT

Hello,

t know enough to say for sure, but if you for instance notify every 40 ms and have a connection interval of 40 ms (?) then a possible explanation is that the code snippet that checks buffer filing runs after you have queued a packet but before it has been sent. And this would just continue... Is that what is happening here?

RE: How to check if the NUS TX Buffer is empty?

Michael01101 — Mon, 21 Dec 2020 13:34:56 GMT

Hello,

we implemented the counter for the amount of NUS packets in the TX buffer (BLE_NUS_TX_BUFFER) as recommended:

static int16_t BLE_NUS_TX_BUFFER = 0;


void saadc_callback(nrf_drv_saadc_evt_t const * p_event)    // Every 40ms
{
    ...
    if(BLE_NUS_TX_BUFFER == 0)
    {
        err_code = ble_nus_data_send(&m_nus, stringA, &bytes_to_send, m_conn_handle);
    }
    else
    {
        err_code = ble_nus_data_send(&m_nus, stringB, &bytes_to_send, m_conn_handle);   
    }
    BLE_NUS_TX_BUFFER++;
    ...
}


static void ble_evt_handler(ble_evt_t const * p_ble_evt, void * p_context)
{
    switch (p_ble_evt->header.evt_id)
    {
        case BLE_GATTS_EVT_HVN_TX_COMPLETE:
             BLE_NUS_TX_BUFFER -= p_ble_evt->evt.gatts_evt.params.hvn_tx_complete.count;
             break;
        ...
    ...
    }
...
}

We use a Connection Interval of 40ms (min. and max.). Equal to the SAADC callback.

Our problem is, that our counter BLE_NUS_TX_BUFFER contains always "1" or higher. It seems, that the TX Buffer is never empty. So the "stringA" gets never sent.

Do you know, why the counter BLE_NUS_TX_BUFFER never reach "zero"?

Thank you very much in advance.

RE: How to check if the NUS TX Buffer is empty?

Einar Thorsrud — Fri, 18 Dec 2020 13:59:01 GMT

Yes, that looks correct.

RE: How to check if the NUS TX Buffer is empty?

Michael01101 — Fri, 18 Dec 2020 13:45:54 GMT

Thank you again for your fast reply.

So when the BLE_NUS_TX_BUFFER variable is the counter for the amount of NUS packets in the TX buffer, then this would be the right implementation?

static void ble_evt_handler(ble_evt_t const * p_ble_evt, void * p_context)
{
    uint32_t err_code;

    switch (p_ble_evt->header.evt_id)
    {
        case BLE_GATTS_EVT_HVN_TX_COMPLETE:
             BLE_NUS_TX_BUFFER -= p_ble_evt->evt.gatts_evt.params.hvn_tx_complete.count;
             break;
             
        ...
    ...
...

Thank you very much in advance.

RE: How to check if the NUS TX Buffer is empty?

Einar Thorsrud — Fri, 18 Dec 2020 13:33:02 GMT

Yes, that is correct. (Just make sure to check the count when you get a TX complete event, so that if e.g. it is 2, then you decrement the counter by 2 instead of 1.)

RE: How to check if the NUS TX Buffer is empty?

Michael01101 — Fri, 18 Dec 2020 13:30:39 GMT

Thank you very much for your quick response.

So we have to add the BLE_GATTS_EVT_HVN_TX_COMPLETE case in the ble_evt_handler(...).

Then we can create a counter which increases every time we call ble_nus_data_send() and decreases every time a BLE_GATTS_EVT_HVN_TX_COMPLETE event occured, is that right?

Finally we can use the counter as an indicator for the amount of NUS packets in the TX buffer?

Thank you very much in advance.

RE: How to check if the NUS TX Buffer is empty?

Einar Thorsrud — Fri, 18 Dec 2020 13:18:09 GMT

Hi,

There is unfortunately no SoftDevice API to check the number of packets in the Tx buffer. The only way to know this is to count the number of packets you queue and the BLE_GATTS_EVT_HVN_TX_COMPLETE events (note that the count can be higher than 1).