bluetooth: services: ble_hrs_client: add bsl event

Add body sensor location event.

Signed-off-by: Sondre Pettersen <sondre.pettersen@nordicsemi.no>
Co-authored-by: Asil Zogby <asil.zogby@nordicsemi.no>

Author: sondrep

include/bm/bluetooth/services/ble_hrs_client.h

@@ -56,6 +56,10 @@ enum ble_hrs_client_evt_type {
 	 *  received from the peer.
 	 */
 	BLE_HRS_CLIENT_EVT_HRM_NOTIFICATION,
+	/** Event indicating that the Body Sensor Location characteristic was received from the
+	 *  peer
+	 */
+	BLE_HRS_CLIENT_EVT_BSL_UPDATE,
 	/** Error. */
 	BLE_HRS_CLIENT_EVT_ERROR,
 };
@@ -80,6 +84,8 @@ struct ble_hrs_handles {
 	uint16_t hrm_cccd_handle;
 	/** Handle of the Heart Rate Measurement characteristic, as provided by the SoftDevice. */
 	uint16_t hrm_handle;
+	/** Handle of the Body Sensor Location characteristic, as provided by the SoftDevice */
+	uint16_t bsl_handle;
 };
 
 /**
@@ -98,6 +104,8 @@ struct ble_hrs_client_evt {
 		} discovery_complete;
 		/** @ref BLE_HRS_CLIENT_EVT_HRM_NOTIFICATION event data. */
 		struct ble_hrs_measurement hrm_notification;
+		/** @ref BLE_HRS_CLIENT_EVT_BSL_UPDATE event data. */
+		uint8_t body_sensor_location;
 		/** @ref BLE_HRS_CLIENT_EVT_ERROR event data. */
 		struct {
 			/** Error reason */

subsys/bluetooth/services/ble_hrs_client/ble_hrs_client.c

@@ -119,6 +119,7 @@ static void on_disconnected(struct ble_hrs_client *hrs_client, const ble_evt_t *
 		hrs_client->conn_handle = BLE_CONN_HANDLE_INVALID;
 		hrs_client->handles.hrm_cccd_handle = BLE_GATT_HANDLE_INVALID;
 		hrs_client->handles.hrm_handle = BLE_GATT_HANDLE_INVALID;
+		hrs_client->handles.bsl_handle = BLE_GATT_HANDLE_INVALID;
 	}
 }
 
@@ -128,6 +129,7 @@ void ble_hrs_on_db_disc_evt(struct ble_hrs_client *hrs_client,
 	__ASSERT(hrs_client, "HRS client instance is NULL");
 	__ASSERT(db_discovery_evt, "Discovery event is NULL");
 
+	uint32_t nrf_err;
 	const struct ble_gatt_db_char *db_char;
 	struct ble_hrs_client_evt hrs_client_evt = {
 		.evt_type = BLE_HRS_CLIENT_EVT_DISCOVERY_COMPLETE,
@@ -142,7 +144,11 @@ void ble_hrs_on_db_disc_evt(struct ble_hrs_client *hrs_client,
 		return;
 	}
 
-	/* Find the CCCD Handle of the Heart Rate Measurement characteristic. */
+	/* Iterate discovered characteristics:
+	 * - HRM:  store value handle and CCCD handle for later notification setup.
+	 * - BSL:  store value handle and issue a Read Request; the value will
+	 *         arrive asynchronously via BLE_GATTC_EVT_READ_RSP.
+	 */
 	for (uint32_t i = 0; i < db_discovery_evt->discovered_db->char_count; i++) {
 		db_char = &db_discovery_evt->discovered_db->characteristics[i];
 
@@ -152,7 +158,18 @@ void ble_hrs_on_db_disc_evt(struct ble_hrs_client *hrs_client,
 				db_char->cccd_handle;
 			hrs_client_evt.discovery_complete.handles.hrm_handle =
 				db_char->characteristic.handle_value;
-			break;
+		} else if (db_char->characteristic.uuid.uuid ==
+			BLE_UUID_BODY_SENSOR_LOCATION_CHAR) {
+			hrs_client_evt.discovery_complete.handles.bsl_handle =
+				db_char->characteristic.handle_value;
+			nrf_err = sd_ble_gattc_read(db_discovery_evt->conn_handle,
+				db_char->characteristic.handle_value, 0);
+				if (nrf_err) {
+					hrs_client_evt.evt_type = BLE_HRS_CLIENT_EVT_ERROR;
+					hrs_client_evt.error.reason = nrf_err;
+					hrs_client->evt_handler(hrs_client, &hrs_client_evt);
+					return;
+				}
 		}
 	}
 
@@ -194,18 +211,52 @@ uint32_t ble_hrs_client_init(struct ble_hrs_client *hrs_client,
 	return ble_db_discovery_service_register(hrs_client_config->db_discovery, &hrs_uuid);
 }
 
+static void on_read_rsp(struct ble_hrs_client *hrs_client, const ble_evt_t *ble_evt)
+{
+	const ble_gattc_evt_read_rsp_t *read_rsp = &ble_evt->evt.gattc_evt.params.read_rsp;
+	struct ble_hrs_client_evt evt = {
+		.conn_handle = ble_evt->evt.gattc_evt.conn_handle,
+		.evt_type = BLE_HRS_CLIENT_EVT_BSL_UPDATE,
+		.body_sensor_location = ble_evt->evt.gattc_evt.params.read_rsp.data[0],
+	};
+
+	if (hrs_client->conn_handle != ble_evt->evt.gattc_evt.conn_handle) {
+		return;
+	}
+
+	/* Check if this is a body sensor location response. */
+	if (read_rsp->handle != hrs_client->handles.bsl_handle) {
+		return;
+	}
+
+	if (read_rsp->len < 1) {
+		return;
+	}
+
+	hrs_client->evt_handler(hrs_client, &evt);
+}
+
 void ble_hrs_client_on_ble_evt(const ble_evt_t *ble_evt, void *hrs_client)
 {
 	__ASSERT(ble_evt, "BLE event is NULL");
 	__ASSERT(hrs_client, "HRS central instance is NULL");
 
+	if (ble_evt->header.evt_id >= BLE_GAP_EVT_BASE &&
+	    ble_evt->header.evt_id <= BLE_GAP_EVT_LAST &&
+	    ble_evt->evt.gap_evt.params.connected.role == BLE_GAP_ROLE_PERIPH) {
+		return;
+	}
+
 	switch (ble_evt->header.evt_id) {
 	case BLE_GATTC_EVT_HVX:
 		on_hvx(hrs_client, ble_evt);
 		break;
 	case BLE_GAP_EVT_DISCONNECTED:
 		on_disconnected(hrs_client, ble_evt);
 		break;
+	case BLE_GATTC_EVT_READ_RSP:
+		on_read_rsp(hrs_client, ble_evt);
+		break;
 	default:
 		break;
 	}

tests/unit/subsys/bluetooth/services/ble_hrs_client/CMakeLists.txt

@@ -10,7 +10,7 @@ find_package(Zephyr REQUIRED HINTS $ENV{ZEPHYR_BASE})
 
 project(unit_test_ble_hrs_client)
 
-set(SOFTDEVICE_VARIANT "s115")
+set(SOFTDEVICE_VARIANT "s145")
 set(SOFTDEVICE_INCLUDE_DIR "${ZEPHYR_NRF_BM_MODULE_DIR}/components/softdevice/nrf54l/\
 ${SOFTDEVICE_VARIANT}/${SOFTDEVICE_VARIANT}_API/include")
 

tests/unit/subsys/bluetooth/services/ble_hrs_client/src/unity_test.c

@@ -520,6 +520,7 @@ void test_ble_hrs_on_db_disc_evt_hrm_char_at_index_one(void)
 
 	__cmock_ble_db_discovery_service_register_ExpectAndReturn(&db_discovery, NULL, NRF_SUCCESS);
 	__cmock_ble_db_discovery_service_register_IgnoreArg_uuid();
+	__cmock_sd_ble_gattc_read_ExpectAnyArgsAndReturn(NRF_SUCCESS);
 
 	nrf_err = ble_hrs_client_init(&hrs_client, &config);
 	TEST_ASSERT_EQUAL(NRF_SUCCESS, nrf_err);