Skip to content

Commit 9c21838

Browse files
committed
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 <[email protected]>
1 parent 0e0ff29 commit 9c21838

File tree

1 file changed

+0
-12
lines changed

1 file changed

+0
-12
lines changed

tests/bluetooth/tester/src/audio/btp_bap_broadcast.c

Lines changed: 0 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1680,12 +1680,6 @@ uint8_t btp_bap_broadcast_assistant_add_src(const void *cmd, uint16_t cmd_len, v
16801680
struct bt_bap_bass_subgroup *subgroup = &delegator_subgroups[i];
16811681

16821682
subgroup->bis_sync = sys_get_le32(ptr);
1683-
if (subgroup->bis_sync != BT_BAP_BIS_SYNC_NO_PREF) {
1684-
/* For semantic purposes Zephyr API uses BIS Index bitfield
1685-
* where BIT(1) means BIS Index 1
1686-
*/
1687-
subgroup->bis_sync <<= 1;
1688-
}
16891683

16901684
ptr += sizeof(subgroup->bis_sync);
16911685
subgroup->metadata_len = *ptr;
@@ -1751,12 +1745,6 @@ uint8_t btp_bap_broadcast_assistant_modify_src(const void *cmd, uint16_t cmd_len
17511745
struct bt_bap_bass_subgroup *subgroup = &delegator_subgroups[i];
17521746

17531747
subgroup->bis_sync = sys_get_le32(ptr);
1754-
if (subgroup->bis_sync != BT_BAP_BIS_SYNC_NO_PREF) {
1755-
/* For semantic purposes Zephyr API uses BIS Index bitfield
1756-
* where BIT(1) means BIS Index 1
1757-
*/
1758-
subgroup->bis_sync <<= 1;
1759-
}
17601748

17611749
ptr += sizeof(subgroup->bis_sync);
17621750
subgroup->metadata_len = *ptr;

0 commit comments

Comments
 (0)