Hi Sven, On Fri, Nov 4, 2022 at 3:15 PM <bluez.test.bot@xxxxxxxxx> wrote: > > 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=692274 > > ---Test result--- > > Test Summary: > CheckPatch FAIL 7.46 seconds > GitLint FAIL 2.01 seconds > SubjectPrefix FAIL 2.61 seconds > BuildKernel PASS 34.62 seconds > BuildKernel32 PASS 31.22 seconds > Incremental Build with patchesPASS 151.12 seconds > TestRunner: Setup PASS 513.16 seconds > TestRunner: l2cap-tester PASS 17.36 seconds > TestRunner: iso-tester PASS 16.95 seconds > TestRunner: bnep-tester PASS 6.66 seconds > TestRunner: mgmt-tester PASS 107.82 seconds > TestRunner: rfcomm-tester PASS 10.55 seconds > TestRunner: sco-tester PASS 9.93 seconds > TestRunner: ioctl-tester PASS 11.21 seconds > TestRunner: mesh-tester PASS 8.08 seconds > TestRunner: smp-tester PASS 9.90 seconds > TestRunner: userchan-tester PASS 6.88 seconds > > Details > ############################## > Test: CheckPatch - FAIL - 7.46 seconds > Run checkpatch.pl script with rule in .checkpatch.conf > [v5,1/7] dt-bindings: net: Add generic Bluetooth controller\WARNING:FILE_PATH_CHANGES: added, moved or deleted file(s), does MAINTAINERS need updating? > #115: > .../{ => bluetooth}/qualcomm-bluetooth.yaml | 6 ++-- > > WARNING:DT_SPLIT_BINDING_PATCH: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst > > WARNING:DT_SPLIT_BINDING_PATCH: DT binding docs and includes should be a separate patch. See: Documentation/devicetree/bindings/submitting-patches.rst > > total: 0 errors, 3 warnings, 71 lines checked The other errors you can probably ignore but this one above got my attention, it seems we are doing the right by having the documentation changes as a separate patch but checkpatch is still complaining? > > 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/13032592.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. > > [v5,3/7] arm64: dts: apple: t8103: Add Bluetooth controller\WARNING:UNDOCUMENTED_DT_STRING: DT compatible string vendor "pci14e4" appears un-documented -- check ./Documentation/devicetree/bindings/vendor-prefixes.yaml > #210: FILE: arch/arm64/boot/dts/apple/t8103-jxxx.dtsi:83: > + compatible = "pci14e4,5f69"; > > total: 0 errors, 1 warnings, 68 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/13032594.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. > > > ############################## > Test: GitLint - FAIL - 2.01 seconds > Run gitlint with rule in .gitlint > [v5,1/7] dt-bindings: net: Add generic Bluetooth controller > 15: B1 Line exceeds max length (93>80): " create mode 100644 Documentation/devicetree/bindings/net/bluetooth/bluetooth-controller.yaml" > 16: B1 Line exceeds max length (91>80): " rename Documentation/devicetree/bindings/net/{ => bluetooth}/qualcomm-bluetooth.yaml (96%)" > > [v5,2/7] dt-bindings: net: Add Broadcom BCM4377 family PCIe Bluetooth > 14: B1 Line exceeds max length (95>80): " create mode 100644 Documentation/devicetree/bindings/net/bluetooth/brcm,bcm4377-bluetooth.yaml" > > > ############################## > Test: SubjectPrefix - FAIL - 2.61 seconds > Check subject contains "Bluetooth" prefix > "Bluetooth: " is not specified in the subject > "Bluetooth: " is not specified in the subject > "Bluetooth: " is not specified in the subject > > > > --- > Regards, > Linux Bluetooth > -- Luiz Augusto von Dentz
