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=531151 ---Test result--- Test Summary: CheckPatch FAIL 12.00 seconds GitLint FAIL 6.07 seconds Prep - Setup ELL PASS 39.11 seconds Build - Prep PASS 0.09 seconds Build - Configure PASS 6.90 seconds Build - Make PASS 165.81 seconds Make Check PASS 9.01 seconds Make Distcheck PASS 201.08 seconds Build w/ext ELL - Configure PASS 6.92 seconds Build w/ext ELL - Make PASS 155.03 seconds Details ############################## Test: CheckPatch - FAIL Desc: Run checkpatch.pl script with rule in .checkpatch.conf Output: lib: Inclusive language in HCI commands and events WARNING:PREFER_DEFINED_ATTRIBUTE_MACRO: Prefer __packed over __attribute__((packed)) #25: FILE: lib/hci.h:427: +} __attribute__ ((packed)) central_link_key_cp; WARNING:PREFER_DEFINED_ATTRIBUTE_MACRO: Prefer __packed over __attribute__((packed)) #42: FILE: lib/hci.h:1812: +} __attribute__ ((packed)) evt_central_link_key_complete; - total: 0 errors, 2 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. "[PATCH] lib: Inclusive language in HCI commands and events" 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. lib: Inclusive language in consts and strings ERROR:SPACING: space required after that ',' (ctx:VxV) #49: FILE: lib/hci.c:739: + { "<AFH class. perip.>",LMP_AFH_CLS_PRP }, /* Bit 4 */ ^ - total: 1 errors, 0 warnings, 74 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. "[PATCH] lib: Inclusive language in consts and strings" 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. lib: Inclusive language for filtering devices WARNING:PREFER_DEFINED_ATTRIBUTE_MACRO: Prefer __packed over __attribute__((packed)) #133: FILE: lib/hci.h:1568: +} __attribute__ ((packed)) le_read_accept_list_size_rp; WARNING:PREFER_DEFINED_ATTRIBUTE_MACRO: Prefer __packed over __attribute__((packed)) #146: FILE: lib/hci.h:1577: +} __attribute__ ((packed)) le_add_device_to_accept_list_cp; WARNING:PREFER_DEFINED_ATTRIBUTE_MACRO: Prefer __packed over __attribute__((packed)) #156: FILE: lib/hci.h:1584: +} __attribute__ ((packed)) le_remove_device_from_accept_list_cp; WARNING:LONG_LINE: line length of 81 exceeds 80 columns #173: FILE: lib/hci_lib.h:120: +int hci_le_add_accept_list(int dd, const bdaddr_t *bdaddr, uint8_t type, int to); - total: 0 errors, 4 warnings, 195 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. "[PATCH] lib: Inclusive language for filtering devices" 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. shared/ad: Inclusive language changes WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #8: https://specificationrefs.bluetooth.com/language-mapping/Appropriate_Language_Mapping_Table.pdf - total: 0 errors, 1 warnings, 34 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. "[PATCH] shared/ad: Inclusive language changes" 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. shared/hfp: Inclusive language changes WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #7: https://specificationrefs.bluetooth.com/language-mapping/Appropriate_Language_Mapping_Table.pdf - total: 0 errors, 1 warnings, 32 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. "[PATCH] shared/hfp: Inclusive language changes" 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. monitor: Inclusive language in peripheral broadcast WARNING:LONG_LINE: line length of 84 exceeds 80 columns #284: FILE: monitor/packet.c:10248: + const struct bt_hci_evt_peripheral_broadcast_channel_map_change *evt = data; - total: 0 errors, 1 warnings, 378 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. "[PATCH] monitor: Inclusive language in peripheral broadcast" 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. monitor: Inclusive language in consts and strings WARNING:LONG_LINE_STRING: line length of 83 exceeds 80 columns #302: FILE: monitor/packet.c:3674: + print_field("Peripheral Conn. Interval: 0x%4.4x - 0x%4.4x", - total: 0 errors, 1 warnings, 408 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. "[PATCH] monitor: Inclusive language in consts and strings" 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. emulator: Inclusive language in strings WARNING:LONG_LINE_COMMENT: line length of 84 exceeds 80 columns #90: FILE: emulator/btdev.c:6130: + btdev->le_features[0] |= 0x08; /* Peripheral-initiated Features Exchange */ WARNING:LONG_LINE_COMMENT: line length of 84 exceeds 80 columns #103: FILE: emulator/le.c:346: + //hci->le_features[0] |= 0x08; /* Peripheral-initiated Features Exchange */ WARNING:LONG_LINE_COMMENT: line length of 84 exceeds 80 columns #113: FILE: emulator/le.c:402: + hci->le_states[0] |= 0x40; /* Initiating + Connection (Central Role) */ - total: 0 errors, 3 warnings, 116 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. "[PATCH] emulator: Inclusive language in strings" 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. emulator: Inclusive language in filtering device WARNING:LONG_LINE_COMMENT: line length of 81 exceeds 80 columns #115: FILE: emulator/btdev.c:3553: + * • any advertising filter policy uses the Accept List and advertising WARNING:LONG_LINE_COMMENT: line length of 81 exceeds 80 columns #162: FILE: emulator/btdev.c:3590: + * • any advertising filter policy uses the Accept List and advertising WARNING:LONG_LINE_COMMENT: line length of 81 exceeds 80 columns #217: FILE: emulator/btdev.c:3641: + * • any advertising filter policy uses the Accept List and advertising - total: 0 errors, 3 warnings, 468 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. "[PATCH] emulator: Inclusive language in filtering device" 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. tools/parser/hci: Inclusive language changes WARNING:LONG_LINE: line length of 81 exceeds 80 columns #87: FILE: tools/parser/hci.c:1643: + (cp->filter == 0x01 ? "accept list only" : "reserved"))); - total: 0 errors, 1 warnings, 104 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. "[PATCH] tools/parser/hci: Inclusive language changes" 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. tools/mgmt-tester: Inclusive language changes WARNING:LONG_LINE_STRING: line length of 86 exceeds 80 columns #194: FILE: tools/mgmt-tester.c:11551: + test_bredrle50_full("Ext Adv. connectable & connected (peripheral) - Success", WARNING:LONG_LINE_STRING: line length of 90 exceeds 80 columns #201: FILE: tools/mgmt-tester.c:11556: + test_bredrle50_full("Ext Adv. non-connectable & connected (peripheral) - Success", WARNING:LONG_LINE_STRING: line length of 83 exceeds 80 columns #208: FILE: tools/mgmt-tester.c:11561: + test_bredrle50_full("Ext Adv. connectable & connected (central) - Success", WARNING:LONG_LINE_STRING: line length of 87 exceeds 80 columns #215: FILE: tools/mgmt-tester.c:11566: + test_bredrle50_full("Ext Adv. non-connectable & connected (central) - Success", - total: 0 errors, 4 warnings, 197 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. "[PATCH] tools/mgmt-tester: Inclusive language changes" 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. tools/hciconfig: Inclusive language changes WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #8: https://specificationrefs.bluetooth.com/language-mapping/Appropriate_Language_Mapping_Table.pdf WARNING:LONG_LINE: line length of 92 exceeds 80 columns #58: FILE: tools/hciconfig.c:1925: + { "block", cmd_block, "<bdaddr>", "Add a device to the reject list" }, WARNING:LONG_LINE: line length of 97 exceeds 80 columns #59: FILE: tools/hciconfig.c:1926: + { "unblock", cmd_unblock, "<bdaddr>", "Remove a device from the reject list" }, - total: 0 errors, 3 warnings, 58 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. "[PATCH] tools/hciconfig: Inclusive language changes" 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. tools/parser/avdtp: Inclusive language changes WARNING:LONG_LINE_STRING: line length of 86 exceeds 80 columns #27: FILE: tools/parser/avdtp.c:662: + printf("AVDTP(c): ver %d %s%scc %d %spt %d seqn %d time %d ssrc %d\n", - total: 0 errors, 1 warnings, 16 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. "[PATCH] tools/parser/avdtp: Inclusive language changes" 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. tools/l2test: Inclusive language changes ERROR:INITIALISED_STATIC: do not initialise statics to 0 #17: FILE: tools/l2test.c:113: +static int central = 0; WARNING:MISSING_SPACE: break quoted strings at a space character #62: FILE: tools/l2test.c:1340: while ((opt = getopt(argc, argv, "a:b:cde:g:i:mnpqrstuwxyz" + "AB:CD:EF:GH:I:J:K:L:M:N:O:P:Q:RSTUV:W:X:Y:Z:")) != EOF) { - total: 1 errors, 1 warnings, 66 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. "[PATCH] tools/l2test: Inclusive language changes" 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. tools/rctest: Inclusive language changes ERROR:INITIALISED_STATIC: do not initialise statics to 0 #17: FILE: tools/rctest.c:79: +static int central = 0; WARNING:LONG_LINE: line length of 90 exceeds 80 columns #60: FILE: tools/rctest.c:700: + while ((opt=getopt(argc,argv,"rdscuwmna:b:i:P:U:B:O:N:CAESL:W:M:D:Y:T")) != EOF) { ERROR:SPACING: spaces required around that '=' (ctx:VxV) #60: FILE: tools/rctest.c:700: + while ((opt=getopt(argc,argv,"rdscuwmna:b:i:P:U:B:O:N:CAESL:W:M:D:Y:T")) != EOF) { ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #60: FILE: tools/rctest.c:700: + while ((opt=getopt(argc,argv,"rdscuwmna:b:i:P:U:B:O:N:CAESL:W:M:D:Y:T")) != EOF) { ^ ERROR:SPACING: space required after that ',' (ctx:VxV) #60: FILE: tools/rctest.c:700: + while ((opt=getopt(argc,argv,"rdscuwmna:b:i:P:U:B:O:N:CAESL:W:M:D:Y:T")) != EOF) { ^ - total: 4 errors, 1 warnings, 81 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. "[PATCH] tools/rctest: Inclusive language changes" 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. tools/rfcomm: Inclusive language changes ERROR:INITIALISED_STATIC: do not initialise statics to 0 #17: FILE: tools/rfcomm.c:40: +static int central = 0; WARNING:LONG_LINE: line length of 91 exceeds 80 columns #53: FILE: tools/rfcomm.c:683: + while ((opt = getopt_long(argc, argv, "+i:rahAESCL:", main_options, NULL)) != -1) { - total: 1 errors, 1 warnings, 58 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. "[PATCH] tools/rfcomm: Inclusive language changes" 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. profiles/health: Inclusive language changes WARNING:LONG_LINE_COMMENT: line length of 85 exceeds 80 columns #27: FILE: profiles/health/mcap.c:56: + guint remote_caps; /* CSP-Peripheral: remote central got caps */ WARNING:LONG_LINE_COMMENT: line length of 90 exceeds 80 columns #28: FILE: profiles/health/mcap.c:57: + guint rem_req_acc; /* CSP-Peripheral: accuracy required by central */ WARNING:LONG_LINE_COMMENT: line length of 81 exceeds 80 columns #34: FILE: profiles/health/mcap.c:63: + void *csp_priv_data; /* CSP-Central: In-flight request data */ WARNING:LONG_LINE_COMMENT: line length of 100 exceeds 80 columns #56: FILE: profiles/health/mcap.h:273: + mcap_info_ind_event_cb mcl_sync_infoind_cb; /* (CSP Central) Received info indication */ - total: 0 errors, 4 warnings, 40 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. "[PATCH] profiles/health: Inclusive language changes" 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. adapter: Inclusive language for central and peripheral ERROR:INITIALISED_STATIC: do not initialise statics to NULL #19: FILE: src/adapter.c:115: +static struct mgmt *mgmt_primary = NULL; - total: 1 errors, 0 warnings, 285 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. "[PATCH] adapter: Inclusive language for central and peripheral" 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. adapter: Inclusive language for device filtering WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #8: https://specificationrefs.bluetooth.com/language-mapping/Appropriate_Language_Mapping_Table.pdf - total: 0 errors, 1 warnings, 89 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. "[PATCH] adapter: Inclusive language for device filtering" 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. mesh: Inclusive language changes WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #8: https://specificationrefs.bluetooth.com/language-mapping/Appropriate_Language_Mapping_Table.pdf ERROR:INITIALISED_STATIC: do not initialise statics to 0 #28: FILE: mesh/net-keys.c:56: +static uint32_t last_flooding_id = 0; - total: 1 errors, 1 warnings, 146 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. "[PATCH] mesh: Inclusive language changes" 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. tools/mesh: Inclusive language changes WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #7: https://specificationrefs.bluetooth.com/language-mapping/Appropriate_Language_Mapping_Table.pdf - total: 0 errors, 1 warnings, 239 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. "[PATCH] tools/mesh: Inclusive language changes" 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. tools/mesh-gatt: Inclusive language changes WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #7: https://specificationrefs.bluetooth.com/language-mapping/Appropriate_Language_Mapping_Table.pdf ERROR:FUNCTION_WITHOUT_ARGS: Bad function definition - void setup_accept_list() should probably be void setup_accept_list(void) #90: FILE: tools/mesh-gatt/net.c:944: +static void setup_accept_list() - total: 1 errors, 1 warnings, 127 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. "[PATCH] tools/mesh-gatt: Inclusive language changes" 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. unit/mesh: Inclusive language changes WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #7: https://specificationrefs.bluetooth.com/language-mapping/Appropriate_Language_Mapping_Table.pdf - total: 0 errors, 1 warnings, 16 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. "[PATCH] unit/mesh: Inclusive language changes" 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. doc/mesh: Inclusive language changes WARNING:COMMIT_LOG_LONG_LINE: Possible unwrapped commit description (prefer a maximum 75 chars per line) #7: https://specificationrefs.bluetooth.com/language-mapping/Appropriate_Language_Mapping_Table.pdf - total: 0 errors, 1 warnings, 8 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. "[PATCH] doc/mesh: Inclusive language changes" 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. android: Inclusive language for volume control WARNING:LEADING_SPACE: please, no spaces at the start of a line #115: FILE: android/hardware/audio.h:517: + int (*set_main_volume)(struct audio_hw_device *dev, float volume);$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #128: FILE: android/hardware/audio.h:526: + int (*get_main_volume)(struct audio_hw_device *dev, float *volume);$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #137: FILE: android/hardware/audio.h:595: + int (*set_main_mute)(struct audio_hw_device *dev, bool mute);$ WARNING:LEADING_SPACE: please, no spaces at the start of a line #150: FILE: android/hardware/audio.h:604: + int (*get_main_mute)(struct audio_hw_device *dev, bool *mute);$ - total: 0 errors, 4 warnings, 124 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. "[PATCH] android: Inclusive language for volume control" 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. android: Inclusive language for filtering devices WARNING:LEADING_SPACE: please, no spaces at the start of a line #108: FILE: android/hardware/bt_hf_client.h:101: + BTHF_CLIENT_CMD_COMPLETE_ERROR_BLOCKED,$ - total: 0 errors, 1 warnings, 78 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. "[PATCH] android: Inclusive language for filtering devices" 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 with rule in .gitlint Output: shared/ad: Inclusive language changes 5: B1 Line exceeds max length (95>80): "https://specificationrefs.bluetooth.com/language-mapping/Appropriate_Language_Mapping_Table.pdf"; shared/hfp: Inclusive language changes 4: B1 Line exceeds max length (95>80): "https://specificationrefs.bluetooth.com/language-mapping/Appropriate_Language_Mapping_Table.pdf"; tools/hciconfig: Inclusive language changes 5: B1 Line exceeds max length (95>80): "https://specificationrefs.bluetooth.com/language-mapping/Appropriate_Language_Mapping_Table.pdf"; adapter: Inclusive language for device filtering 5: B1 Line exceeds max length (95>80): "https://specificationrefs.bluetooth.com/language-mapping/Appropriate_Language_Mapping_Table.pdf"; mesh: Inclusive language changes 5: B1 Line exceeds max length (95>80): "https://specificationrefs.bluetooth.com/language-mapping/Appropriate_Language_Mapping_Table.pdf"; tools/mesh: Inclusive language changes 4: B1 Line exceeds max length (95>80): "https://specificationrefs.bluetooth.com/language-mapping/Appropriate_Language_Mapping_Table.pdf"; tools/mesh-gatt: Inclusive language changes 4: B1 Line exceeds max length (95>80): "https://specificationrefs.bluetooth.com/language-mapping/Appropriate_Language_Mapping_Table.pdf"; unit/mesh: Inclusive language changes 4: B1 Line exceeds max length (95>80): "https://specificationrefs.bluetooth.com/language-mapping/Appropriate_Language_Mapping_Table.pdf"; doc/mesh: Inclusive language changes 4: B1 Line exceeds max length (95>80): "https://specificationrefs.bluetooth.com/language-mapping/Appropriate_Language_Mapping_Table.pdf"; ############################## Test: Prep - Setup ELL - PASS Desc: Clone, build, and install ELL ############################## Test: Build - Prep - PASS Desc: Prepare environment for build ############################## Test: Build - Configure - PASS Desc: Configure the BlueZ source tree ############################## Test: Build - Make - PASS Desc: Build the BlueZ source tree ############################## Test: Make Check - PASS Desc: Run 'make check' ############################## Test: Make Distcheck - PASS Desc: Run distcheck to check the distribution ############################## Test: Build w/ext ELL - Configure - PASS Desc: Configure BlueZ source with '--enable-external-ell' configuration ############################## Test: Build w/ext ELL - Make - PASS Desc: Build BlueZ source with '--enable-external-ell' configuration --- Regards, Linux Bluetooth
