Can't get BLE extended connectable advertising timeout callback

RE: Can't get BLE extended connectable advertising timeout callback

Simonr — Tue, 28 May 2024 09:29:47 GMT

Okay, so we were able to add this on our end after a bit trial and error. Here is the relevant source code and configs (tested on NCS 2.1.1):

main.c

/*
 * Copyright (c) 2021 Nordic Semiconductor ASA
 *
 * SPDX-License-Identifier: LicenseRef-Nordic-5-Clause
 */

/** @file
 *  @brief Peripheral example
 */

#include <zephyr/types.h>
#include <stddef.h>
#include <string.h>
#include <errno.h>
#include <sys/printk.h>
#include <zephyr.h>

#include <bluetooth/bluetooth.h>
#include <bluetooth/conn.h>
#include <bluetooth/uuid.h>
#include <bluetooth/gatt.h>

#include <logging/log.h>
LOG_MODULE_REGISTER(app, CONFIG_LOG_DEFAULT_LEVEL);


#define BLE_INT                         (0x140) // N * 0.625. corresponds to dec. 320; 320*0.625 = 200ms
#define BLE_ADV_TIMEOUT                 (1000)  // N * 10ms for advertiser timeout
#define BLE_ADV_EVENTS                  (0)

static struct k_work start_advertising_worker;

static struct bt_le_ext_adv *adv;

static const struct bt_data ad[] = {
    BT_DATA_BYTES(BT_DATA_FLAGS, (BT_LE_AD_GENERAL | BT_LE_AD_NO_BREDR)),
    BT_DATA_BYTES(BT_DATA_UUID16_ALL, 0x0d, 0x18, 0x0f, 0x18, 0x0a, 0x18),
};

#if defined(CONFIG_BT_PERIPHERAL)

static void connected(struct bt_conn *conn, uint8_t conn_err)
{
    int err;
    struct bt_conn_info info;
    char addr[BT_ADDR_LE_STR_LEN];

    bt_addr_le_to_str(bt_conn_get_dst(conn), addr, sizeof(addr));

    if (conn_err) {
        printk("Connection failed (err %d)\n", conn_err);
        return;
    }

    err = bt_conn_get_info(conn, &info);

    if (err) {
        printk("Failed to get connection info\n");
    } else {
        const struct bt_conn_le_phy_info *phy_info;
        phy_info = info.le.phy;

        printk("Connected: %s, tx_phy %u, rx_phy %u\n",
               addr, phy_info->tx_phy, phy_info->rx_phy);
    }
}

static void disconnected(struct bt_conn *conn, uint8_t reason)
{
    printk("Disconnected (reason 0x%02x)\n", reason);

    k_work_submit(&start_advertising_worker);
}

static struct bt_conn_cb conn_callbacks = {
    .connected = connected,
    .disconnected = disconnected,
    
};

#endif /* defined(CONFIG_BT_PERIPHERAL) */


static void adv_sent(struct bt_le_ext_adv *instance,
             struct bt_le_ext_adv_sent_info *info)
{   
    //printk("Advertising stopped. num_sent: %d ", info->num_sent);
    LOG_INF("Advertising stopped");
}

static int create_advertising(void)
{
    int err;
    struct bt_le_adv_param param =
        BT_LE_ADV_PARAM_INIT(
                     BT_LE_ADV_OPT_EXT_ADV | BT_LE_ADV_OPT_USE_NAME,
                     BLE_INT,
                     BLE_INT,
                     NULL);

    static const struct bt_le_ext_adv_cb adv_cb = {
        .sent = adv_sent,
    };

    err = bt_le_ext_adv_create(&param, &adv_cb, &adv);
    if (err) {
        printk("Failed to create advertiser set (%d)\n", err);
        return err;
    }

    printk("Created adv: %p\n", adv);

    err = bt_le_ext_adv_set_data(adv, ad, ARRAY_SIZE(ad), NULL, 0);
    if (err) {
        printk("Failed to set advertising data (%d)\n", err);
        return err;
    }

    return 0;
}

static void start_advertising(struct k_work *item)
{
    int err;

    err = bt_le_ext_adv_start(adv, BT_LE_EXT_ADV_START_PARAM(BLE_ADV_TIMEOUT, BLE_ADV_EVENTS));
    if (err) {
        printk("Failed to start advertising set (%d)\n", err);
        return;
    }

    //printk("Advertiser %p set started\n", adv);
    LOG_INF("Advertising started");
}

static void bt_ready(void)
{
    int err = 0;

    printk("Bluetooth initialized\n");

    k_work_init(&start_advertising_worker, start_advertising);

    err = create_advertising();
    if (err) {
        printk("Advertising failed to create (err %d)\n", err);
        return;
    }

    k_work_submit(&start_advertising_worker);
}

void radio_handler(const void *context)
{

    printk("radio_handler");

}

void main(void)
{
    int err;

    printk("Main start");

    err = bt_enable(NULL);
    if (err) {
        printk("Bluetooth init failed (err %d)\n", err);
        return;
    }

    bt_ready();

    bt_conn_cb_register(&conn_callbacks);
}

prj.conf

CONFIG_BT=y
CONFIG_BT_DEBUG_LOG=y
CONFIG_BT_PERIPHERAL=y

CONFIG_BT_CTLR_PHY_CODED=y
CONFIG_BT_EXT_ADV=y
CONFIG_BT_USER_PHY_UPDATE=y

# Log
CONFIG_LOG=y
CONFIG_LOG_DEFAULT_LEVEL=3

Here we get the callback after ~10 seconds when using BT_LE_EXT_ADV_START_PARAM(1000, 0)

Best regards,

Simon

RE: Can't get BLE extended connectable advertising timeout callback

Simonr — Fri, 24 May 2024 12:46:19 GMT

I'll get back to you next week with a decisive answer here, as I'll need some more time to look into/ask the devs about how to implement this correctly. Thank you for your patience!

Best regards,

Simon

RE: Can't get BLE extended connectable advertising timeout callback

Alex_S@Aperia — Thu, 23 May 2024 15:44:32 GMT

Unfortunately I couldn't find (in SDK 2.1.2) any examples of using timeout in BT_LE_EXT_ADV_START_PARAM
All examples use BT_LE_EXT_ADV_START_DEFAULT, which is BT_LE_EXT_ADV_START_PARAM(0, 0)

RE: Can't get BLE extended connectable advertising timeout callback

Alex_S@Aperia — Thu, 23 May 2024 15:41:01 GMT

Also see

struct bt_le_per_adv_sync_param {
/**
* @brief Periodic Advertiser Address
*
* Only valid if not using the periodic advertising list
* (BT_LE_PER_ADV_SYNC_OPT_USE_PER_ADV_LIST)
*/
bt_addr_le_t addr;

/**
* @brief Advertiser SID
*
* Only valid if not using the periodic advertising list
* (BT_LE_PER_ADV_SYNC_OPT_USE_PER_ADV_LIST)
*/
uint8_t sid;

/** Bit-field of periodic advertising sync options. */
uint32_t options;

/**
* @brief Maximum event skip
*
* Maximum number of periodic advertising events that can be
* skipped after a successful receive.
* Range: 0x0000 to 0x01F3
*/
uint16_t skip;

/**
* @brief Synchronization timeout (N * 10 ms)
*
* Synchronization timeout for the periodic advertising sync.
* Range 0x000A to 0x4000 (100 ms to 163840 ms)
*/
uint16_t timeout;
};

RE: Can't get BLE extended connectable advertising timeout callback

Alex_S@Aperia — Thu, 23 May 2024 15:39:12 GMT

> From where do you see that this equals 10ms units?
From bluetooth.h. Perhaps I misunderstood the comments ?
What should the unit be then ?

struct bt_le_ext_adv_start_param {
/**
* @brief Advertiser timeout (N * 10 ms).
*
* Application will be notified by the advertiser sent callback.
* Set to zero for no timeout.
*
* When using high duty cycle directed connectable advertising then
* this parameters must be set to a non-zero value less than or equal
* to the maximum of @ref BT_GAP_ADV_HIGH_DUTY_CYCLE_MAX_TIMEOUT.
*
* If privacy @kconfig{CONFIG_BT_PRIVACY} is enabled then the timeout
* must be less than @kconfig{CONFIG_BT_RPA_TIMEOUT}.
*/
uint16_t timeout;
/**
* @brief Number of advertising events.
*
* Application will be notified by the advertiser sent callback.
* Set to zero for no limit.
*/
uint8_t num_events;
};

RE: Can't get BLE extended connectable advertising timeout callback

Simonr — Tue, 21 May 2024 10:42:45 GMT

So, from bluetooth.h, the bt_le_ext_adv_cb() returns when the advertising set has finished, either by way of a timeout or because the specified number of advertising events have been reached. I think the reason you're not seeing the timeout being trigged is because your define of the BLE_CONN_ADV_TIMEOUT doesn't really specify what the setting is. From where do you see that this equals 10ms units?

#define BLE_CONN_ADV_TIMEOUT 4 * 100; // 4 sec in 10ms units

Best regards,

Simon

RE: Can't get BLE extended connectable advertising timeout callback

Alex_S@Aperia — Thu, 16 May 2024 23:02:48 GMT

Nah, looking at SDK code, the only place this BT_GAP_ADV_HIGH_DUTY_CYCLE_MAX_TIMEOUT is used is for direct high duty advertisement, mine is regular connectable advertisement, not direct)

So, back to original question, why

param = BT_LE_EXT_ADV_START_PARAM(1000, 0);

doesn't generate bt_le_ext_adv_cb.sent callback after 10 sec timeout, but

param = BT_LE_EXT_ADV_START_PARAM(0, 3);

does after 3 adverisements sent

RE: Can't get BLE extended connectable advertising timeout callback

Alex_S@Aperia — Thu, 16 May 2024 17:26:49 GMT

Perhaps this is my problem
/* The maximum allowed high duty cycle directed advertising timeout, 1.28
* seconds in 10 ms unit.
*/
#define BT_GAP_ADV_HIGH_DUTY_CYCLE_MAX_TIMEOUT 128

I'm trying much longer timeouts...

Is there a way to translate timeout in seconds to number of events ?