michaelh/zephyr
1
0
Fork 0
Code Releases Activity

Bluetooth: BAP: Add PSN debug log support

Browse source 
Add a Kconfig that enables support for logging and
debugging invalid sequence numbers.

Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no>
This commit is contained in:
Emil Gydesen 2023-10-06 11:16:20 +02:00 • committed by Carles Cufí
commit 6054714a08
3 changed files with 23 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

4
include/zephyr/bluetooth/audio/bap.h
View file

@ -484,6 +484,10 @@ struct bt_bap_stream {
/** Stream user data */
void *user_data;
#if defined(CONFIG_BT_BAP_DEBUG_STREAM_SEQ_NUM)
uint16_t _prev_seq_num;
#endif /* CONFIG_BT_BAP_DEBUG_STREAM_SEQ_NUM */
/* Internally used list node */
sys_snode_t _node;
};

9
subsys/bluetooth/audio/Kconfig.bap
View file

@ -277,5 +277,14 @@ config BT_BAP_STREAM
default y if BT_ASCS || BT_BAP_UNICAST_CLIENT || \
BT_BAP_BROADCAST_SOURCE || BT_BAP_BROADCAST_SINK
config BT_BAP_DEBUG_STREAM_SEQ_NUM
bool "Bluetooth Audio Stream sequence number debug"
depends on BT_BAP_STREAM_LOG_LEVEL >= BT_BAP_STREAM_LOG_LEVEL_WRN
default y
help
Use this option to enable Bluetooth Audio Stream sequence number debugging logs for
the Bluetooth Audio functionality. This will provide a warning if the application
provides unexpected sequence numbers.
rsource "Kconfig.pacs"
rsource "Kconfig.ascs"

10
subsys/bluetooth/audio/bap_stream.c
View file

@ -277,6 +277,16 @@ int bt_bap_stream_send(struct bt_bap_stream *stream, struct net_buf *buf,
return -EBADMSG;
}
#if defined(CONFIG_BT_BAP_DEBUG_STREAM_SEQ_NUM)
if (stream->_prev_seq_num != 0U && seq_num != 0U &&
(stream->_prev_seq_num + 1U) != seq_num) {
LOG_WRN("Unexpected seq_num diff between %u and %u for %p", stream->_prev_seq_num,
seq_num, stream);
}
stream->_prev_seq_num = seq_num;
#endif /* CONFIG_BT_BAP_DEBUG_STREAM_SEQ_NUM */
/* TODO: Add checks for broadcast sink */
return bt_iso_chan_send(bt_bap_stream_iso_chan_get(stream),