NCS BLE Disconnect issue.

[Wangshuhai]

Hi All.

NCS 2.3.0,  peripheral LBS example.

   I want try to disconnect session between peripheral and central . 

Then I call the bt_conn_disconnect(default_conn, BT_HCI_ERR_REMOTE_USER_TERM_CONN);  to disconnect .It's worked .

But i added the the bt_conn_disconnect fuction into k_work,   when i call disconnect fuction disconnected callback  will be happen after 5S .

But it's  weird , i added the the bt_conn_disconnect fuction into button callback , the disconnected callback  will happen immediate.

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

#include <zephyr/types.h>
#include <stddef.h>
#include <string.h>
#include <errno.h>
#include <zephyr/sys/printk.h>
#include <zephyr/sys/byteorder.h>
#include <zephyr/kernel.h>
#include <zephyr/drivers/gpio.h>
#include <soc.h>

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

#include <bluetooth/services/lbs.h>

#include <zephyr/settings/settings.h>

#include <dk_buttons_and_leds.h>

#define DEVICE_NAME             CONFIG_BT_DEVICE_NAME
#define DEVICE_NAME_LEN         (sizeof(DEVICE_NAME) - 1)


#define RUN_STATUS_LED          DK_LED1
#define CON_STATUS_LED          DK_LED2
#define RUN_LED_BLINK_INTERVAL  1000

#define USER_LED                DK_LED3

#define USER_BUTTON             DK_BTN1_MSK

static bool app_button_state;
struct bt_conn *default_conn;

static const struct bt_data ad[] = {
	BT_DATA_BYTES(BT_DATA_FLAGS, (BT_LE_AD_GENERAL | BT_LE_AD_NO_BREDR)),
	BT_DATA(BT_DATA_NAME_COMPLETE, DEVICE_NAME, DEVICE_NAME_LEN),
};

static const struct bt_data sd[] = {
	BT_DATA_BYTES(BT_DATA_UUID128_ALL, BT_UUID_LBS_VAL),
};
static struct k_work advertise_work;
static struct k_work_delayable nus_msg_timer;




/**
 * @brief 连接之后没有
 * 
 */
void nus_msg_connect_timeout(struct k_work *work)
{
	int err;

	printk("time %d\r\n",k_uptime_get_32());
	if(default_conn != NULL)
	{
		err = bt_conn_disconnect(default_conn, BT_HCI_ERR_REMOTE_USER_TERM_CONN);
		if (err) {
			printk("Disconnection failed (err %d)\r\n", err);
			return ;
		}
		printk("call Disconnect!\r\n");
	}
}

void nus_msg_connect(void)
{
    k_work_cancel_delayable(&nus_msg_timer);

    k_work_schedule(&nus_msg_timer, K_MSEC(5000));
}



static void connected(struct bt_conn *conn, uint8_t err)
{
	if (err) {
		printk("Connection failed (err %u)\n", err);
		return;
	}

	if (!default_conn) {
		default_conn = bt_conn_ref(conn);
	}

	printk("Connected\n");

	dk_set_led_on(CON_STATUS_LED);
	
	nus_msg_connect();
}

static void disconnected(struct bt_conn *conn, uint8_t reason)
{
	printk("Disconnected (reason %u)\n", reason);
	printk("time %d\r\n",k_uptime_get_32());
	if (default_conn == conn) {
		bt_conn_unref(default_conn);
		default_conn = NULL;
	}

	dk_set_led_off(CON_STATUS_LED);

	k_work_submit(&advertise_work);
}

#ifdef CONFIG_BT_LBS_SECURITY_ENABLED
static void security_changed(struct bt_conn *conn, bt_security_t level,
			     enum bt_security_err err)
{
	char addr[BT_ADDR_LE_STR_LEN];

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

	if (!err) {
		printk("Security changed: %s level %u\n", addr, level);
	} else {
		printk("Security failed: %s level %u err %d\n", addr, level,
			err);
	}
}
#endif

BT_CONN_CB_DEFINE(conn_callbacks) = {
	.connected        = connected,
	.disconnected     = disconnected,
#ifdef CONFIG_BT_LBS_SECURITY_ENABLED
	.security_changed = security_changed,
#endif
};

#if defined(CONFIG_BT_LBS_SECURITY_ENABLED)
static void auth_passkey_display(struct bt_conn *conn, unsigned int passkey)
{
	char addr[BT_ADDR_LE_STR_LEN];

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

	printk("Passkey for %s: %06u\n", addr, passkey);
}

static void auth_cancel(struct bt_conn *conn)
{
	char addr[BT_ADDR_LE_STR_LEN];

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

	printk("Pairing cancelled: %s\n", addr);
}

static void pairing_complete(struct bt_conn *conn, bool bonded)
{
	char addr[BT_ADDR_LE_STR_LEN];

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

	printk("Pairing completed: %s, bonded: %d\n", addr, bonded);
}

static void pairing_failed(struct bt_conn *conn, enum bt_security_err reason)
{
	char addr[BT_ADDR_LE_STR_LEN];

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

	printk("Pairing failed conn: %s, reason %d\n", addr, reason);
}

static struct bt_conn_auth_cb conn_auth_callbacks = {
	.passkey_display = auth_passkey_display,
	.cancel = auth_cancel,
};

static struct bt_conn_auth_info_cb conn_auth_info_callbacks = {
	.pairing_complete = pairing_complete,
	.pairing_failed = pairing_failed
};
#else
static struct bt_conn_auth_cb conn_auth_callbacks;
#endif

static void app_led_cb(bool led_state)
{
	dk_set_led(USER_LED, led_state);
}

static bool app_button_cb(void)
{
	return app_button_state;
}

static struct bt_lbs_cb lbs_callbacs = {
	.led_cb    = app_led_cb,
	.button_cb = app_button_cb,
};

static void button_changed(uint32_t button_state, uint32_t has_changed)
{
	int err;

	if (has_changed & USER_BUTTON) {
		uint32_t user_button_state = button_state & USER_BUTTON;

		bt_lbs_send_button_state(user_button_state);
		app_button_state = user_button_state ? true : false;
	}
	else if(has_changed & DK_BTN2_MSK)
	{
		if(default_conn != NULL)
		{
			printk("time %d\r\n",k_uptime_get_32());
			err = bt_conn_disconnect(default_conn, BT_HCI_ERR_REMOTE_USER_TERM_CONN);
			if (err) {
				printk("Disconnection failed (err %d)\r\n", err);
				return ;
			}
			printk("Disconnect!\r\n");
		}
	}
}

static int init_button(void)
{
	int err;

	err = dk_buttons_init(button_changed);
	if (err) {
		printk("Cannot init buttons (err: %d)\n", err);
	}

	return err;
}



static void advertise(struct k_work *work)
{
    int ret;

    bt_le_adv_stop();

    /* Check UICR to see what channel is selected for this HW */
    ret = bt_le_adv_start(BT_LE_ADV_CONN, ad, ARRAY_SIZE(ad), NULL, 0);

    if (ret) {
        printk("Advertising failed to start (ret %d)\r\n", ret);
		return ;
    }

    printk("Advertising successfully started\r\n");
}


void main(void)
{
	int blink_status = 0;
	int err;

	printk("Starting Bluetooth Peripheral LBS example\n");

	k_work_init(&advertise_work, advertise);
    k_work_init_delayable(&nus_msg_timer , nus_msg_connect_timeout);

	err = dk_leds_init();
	if (err) {
		printk("LEDs init failed (err %d)\n", err);
		return;
	}

	err = init_button();
	if (err) {
		printk("Button init failed (err %d)\n", err);
		return;
	}

	if (IS_ENABLED(CONFIG_BT_LBS_SECURITY_ENABLED)) {
		err = bt_conn_auth_cb_register(&conn_auth_callbacks);
		if (err) {
			printk("Failed to register authorization callbacks.\n");
			return;
		}

		err = bt_conn_auth_info_cb_register(&conn_auth_info_callbacks);
		if (err) {
			printk("Failed to register authorization info callbacks.\n");
			return;
		}
	}

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

	printk("Bluetooth initialized\n");

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

	err = bt_lbs_init(&lbs_callbacs);
	if (err) {
		printk("Failed to init LBS (err:%d)\n", err);
		return;
	}

	err = bt_le_adv_start(BT_LE_ADV_CONN, ad, ARRAY_SIZE(ad),
			      sd, ARRAY_SIZE(sd));
	if (err) {
		printk("Advertising failed to start (err %d)\n", err);
		return;
	}

	printk("Advertising successfully started\n");

	for (;;) {
		dk_set_led(RUN_STATUS_LED, (++blink_status) % 2);
		k_sleep(K_MSEC(RUN_LED_BLINK_INTERVAL));
	}
}

may you give me some advice?

Thank you very much.

Wang

[Kazi Afroza Sultana]

Hello Wang,

Can you share the whole project so I can reproduce this here?

Thanks.

BR

Kazi

[Wangshuhai]

Hi Kazi

   Thanks for your replay. 

My project is lbs 

0027.peripheral_lbs.zip

Thanks !

Wang

[Kazi Afroza Sultana]

Hello Wang,

I have run your sample on NCS v2.3.0 and that is the output on serial terminal. 

The advertising from nRF52840 DK started successfully and connected to central. I disconnected this from nRF connect app from mobile and it is disconnected by reason 19, that is normal. 

You got error reason 34 (0x22 in hex) and 22 (0x16); according to BLUETOOTH CORE SPECIFICATION Version 5.2 | Vol 1, Part F page 371.

Could you please send us the sniffer log (https://www.nordicsemi.com/Products/Development-tools/nrf-sniffer-for-bluetooth-le) so we can see what the cause is? 

Thanks.

BR

Kazi

[Wangshuhai]

Hi Kazi

   Happy to hear your reply， I forget tell you that i using nRF5340-DK . Could your try on nRF5340?

Thanks 

Wang

[Kazi Afroza Sultana]

Hello Wang,

I tried this on nRF52840. I have tried to run the application on nRF5340. It seems peripheral can advertise. Because both on BLE app and nRF connect for mobile can not see the device while scanning. I will test more tomorrow and will give you the feedback.