LMP Responese Timeout - nRF52840 S140v6.1.0 SDK 15.2.0

RE: LMP Responese Timeout - nRF52840 S140v6.1.0 SDK 15.2.0

Fabian Hauschild — Mon, 11 Mar 2019 13:15:40 GMT

Hi,

I have tried out the new SoftDevice S140v6.1.1.

Using this new SoftSevice solved the problem for me.

RE: LMP Responese Timeout - nRF52840 S140v6.1.0 SDK 15.2.0

Fabian Hauschild — Mon, 11 Mar 2019 13:09:46 GMT

Yes it works with the PCA10040 project but not with the PCA10056 project.

RE: LMP Responese Timeout - nRF52840 S140v6.1.0 SDK 15.2.0

Mttrinh — Mon, 11 Mar 2019 12:56:32 GMT

So it works when you use the PCA10040 project but not when you are using the PCA10056 project?

No, it isn't a typo.

RE: LMP Responese Timeout - nRF52840 S140v6.1.0 SDK 15.2.0

Fabian Hauschild — Mon, 04 Mar 2019 12:59:59 GMT

Hi,

I have added these two lines but it didn't help me.

The Problem is still the same.

The error code after sd_ble_gap_data_length_update(conn_handle, &dlp, &dll) is NRF_SUCCESS so the added lines of code won't get executed

However I have found out that there are no problems with SD132v6.1.0.

I have tried the app_blinky example for pca10040 on the pca10056 with the same modifications i have mentioned above. (changed NRF_SDH_BLE_GATT_MAX_MTU_SIZE to 138 and NRF_SDH_BLE_GAP_DATA_LENGTH to 142)

With this example the S7 didn't disconnect with LMP Response Timeout.

P.S.

I think there is a typo in the two lines I should add.

I think they schould be like this:

err_code = sd_ble_gap_data_length_update(conn_handle, &dlp, &dll);
return err_code;

RE: LMP Responese Timeout - nRF52840 S140v6.1.0 SDK 15.2.0

Mttrinh — Fri, 01 Mar 2019 13:55:56 GMT

Hi,

In nrf_ble_gatt.c, inside data_length_update(..), could you add these two lines:

err_code = sd_ble_gap_data_length_update(conn_handle, &dll, &dll);
return err_code;

Like this:

static ret_code_t data_length_update(uint16_t conn_handle, uint16_t data_length)
{
    NRF_LOG_DEBUG("Updating data length to %u on connection 0x%x.",
                  data_length, conn_handle);

    ble_gap_data_length_params_t const dlp =
    {
        .max_rx_octets  = data_length,
        .max_tx_octets  = data_length,
        .max_rx_time_us = BLE_GAP_DATA_LENGTH_AUTO,
        .max_tx_time_us = BLE_GAP_DATA_LENGTH_AUTO,
    };

    ble_gap_data_length_limitation_t dll = {0};

    ret_code_t err_code = sd_ble_gap_data_length_update(conn_handle, &dlp, &dll);
    if (err_code != NRF_SUCCESS)
    {
        NRF_LOG_ERROR("sd_ble_gap_data_length_update() (request) on connection 0x%x returned %s.",
                      conn_handle, nrf_strerror_get(err_code));

        if (   (dll.tx_payload_limited_octets != 0)
            || (dll.rx_payload_limited_octets != 0))
        {
            NRF_LOG_ERROR("The requested TX/RX packet length is too long by %u/%u octets.",
                          dll.tx_payload_limited_octets, dll.rx_payload_limited_octets);
+                         err_code = sd_ble_gap_data_length_update(conn_handle, &dll, &dll);
+                         return err_code;
        }

        if (dll.tx_rx_time_limited_us != 0)
        {
            NRF_LOG_ERROR("The requested combination of TX and RX packet lengths "
                          "is too long by %u microseconds.",
                          dll.tx_rx_time_limited_us);
        }
    }

    return err_code;
}

See if this helps you solve the problem?

RE: LMP Responese Timeout - nRF52840 S140v6.1.0 SDK 15.2.0

Fabian Hauschild — Mon, 25 Feb 2019 13:39:48 GMT

Hi ,

I have only changed the MTU.

I Havn't toched the connection interval.

#define MIN_CONN_INTERVAL               MSEC_TO_UNITS(100, UNIT_1_25_MS)        /**< Minimum acceptable connection interval (0.5 seconds). */
#define MAX_CONN_INTERVAL               MSEC_TO_UNITS(200, UNIT_1_25_MS)        /**< Maximum acceptable connection interval (1 second). */
#define SLAVE_LATENCY                   0                                       /**< Slave latency. */
#define CONN_SUP_TIMEOUT                MSEC_TO_UNITS(4000, UNIT_10_MS)         /**< Connection supervisory time-out (4 seconds). */

#define FIRST_CONN_PARAMS_UPDATE_DELAY  APP_TIMER_TICKS(20000)                  /**< Time from initiating event (connect or start of notification) to first time sd_ble_gap_conn_param_update is called (15 seconds). */
#define NEXT_CONN_PARAMS_UPDATE_DELAY   APP_TIMER_TICKS(5000)                   /**< Time between each call to sd_ble_gap_conn_param_update after the first call (5 seconds). */
#define MAX_CONN_PARAMS_UPDATE_COUNT    3                                       /**< Number of attempts before giving up the connection parameter negotiation. */

#define BUTTON_DETECTION_DELAY          APP_TIMER_TICKS(50)                     /**< Delay from a GPIOTE event until a button is reported as pushed (in number of timer ticks). */

These are the original connection Parameter from the example.

RE: LMP Responese Timeout - nRF52840 S140v6.1.0 SDK 15.2.0

Mttrinh — Mon, 25 Feb 2019 13:28:33 GMT

Hi,

I have talked to one of our Android developers and it seems like that there shouldn't be different MTU size for different phones. Android supports MTU from 23 up to 517. What other changes have you done to the example? What is your connection interval?