When notification with big data, only maximum 4 notification is sent.

RE: When notification with big data, only maximum 4 notification is sent.

Quangca — Thu, 15 Dec 2022 10:19:02 GMT

Hi Sigurd,

I understand, thank you!

RE: When notification with big data, only maximum 4 notification is sent.

Sigurd — Thu, 15 Dec 2022 09:19:40 GMT

Hi,

Somewhere after you call ble_stack_init()

RE: When notification with big data, only maximum 4 notification is sent.

Quangca — Thu, 15 Dec 2022 08:44:01 GMT

hi Sigurd,

where to add this code below?

    ret_code_t err_code;
    ble_opt_t  opt;

    memset(&opt, 0x00, sizeof(opt));
    opt.common_opt.conn_evt_ext.enable = 1;

    err_code = sd_ble_opt_set(BLE_COMMON_OPT_CONN_EVT_EXT, &opt);
    APP_ERROR_CHECK(err_code);

RE: When notification with big data, only maximum 4 notification is sent.

Sigurd — Wed, 14 Dec 2022 13:14:04 GMT

Try to set the hvn_tx_queue_size directly as well, and see if it improves anything, snippet:

/**@brief Function for the SoftDevice initialization.
 *
 * @details This function initializes the SoftDevice and the BLE event interrupt.
 */
static void ble_stack_init(void)
{
    ret_code_t err_code;

    err_code = nrf_sdh_enable_request();
    APP_ERROR_CHECK(err_code);

    // Configure the BLE stack using the default settings.
    // Fetch the start address of the application RAM.
    uint32_t ram_start = 0;
    err_code = nrf_sdh_ble_default_cfg_set(APP_BLE_CONN_CFG_TAG, &ram_start);
    APP_ERROR_CHECK(err_code);

    ble_cfg_t ble_cfg;
    memset(&ble_cfg, 0, sizeof(ble_cfg));
    ble_cfg.conn_cfg.conn_cfg_tag = APP_BLE_CONN_CFG_TAG;
    ble_cfg.conn_cfg.params.gatts_conn_cfg.hvn_tx_queue_size = 10;

    err_code = sd_ble_cfg_set(BLE_CONN_CFG_GATTS, &ble_cfg, ram_start);
    if (err_code != NRF_SUCCESS)
    {
        NRF_LOG_ERROR("sd_ble_cfg_set() returned %s when attempting to set BLE_CONN_CFG_GATTS.",
                      nrf_strerror_get(err_code));
    }



    // Enable BLE stack.
    err_code = nrf_sdh_ble_enable(&ram_start);
    APP_ERROR_CHECK(err_code);

    // Register a handler for BLE events.
    NRF_SDH_BLE_OBSERVER(m_ble_observer, APP_BLE_OBSERVER_PRIO, ble_evt_handler, NULL);
}

And also try to enable Connection Event Length Extension, snippet:

    ret_code_t err_code;
    ble_opt_t  opt;

    memset(&opt, 0x00, sizeof(opt));
    opt.common_opt.conn_evt_ext.enable = 1;

    err_code = sd_ble_opt_set(BLE_COMMON_OPT_CONN_EVT_EXT, &opt);
    APP_ERROR_CHECK(err_code);

RE: When notification with big data, only maximum 4 notification is sent.

Quangca — Wed, 14 Dec 2022 09:58:39 GMT

yes, i tried but the same.

Do i need change more parameter?

RE: When notification with big data, only maximum 4 notification is sent.

Sigurd — Wed, 14 Dec 2022 09:52:52 GMT

Hi,

[quote user="lta2"] Do you have sugestion for me?[/quote]

Yes, as mentioned:

[quote user="sigurdon"]You could try to set NRF_SDH_BLE_GAP_EVENT_LENGTH in sdk_config.h to a higher value, e.g. 320.[/quote]

RE: When notification with big data, only maximum 4 notification is sent.

Quangca — Wed, 14 Dec 2022 09:41:53 GMT

hi,

I get log error code return in dataTransferHandler() function, and see that it return 0(NRF_SUCCESS) total 4 times, and 20 times remain return 19(NRF_ERROR_RESOURCES), NRF_ERROR_RESOURCES mean Not enough resources like that below:

#define NRF_SUCCESS (NRF_ERROR_BASE_NUM + 0) ///< Successful command

#define NRF_ERROR_RESOURCES (NRF_ERROR_BASE_NUM + 19) ///< Not enough resources for operation

So i think this is the main reason it only send 4 notifications to smart phone app..

=> Do you have sugestion for me?

Below is my RAM and Flash configure:

linker_section_placement_macros="FLASH_PH_START=0x0;FLASH_PH_SIZE=0x30000;RAM_PH_START=0x20000000;RAM_PH_SIZE=0x6000;FLASH_START=0x19000;FLASH_SIZE=0x17000;RAM_START=0x20002000;RAM_SIZE=0x6000"
linker_section_placements_segments="FLASH RX 0x0 0x30000;RAM1 RWX 0x20000000 0x6000"

dataTransferHandler() function:

static void dataTransferHandler(void * p_context)
{
    UNUSED_PARAMETER(p_context);
    
    uint8_t dataLengthTransfer = 128;
    ret_code_t err_code;
    uint16_t i = 0, j;

    for(i = 0; i< 24; i++)
     {
        for(j = 0; j<128; j++)

        {
            dataTransfer[j] = arrayTransfer[i*128+j+i];
        }
        err_code = ble_TWIs_temperature_update(&m_TWIs, dataTransfer, BLE_CONN_HANDLE_ALL, dataLengthTransfer);
        NRF_LOG_INFO("err_code: %d.", err_code); //get erro code to check
        
        if ((err_code != NRF_SUCCESS) && (err_code != NRF_ERROR_INVALID_STATE) &&
            (err_code != NRF_ERROR_RESOURCES) && (err_code != NRF_ERROR_BUSY) &&
            (err_code != BLE_ERROR_GATTS_SYS_ATTR_MISSING))
        {
            APP_ERROR_HANDLER(err_code);
        }
    }
}

RE: When notification with big data, only maximum 4 notification is sent.

Sigurd — Tue, 13 Dec 2022 10:09:08 GMT

You might be filling up the SoftDevice buffer to fast. After 4 notifications queued/buffered, you are likely getting NRF_ERROR_RESOURCES returned. You could try to set NRF_SDH_BLE_GAP_EVENT_LENGTH in sdk_config.h to a higher value, e.g. 320. After this change, you likely need to allocate more RAM to the SoftDevice.

RE: When notification with big data, only maximum 4 notification is sent.

Quangca — Tue, 13 Dec 2022 02:46:27 GMT

hello Sigurd,

Bellow is the ble_TWIs_temperature_update()

ret_code_t ble_TWIs_temperature_update(ble_TWIs_t * p_TWIs, uint8_t     *pData, uint16_t    conn_handle, uint16_t datalength)
{
    if (p_TWIs == NULL)
    {
        return NRF_ERROR_NULL;
    }

    ret_code_t         err_code = NRF_SUCCESS;
    ble_gatts_value_t  gatts_value;

    //if (currentVal != lastVal)
    {
        // Initialize value struct.
        memset(&gatts_value, 0, sizeof(gatts_value));

        gatts_value.len     = datalength * sizeof(uint8_t);
        gatts_value.offset  = 0;
        gatts_value.p_value = pData;

        // Send value if connected and notifying.
        if (p_TWIs->is_notification_supported)
        {
            ble_gatts_hvx_params_t hvx_params;

            memset(&hvx_params, 0, sizeof(hvx_params));

            hvx_params.handle = p_TWIs->TWI2C_level_handles.value_handle;
            hvx_params.type   = BLE_GATT_HVX_NOTIFICATION;
            hvx_params.offset = gatts_value.offset;
            hvx_params.p_len  = &datalength;
            hvx_params.p_data = gatts_value.p_value;

            if (conn_handle == BLE_CONN_HANDLE_ALL)
            {
                NRF_LOG_DEBUG("conn_handle == BLE_CONN_HANDLE_ALL")
                ble_conn_state_conn_handle_list_t conn_handles = ble_conn_state_conn_handles();
                // Try sending notifications to all valid connection handles.
                for (uint32_t i = 0; i < conn_handles.len; i++)
                {
                    if (ble_conn_state_status(conn_handles.conn_handles[i]) == BLE_CONN_STATUS_CONNECTED)
                    {
                        if (err_code == NRF_SUCCESS)
                        {
                            err_code = temperature_notification_send(&hvx_params,
                                                                 conn_handles.conn_handles[i]);
                             NRF_LOG_DEBUG("Try sending notifications to all valid connection handles")
                        }
                        else
                        {
                            // Preserve the first non-zero error code
                            UNUSED_RETURN_VALUE(temperature_notification_send(&hvx_params, conn_handles.conn_handles[i]));
                        }
                    }
                }
            }
            else
            {
                NRF_LOG_DEBUG("conn_handle != BLE_CONN_HANDLE_ALL")
                err_code = temperature_notification_send(&hvx_params, conn_handle);
            }
        }
        else
        {
            err_code = NRF_ERROR_INVALID_STATE;
        }
    }
    return err_code;
}

RE: When notification with big data, only maximum 4 notification is sent.

Sigurd — Mon, 12 Dec 2022 13:25:24 GMT

Hi,

How does the function ble_TWIs_temperature_update() look like?

Can you print what the function is returning?