RE: Add mesh testing support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



This is automated email and please do not reply to this email!

Dear submitter,

Thank you for submitting the patches to the linux bluetooth mailing list.
This is a CI test results with your patch series:
PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=672980

---Test result---

Test Summary:
CheckPatch                    FAIL      2.87 seconds
GitLint                       PASS      0.96 seconds
Prep - Setup ELL              PASS      31.96 seconds
Build - Prep                  PASS      0.66 seconds
Build - Configure             PASS      10.04 seconds
Build - Make                  PASS      981.55 seconds
Make Check                    PASS      12.52 seconds
Make Check w/Valgrind         PASS      347.96 seconds
Make Distcheck                PASS      292.35 seconds
Build w/ext ELL - Configure   PASS      10.16 seconds
Build w/ext ELL - Make        PASS      101.38 seconds
Incremental Build w/ patches  PASS      238.65 seconds
Scan Build                    PASS      784.56 seconds

Details
##############################
Test: CheckPatch - FAIL
Desc: Run checkpatch.pl script with rule in .checkpatch.conf
Output:
[BlueZ,v3,2/2] tools: Add mesh-tester to test Kernel mesh support
WARNING:LONG_LINE: line length of 81 exceeds 80 columns
#362: FILE: tools/mesh-tester.c:252:
+			sizeof(set_exp_feat_param_mesh), set_exp_feat_param_mesh,

WARNING:LONG_LINE: line length of 83 exceeds 80 columns
#502: FILE: tools/mesh-tester.c:392:
+			sizeof(set_exp_feat_param_debug), set_exp_feat_param_debug,

WARNING:LONG_LINE: line length of 81 exceeds 80 columns
#592: FILE: tools/mesh-tester.c:482:
+					test_pre_setup, test_setup, func, NULL, \

WARNING:LONG_LINE: line length of 83 exceeds 80 columns
#593: FILE: tools/mesh-tester.c:483:
+					test_post_teardown, timeout, user, free); \

WARNING:BLOCK_COMMENT_STYLE: Block comments should align the * on each line
#728: FILE: tools/mesh-tester.c:618:
+/* Read HCI commands in the expect_hci_list and add it to the queue
+*/

ERROR:SWITCH_CASE_INDENT_LEVEL: switch and case should be at the same indent
#1040: FILE: tools/mesh-tester.c:930:
+	switch (opcode) {
+		case BT_HCI_CMD_WRITE_SCAN_ENABLE:
+		case BT_HCI_CMD_LE_SET_ADV_ENABLE:
+		case BT_HCI_CMD_LE_SET_EXT_ADV_ENABLE:
[...]
+		case BT_HCI_CMD_WRITE_SIMPLE_PAIRING_MODE:
[...]
+		default:

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#1249: FILE: tools/mesh-tester.c:1139:
+		 * for a callback. */

WARNING:LONG_LINE: line length of 81 exceeds 80 columns
#1441: FILE: tools/mesh-tester.c:1331:
+			sizeof(set_exp_feat_param_mesh), set_exp_feat_param_mesh,

/github/workspace/src/12961464.patch total: 1 errors, 7 warnings, 1474 lines checked

NOTE: For some of the reported defects, checkpatch may be able to
      mechanically convert to the typical style using --fix or --fix-inplace.

/github/workspace/src/12961464.patch has style problems, please review.

NOTE: Ignored message types: COMMIT_MESSAGE COMPLEX_MACRO CONST_STRUCT FILE_PATH_CHANGES MISSING_SIGN_OFF PREFER_PACKED SPDX_LICENSE_TAG SPLIT_STRING SSCANF_TO_KSTRTO

NOTE: If any of the errors are false positives, please report
      them to the maintainer, see CHECKPATCH in MAINTAINERS.




---
Regards,
Linux Bluetooth


[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux