Skip to content

tests: bsim: Bluetooth: Enable some high reliability CAP tests #158703

tests: bsim: Bluetooth: Enable some high reliability CAP tests

tests: bsim: Bluetooth: Enable some high reliability CAP tests #158703

Triggered via pull request November 3, 2024 21:57
Status Success
Total duration 2m 36s
Artifacts 1

compliance.yml

on: pull_request
Run compliance checks on patch series (PR)
2m 28s
Run compliance checks on patch series (PR)
Fit to window
Zoom out
Zoom in

Annotations

1 error, 1 warning, and 9 notices
Run compliance checks on patch series (PR)
Process completed with exit code 1.
Run compliance checks on patch series (PR): ClangFormat.txt#L1
See https://docs.zephyrproject.org/latest/contribute/guidelines.html#clang-format for more details. You may want to run clang-format on this change: -static void isr_rx_iso_data_invalid(const struct lll_sync_iso *const lll, - uint16_t latency, uint8_t bn, - uint16_t handle, - struct node_rx_pdu *node_rx); +static void isr_rx_iso_data_invalid(const struct lll_sync_iso *const lll, uint16_t latency, + uint8_t bn, uint16_t handle, struct node_rx_pdu *node_rx); File:subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c Line:62 You may want to run clang-format on this change: -static void isr_rx_iso_data_invalid(const struct lll_sync_iso *const lll, - uint16_t latency, uint8_t bn, - uint16_t handle, - struct node_rx_pdu *node_rx) +static void isr_rx_iso_data_invalid(const struct lll_sync_iso *const lll, uint16_t latency, + uint8_t bn, uint16_t handle, struct node_rx_pdu *node_rx) File:subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c Line:1389 You may want to run clang-format on this change: - iso_meta->timestamp = HAL_TICKER_TICKS_TO_US(radio_tmr_start_get()) + - radio_tmr_aa_restore() - addr_us_get(lll->phy) - - ((stream->bis_index - 1U) * - lll->sub_interval * ((lll->irc * lll->bn) + - lll->ptc)) - - (latency * lll->iso_interval * - PERIODIC_INT_UNIT_US); + iso_meta->timestamp = + HAL_TICKER_TICKS_TO_US(radio_tmr_start_get()) + radio_tmr_aa_restore() - + addr_us_get(lll->phy) - + ((stream->bis_index - 1U) * lll->sub_interval * ((lll->irc * lll->bn) + lll->ptc)) - + (latency * lll->iso_interval * PERIODIC_INT_UNIT_US); File:subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c Line:1407 You may want to run clang-format on this change: - HAL_TICKER_US_TO_TICKS((uint64_t)interval * - ADV_INT_UNIT_US); + HAL_TICKER_US_TO_TICKS((uint64_t)interval * ADV_INT_UNIT_US); File:subsys/bluetooth/controller/ll_sw/ull_adv.c Line:1538 You may want to run clang-format on this change: -#if defined(CONFIG_BT_CTLR_ADV_AUX_SLOT_WINDOW) || \ - (defined(CONFIG_BT_CTLR_ADV_PERIODIC) && \ - defined(CONFIG_BT_TICKER_EXT_EXPIRE_INFO)) -#if defined(CONFIG_BT_CTLR_ADV_PERIODIC) && \ - defined(CONFIG_BT_TICKER_EXT_EXPIRE_INFO) +#if defined(CONFIG_BT_CTLR_ADV_AUX_SLOT_WINDOW) || \ + (defined(CONFIG_BT_CTLR_ADV_PERIODIC) && defined(CONFIG_BT_TICKER_EXT_EXPIRE_INFO)) +#if defined(CONFIG_BT_CTLR_ADV_PERIODIC) && defined(CONFIG_BT_TICKER_EXT_EXPIRE_INFO) static void ticker_update_op_cb(uint32_t status, void *param); #endif /* CONFIG_BT_CTLR_ADV_PERIODIC && CONFIG_BT_TICKER_EXT_EXPIRE_INFO */ static struct ticker_ext ll_adv_aux_ticker_ext[CONFIG_BT_CTLR_ADV_AUX_SET]; -#endif /* CONFIG_BT_CTLR_ADV_AUX_SLOT_WINDOW || - * (CONFIG_BT_CTLR_ADV_PERIODIC && CONFIG_BT_TICKER_EXT_EXPIRE_INFO) +#endif /* CONFIG_BT_CTLR_ADV_AUX_SLOT_WINDOW || \ + * (CONFIG_BT_CTLR_ADV_PERIODIC && CONFIG_BT_TICKER_EXT_EXPIRE_INFO) \ File:subsys/bluetooth/controller/ll_sw/ull_adv_aux.c Line:85 You may want to run clang-format on this change: - ll_adv_aux_ticker_ext[aux_handle].ticks_slot_window += - HAL_TICKER_US_TO_TICKS(interval_us); + ll_adv_aux_ticker_ext[aux_handle].ticks_slot_window += HAL_TICKER_US_TO_TICKS(interval_us); File:subsys/bluetooth/controller/ll_sw/ull_adv_aux.c Line:2617 You may want to run clang-format on this change: -#if defined(CONFIG_BT_CTLR_ADV_AUX_SLOT_WINDOW) || \ - (defined(CONFIG_BT_CTLR_ADV_PERIODIC) && \ - defined(CONFIG_BT_TICKER_EXT_EXPIRE_INFO)) +#if defined(CONFIG_BT_CTLR_ADV_AUX_SLOT_WINDOW) || \ + (defined(CONFIG_BT_CTLR_ADV_PERIODIC) && defined(CONFIG_BT_TICKER_EXT_EXPIRE_INFO)) ret_cb = TICKER_STATUS_BUSY; ret = ticker_start_ext( -#else /* !CONFIG_BT_CTLR_ADV_AUX_SLOT_WINDOW && - * !(CONFIG_BT_CTLR_ADV_PERIODIC && CONFIG_BT_TICKER_EXT_EXPIRE_INFO) +#else /* !CONFIG_BT_CTLR_ADV_AUX_SLOT_WINDOW &&
You may want to run clang-format on this change: subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c#L62
subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c:62 -static void isr_rx_iso_data_invalid(const struct lll_sync_iso *const lll, - uint16_t latency, uint8_t bn, - uint16_t handle, - struct node_rx_pdu *node_rx); +static void isr_rx_iso_data_invalid(const struct lll_sync_iso *const lll, uint16_t latency, + uint8_t bn, uint16_t handle, struct node_rx_pdu *node_rx);
You may want to run clang-format on this change: subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c#L1389
subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c:1389 -static void isr_rx_iso_data_invalid(const struct lll_sync_iso *const lll, - uint16_t latency, uint8_t bn, - uint16_t handle, - struct node_rx_pdu *node_rx) +static void isr_rx_iso_data_invalid(const struct lll_sync_iso *const lll, uint16_t latency, + uint8_t bn, uint16_t handle, struct node_rx_pdu *node_rx)
You may want to run clang-format on this change: subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c#L1407
subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c:1407 - iso_meta->timestamp = HAL_TICKER_TICKS_TO_US(radio_tmr_start_get()) + - radio_tmr_aa_restore() - addr_us_get(lll->phy) - - ((stream->bis_index - 1U) * - lll->sub_interval * ((lll->irc * lll->bn) + - lll->ptc)) - - (latency * lll->iso_interval * - PERIODIC_INT_UNIT_US); + iso_meta->timestamp = + HAL_TICKER_TICKS_TO_US(radio_tmr_start_get()) + radio_tmr_aa_restore() - + addr_us_get(lll->phy) - + ((stream->bis_index - 1U) * lll->sub_interval * ((lll->irc * lll->bn) + lll->ptc)) - + (latency * lll->iso_interval * PERIODIC_INT_UNIT_US);
You may want to run clang-format on this change: subsys/bluetooth/controller/ll_sw/ull_adv.c#L1538
subsys/bluetooth/controller/ll_sw/ull_adv.c:1538 - HAL_TICKER_US_TO_TICKS((uint64_t)interval * - ADV_INT_UNIT_US); + HAL_TICKER_US_TO_TICKS((uint64_t)interval * ADV_INT_UNIT_US);
You may want to run clang-format on this change: subsys/bluetooth/controller/ll_sw/ull_adv_aux.c#L85
subsys/bluetooth/controller/ll_sw/ull_adv_aux.c:85 -#if defined(CONFIG_BT_CTLR_ADV_AUX_SLOT_WINDOW) || \ - (defined(CONFIG_BT_CTLR_ADV_PERIODIC) && \ - defined(CONFIG_BT_TICKER_EXT_EXPIRE_INFO)) -#if defined(CONFIG_BT_CTLR_ADV_PERIODIC) && \ - defined(CONFIG_BT_TICKER_EXT_EXPIRE_INFO) +#if defined(CONFIG_BT_CTLR_ADV_AUX_SLOT_WINDOW) || \ + (defined(CONFIG_BT_CTLR_ADV_PERIODIC) && defined(CONFIG_BT_TICKER_EXT_EXPIRE_INFO)) +#if defined(CONFIG_BT_CTLR_ADV_PERIODIC) && defined(CONFIG_BT_TICKER_EXT_EXPIRE_INFO) static void ticker_update_op_cb(uint32_t status, void *param); #endif /* CONFIG_BT_CTLR_ADV_PERIODIC && CONFIG_BT_TICKER_EXT_EXPIRE_INFO */ static struct ticker_ext ll_adv_aux_ticker_ext[CONFIG_BT_CTLR_ADV_AUX_SET]; -#endif /* CONFIG_BT_CTLR_ADV_AUX_SLOT_WINDOW || - * (CONFIG_BT_CTLR_ADV_PERIODIC && CONFIG_BT_TICKER_EXT_EXPIRE_INFO) +#endif /* CONFIG_BT_CTLR_ADV_AUX_SLOT_WINDOW || \ + * (CONFIG_BT_CTLR_ADV_PERIODIC && CONFIG_BT_TICKER_EXT_EXPIRE_INFO) \
You may want to run clang-format on this change: subsys/bluetooth/controller/ll_sw/ull_adv_aux.c#L2617
subsys/bluetooth/controller/ll_sw/ull_adv_aux.c:2617 - ll_adv_aux_ticker_ext[aux_handle].ticks_slot_window += - HAL_TICKER_US_TO_TICKS(interval_us); + ll_adv_aux_ticker_ext[aux_handle].ticks_slot_window += HAL_TICKER_US_TO_TICKS(interval_us);
You may want to run clang-format on this change: subsys/bluetooth/controller/ll_sw/ull_adv_aux.c#L2665
subsys/bluetooth/controller/ll_sw/ull_adv_aux.c:2665 -#if defined(CONFIG_BT_CTLR_ADV_AUX_SLOT_WINDOW) || \ - (defined(CONFIG_BT_CTLR_ADV_PERIODIC) && \ - defined(CONFIG_BT_TICKER_EXT_EXPIRE_INFO)) +#if defined(CONFIG_BT_CTLR_ADV_AUX_SLOT_WINDOW) || \ + (defined(CONFIG_BT_CTLR_ADV_PERIODIC) && defined(CONFIG_BT_TICKER_EXT_EXPIRE_INFO)) ret_cb = TICKER_STATUS_BUSY; ret = ticker_start_ext( -#else /* !CONFIG_BT_CTLR_ADV_AUX_SLOT_WINDOW && - * !(CONFIG_BT_CTLR_ADV_PERIODIC && CONFIG_BT_TICKER_EXT_EXPIRE_INFO) +#else /* !CONFIG_BT_CTLR_ADV_AUX_SLOT_WINDOW && \ + * !(CONFIG_BT_CTLR_ADV_PERIODIC && CONFIG_BT_TICKER_EXT_EXPIRE_INFO) \ */ ret_cb = TICKER_STATUS_BUSY; ret = ticker_start( -#endif /* !CONFIG_BT_CTLR_ADV_AUX_SLOT_WINDOW && - * !(CONFIG_BT_CTLR_ADV_PERIODIC && CONFIG_BT_TICKER_EXT_EXPIRE_INFO) +#endif /* !CONFIG_BT_CTLR_ADV_AUX_SLOT_WINDOW && \ + * !(CONFIG_BT_CTLR_ADV_PERIODIC && CONFIG_BT_TICKER_EXT_EXPIRE_INFO) \ */ - TICKER_INSTANCE_ID_CTLR, TICKER_USER_ID_THREAD, - (TICKER_ID_ADV_AUX_BASE + aux_handle), - ticks_anchor, 0U, - HAL_TICKER_US_TO_TICKS(interval_us), - HAL_TICKER_REMAINDER(interval_us), TICKER_NULL_LAZY, - (aux->ull.ticks_slot + ticks_slot_overhead), - ticker_cb, aux, - ull_ticker_status_give, (void *)&ret_cb -#if defined(CONFIG_BT_CTLR_ADV_AUX_SLOT_WINDOW) || \ - (defined(CONFIG_BT_CTLR_ADV_PERIODIC) && \ - defined(CONFIG_BT_TICKER_EXT_EXPIRE_INFO)) - , - &ll_adv_aux_ticker_ext[aux_handle] -#endif /* CONFIG_BT_CTLR_ADV_AUX_SLOT_WINDOW || - * (CONFIG_BT_CTLR_ADV_PERIODIC && CONFIG_BT_TICKER_EXT_EXPIRE_INFO) + TICKER_INSTANCE_ID_CTLR, TICKER_USER_ID_THREAD, + (TICKER_ID_ADV_AUX_BASE + aux_handle), ticks_anchor, 0U, + HAL_TICKER_US_TO_TICKS(interval_us), HAL_TICKER_REMAINDER(interval_us), + TICKER_NULL_LAZY, (aux->ull.ticks_slot + ticks_slot_overhead), ticker_cb, aux, + ull_ticker_status_give, (void *)&ret_cb +#if defined(CONFIG_BT_CTLR_ADV_AUX_SLOT_WINDOW) || \ + (defined(CONFIG_BT_CTLR_ADV_PERIODIC) && defined(CONFIG_BT_TICKER_EXT_EXPIRE_INFO)) + , + &ll_adv_aux_ticker_ext[aux_handle] +#endif /* CONFIG_BT_CTLR_ADV_AUX_SLOT_WINDOW || \ + * (CONFIG_BT_CTLR_ADV_PERIODIC && CONFIG_BT_TICKER_EXT_EXPIRE_INFO) \ */ - ); + );
You may want to run clang-format on this change: subsys/bluetooth/controller/ll_sw/ull_sync_iso.c#L614
subsys/bluetooth/controller/ll_sw/ull_sync_iso.c:614 - ctrl_spacing = PDU_BIS_US(sizeof(struct pdu_big_ctrl), lll->enc, - lll->phy, PHY_FLAGS_S8); + ctrl_spacing = + PDU_BIS_US(sizeof(struct pdu_big_ctrl), lll->enc, lll->phy, PHY_FLAGS_S8);
You may want to run clang-format on this change: tests/bsim/bluetooth/audio/src/cap_initiator_broadcast_test.c#L49
tests/bsim/bluetooth/audio/src/cap_initiator_broadcast_test.c:49 -#define BT_LE_EXT_ADV_CUSTOM \ - BT_LE_ADV_PARAM(BT_LE_ADV_OPT_EXT_ADV, \ - 0x00e0, 0x00e0, NULL) - -#define BT_LE_PER_ADV_CUSTOM \ - BT_LE_PER_ADV_PARAM(0x0078, \ - 0x0078, \ - BT_LE_PER_ADV_OPT_NONE) +#define BT_LE_EXT_ADV_CUSTOM BT_LE_ADV_PARAM(BT_LE_ADV_OPT_EXT_ADV, 0x00e0, 0x00e0, NULL) + +#define BT_LE_PER_ADV_CUSTOM BT_LE_PER_ADV_PARAM(0x0078, 0x0078, BT_LE_PER_ADV_OPT_NONE)

Artifacts

Produced during runtime
Name Size
compliance.xml
1.95 KB