Peripheral fails to re-start advertising after disconnecting from Client

RE: Peripheral fails to re-start advertising after disconnecting from Client

Mike Austin (LPI) — Wed, 12 Jul 2023 06:08:24 GMT

Hi Joakim,

Seems I spoke too soon. I'm still having this issue.

For whatever reason, when I have an iOS Client (such as the nRF Connect App) attempt to disconnect from my device, the connection remains valid and I cannot see it re-advertise.

The only way I seem to be able to get the disconnection to work successfully is to actually have a Disconnect Service and Characteristic, and when the Client writes to the Characteristic, I call:

// Function to update the value of disconnect_status when Client writes to it
static ssize_t write_disconnect_status(struct bt_conn *conn,
			       const struct bt_gatt_attr *attr, const void *buf,
			       uint16_t len, uint16_t offset, uint8_t flags)
{
	LOG_DBG("INFO: Received value for disconnect_status: %d", ((uint8_t *) buf)[0]);
	disconnect_status = ((uint8_t *) buf)[0];
	if (disconnect_status)
	{
		uint8_t rv = bt_conn_disconnect(conn, BT_HCI_ERR_REMOTE_USER_TERM_CONN);
		if (rv == 0) {
			LOG_DBG("INFO: Peripheral successfully disconencted from Central");
			disconnect_status = 0;
		}
		else {
			LOG_DBG("ERROR: Peripheral failed to disconenct from Central");
		}
	}
	return len;
}

This then successfully forces a disconnection of the current connection.

Yet, if the iOS Client simply issues a disconnect request, the disconnect callback I have defined doesn't appear to get called.

My Peripheral is Paired and Bonded to the Client (I need this for getting the CTS). Not sure if this is part of the problem?

I recall seeing this issue about a year ago (refer to this link) and was never really able to resolve it beyond have a specific Service/Characteristic that would enable me to force a disconnection.

If I used an nRF52-DK as my Client, and have this interfaced to the nRF Connect for Desktop Bluetooth Low Energy Standalone App on my Windows machine, then when I press the "disconnect" button on that, it successfully disconnects from the Peripheral and the Peripheral will start to advertise again.

Regards,

Mike

RE: Peripheral fails to re-start advertising after disconnecting from Client

Mike Austin (LPI) — Sun, 11 Jun 2023 06:30:50 GMT

Hi Joakim,

I eventually got to the bottom of the issue. I'd blindly copied some code from another example that was providing notifications on a characteristic, and in the connection callback for that example, they had the following line:

current_conn = bt_conn_ref(conn);

The current_conn parameter was used to make changes to the MTU and a few other connection parameters. But calling bt_conn_ref() I believe is incrementing the connection counter, so when I disconnected, I believe I was only disconnecting from one connection, or the connection counter was such that it was indicating there was still a valid connection. So, when I went to update the advertisting data, it still was effectively connected, not advertising, hence the error messages.

I can fix this issue in two ways:

1. Add bt_conn_unref(current_con) in my disconnect callback

2. Calling current_con = conn in my connection callback, and excluding the call to bt_conn_ref().

I'm not exactly sure why you could use bt_conn_ref() and I'm not sure which of the above solutions is actually the correct (or if they both are, then better) option.

Regards,

Mike

RE: Peripheral fails to re-start advertising after disconnecting from Client

Joakim Jakobsen — Fri, 09 Jun 2023 07:55:41 GMT

Hi Mike.

[quote user=""]I would expect that once the Peripheral disconnects, it should automatically restart advertising.[/quote]

This is partially true. Most of the samples implements this functionality, because in most use cases it makes sense to start advertising once the device disconnects.

However, this is controlled by the application. So after your device disconnects to have to make sure that bt_le_adv_start() is called.

You can use the function bt_le_adv_update_data() if your advertising is already started.

Br,
Joakim