Changing advertising modes from IDLE to FAST SDK V15.0

RE: Changing advertising modes from IDLE to FAST SDK V15.0

amorgan — Fri, 27 Jul 2018 17:14:58 GMT

Håvard,

Thanks for the reply! This resolved the error I was getting, but it was not setting the duration correctly.

Stepping through ble_advertising_start, you will see that

memset(&p_advertising->adv_params, 0, sizeof(p_advertising->adv_params));

is called, which clears out the adv_params. Therefore, the following is what was needed to get the duration / timeout to fully work and to not get the original error message:

static void sleep_mode_enter(void)
{
    ret_code_t err_code;
    
    m_advertising.adv_mode_current = BLE_ADV_MODE_FAST;
    m_advertising.adv_modes_config.ble_adv_fast_timeout = 10000;
    m_advertising.adv_modes_config.ble_adv_fast_interval = APP_ADV_INTERVAL;
    
    m_advertising.adv_params.duration = 10000;
    m_advertising.adv_params.interval = APP_ADV_INTERVAL;

    err_code = ble_advertising_restart_without_whitelist(&m_advertising);
    if (err_code != NRF_ERROR_INVALID_STATE)
    {
        APP_ERROR_CHECK(err_code);
    }
}

Note also that if m_advertising.adv_params.interval = APP_ADV_INTERVAL; is not set before calling ble_advertising_restart_without_whitelist, I still get the invalid parameters error. Even though I was not intending to change the interval, only duration, this still was needed.

Is there any documentation on how to correctly change these parameters? It seems as if the new SDK does not handle this as seamlessly as one would expect.

Also, I am using a whitelist, but in this specific case, where there are no bonds, I do not want the device to be advertising, which does occur when using a whitelist when there are no bonds. Therefore I prevent this from happening by forcing it to go to idle, then on user input, change to advertise without whitelist to get the initial bond.

Either way, thank you for helping out, this has now solved my issue.

RE: Changing advertising modes from IDLE to FAST SDK V15.0

Håvard — Thu, 26 Jul 2018 22:43:49 GMT

static void sleep_mode_enter(void)
{
    ret_code_t err_code;
    
    m_advertising.adv_mode_current = BLE_ADV_MODE_FAST;
    m_advertising.adv_params.duration = 10000;
    m_advertising.adv_params.interval = APP_ADV_INTERVAL;

    err_code = ble_advertising_restart_without_whitelist(&m_advertising);
    if (err_code != NRF_ERROR_INVALID_STATE)
    {
        APP_ERROR_CHECK(err_code);
    }
}

Hi amorgan, try the code above instead. You were setting configs that are only being read during ble_advertising_init().

On a related note, if you are not using whitelist, you should never have to call ble_advertising_restart...

instead you can just call ble_advertising_start(&m_advertising,APP_ADV_MODE_FAST) when you get the idle event.

Good luck!

RE: Changing advertising modes from IDLE to FAST SDK V15.0

amorgan — Mon, 16 Jul 2018 14:17:28 GMT

Sorry for the delay, Attached are the before and after. Note I get the error when ble_advertising_restart_without_whitelist calls flags_set.

Before update m_advertising coming from IDLE:

Before ble_advertising_restart_without_whitelist calls flags_set:

After flags_set returns Error (0x07):

LLet me know if this helps.

TThank you.

RE: Changing advertising modes from IDLE to FAST SDK V15.0

FormerMember — Wed, 04 Jul 2018 08:23:31 GMT

I cannot see directly from the main.c what is causing the issue. When you have the failing and working parameters avaiable, please upload them here.

RE: Changing advertising modes from IDLE to FAST SDK V15.0

amorgan — Tue, 03 Jul 2018 10:45:21 GMT

The file main.c I linked duplicates the issue, which occurs when it is in idle then ble_advertising_restart_without_whitelist is called to switch to fast mode. Would you be able to evaluate this?

I will send the results of your request when i get back in the office.

RE: Changing advertising modes from IDLE to FAST SDK V15.0

FormerMember — Tue, 03 Jul 2018 08:32:46 GMT

It should not be necessary to comment out flags_set.

When the advertising mode is BLE_ADV_MODE_FAST, could you upload a snippet of the current advertising parameters, like then one I have added below?

(I have tested a simple example going from BLE_ADV_MODE_IDLE to BLE_ADV_MODE_FAST and that works. I think it's hard to see just from the code what is wrong, when the problem is invalid parameters, it is better to see what the parameters are.)

RE: Changing advertising modes from IDLE to FAST SDK V15.0

amorgan — Mon, 02 Jul 2018 19:08:02 GMT

thanks for the response.

It doesn't seem to be related to the whitelist, since I am able to restart without a whitelist if I am in BLE_ADV_MODE_FAST and then call ble_advertising_restart_without_whitelist().

It seems to be related to going from BLE_ADV_MODE_IDLE to BLE_ADV_MODE_FAST mode.

If I comment out:

ret = flags_set(p_advertising, BLE_GAP_ADV_FLAGS_LE_ONLY_GENERAL_DISC_MODE);
VERIFY_SUCCESS(ret);

in ble_advertising_restart_without_whitelist function, I no longer get the issue and it works correctly.

Full modified function:

uint32_t ble_advertising_restart_without_whitelist(ble_advertising_t * const p_advertising)
{
    ret_code_t ret;

    (void) sd_ble_gap_adv_stop(p_advertising->adv_handle);

    p_advertising->whitelist_temporarily_disabled = true;
    p_advertising->whitelist_in_use               = false;
    p_advertising->adv_params.filter_policy       = BLE_GAP_ADV_FP_ANY;
    // Set correct flags.
    //ret = flags_set(p_advertising, BLE_GAP_ADV_FLAGS_LE_ONLY_GENERAL_DISC_MODE);
    //VERIFY_SUCCESS(ret);

    ret = ble_advertising_start(p_advertising, p_advertising->adv_mode_current);
    if ((ret != NRF_SUCCESS) && (p_advertising->error_handler != NULL))
    {
        p_advertising->error_handler(ret);
    }

    return NRF_SUCCESS;
}

It is unclear which config is causing the issue or how to resolve this error without modified the Nordic SDK source code, since ble_advertising_start() seems to set the rest of the parameters correctly.

Have you evaluated the main.c file above using the example I referenced?

Thank you.

RE: Changing advertising modes from IDLE to FAST SDK V15.0

FormerMember — Fri, 29 Jun 2018 12:41:30 GMT

In ble_advertising_restart_without_whitelist(), sd_ble_gap_adv_set_configure() returns NRF_ERROR_INVALID_PARAM.

According to the doumentation, sd_ble_gap_adv_set_configure() returns NRF_ERROR_INVALID_PARAM if:

- Invalid configuration of p_adv_params. See @ref ble_gap_adv_params_t.
- Use of whitelist requested but whitelist has not been set, see @ref sd_ble_gap_whitelist_set.

Did you remember to disable the whitelist setting in m_advertising.adv_modes_config.ble_adv_whitelist_enabled?

Could you run the chip in debug mode and compare the settings of m_advertising for a non-working and a working case* ?

*working case: advertising without whitelist

RE: Changing advertising modes from IDLE to FAST SDK V15.0

amorgan — Fri, 22 Jun 2018 13:48:58 GMT

Joakim Jakobsen Was my reply helpful or any updates?

Thank you

RE: Changing advertising modes from IDLE to FAST SDK V15.0

amorgan — Mon, 18 Jun 2018 18:21:51 GMT

I have duplicated the issue using the SDK v15.0.0 example: ble_app_template using IAR (using ble_app_template_pca10040e_S112 with a Rigado dev board similar to the pca10040). Attached is the modified main.c file.

I changed it so that instead of going into a sleep mode, it tries to restart without advertising. This produces the same error. Maybe I'm doing something fundamentally wrong, but this seems to duplicate the similar behavior in my application code.

The only section that should be different than the original example is the sleep_mode_enter() function.

/cfs-file/__key/communityserver-discussions-components-files/4/1565.main.c

RE: Changing advertising modes from IDLE to FAST SDK V15.0

Joakim Jakobsen — Mon, 18 Jun 2018 13:44:39 GMT

Hi!

That sounds strange.

Is is possible for you to upload your code? So that I can get a better overview of what is going on.

If your code contains any sensitive information and you don't want to upload your code to the public forum, it is possible to turn the case private first.

Best regards,
Joakim.