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=657598 ---Test result--- Test Summary: CheckPatch FAIL 4.51 seconds GitLint PASS 3.02 seconds SubjectPrefix PASS 2.72 seconds BuildKernel PASS 32.04 seconds BuildKernel32 PASS 27.65 seconds Incremental Build with patchesPASS 52.40 seconds TestRunner: Setup PASS 474.56 seconds TestRunner: l2cap-tester PASS 16.85 seconds TestRunner: bnep-tester PASS 5.82 seconds TestRunner: mgmt-tester PASS 97.85 seconds TestRunner: rfcomm-tester PASS 9.35 seconds TestRunner: sco-tester PASS 9.12 seconds TestRunner: smp-tester PASS 9.11 seconds TestRunner: userchan-tester PASS 6.05 seconds Details ############################## Test: CheckPatch - FAIL - 4.51 seconds Run checkpatch.pl script with rule in .checkpatch.conf [v3,1/3] Bluetooth: Add support for devcoredump\Traceback (most recent call last): File "scripts/spdxcheck.py", line 6, in <module> from ply import lex, yacc ModuleNotFoundError: No module named 'ply' WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? #133: new file mode 100644 WARNING:SPDX_LICENSE_TAG: Improper SPDX comment style for 'include/net/bluetooth/coredump.h', please use '/*' instead #138: FILE: include/net/bluetooth/coredump.h:1: +// SPDX-License-Identifier: GPL-2.0-only WARNING:SPDX_LICENSE_TAG: Missing or malformed SPDX-License-Identifier tag in line 1 #138: FILE: include/net/bluetooth/coredump.h:1: +// SPDX-License-Identifier: GPL-2.0-only WARNING:SPLIT_STRING: quoted string split across lines #589: FILE: net/bluetooth/coredump.c:300: + "Devcoredump complete with size %u " + "(expect %u)", WARNING:SPLIT_STRING: quoted string split across lines #608: FILE: net/bluetooth/coredump.c:319: + "Devcoredump aborted with size %u " + "(expect %u)", WARNING:OOM_MESSAGE: Possible unnecessary 'out of memory' message #691: FILE: net/bluetooth/coredump.c:402: + if (!skb) { + bt_dev_err(hdev, "Failed to allocate devcoredump init"); WARNING:OOM_MESSAGE: Possible unnecessary 'out of memory' message #734: FILE: net/bluetooth/coredump.c:445: + if (!skb) { + bt_dev_err(hdev, "Failed to allocate devcoredump pattern"); WARNING:OOM_MESSAGE: Possible unnecessary 'out of memory' message #760: FILE: net/bluetooth/coredump.c:471: + if (!skb) { + bt_dev_err(hdev, "Failed to allocate devcoredump complete"); WARNING:OOM_MESSAGE: Possible unnecessary 'out of memory' message #782: FILE: net/bluetooth/coredump.c:493: + if (!skb) { + bt_dev_err(hdev, "Failed to allocate devcoredump abort"); total: 0 errors, 9 warnings, 0 checks, 670 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/12910303.patch has style problems, please review. NOTE: Ignored message types: UNKNOWN_COMMIT_ID NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS. [v3,2/3] Bluetooth: Add sysfs entry to enable/disable devcoredump\WARNING:CONSIDER_KSTRTO: simple_strtol is obsolete, use kstrtol instead #131: FILE: net/bluetooth/hci_sysfs.c:111: + if (simple_strtol(buf, NULL, 10)) total: 0 errors, 1 warnings, 0 checks, 45 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/12910304.patch has style problems, please review. NOTE: Ignored message types: UNKNOWN_COMMIT_ID NOTE: If any of the errors are false positives, please report them to the maintainer, see CHECKPATCH in MAINTAINERS. --- Regards, Linux Bluetooth