samples: Bluetooth: BAP Unicast Client improve reporting

Make the reporting interval configurable to make it easier
to actually follow. Improve what gets reported.

Add check for SDU size to ensure correctness when not
using LIBLC3.

Signed-off-by: Emil Gydesen <[email protected]>

Author: Thalley

samples/bluetooth/bap_unicast_client/Kconfig

@@ -0,0 +1,13 @@
+# Copyright (c) 2025 Nordic Semiconductor ASA
+# SPDX-License-Identifier: Apache-2.0
+
+mainmenu "Bluetooth: BAP Unicast Client"
+
+config INFO_REPORTING_INTERVAL
+	int "Number of SDUs received between each information report"
+	default 1000
+	help
+	  Determines how often information about received data is logged.
+	  Set to 0 to disable reporting.
+
+source "Kconfig.zephyr"

samples/bluetooth/bap_unicast_client/src/main.c

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2021-2024 Nordic Semiconductor ASA
+ * Copyright (c) 2021-2025 Nordic Semiconductor ASA
  *
  * SPDX-License-Identifier: Apache-2.0
  */
@@ -270,6 +270,8 @@ static void stream_connected_cb(struct bt_bap_stream *stream)
 static void stream_started(struct bt_bap_stream *stream)
 {
 	printk("Audio Stream %p started\n", stream);
+	unicast_audio_recv_ctr = 0U;
+
 	/* Register the stream for TX if it can send */
 	if (IS_ENABLED(CONFIG_BT_AUDIO_TX) && stream_tx_can_send(stream)) {
 		const int err = stream_tx_register(stream);
@@ -317,8 +319,12 @@ static void stream_recv(struct bt_bap_stream *stream,
 {
 	if (info->flags & BT_ISO_FLAGS_VALID) {
 		unicast_audio_recv_ctr++;
-		printk("Incoming audio on stream %p len %u (%"PRIu64")\n", stream, buf->len,
-			unicast_audio_recv_ctr);
+
+		if (CONFIG_INFO_REPORTING_INTERVAL > 0 &&
+		    (unicast_audio_recv_ctr % CONFIG_INFO_REPORTING_INTERVAL) == 0U) {
+			printk("Incoming audio on stream %p len %u (%" PRIu64 ")\n", stream,
+			       buf->len, unicast_audio_recv_ctr);
+		}
 	}
 }
 

samples/bluetooth/bap_unicast_client/src/stream_lc3.c

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2024 Nordic Semiconductor ASA
+ * Copyright (c) 2024-2025 Nordic Semiconductor ASA
  *
  * SPDX-License-Identifier: Apache-2.0
  */
@@ -158,6 +158,7 @@ static bool encode_frame_block(struct tx_stream *stream, struct net_buf *out_buf
 		 * purposes
 		 */
 		if (!encode_frame(stream, i, out_buf)) {
+			LOG_WRN("Failed to encode frame %u", i);
 			return false;
 		}
 	}
@@ -169,6 +170,7 @@ void stream_lc3_add_data(struct tx_stream *stream, struct net_buf *buf)
 {
 	for (uint8_t i = 0U; i < stream->lc3_tx.frame_blocks_per_sdu; i++) {
 		if (!encode_frame_block(stream, buf)) {
+			LOG_WRN("Failed to encode frame block %u", i);
 			break;
 		}
 	}

samples/bluetooth/bap_unicast_client/src/stream_tx.c

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2024 Nordic Semiconductor ASA
+ * Copyright (c) 2024-2025 Nordic Semiconductor ASA
  *
  * SPDX-License-Identifier: Apache-2.0
  */
@@ -20,6 +20,7 @@
 #include <zephyr/logging/log.h>
 #include <zephyr/logging/log_core.h>
 #include <zephyr/net_buf.h>
+#include <zephyr/sys/__assert.h>
 #include <zephyr/sys/byteorder.h>
 #include <zephyr/sys/util.h>
 #include <zephyr/sys/util_macro.h>
@@ -79,6 +80,7 @@ static void tx_thread_func(void *arg1, void *arg2, void *arg3)
 			struct bt_bap_stream *bap_stream = tx_streams[i].bap_stream;
 
 			if (stream_is_streaming(bap_stream)) {
+				uint16_t sdu_len;
 				struct net_buf *buf;
 
 				buf = net_buf_alloc(&tx_pool, K_FOREVER);
@@ -88,12 +90,25 @@ static void tx_thread_func(void *arg1, void *arg2, void *arg3)
 				    bap_stream->codec_cfg->id == BT_HCI_CODING_FORMAT_LC3) {
 					stream_lc3_add_data(&tx_streams[i], buf);
 				} else {
+					__ASSERT(bap_stream->qos->sdu <= ARRAY_SIZE(mock_data),
+						 "Configured codec SDU len %u does not match mock "
+						 "data size %zu",
+						 bap_stream->qos->sdu, ARRAY_SIZE(mock_data));
 					net_buf_add_mem(buf, mock_data, bap_stream->qos->sdu);
 				}
 
+				sdu_len = buf->len;
+
 				err = bt_bap_stream_send(bap_stream, buf, tx_streams[i].seq_num);
 				if (err == 0) {
 					tx_streams[i].seq_num++;
+
+					if (CONFIG_INFO_REPORTING_INTERVAL > 0 &&
+					    (tx_streams[i].seq_num %
+					     CONFIG_INFO_REPORTING_INTERVAL) == 0U) {
+						LOG_INF("Stream %p: Sent %u total SDUs of size %u",
+							bap_stream, tx_streams[i].seq_num, sdu_len);
+					}
 				} else {
 					LOG_ERR("Unable to send: %d", err);
 					net_buf_unref(buf);
@@ -131,6 +146,11 @@ int stream_tx_register(struct bt_bap_stream *bap_stream)
 			}
 
 			LOG_INF("Registered %p for TX", bap_stream);
+			if (bap_stream->qos->sdu > CONFIG_BT_ISO_TX_MTU) {
+				LOG_WRN("Stream configured for SDUs larger (%u) than "
+					"CONFIG_BT_ISO_TX_MTU (%d)",
+					bap_stream->qos->sdu, CONFIG_BT_ISO_TX_MTU);
+			}
 
 			return 0;
 		}

tests/bsim/bluetooth/audio_samples/bap_unicast_client/Kconfig

@@ -0,0 +1,4 @@
+# Copyright (c) 2024 Nordic Semiconductor ASA
+# SPDX-License-Identifier: Apache-2.0
+
+source "${ZEPHYR_BASE}/samples/bluetooth/bap_unicast_client/Kconfig"