Error when trying to enable extended advertising on code based on NUS example

RE: Error when trying to enable extended advertising on code based on NUS example

Glen M — Thu, 02 Dec 2021 22:26:22 GMT

I finally got somewhere with this - Instead of always using "Debug" I tried "Erase and Flash" from the actions menu and now the example works (both after Erase and Flash and now that I just debug it. Maybe I had some old stuff in the network core or somewhere that was cleaned out by that action?

Glad to have it going!

RE: Error when trying to enable extended advertising on code based on NUS example

Susheel Nuguru — Mon, 22 Nov 2021 06:52:19 GMT

[quote user="Aryan"]I tried to use your code in hr_coded example and adding settings or BT_LE_EXT_ADV_NCONN_NAME in bt_le_ext_adv_create passes normally. Not sure what else is different in your setup but i cannot find anything that is relevant to this issue to be different than your code snippet and this hr_coded exampe [/quote]

I tried hr_coded example with changes to the create function to match yours and the error code returned was 0.

[quote user="Glen M"]Maybe the return code doesn't matter and it'll end up working anyway if I just let it run?[/quote]

That odes not sound right, error codes other than 0 shows failure and letting it run anyway to expect it to work is very risky.

Have you tried the hr_coded example with minor changes to the arguments to bt_le_ext_adv_create and run it in the debugger, set a breakpoint at this function and see what error code you get? If this example works for you, then we need to take a closer look at your setup.

RE: Error when trying to enable extended advertising on code based on NUS example

Glen M — Fri, 19 Nov 2021 13:27:56 GMT

Yes CONFIG_BT_EXT_ADV is y. I'm not sure what to try next either as we're running the same HR CODED example using the same hardware and tools and getting different results.

How are you verifying that it works - is it just working, or are you looking at the return codes? Maybe the return code doesn't matter and it'll end up working anyway if I just let it run?

RE: Error when trying to enable extended advertising on code based on NUS example

Susheel Nuguru — Fri, 19 Nov 2021 09:16:01 GMT

sorry for the late reply Glen,

I need to get your full setup, obviously there is some difference between your and my setup.

Have you set CONFIG_BT_EXT_ADV=y in your config? Not sure what else is different.

RE: Error when trying to enable extended advertising on code based on NUS example

Glen M — Fri, 12 Nov 2021 23:07:48 GMT

I upgraded to all the latest stuff and I get the same thing... at line 83 of main.c in the create_advertising_coded function I get -5 back from bt_le_ext_adv_create().

RE: Error when trying to enable extended advertising on code based on NUS example

Glen M — Fri, 12 Nov 2021 20:19:21 GMT

I just tried it and I get the same error from bt_le_ext_adv_create. I'm using VSCode with SDK 1.7.0. The newer SDK isn't supposed to have any relevant changes but I guess I could upgrade and see what happens.

RE: Error when trying to enable extended advertising on code based on NUS example

Susheel Nuguru — Fri, 12 Nov 2021 10:04:02 GMT

i used 5340 DK, so not sure why you are seeing any difference. Have you run the hr_coded example (unedited) on your eval board? does it run normally?

RE: Error when trying to enable extended advertising on code based on NUS example

Glen M — Thu, 11 Nov 2021 14:27:54 GMT

Will your example work on my nRF5340 eval board where I could debug it?

RE: Error when trying to enable extended advertising on code based on NUS example

Susheel Nuguru — Thu, 11 Nov 2021 11:45:20 GMT

I tried to use your code in hr_coded example and adding settings or BT_LE_EXT_ADV_NCONN_NAME in bt_le_ext_adv_create passes normally. Not sure what else is different in your setup but i cannot find anything that is relevant to this issue to be different than your code snippet and this hr_coded exampe

RE: Error when trying to enable extended advertising on code based on NUS example

Glen M — Tue, 09 Nov 2021 22:40:37 GMT

If I call bt_le_ext_adv_create after settings_load the error is -5 (-EIO), if I call it before settings_load the return code is -11 (-EAGAIN)

RE: Error when trying to enable extended advertising on code based on NUS example

Glen M — Tue, 09 Nov 2021 15:20:57 GMT

Thanks Susheel

RE: Error when trying to enable extended advertising on code based on NUS example

Susheel Nuguru — Tue, 09 Nov 2021 14:43:58 GMT

sorry Glen, I did not manage to test this as i am OoO due to sick child. I will test this as soon as i am back to office. Since this is an public ticket, hopefully someone chimes in some knowledge before i can test.

RE: Error when trying to enable extended advertising on code based on NUS example

Glen M — Fri, 05 Nov 2021 15:12:57 GMT

Here's a trimmed down version with the problem section. Most of this is from the peripheral_uart (NUS) example that I used as a starting point, I just added the bt_le_ext_adv_create stuff.

static void connected(struct bt_conn *conn, uint8_t err);
static void disconnected(struct bt_conn *conn, uint8_t reason);
#ifdef CONFIG_BT_NUS_SECURITY_ENABLED
static void security_changed(struct bt_conn *conn, bt_security_t level,enum bt_security_err err);
#endif
#if defined(CONFIG_BT_NUS_SECURITY_ENABLED)
static void auth_passkey_display(struct bt_conn *conn, unsigned int passkey);
static void auth_passkey_confirm(struct bt_conn *conn, unsigned int passkey);
static void auth_cancel(struct bt_conn *conn);
static void pairing_confirm(struct bt_conn *conn);
static void pairing_complete(struct bt_conn *conn, bool bonded);
static void pairing_failed(struct bt_conn *conn, enum bt_security_err reason);
#endif
static void bt_receive_cb(struct bt_conn *conn, const uint8_t *const data, uint16_t len);
void error(void);

//== Global Variables ========================================================

static K_SEM_DEFINE(ble_init_ok, 0, 1);

static const struct bt_data m_scan_response_data[] = 
{
   BT_DATA_BYTES(BT_DATA_UUID128_ALL, BT_UUID_NUS_VAL),
};

static struct bt_conn *current_conn;
static struct bt_conn *auth_conn;

static struct bt_conn_cb conn_callbacks = 
{
   .connected    = connected,
   .disconnected = disconnected,
   #ifdef CONFIG_BT_NUS_SECURITY_ENABLED
      .security_changed = security_changed,
   #endif
};

#if defined(CONFIG_BT_NUS_SECURITY_ENABLED)
static struct bt_conn_auth_cb conn_auth_callbacks = 
{
   .passkey_display = auth_passkey_display,
   .passkey_confirm = auth_passkey_confirm,
   .cancel = auth_cancel,
   .pairing_confirm = pairing_confirm,
   .pairing_complete = pairing_complete,
   .pairing_failed = pairing_failed
};
#else
static struct bt_conn_auth_cb conn_auth_callbacks;
#endif

static struct bt_nus_cb nus_cb = 
{
   .received = bt_receive_cb,
};

static struct bt_le_ext_adv *m_extended_advertising_set;
static struct bt_le_ext_adv_start_param ext_adv_start_param = {
	.timeout = 0,
	.num_events = 0,
};

//============================================================================

//----------------------------------------------------------------------------
// Name  : bluetooth_init
// Desc  : Configure Bluetooth services and start advertising
// Ins   : none
// Outs  : none
//----------------------------------------------------------------------------
void bluetooth_init(void)
{
   int err = 0;

   bt_conn_cb_register(&conn_callbacks);

   if (IS_ENABLED(CONFIG_BT_NUS_SECURITY_ENABLED)) 
   {
      bt_conn_auth_cb_register(&conn_auth_callbacks);
   }

   err = bt_enable(NULL);
   if (err) 
   {
      error();
   }

   printk("Bluetooth initialized");

   k_sem_give(&ble_init_ok);

   if (IS_ENABLED(CONFIG_SETTINGS)) 
   {
      settings_load();
   }

   err = bt_le_ext_adv_create(BT_LE_EXT_ADV_NCONN_NAME, NULL, &m_extended_advertising_set);  
   if (err) 
   {
      error();
   }

   err = bt_nus_init(&nus_cb);
   if (err) 
   {
      LOG_ERR("Failed to initialize UART service (err: %d)", err);
      return;
   }
}

RE: Error when trying to enable extended advertising on code based on NUS example

Susheel Nuguru — Fri, 05 Nov 2021 14:04:41 GMT

can you show some code snippets on the details of arguments to bt_le_ext_adv_create I can try to replicate that on my end. That error looks like a missing feature but cannot confirm it before I can see it.