Received-SPF: softfail (google.com: domain of transitioning linux-bluetooth-owner@vger.kernel.org does not designate 23.128.96.19 as permitted sender) client-ip=23.128.96.19;
Message-ID: <6259192b.1c69fb81.bf4ba.d8e9@mx.google.com>
Date:   Fri, 15 Apr 2022 00:05:15 -0700 (PDT)
Content-Type: multipart/mixed; boundary="===============7603793001052204614=="
MIME-Version: 1.0
From:   bluez.test.bot@gmail.com
To:     linux-bluetooth@vger.kernel.org, luiz.dentz@gmail.com
Subject: RE: [v2,BlueZ,1/2] client/player: Add endpoint menu
Reply-To: linux-bluetooth@vger.kernel.org
In-Reply-To: <20220413211630.171843-1-luiz.dentz@gmail.com>
References: <20220413211630.171843-1-luiz.dentz@gmail.com>
Precedence: bulk

--===============7603793001052204614==
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=632031

---Test result---

Test Summary:
CheckPatch                    FAIL      2.97 seconds
GitLint                       PASS      0.91 seconds
Prep - Setup ELL              PASS      41.43 seconds
Build - Prep                  PASS      0.45 seconds
Build - Configure             PASS      8.12 seconds
Build - Make                  PASS      1247.20 seconds
Make Check                    PASS      11.23 seconds
Make Check w/Valgrind         PASS      435.67 seconds
Make Distcheck                PASS      223.18 seconds
Build w/ext ELL - Configure   PASS      8.22 seconds
Build w/ext ELL - Make        PASS      1229.19 seconds
Incremental Build with patchesPASS      2507.39 seconds

Details
##############################
Test: CheckPatch - FAIL
Desc: Run checkpatch.pl script with rule in .checkpatch.conf
Output:
[v2,BlueZ,1/2] client/player: Add endpoint menu
ERROR:INITIALISED_STATIC: do not initialise statics to NULL
#152: FILE: client/player.c:72:
+static GList *medias = NULL;

ERROR:INITIALISED_STATIC: do not initialise statics to NULL
#156: FILE: client/player.c:76:
+static GList *endpoints = NULL;

ERROR:INITIALISED_STATIC: do not initialise statics to NULL
#157: FILE: client/player.c:77:
+static GList *local_endpoints = NULL;

ERROR:INITIALISED_STATIC: do not initialise statics to 0
#251: FILE: client/player.c:1020:
+	static int index = 0;

WARNING:LINE_SPACING: Missing a blank line after declarations
#303: FILE: client/player.c:1072:
+		GDBusProxy *proxy = l->data;
+		print_endpoint(proxy, NULL);

ERROR:SPACING: space prohibited before that close parenthesis ')'
#687: FILE: client/player.c:1456:
+						{ "properties", "a{sv}" } ),

ERROR:SPACING: space prohibited before that close parenthesis ')'
#690: FILE: client/player.c:1459:
+					GDBUS_ARGS({ "caps", "ay" } ),

ERROR:SPACING: space prohibited before that close parenthesis ')'
#693: FILE: client/player.c:1462:
+					GDBUS_ARGS({ "transport", "o" } ),

ERROR:INITIALISED_STATIC: do not initialise statics to 0
#870: FILE: client/player.c:1639:
+	static int index = 0;

/github/workspace/src/12812703.patch total: 8 errors, 1 warnings, 1314 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/12812703.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.

[v2,BlueZ,2/2] client/player: Add transport menu
ERROR:INITIALISED_STATIC: do not initialise statics to NULL
#113: FILE: client/player.c:79:
+static GList *transports = NULL;

WARNING:LINE_SPACING: Missing a blank line after declarations
#350: FILE: client/player.c:2462:
+		GDBusProxy *proxy = l->data;
+		print_transport(proxy, NULL);

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


--===============7603793001052204614==--