Skip to content

tests: Bluetooth: ISO: Add validation of broadcast info #80999

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 30, 2025
Merged

tests: Bluetooth: ISO: Add validation of broadcast info #80999

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
7 changes: 7 additions & 0 deletions include/zephyr/bluetooth/hci_types.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3403,6 +3403,13 @@ struct bt_hci_evt_le_cis_req {
uint8_t cis_id;
} __packed;

#define BT_HCI_LE_BIG_HANDLE_MIN 0x00U
#define BT_HCI_LE_BIG_HANDLE_MAX 0xEFU
#define BT_HCI_LE_BIG_SYNC_DELAY_MIN 0x000030U
#define BT_HCI_LE_BIG_SYNC_DELAY_MAX 0x7FFFFFU
#define BT_HCI_LE_TRANSPORT_LATENCY_BIG_MIN 0x000030U
#define BT_HCI_LE_TRANSPORT_LATENCY_BIG_MAX 0x7FFFFFU

#define BT_HCI_EVT_LE_BIG_COMPLETE 0x1b
struct bt_hci_evt_le_big_complete {
uint8_t status;
Expand Down
36 changes: 36 additions & 0 deletions tests/bsim/bluetooth/host/iso/bis/src/bis_broadcaster.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -104,10 +104,46 @@ static void iso_connected_cb(struct bt_iso_chan *chan)
.pid = BT_ISO_DATA_PATH_HCI,
.format = BT_HCI_CODING_FORMAT_TRANSPARENT,
};
struct bt_iso_info info;
int err;

LOG_INF("ISO Channel %p connected", chan);

err = bt_iso_chan_get_info(chan, &info);
TEST_ASSERT(err == 0, "Failed to get BIS info: %d", err);

TEST_ASSERT(!info.can_recv);
TEST_ASSERT(info.can_send);
TEST_ASSERT(info.type == BT_ISO_CHAN_TYPE_BROADCASTER);
TEST_ASSERT((BT_GAP_ISO_INTERVAL_TO_US(info.iso_interval) % SDU_INTERVAL_US) == 0U,
"ISO interval %u (%u) shall be a multiple of the SDU interval %u",
BT_GAP_ISO_INTERVAL_TO_US(info.iso_interval), info.iso_interval,
SDU_INTERVAL_US);
TEST_ASSERT(IN_RANGE(info.iso_interval, BT_ISO_ISO_INTERVAL_MIN, BT_ISO_ISO_INTERVAL_MAX),
"Invalid ISO interval 0x%04x", info.iso_interval);
TEST_ASSERT(IN_RANGE(info.max_subevent, BT_ISO_NSE_MIN, BT_ISO_NSE_MAX),
"Invalid subevent number 0x%02x", info.max_subevent);
TEST_ASSERT(IN_RANGE(info.broadcaster.sync_delay, BT_HCI_LE_BIG_SYNC_DELAY_MIN,
BT_HCI_LE_BIG_SYNC_DELAY_MAX),
"Invalid sync delay 0x%06x", info.broadcaster.sync_delay);
TEST_ASSERT(IN_RANGE(info.broadcaster.latency, BT_HCI_LE_TRANSPORT_LATENCY_BIG_MIN,
BT_HCI_LE_TRANSPORT_LATENCY_BIG_MAX),
"Invalid transport latency 0x%06x", info.broadcaster.latency);
TEST_ASSERT((info.broadcaster.pto % info.iso_interval) == 0U,
"PTO in ms %u shall be a multiple of the ISO interval %u", info.broadcaster.pto,
info.iso_interval);
TEST_ASSERT(IN_RANGE((info.broadcaster.pto / info.iso_interval), BT_ISO_PTO_MIN,
BT_ISO_PTO_MAX),
"Invalid PTO 0x%x", (info.broadcaster.pto / info.iso_interval));
TEST_ASSERT(info.broadcaster.phy == BT_GAP_LE_PHY_1M ||
info.broadcaster.phy == BT_GAP_LE_PHY_2M ||
info.broadcaster.phy == BT_GAP_LE_PHY_CODED,
"Invalid PHY 0x%02x", info.broadcaster.phy);
TEST_ASSERT(IN_RANGE(info.broadcaster.bn, BT_ISO_BN_MIN, BT_ISO_BN_MAX),
"Invalid BN 0x%02x", info.broadcaster.bn);
TEST_ASSERT(IN_RANGE(info.broadcaster.irc, BT_ISO_IRC_MIN, BT_ISO_IRC_MAX),
"Invalid IRC 0x%02x", info.broadcaster.irc);

if (chan == default_chan) {
seq_num = 0U;

Expand Down
25 changes: 25 additions & 0 deletions tests/bsim/bluetooth/host/iso/bis/src/bis_receiver.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -87,10 +87,35 @@ static void iso_connected(struct bt_iso_chan *chan)
.pid = BT_ISO_DATA_PATH_HCI,
.format = BT_HCI_CODING_FORMAT_TRANSPARENT,
};
struct bt_iso_info info;
int err;

LOG_INF("ISO Channel %p connected", chan);

err = bt_iso_chan_get_info(chan, &info);
TEST_ASSERT(err == 0, "Failed to get BIS info: %d", err);

TEST_ASSERT(info.can_recv);
TEST_ASSERT(!info.can_send);
TEST_ASSERT(info.type == BT_ISO_CHAN_TYPE_SYNC_RECEIVER);
TEST_ASSERT(IN_RANGE(info.iso_interval, BT_ISO_ISO_INTERVAL_MIN, BT_ISO_ISO_INTERVAL_MAX),
"Invalid ISO interval 0x%04x", info.iso_interval);
TEST_ASSERT(IN_RANGE(info.max_subevent, BT_ISO_NSE_MIN, BT_ISO_NSE_MAX),
"Invalid subevent number 0x%02x", info.max_subevent);
TEST_ASSERT(IN_RANGE(info.sync_receiver.latency, BT_HCI_LE_TRANSPORT_LATENCY_BIG_MIN,
BT_HCI_LE_TRANSPORT_LATENCY_BIG_MAX),
"Invalid transport latency 0x%06x", info.sync_receiver.latency);
TEST_ASSERT((info.sync_receiver.pto % info.iso_interval) == 0U,
"PTO in ms %u shall be a multiple of the ISO interval %u",
info.sync_receiver.pto, info.iso_interval);
TEST_ASSERT(IN_RANGE((info.sync_receiver.pto / info.iso_interval), BT_ISO_PTO_MIN,
BT_ISO_PTO_MAX),
"Invalid PTO 0x%x", (info.sync_receiver.pto / info.iso_interval));
TEST_ASSERT(IN_RANGE(info.sync_receiver.bn, BT_ISO_BN_MIN, BT_ISO_BN_MAX),
"Invalid BN 0x%02x", info.sync_receiver.bn);
TEST_ASSERT(IN_RANGE(info.sync_receiver.irc, BT_ISO_IRC_MIN, BT_ISO_IRC_MAX),
"Invalid IRC 0x%02x", info.sync_receiver.irc);

SET_FLAG(flag_iso_connected);

err = bt_iso_setup_data_path(chan, BT_HCI_DATAPATH_DIR_CTLR_TO_HOST, &hci_path);
Expand Down