tests: Bluetooth: ISO: BIS: Add fragmented data test #162711
Annotations
3 errors, 1 warning, and 8 notices
Run Compliance Tests
Process completed with exit code 2.
|
check-warns:
Gitlint.txt#L1
See https://docs.zephyrproject.org/latest/contribute/guidelines.html#commit-guidelines for more details
Commit d3b8065e41:
1: UC5 Commit title exceeds max length (76>75): "fixup! Bluetooth: Controller: Fix ISO Tx PDU buffer counts for fragmentation"
|
check-warns
Process completed with exit code 1.
|
check-warns:
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:63
You may want to run clang-format on this change:
- err = lll_prepare(is_abort_cb, abort_cb, create_prepare_cb, 0U,
- param);
+ err = lll_prepare(is_abort_cb, abort_cb, create_prepare_cb, 0U, param);
File:subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c
Line:153
You may want to run clang-format on this change:
- iso_meta->payload_number = lll->payload_count + (lll->bn_curr - 1U) +
- (lll->ptc_curr * lll->pto);
+ iso_meta->payload_number =
+ lll->payload_count + (lll->bn_curr - 1U) + (lll->ptc_curr * lll->pto);
File:subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c
Line:1382
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:1404
You may want to run clang-format on this change:
- iso_meta->timestamp -= (latency * lll->iso_interval *
- PERIODIC_INT_UNIT_US);
+ iso_meta->timestamp -= (latency * lll->iso_interval * PERIODIC_INT_UNIT_US);
File:subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c
Line:1424
You may want to run clang-format on this change:
-#define BT_CTLR_ISO_SDU_LEN_MAX ((CONFIG_BT_CTLR_ISO_TX_BUFFER_SIZE * \
- CONFIG_BT_CTLR_ISO_TX_BUFFERS) - \
- BT_HCI_ISO_SDU_HDR_SIZE)
+#define BT_CTLR_ISO_SDU_LEN_MAX \
+ ((CONFIG_BT_CTLR_ISO_TX_BUFFER_SIZE * CONFIG_BT_CTLR_ISO_TX_BUFFERS) - \
+ BT_HCI_ISO_SDU_HDR_SIZE)
#if defined(CONFIG_BT_CTLR_ADV_ISO) && defined(CONFIG_BT_CTLR_CONN_ISO)
-#define BT_CTLR_ISO_TX_BUFFERS (DIV_ROUND_UP(BT_CTLR_ISO_SDU_LEN_MAX, \
- MIN((CONFIG_BT_CTLR_ISO_TX_BUFFER_SIZE - \
- BT_HCI_ISO_SDU_HDR_SIZE), \
- MIN(CONFIG_BT_CTLR_ADV_ISO_PDU_LEN_MAX, \
- CONFIG_BT_CTLR_CONN_ISO_PDU_LEN_MAX))) * \
- CONFIG_BT_CTLR_ISO_TX_BUFFERS)
+#define BT_CTLR_ISO_TX_BUFFERS \
+ (DIV_ROUND_UP(BT_CTLR_ISO_SDU_LEN_MAX, \
+ MIN((CONFIG_BT_CTLR_ISO_TX_BUFFER_SIZE - BT_HCI_ISO_SDU_HDR_SIZE), \
+ MIN(CONFIG_BT_CTLR_ADV_ISO_PDU_LEN_MAX, \
+ CONFIG_BT_CTLR_CONN_ISO_PDU_LEN_MAX))) * \
+ CONFIG_BT_CTLR_ISO_TX_BUFFERS)
#elif defined(CONFIG_BT_CTLR_ADV_ISO)
-#define BT_CTLR_ISO_TX_BUFFERS (DIV_ROUND_UP(BT_CTLR_ISO_SDU_LEN_MAX, \
- MIN((CONFIG_BT_CTLR_ISO_TX_BUFFER_SIZE - \
- BT_HCI_ISO_SDU_HDR_SIZE), \
- CONFIG_BT_CTLR_ADV_ISO_PDU_LEN_MAX)) * \
- CONFIG_BT_CTLR_ISO_TX_BUFFERS)
+#define BT_CTLR_ISO_TX_BUFFERS \
+ (DIV_ROUND_UP(BT_CTLR_ISO_SDU_LEN_MAX, \
+ MIN((CONFIG_BT_CTLR_ISO_TX_BUFFER_SIZE - BT_HCI_ISO_SDU_HDR_SIZE), \
+ CONFIG_BT_CTLR_ADV_ISO_PDU_LEN_MAX)) * \
+ CONFIG_BT_CTLR_ISO_TX_BUFFERS)
#else /* CONFIG_BT_CTLR_CONN_ISO */
-#define BT_CTLR_ISO_TX_BUFFERS (DIV_ROUND_UP(BT_CTLR_ISO_SDU_LEN_MAX, \
- MIN((CONFIG_BT_CTLR_ISO_TX_BUFFER_SIZE - \
- BT_HCI_ISO_SDU_
|
Run Compliance Tests:
subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c#L63
subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c:63
-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);
|
Run Compliance Tests:
subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c#L153
subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c:153
- err = lll_prepare(is_abort_cb, abort_cb, create_prepare_cb, 0U,
- param);
+ err = lll_prepare(is_abort_cb, abort_cb, create_prepare_cb, 0U, param);
|
Run Compliance Tests:
subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c#L1382
subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c:1382
- iso_meta->payload_number = lll->payload_count + (lll->bn_curr - 1U) +
- (lll->ptc_curr * lll->pto);
+ iso_meta->payload_number =
+ lll->payload_count + (lll->bn_curr - 1U) + (lll->ptc_curr * lll->pto);
|
Run Compliance Tests:
subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c#L1404
subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c:1404
-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)
|
Run Compliance Tests:
subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c#L1424
subsys/bluetooth/controller/ll_sw/nordic/lll/lll_sync_iso.c:1424
- iso_meta->timestamp -= (latency * lll->iso_interval *
- PERIODIC_INT_UNIT_US);
+ iso_meta->timestamp -= (latency * lll->iso_interval * PERIODIC_INT_UNIT_US);
|
Run Compliance Tests:
subsys/bluetooth/controller/ll_sw/ull_iso_internal.h#L31
subsys/bluetooth/controller/ll_sw/ull_iso_internal.h:31
-#define BT_CTLR_ISO_SDU_LEN_MAX ((CONFIG_BT_CTLR_ISO_TX_BUFFER_SIZE * \
- CONFIG_BT_CTLR_ISO_TX_BUFFERS) - \
- BT_HCI_ISO_SDU_HDR_SIZE)
+#define BT_CTLR_ISO_SDU_LEN_MAX \
+ ((CONFIG_BT_CTLR_ISO_TX_BUFFER_SIZE * CONFIG_BT_CTLR_ISO_TX_BUFFERS) - \
+ BT_HCI_ISO_SDU_HDR_SIZE)
#if defined(CONFIG_BT_CTLR_ADV_ISO) && defined(CONFIG_BT_CTLR_CONN_ISO)
-#define BT_CTLR_ISO_TX_BUFFERS (DIV_ROUND_UP(BT_CTLR_ISO_SDU_LEN_MAX, \
- MIN((CONFIG_BT_CTLR_ISO_TX_BUFFER_SIZE - \
- BT_HCI_ISO_SDU_HDR_SIZE), \
- MIN(CONFIG_BT_CTLR_ADV_ISO_PDU_LEN_MAX, \
- CONFIG_BT_CTLR_CONN_ISO_PDU_LEN_MAX))) * \
- CONFIG_BT_CTLR_ISO_TX_BUFFERS)
+#define BT_CTLR_ISO_TX_BUFFERS \
+ (DIV_ROUND_UP(BT_CTLR_ISO_SDU_LEN_MAX, \
+ MIN((CONFIG_BT_CTLR_ISO_TX_BUFFER_SIZE - BT_HCI_ISO_SDU_HDR_SIZE), \
+ MIN(CONFIG_BT_CTLR_ADV_ISO_PDU_LEN_MAX, \
+ CONFIG_BT_CTLR_CONN_ISO_PDU_LEN_MAX))) * \
+ CONFIG_BT_CTLR_ISO_TX_BUFFERS)
#elif defined(CONFIG_BT_CTLR_ADV_ISO)
-#define BT_CTLR_ISO_TX_BUFFERS (DIV_ROUND_UP(BT_CTLR_ISO_SDU_LEN_MAX, \
- MIN((CONFIG_BT_CTLR_ISO_TX_BUFFER_SIZE - \
- BT_HCI_ISO_SDU_HDR_SIZE), \
- CONFIG_BT_CTLR_ADV_ISO_PDU_LEN_MAX)) * \
- CONFIG_BT_CTLR_ISO_TX_BUFFERS)
+#define BT_CTLR_ISO_TX_BUFFERS \
+ (DIV_ROUND_UP(BT_CTLR_ISO_SDU_LEN_MAX, \
+ MIN((CONFIG_BT_CTLR_ISO_TX_BUFFER_SIZE - BT_HCI_ISO_SDU_HDR_SIZE), \
+ CONFIG_BT_CTLR_ADV_ISO_PDU_LEN_MAX)) * \
+ CONFIG_BT_CTLR_ISO_TX_BUFFERS)
#else /* CONFIG_BT_CTLR_CONN_ISO */
-#define BT_CTLR_ISO_TX_BUFFERS (DIV_ROUND_UP(BT_CTLR_ISO_SDU_LEN_MAX, \
- MIN((CONFIG_BT_CTLR_ISO_TX_BUFFER_SIZE - \
- BT_HCI_ISO_SDU_HDR_SIZE), \
- CONFIG_BT_CTLR_CONN_ISO_PDU_LEN_MAX)) * \
- CONFIG_BT_CTLR_ISO_TX_BUFFERS)
+#define BT_CTLR_ISO_TX_BUFFERS \
+ (DIV_ROUND_UP(BT_CTLR_ISO_SDU_LEN_MAX, \
+ MIN((CONFIG_BT_CTLR_ISO_TX_BUFFER_SIZE - BT_HCI_ISO_SDU_HDR_SIZE), \
+ CONFIG_BT_CTLR_CONN_ISO_PDU_LEN_MAX)) * \
+ CONFIG_BT_CTLR_ISO_TX_BUFFERS)
|
Run Compliance Tests:
subsys/bluetooth/controller/ll_sw/ull_sync_iso.c#L614
subsys/bluetooth/controller/ll_sw/ull_sync_iso.c:614
- ctrl_spacing_us = PDU_BIS_US(sizeof(struct pdu_big_ctrl),
- lll->enc, lll->phy, PHY_FLAGS_S8);
+ ctrl_spacing_us =
+ PDU_BIS_US(sizeof(struct pdu_big_ctrl), lll->enc, lll->phy, PHY_FLAGS_S8);
|
Run Compliance Tests:
tests/bsim/bluetooth/audio/src/cap_acceptor_test.c#L52
tests/bsim/bluetooth/audio/src/cap_acceptor_test.c:52
-#define BT_LE_EXT_ADV_CONN_CUSTOM \
- BT_LE_ADV_PARAM(BT_LE_ADV_OPT_EXT_ADV | BT_LE_ADV_OPT_CONN, \
- BT_GAP_MS_TO_ADV_INTERVAL(140), \
- BT_GAP_MS_TO_ADV_INTERVAL(140), \
- NULL)
+#define BT_LE_EXT_ADV_CONN_CUSTOM \
+ BT_LE_ADV_PARAM(BT_LE_ADV_OPT_EXT_ADV | BT_LE_ADV_OPT_CONN, \
+ BT_GAP_MS_TO_ADV_INTERVAL(140), BT_GAP_MS_TO_ADV_INTERVAL(140), NULL)
|
Loading