DNM Add support for PBP testing (CI test) #8332
Annotations
4 errors, 1 warning, and 10 notices
Run Compliance Tests
Process completed with exit code 3.
|
check-warns:
Identity.txt#L1
See https://docs.zephyrproject.org/latest/contribute/guidelines.html#commit-guidelines for more details
ecf3155d3f436c981ad7732c48fc41c2688985ee: author email (Frode van der Meeren <[email protected]>) needs to match one of the signed-off-by entries.
|
check-warns:
Gitlint.txt#L1
See https://docs.zephyrproject.org/latest/contribute/guidelines.html#commit-guidelines for more details
Commit ecf3155d3f:
1: UC2 Commit message does not contain a 'Signed-off-by:' line
1: UC3 Commit title does not follow [subsystem]: [subject] (and should not start with literal subsys or treewide): "temporarely run with more logging"
1: UC6 Commit message body is empty, should at least have 1 line(s).
3: B6 Body message is missing
|
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:
-#define BTP_PBP_READ_SUPPORTED_COMMANDS 0x01
+#define BTP_PBP_READ_SUPPORTED_COMMANDS 0x01
File:tests/bluetooth/tester/src/audio/btp/btp_pbp.h
Line:10
You may want to run clang-format on this change:
- if ((info->adv_props & BT_GAP_ADV_PROP_CONNECTABLE) ||
- info->interval == 0) {
+ if ((info->adv_props & BT_GAP_ADV_PROP_CONNECTABLE) || info->interval == 0) {
File:tests/bluetooth/tester/src/audio/btp_pbp.c
Line:65
You may want to run clang-format on this change:
-static uint8_t pbp_read_supported_commands(const void *cmd, uint16_t cmd_len,
- void *rsp, uint16_t *rsp_len)
+static uint8_t pbp_read_supported_commands(const void *cmd, uint16_t cmd_len, void *rsp,
+ uint16_t *rsp_len)
File:tests/bluetooth/tester/src/audio/btp_pbp.c
Line:97
You may want to run clang-format on this change:
- struct bt_le_adv_param param = BT_LE_ADV_PARAM_INIT(0, BT_GAP_ADV_FAST_INT_MIN_2,
- BT_GAP_ADV_FAST_INT_MAX_2, NULL);
- uint32_t gap_settings = BIT(BTP_GAP_SETTINGS_DISCOVERABLE) |
- BIT(BTP_GAP_SETTINGS_EXTENDED_ADVERTISING);
+ struct bt_le_adv_param param =
+ BT_LE_ADV_PARAM_INIT(0, BT_GAP_ADV_FAST_INT_MIN_2, BT_GAP_ADV_FAST_INT_MAX_2, NULL);
+ uint32_t gap_settings =
+ BIT(BTP_GAP_SETTINGS_DISCOVERABLE) | BIT(BTP_GAP_SETTINGS_EXTENDED_ADVERTISING);
uint32_t broadcast_id;
- NET_BUF_SIMPLE_DEFINE(ad_buf,
- BT_UUID_SIZE_16 + BT_AUDIO_BROADCAST_ID_SIZE);
+ NET_BUF_SIMPLE_DEFINE(ad_buf, BT_UUID_SIZE_16 + BT_AUDIO_BROADCAST_ID_SIZE);
File:tests/bluetooth/tester/src/audio/btp_pbp.c
Line:123
You may want to run clang-format on this change:
-static uint8_t pbp_set_public_broadcast_announcement(const void *cmd, uint16_t cmd_len,
- void *rsp, uint16_t *rsp_len)
+static uint8_t pbp_set_public_broadcast_announcement(const void *cmd, uint16_t cmd_len, void *rsp,
+ uint16_t *rsp_len)
File:tests/bluetooth/tester/src/audio/btp_pbp.c
Line:164
You may want to run clang-format on this change:
-static uint8_t pbp_set_broadcast_name(const void *cmd, uint16_t cmd_len,
- void *rsp, uint16_t *rsp_len)
+static uint8_t pbp_set_broadcast_name(const void *cmd, uint16_t cmd_len, void *rsp,
+ uint16_t *rsp_len)
File:tests/bluetooth/tester/src/audio/btp_pbp.c
Line:184
You may want to run clang-format on this change:
-static uint8_t pbp_broadcast_scan_start(const void *cmd, uint16_t cmd_len,
- void *rsp, uint16_t *rsp_len)
+static uint8_t pbp_broadcast_scan_start(const void *cmd, uint16_t cmd_len, void *rsp,
+ uint16_t *rsp_len)
File:tests/bluetooth/tester/src/audio/btp_pbp.c
Line:204
You may want to run clang-format on this change:
-static uint8_t pbp_broadcast_scan_stop(const void *cmd, uint16_t cmd_len,
- void *rsp, uint16_t *rsp_len)
+static uint8_t pbp_broadcast_scan_stop(const void *cmd, uint16_t cmd_len, void *rsp,
+ uint16_t *rsp_len)
File:tests/bluetooth/tester/src/audio/btp_pbp.c
Line:221
You may want to run clang-format on this change:
- {
- .opcode = BTP_PBP_READ_SUPPORTED_COMMANDS,
- .index = BTP_INDEX_NONE,
- .expect_len = 0,
- .func = pbp_read_supported_commands
- },
- {
- .opcode = BTP_PBP_SET_PUBLIC_BROADCAST_ANNOUNCEMENT,
- .expect_len = sizeof(struct btp_pbp_set_public_broadcast_announcement_cmd),
- .func = pbp_set_public_broadcast_announcement
- },
- {
- .opcode = BTP_PBP_SET_BROADCAST_NAME,
- .expect_len = sizeof(struct btp_pbp_set_broadcast_name_cmd),
- .func = pbp_set_broadcast_name
- },
- {
- .opcode = BTP_PBP_BROADCAST_SCAN_START,
- .expect_len = sizeof(struct btp_pbp_broadcast_scan_start_cmd),
- .func = pbp_broadcast_scan_start
- },
- {
- .opcode = BTP_PBP_BROADCAST_SCAN_STOP,
- .expect_len = sizeof(struct btp_pbp_broadcast_scan_stop_cmd),
- .func = pbp_broadcast_scan_stop
- }
-};
+ {.opcode = BTP_PBP_READ_SUPPORTED_COMMANDS,
+ .index = BTP_INDEX_NONE,
+ .expect_len = 0,
+ .func = pbp_read_supported_commands},
+ {.opcode = BTP_PBP_SET_PUBLI
|
Run Compliance Tests:
tests/bluetooth/tester/src/audio/btp/btp_pbp.h#L10
tests/bluetooth/tester/src/audio/btp/btp_pbp.h:10
-#define BTP_PBP_READ_SUPPORTED_COMMANDS 0x01
+#define BTP_PBP_READ_SUPPORTED_COMMANDS 0x01
|
Run Compliance Tests:
tests/bluetooth/tester/src/audio/btp_pbp.c#L65
tests/bluetooth/tester/src/audio/btp_pbp.c:65
- if ((info->adv_props & BT_GAP_ADV_PROP_CONNECTABLE) ||
- info->interval == 0) {
+ if ((info->adv_props & BT_GAP_ADV_PROP_CONNECTABLE) || info->interval == 0) {
|
Run Compliance Tests:
tests/bluetooth/tester/src/audio/btp_pbp.c#L97
tests/bluetooth/tester/src/audio/btp_pbp.c:97
-static uint8_t pbp_read_supported_commands(const void *cmd, uint16_t cmd_len,
- void *rsp, uint16_t *rsp_len)
+static uint8_t pbp_read_supported_commands(const void *cmd, uint16_t cmd_len, void *rsp,
+ uint16_t *rsp_len)
|
Run Compliance Tests:
tests/bluetooth/tester/src/audio/btp_pbp.c#L123
tests/bluetooth/tester/src/audio/btp_pbp.c:123
- struct bt_le_adv_param param = BT_LE_ADV_PARAM_INIT(0, BT_GAP_ADV_FAST_INT_MIN_2,
- BT_GAP_ADV_FAST_INT_MAX_2, NULL);
- uint32_t gap_settings = BIT(BTP_GAP_SETTINGS_DISCOVERABLE) |
- BIT(BTP_GAP_SETTINGS_EXTENDED_ADVERTISING);
+ struct bt_le_adv_param param =
+ BT_LE_ADV_PARAM_INIT(0, BT_GAP_ADV_FAST_INT_MIN_2, BT_GAP_ADV_FAST_INT_MAX_2, NULL);
+ uint32_t gap_settings =
+ BIT(BTP_GAP_SETTINGS_DISCOVERABLE) | BIT(BTP_GAP_SETTINGS_EXTENDED_ADVERTISING);
uint32_t broadcast_id;
- NET_BUF_SIMPLE_DEFINE(ad_buf,
- BT_UUID_SIZE_16 + BT_AUDIO_BROADCAST_ID_SIZE);
+ NET_BUF_SIMPLE_DEFINE(ad_buf, BT_UUID_SIZE_16 + BT_AUDIO_BROADCAST_ID_SIZE);
|
Run Compliance Tests:
tests/bluetooth/tester/src/audio/btp_pbp.c#L164
tests/bluetooth/tester/src/audio/btp_pbp.c:164
-static uint8_t pbp_set_public_broadcast_announcement(const void *cmd, uint16_t cmd_len,
- void *rsp, uint16_t *rsp_len)
+static uint8_t pbp_set_public_broadcast_announcement(const void *cmd, uint16_t cmd_len, void *rsp,
+ uint16_t *rsp_len)
|
Run Compliance Tests:
tests/bluetooth/tester/src/audio/btp_pbp.c#L184
tests/bluetooth/tester/src/audio/btp_pbp.c:184
-static uint8_t pbp_set_broadcast_name(const void *cmd, uint16_t cmd_len,
- void *rsp, uint16_t *rsp_len)
+static uint8_t pbp_set_broadcast_name(const void *cmd, uint16_t cmd_len, void *rsp,
+ uint16_t *rsp_len)
|
Run Compliance Tests:
tests/bluetooth/tester/src/audio/btp_pbp.c#L204
tests/bluetooth/tester/src/audio/btp_pbp.c:204
-static uint8_t pbp_broadcast_scan_start(const void *cmd, uint16_t cmd_len,
- void *rsp, uint16_t *rsp_len)
+static uint8_t pbp_broadcast_scan_start(const void *cmd, uint16_t cmd_len, void *rsp,
+ uint16_t *rsp_len)
|
Run Compliance Tests:
tests/bluetooth/tester/src/audio/btp_pbp.c#L221
tests/bluetooth/tester/src/audio/btp_pbp.c:221
-static uint8_t pbp_broadcast_scan_stop(const void *cmd, uint16_t cmd_len,
- void *rsp, uint16_t *rsp_len)
+static uint8_t pbp_broadcast_scan_stop(const void *cmd, uint16_t cmd_len, void *rsp,
+ uint16_t *rsp_len)
|
Run Compliance Tests:
tests/bluetooth/tester/src/audio/btp_pbp.c#L264
tests/bluetooth/tester/src/audio/btp_pbp.c:264
- {
- .opcode = BTP_PBP_READ_SUPPORTED_COMMANDS,
- .index = BTP_INDEX_NONE,
- .expect_len = 0,
- .func = pbp_read_supported_commands
- },
- {
- .opcode = BTP_PBP_SET_PUBLIC_BROADCAST_ANNOUNCEMENT,
- .expect_len = sizeof(struct btp_pbp_set_public_broadcast_announcement_cmd),
- .func = pbp_set_public_broadcast_announcement
- },
- {
- .opcode = BTP_PBP_SET_BROADCAST_NAME,
- .expect_len = sizeof(struct btp_pbp_set_broadcast_name_cmd),
- .func = pbp_set_broadcast_name
- },
- {
- .opcode = BTP_PBP_BROADCAST_SCAN_START,
- .expect_len = sizeof(struct btp_pbp_broadcast_scan_start_cmd),
- .func = pbp_broadcast_scan_start
- },
- {
- .opcode = BTP_PBP_BROADCAST_SCAN_STOP,
- .expect_len = sizeof(struct btp_pbp_broadcast_scan_stop_cmd),
- .func = pbp_broadcast_scan_stop
- }
-};
+ {.opcode = BTP_PBP_READ_SUPPORTED_COMMANDS,
+ .index = BTP_INDEX_NONE,
+ .expect_len = 0,
+ .func = pbp_read_supported_commands},
+ {.opcode = BTP_PBP_SET_PUBLIC_BROADCAST_ANNOUNCEMENT,
+ .expect_len = sizeof(struct btp_pbp_set_public_broadcast_announcement_cmd),
+ .func = pbp_set_public_broadcast_announcement},
+ {.opcode = BTP_PBP_SET_BROADCAST_NAME,
+ .expect_len = sizeof(struct btp_pbp_set_broadcast_name_cmd),
+ .func = pbp_set_broadcast_name},
+ {.opcode = BTP_PBP_BROADCAST_SCAN_START,
+ .expect_len = sizeof(struct btp_pbp_broadcast_scan_start_cmd),
+ .func = pbp_broadcast_scan_start},
+ {.opcode = BTP_PBP_BROADCAST_SCAN_STOP,
+ .expect_len = sizeof(struct btp_pbp_broadcast_scan_stop_cmd),
+ .func = pbp_broadcast_scan_stop}};
|
Run Compliance Tests:
tests/bluetooth/tester/src/btp/btp.h#L82
tests/bluetooth/tester/src/btp/btp.h:82
-#define BTP_SERVICE_ID_PBP 30
+#define BTP_SERVICE_ID_PBP 30
-#define BTP_SERVICE_ID_MAX BTP_SERVICE_ID_PBP
+#define BTP_SERVICE_ID_MAX BTP_SERVICE_ID_PBP
|
Loading