michaelh/zephyr
1
0
Fork 0
Code Releases Activity

Bluetooth: BAP: Call bis_sync_req for all BIS sync state changes

Browse source 
Previously bis_sync_req was only called when requested_bis_sync
was changed. However that meant that there were cases where the
requested_bis_sync was the same as the new request, but where
the actual BIS sync state were different. This was mostly
the case when the scan delegator had autonousmly
added or modified the receive state.

Signed-off-by: Emil Gydesen <emil.gydesen@nordicsemi.no>
This commit is contained in:
Emil Gydesen 2025-04-01 13:16:50 +02:00 • committed by Benjamin Cabé
commit acd7080350
1 changed files with 8 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

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

@ -793,7 +793,12 @@ static int scan_delegator_mod_src(struct bt_conn *conn,
return BT_GATT_ERR(BT_ATT_ERR_VALUE_NOT_ALLOWED);
}
if (internal_state->requested_bis_sync[i] != requested_bis_sync[i]) {
/* If the BIS sync request is different than what was previously was requested, or
* different than what we are current synced to, we set bis_sync_change_requested to
* let the application know that the state may need a change
*/
if (internal_state->requested_bis_sync[i] != requested_bis_sync[i] ||
internal_state->state.subgroups[i].bis_sync != requested_bis_sync[i]) {
bis_sync_change_requested = true;
}
@ -1005,7 +1010,8 @@ static int scan_delegator_rem_src(struct bt_conn *conn,
bis_sync_was_requested = false;
for (uint8_t i = 0U; i < state->num_subgroups; i++) {
if (internal_state->requested_bis_sync[i] != 0U) {
if (internal_state->requested_bis_sync[i] != 0U &&
internal_state->state.subgroups[i].bis_sync != 0U) {
bis_sync_was_requested = true;
break;
}