Received-SPF: pass (google.com: domain of linux-bluetooth-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <626a99c4.1c69fb81.49cd9.91dd@mx.google.com>
Date:   Thu, 28 Apr 2022 06:42:28 -0700 (PDT)
Content-Type: multipart/mixed; boundary="===============1158593018637342915=="
MIME-Version: 1.0
From:   bluez.test.bot@gmail.com
To:     linux-bluetooth@vger.kernel.org, tangmeng@uniontech.com
Subject: RE: Bluetooth: Add bluetooth error information for error codes
Reply-To: linux-bluetooth@vger.kernel.org
In-Reply-To: <20220428130435.896-1-tangmeng@uniontech.com>
References: <20220428130435.896-1-tangmeng@uniontech.com>
Precedence: bulk

--===============1158593018637342915==
Content-Type: text/plain; charset="us-ascii"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit

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=636605

---Test result---

Test Summary:
CheckPatch                    FAIL      1.81 seconds
GitLint                       PASS      1.05 seconds
SubjectPrefix                 PASS      0.93 seconds
BuildKernel                   PASS      36.51 seconds
BuildKernel32                 PASS      32.21 seconds
Incremental Build with patchesPASS      43.91 seconds
TestRunner: Setup             PASS      545.79 seconds
TestRunner: l2cap-tester      PASS      19.20 seconds
TestRunner: bnep-tester       PASS      7.15 seconds
TestRunner: mgmt-tester       PASS      114.41 seconds
TestRunner: rfcomm-tester     PASS      10.85 seconds
TestRunner: sco-tester        PASS      10.47 seconds
TestRunner: smp-tester        PASS      10.58 seconds
TestRunner: userchan-tester   PASS      7.25 seconds

Details
##############################
Test: CheckPatch - FAIL - 1.81 seconds
Run checkpatch.pl script with rule in .checkpatch.conf
Bluetooth: Add bluetooth error information for error codes\WARNING:PREFER_PR_LEVEL: Prefer [subsystem eg: netdev]_err([subsystem]dev, ... then dev_err(dev, ... then pr_err(...  to printk(KERN_ERR ...
#91: FILE: net/bluetooth/lib.c:126:
+		printk(KERN_ERR "Bluetooth: errno(0x%02x), SCO Interval Rejected", code);

WARNING:ENOSYS: ENOSYS means 'invalid syscall nr' and nothing else
#92: FILE: net/bluetooth/lib.c:127:
+		return ENOSYS;

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOSYS)
#92: FILE: net/bluetooth/lib.c:127:
+		return ENOSYS;

WARNING:PREFER_PR_LEVEL: Prefer [subsystem eg: netdev]_err([subsystem]dev, ... then dev_err(dev, ... then pr_err(...  to printk(KERN_ERR ...
#95: FILE: net/bluetooth/lib.c:130:
+		printk(KERN_ERR "Bluetooth: errno(0x%02x), SCO Air Mode Rejected", code);

WARNING:ENOSYS: ENOSYS means 'invalid syscall nr' and nothing else
#96: FILE: net/bluetooth/lib.c:131:
+		return ENOSYS;

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOSYS)
#96: FILE: net/bluetooth/lib.c:131:
+		return ENOSYS;

WARNING:PREFER_PR_LEVEL: Prefer [subsystem eg: netdev]_err([subsystem]dev, ... then dev_err(dev, ... then pr_err(...  to printk(KERN_ERR ...
#106: FILE: net/bluetooth/lib.c:141:
+		printk(KERN_ERR "Bluetooth: errno(0x%02x), Unspecified Error", code);

WARNING:ENOSYS: ENOSYS means 'invalid syscall nr' and nothing else
#107: FILE: net/bluetooth/lib.c:142:
+		return ENOSYS;

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOSYS)
#107: FILE: net/bluetooth/lib.c:142:
+		return ENOSYS;

WARNING:PREFER_PR_LEVEL: Prefer [subsystem eg: netdev]_err([subsystem]dev, ... then dev_err(dev, ... then pr_err(...  to printk(KERN_ERR ...
#110: FILE: net/bluetooth/lib.c:145:
+		printk(KERN_ERR "Bluetooth: errno(0x%02x), Role Change Not Allowed", code);

WARNING:ENOSYS: ENOSYS means 'invalid syscall nr' and nothing else
#111: FILE: net/bluetooth/lib.c:146:
+		return ENOSYS;

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOSYS)
#111: FILE: net/bluetooth/lib.c:146:
+		return ENOSYS;

WARNING:PREFER_PR_LEVEL: Prefer [subsystem eg: netdev]_err([subsystem]dev, ... then dev_err(dev, ... then pr_err(...  to printk(KERN_ERR ...
#114: FILE: net/bluetooth/lib.c:149:
+		printk(KERN_ERR "Bluetooth: errno(0x%02x), LMP Response Timeout", code);

WARNING:ENOSYS: ENOSYS means 'invalid syscall nr' and nothing else
#115: FILE: net/bluetooth/lib.c:150:
+		return ENOSYS;

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOSYS)
#115: FILE: net/bluetooth/lib.c:150:
+		return ENOSYS;

WARNING:PREFER_PR_LEVEL: Prefer [subsystem eg: netdev]_err([subsystem]dev, ... then dev_err(dev, ... then pr_err(...  to printk(KERN_ERR ...
#118: FILE: net/bluetooth/lib.c:153:
+		printk(KERN_ERR "Bluetooth: errno(0x%02x), Unit Key Used", code);

WARNING:ENOSYS: ENOSYS means 'invalid syscall nr' and nothing else
#119: FILE: net/bluetooth/lib.c:154:
+		return ENOSYS;

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOSYS)
#119: FILE: net/bluetooth/lib.c:154:
+		return ENOSYS;

WARNING:PREFER_PR_LEVEL: Prefer [subsystem eg: netdev]_err([subsystem]dev, ... then dev_err(dev, ... then pr_err(...  to printk(KERN_ERR ...
#122: FILE: net/bluetooth/lib.c:157:
+		printk(KERN_ERR "Bluetooth: errno(0x%02x), Instant Passed", code);

WARNING:ENOSYS: ENOSYS means 'invalid syscall nr' and nothing else
#123: FILE: net/bluetooth/lib.c:158:
+		return ENOSYS;

WARNING:USE_NEGATIVE_ERRNO: return of an errno should typically be negative (ie: return -ENOSYS)
#123: FILE: net/bluetooth/lib.c:158:
+		return ENOSYS;

WARNING:PREFER_PR_LEVEL: Prefer [subsystem eg: netdev]_err([subsystem]dev, ... then dev_err(dev, ... then pr_err(...  to printk(KERN_ERR ...
#126: FILE: net/bluetooth/lib.c:161:
+		printk(KERN_ERR "Bluetooth: errno(0x%02x), Error code unknown", code);

total: 0 errors, 22 warnings, 0 checks, 42 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/12830665.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


--===============1158593018637342915==--