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=713938 ---Test result--- Test Summary: CheckPatch FAIL 9.88 seconds GitLint FAIL 4.26 seconds BuildEll PASS 27.14 seconds BluezMake PASS 835.50 seconds MakeCheck PASS 11.77 seconds MakeDistcheck PASS 147.68 seconds CheckValgrind PASS 243.94 seconds CheckSmatch WARNING 326.37 seconds bluezmakeextell PASS 97.49 seconds IncrementalBuild PASS 8277.83 seconds ScanBuild WARNING 997.72 seconds Details ############################## Test: CheckPatch - FAIL Desc: Run checkpatch.pl script Output: [BlueZ,02/12] mesh: Add Remote Provisioning WARNING:PREFER_FALLTHROUGH: Prefer 'fallthrough;' over fallthrough comment #3305: FILE: mesh/remprv-server.c:238: + /* Fall Through */ WARNING:PREFER_FALLTHROUGH: Prefer 'fallthrough;' over fallthrough comment #3514: FILE: mesh/remprv-server.c:447: + /* Fall Through */ WARNING:PREFER_FALLTHROUGH: Prefer 'fallthrough;' over fallthrough comment #3518: FILE: mesh/remprv-server.c:451: + /* Fall Through */ WARNING:PREFER_FALLTHROUGH: Prefer 'fallthrough;' over fallthrough comment #3797: FILE: mesh/remprv-server.c:730: + /* Fall Through */ /github/workspace/src/src/13108984.patch total: 0 errors, 4 warnings, 3740 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/src/13108984.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. [BlueZ,07/12] mesh: Add Mesh Private Beacon server WARNING:PREFER_FALLTHROUGH: Prefer 'fallthrough;' over fallthrough comment #228: FILE: mesh/prvbeac-server.c:74: + /* Fall Through */ WARNING:PREFER_FALLTHROUGH: Prefer 'fallthrough;' over fallthrough comment #240: FILE: mesh/prvbeac-server.c:86: + /* Fall Through */ WARNING:PREFER_FALLTHROUGH: Prefer 'fallthrough;' over fallthrough comment #247: FILE: mesh/prvbeac-server.c:93: + /* Fall Through */ /github/workspace/src/src/13108981.patch total: 0 errors, 3 warnings, 166 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/src/13108981.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. [BlueZ,08/12] mesh: Add Tx/Rx support of Mesh Private Beacons ERROR:INITIALISED_STATIC: do not initialise statics to NULL #163: FILE: mesh/net-keys.c:70: +static struct l_queue *beacons = NULL; WARNING:LONG_LINE: line length of 81 exceeds 80 columns #981: FILE: mesh/net.c:2834: + beacon_data->net_key_id); /github/workspace/src/src/13108983.patch total: 1 errors, 1 warnings, 1067 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/src/13108983.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. ############################## Test: GitLint - FAIL Desc: Run gitlint Output: [BlueZ,01/12] doc/mesh: Add Remote Provisioning DBus APIs WARNING: I3 - ignore-body-lines: gitlint will be switching from using Python regex 'match' (match beginning) to 'search' (match anywhere) semantics. Please review your ignore-body-lines.regex option accordingly. To remove this warning, set general.regex-style-search=True. More details: https://jorisroovers.github.io/gitlint/configuration/#regex-style-search 11: B3 Line contains hard tab characters (\t): " * Refresh Device Keys" 12: B3 Line contains hard tab characters (\t): " * Reassign Node Addresses" 13: B3 Line contains hard tab characters (\t): " * Refresh Node Composition" ############################## Test: CheckSmatch - WARNING Desc: Run smatch tool with source Output: mesh/manager.c:113:35: warning: non-ANSI function declaration of function 'free_pending_add_call'mesh/crypto.c:1023:30: warning: non-ANSI function declaration of function 'mesh_crypto_check_avail'unit/test-mesh-crypto.c: note: in included file: ############################## Test: ScanBuild - WARNING Desc: Run Scan Build Output: mesh/remprv-server.c:735:12: warning: Assigned value is garbage or undefined msg[n++] = status; ^ ~~~~~~ 1 warning generated. mesh/prov-initiator.c:123:7: warning: Access to field 'server' results in a dereference of a null pointer (loaded from variable 'prov') if (!prov->server) ^~~~~~~~~~~~ 1 warning generated. tools/mesh/mesh-db.c:1978:4: warning: Value stored to 'jobj' is never read jobj = json_object_new_object(); ^ ~~~~~~~~~~~~~~~~~~~~~~~~ 1 warning generated. In file included from unit/test-mesh-crypto.c:20: In file included from ./mesh/crypto.c:18: In file included from ./ell/ell.h:1: ./ell/util.h:187:9: warning: 1st function call argument is an uninitialized value return L_BE32_TO_CPU(L_GET_UNALIGNED((const uint32_t *) ptr)); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ ./ell/util.h:89:28: note: expanded from macro 'L_BE32_TO_CPU' #define L_BE32_TO_CPU(val) bswap_32(val) ^~~~~~~~~~~~~ /usr/include/byteswap.h:34:21: note: expanded from macro 'bswap_32' #define bswap_32(x) __bswap_32 (x) ^~~~~~~~~~~~~~ unit/test-mesh-crypto.c:995:11: warning: The left expression of the compound assignment is an uninitialized value. The computed value will also be garbage key_aid |= KEY_ID_AKF; ~~~~~~~ ^ unit/test-mesh-crypto.c:1038:2: warning: 4th function call argument is an uninitialized value verify_uint8("NID", 0, keys->net_nid, nid); ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ unit/test-mesh-crypto.c:1319:6: warning: Branch condition evaluates to a garbage value if (ctl) { ^~~ unit/test-mesh-crypto.c:1539:7: warning: Branch condition evaluates to a garbage value if (net_ctl) { ^~~~~~~ unit/test-mesh-crypto.c:1806:3: warning: 1st function call argument is an uninitialized value l_put_be64(cmac_tmp, cmac); ^~~~~~~~~~~~~~~~~~~~~~~~~~ 6 warnings generated. --- Regards, Linux Bluetooth
