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=555113 ---Test result--- Test Summary: CheckPatch FAIL 0.88 seconds GitLint PASS 0.59 seconds Prep - Setup ELL PASS 44.86 seconds Build - Prep PASS 0.18 seconds Build - Configure PASS 7.90 seconds Build - Make FAIL 161.42 seconds Make Check FAIL 1.37 seconds Make Distcheck PASS 223.29 seconds Build w/ext ELL - Configure PASS 7.79 seconds Build w/ext ELL - Make FAIL 143.08 seconds Details ############################## Test: CheckPatch - FAIL Desc: Run checkpatch.pl script with rule in .checkpatch.conf Output: [BlueZ,1/2] gatt: allow GATT server to dicate CCC permissions WARNING:LONG_LINE: line length of 81 exceeds 80 columns #144: FILE: src/gatt-database.c:1229: + 0, NULL); WARNING:LONG_LINE: line length of 81 exceeds 80 columns #154: FILE: src/gatt-database.c:1694: + } else if (!strcmp("encrypt-authenticated-asynchronous", flag)) { WARNING:LONG_LINE: line length of 86 exceeds 80 columns #166: FILE: src/gatt-database.c:2797: + ccc_write_cb, chrc, chrc->perm, NULL); /github/workspace/src/12526275.patch total: 0 errors, 3 warnings, 70 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/12526275.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: Build - Make - FAIL Desc: Build the BlueZ source tree Output: src/gatt-database.c: In function ‘service_add_ccc’: src/gatt-database.c:1060:2: error: ISO C90 forbids mixed declarations and code [-Werror=declaration-after-statement] 1060 | uint32_t permissions = BT_ATT_PERM_READ | BT_ATT_PERM_WRITE; | ^~~~~~~~ cc1: all warnings being treated as errors make[1]: *** [Makefile:9212: src/bluetoothd-gatt-database.o] Error 1 make: *** [Makefile:4151: all] Error 2 ############################## Test: Make Check - FAIL Desc: Run 'make check' Output: src/gatt-database.c: In function ‘service_add_ccc’: src/gatt-database.c:1060:2: error: ISO C90 forbids mixed declarations and code [-Werror=declaration-after-statement] 1060 | uint32_t permissions = BT_ATT_PERM_READ | BT_ATT_PERM_WRITE; | ^~~~~~~~ cc1: all warnings being treated as errors make[1]: *** [Makefile:9212: src/bluetoothd-gatt-database.o] Error 1 make: *** [Makefile:10443: check] Error 2 ############################## Test: Build w/ext ELL - Make - FAIL Desc: Build BlueZ source with '--enable-external-ell' configuration Output: src/gatt-database.c: In function ‘service_add_ccc’: src/gatt-database.c:1060:2: error: ISO C90 forbids mixed declarations and code [-Werror=declaration-after-statement] 1060 | uint32_t permissions = BT_ATT_PERM_READ | BT_ATT_PERM_WRITE; | ^~~~~~~~ cc1: all warnings being treated as errors make[1]: *** [Makefile:9212: src/bluetoothd-gatt-database.o] Error 1 make: *** [Makefile:4151: all] Error 2 --- Regards, Linux Bluetooth
