Received-SPF: pass (google.com: domain of linux-bluetooth+bounces-1458-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Message-ID: <65b7e006.050a0220.daa84.c904@mx.google.com>
Date: Mon, 29 Jan 2024 09:27:34 -0800 (PST)
Content-Type: multipart/mixed; boundary="===============6201593272732295382=="
Precedence: bulk
MIME-Version: 1.0
From: bluez.test.bot@gmail.com
To: linux-bluetooth@vger.kernel.org, devnull+emil.l.velikov.gmail.com@kernel.org
Subject: RE: Remove support for external plugins
In-Reply-To: <20240129-rm-ext-plugins-v4-1-bfd1e08c7f99@gmail.com>
References: <20240129-rm-ext-plugins-v4-1-bfd1e08c7f99@gmail.com>
Reply-To: linux-bluetooth@vger.kernel.org

--===============6201593272732295382==
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=820935

---Test result---

Test Summary:
CheckPatch                    FAIL      3.64 seconds
GitLint                       PASS      2.33 seconds
BuildEll                      PASS      24.08 seconds
BluezMake                     PASS      732.29 seconds
MakeCheck                     PASS      13.76 seconds
MakeDistcheck                 PASS      164.26 seconds
CheckValgrind                 PASS      226.32 seconds
CheckSmatch                   PASS      329.54 seconds
bluezmakeextell               PASS      107.29 seconds
IncrementalBuild              PASS      5502.72 seconds
ScanBuild                     PASS      928.77 seconds

Details
##############################
Test: CheckPatch - FAIL
Desc: Run checkpatch.pl script
Output:
[BlueZ,v4,2/8] obexd: factor out external plugin support
WARNING:IS_ENABLED_CONFIG: IS_ENABLED(x) is normally used as IS_ENABLED(CONFIG_x)
#127: FILE: obexd/src/plugin.c:37:
+#define IS_ENABLED(x) (x)

WARNING:IS_ENABLED_CONFIG: IS_ENABLED(EXTERNAL_PLUGINS) is normally used as IS_ENABLED(CONFIG_EXTERNAL_PLUGINS)
#253: FILE: obexd/src/plugin.c:200:
+	if IS_ENABLED(EXTERNAL_PLUGINS)

/github/workspace/src/src/13535810.patch total: 0 errors, 2 warnings, 166 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/src/13535810.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.


[BlueZ,v4,5/8] bluetoothd: factor out external plugin support
WARNING:IS_ENABLED_CONFIG: IS_ENABLED(x) is normally used as IS_ENABLED(CONFIG_x)
#127: FILE: src/plugin.c:29:
+#define IS_ENABLED(x) (x)

WARNING:BLOCK_COMMENT_STYLE: Block comments use a trailing */ on a separate line
#239: FILE: src/plugin.c:186:
+	 * plugins are loaded. */

WARNING:IS_ENABLED_CONFIG: IS_ENABLED(EXTERNAL_PLUGINS) is normally used as IS_ENABLED(CONFIG_EXTERNAL_PLUGINS)
#258: FILE: src/plugin.c:205:
+	if IS_ENABLED(EXTERNAL_PLUGINS)

/github/workspace/src/src/13535813.patch total: 0 errors, 3 warnings, 178 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/src/13535813.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.


[BlueZ,v4,7/8] bluetoothd: change plugin loading alike obexd
WARNING:ENOSYS: ENOSYS means 'invalid syscall nr' and nothing else
#108: FILE: src/plugin.c:52:
+		if (err == -ENOSYS || err == -ENOTSUP)

/github/workspace/src/src/13535815.patch total: 0 errors, 1 warnings, 106 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/src/13535815.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.


---
Regards,
Linux Bluetooth


--===============6201593272732295382==--