From 9c2183806edb95e2936526aec8aa2c8c2a2fdb12 Mon Sep 17 00:00:00 2001 From: Emil Gydesen Date: Thu, 4 Dec 2025 15:37:58 +0100 Subject: [PATCH] tests: Bluetooth: Tester: Fix bad BIS sync shift The shift is an old leftover from when the BAP API was different and BIT(1) means BIS Index 1, but now BIT(0) means BIT Index 1. Signed-off-by: Emil Gydesen --- tests/bluetooth/tester/src/audio/btp_bap_broadcast.c | 12 ------------ 1 file changed, 12 deletions(-) diff --git a/tests/bluetooth/tester/src/audio/btp_bap_broadcast.c b/tests/bluetooth/tester/src/audio/btp_bap_broadcast.c index d301fac6c31f4..ec5125522bd06 100644 --- a/tests/bluetooth/tester/src/audio/btp_bap_broadcast.c +++ b/tests/bluetooth/tester/src/audio/btp_bap_broadcast.c @@ -1680,12 +1680,6 @@ uint8_t btp_bap_broadcast_assistant_add_src(const void *cmd, uint16_t cmd_len, v struct bt_bap_bass_subgroup *subgroup = &delegator_subgroups[i]; subgroup->bis_sync = sys_get_le32(ptr); - if (subgroup->bis_sync != BT_BAP_BIS_SYNC_NO_PREF) { - /* For semantic purposes Zephyr API uses BIS Index bitfield - * where BIT(1) means BIS Index 1 - */ - subgroup->bis_sync <<= 1; - } ptr += sizeof(subgroup->bis_sync); subgroup->metadata_len = *ptr; @@ -1751,12 +1745,6 @@ uint8_t btp_bap_broadcast_assistant_modify_src(const void *cmd, uint16_t cmd_len struct bt_bap_bass_subgroup *subgroup = &delegator_subgroups[i]; subgroup->bis_sync = sys_get_le32(ptr); - if (subgroup->bis_sync != BT_BAP_BIS_SYNC_NO_PREF) { - /* For semantic purposes Zephyr API uses BIS Index bitfield - * where BIT(1) means BIS Index 1 - */ - subgroup->bis_sync <<= 1; - } ptr += sizeof(subgroup->bis_sync); subgroup->metadata_len = *ptr;