From: bluez.test.bot@gmail.com
To: linux-bluetooth@vger.kernel.org, hj.tedd.an@gmail.com
Subject: RE: [RFC,BlueZ,v14,1/5] emulator: Add support to config the accept and resolve list
Date: Fri, 22 Oct 2021 14:34:13 -0700 (PDT) [thread overview]
Message-ID: <61732e55.1c69fb81.68d6d.b06d@mx.google.com> (raw)
In-Reply-To: <20211022204825.397952-1-hj.tedd.an@gmail.com>
[-- Attachment #1: Type: text/plain, Size: 4937 bytes --]
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=568805
---Test result---
Test Summary:
CheckPatch FAIL 9.36 seconds
GitLint FAIL 4.99 seconds
Prep - Setup ELL PASS 46.47 seconds
Build - Prep PASS 0.50 seconds
Build - Configure PASS 8.67 seconds
Build - Make PASS 199.78 seconds
Make Check PASS 9.67 seconds
Make Distcheck PASS 237.61 seconds
Build w/ext ELL - Configure PASS 8.56 seconds
Build w/ext ELL - Make PASS 191.12 seconds
Details
##############################
Test: CheckPatch - FAIL
Desc: Run checkpatch.pl script with rule in .checkpatch.conf
Output:
[RFC,BlueZ,v14,5/5] tools/mgmt-tester: Add LL Privacy test cases
ERROR:SPACING: space prohibited before that ',' (ctx:WxW)
#756: FILE: tools/mgmt-tester.c:10115:
+ MGMT_OP_SET_EXP_FEATURE , 0 };
^
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#1262: FILE: tools/mgmt-tester.c:10620:
+// ^Istruct test_data *data = tester_get_data();$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#1263: FILE: tools/mgmt-tester.c:10621:
+// ^Iunsigned char param[] = { 0x01 };$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#1264: FILE: tools/mgmt-tester.c:10622:
+// ^Iconst uint8_t *ext_feat_param;$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#1265: FILE: tools/mgmt-tester.c:10623:
+// ^Isize_t ext_feat_len;$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#1266: FILE: tools/mgmt-tester.c:10624:
+// ^Iunsigned char privacy_param[] = { 0x01,$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#1267: FILE: tools/mgmt-tester.c:10625:
+// ^I^I^I0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07, 0x08,$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#1268: FILE: tools/mgmt-tester.c:10626:
+// ^I^I^I0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07, 0x08 };$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#1270: FILE: tools/mgmt-tester.c:10628:
+// ^Itester_print("Enabling LL Privacy feature");$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#1272: FILE: tools/mgmt-tester.c:10630:
+// ^Iext_feat_param = set_exp_feat_param_ll_privacy;$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#1273: FILE: tools/mgmt-tester.c:10631:
+// ^Iext_feat_len = sizeof(set_exp_feat_param_ll_privacy);$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#1275: FILE: tools/mgmt-tester.c:10633:
+// ^Imgmt_send(data->mgmt, MGMT_OP_SET_LE, data->mgmt_index,$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#1276: FILE: tools/mgmt-tester.c:10634:
+// ^I^I^I^I^Isizeof(param), ¶m,$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#1277: FILE: tools/mgmt-tester.c:10635:
+// ^I^I^I^I^INULL, NULL, NULL);$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#1279: FILE: tools/mgmt-tester.c:10637:
+// ^Imgmt_send(data->mgmt, MGMT_OP_SET_PRIVACY, data->mgmt_index,$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#1280: FILE: tools/mgmt-tester.c:10638:
+// ^I^I^I^I^Isizeof(privacy_param), privacy_param,$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#1281: FILE: tools/mgmt-tester.c:10639:
+// ^I^I^I^I^INULL, NULL, NULL);$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#1283: FILE: tools/mgmt-tester.c:10641:
+// ^Imgmt_send(data->mgmt, MGMT_OP_SET_EXP_FEATURE, data->mgmt_index,$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#1284: FILE: tools/mgmt-tester.c:10642:
+// ^I^I^I^I^Iext_feat_len, ext_feat_param,$
WARNING:SPACE_BEFORE_TAB: please, no space before tabs
#1285: FILE: tools/mgmt-tester.c:10643:
+// ^I^I^I^I^Isetup_exp_feat_callback, NULL, NULL);$
/github/workspace/src/12579045.patch total: 1 errors, 19 warnings, 2340 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/12579045.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.
##############################
Test: GitLint - FAIL
Desc: Run gitlint with rule in .gitlint
Output:
[RFC,BlueZ,v14,4/5] tools/mgmt-tester: Add support for experimental feature in setup
1: T1 Title exceeds max length (84>80): "[RFC,BlueZ,v14,4/5] tools/mgmt-tester: Add support for experimental feature in setup"
---
Regards,
Linux Bluetooth
prev parent reply other threads:[~2021-10-22 21:34 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-10-22 20:48 [RFC BlueZ PATCH v14 1/5] emulator: Add support to config the accept and resolve list Tedd Ho-Jeong An
2021-10-22 20:48 ` [RFC BlueZ PATCH v14 2/5] emulator: bthost: Add support LE Ext Adv Report Tedd Ho-Jeong An
2021-10-22 20:48 ` [RFC BlueZ PATCH v14 3/5] emulator: Add support to get the advertising address Tedd Ho-Jeong An
2021-10-22 20:48 ` [RFC BlueZ PATCH v14 4/5] tools/mgmt-tester: Add support for experimental feature in setup Tedd Ho-Jeong An
2021-10-22 20:48 ` [RFC BlueZ PATCH v14 5/5] tools/mgmt-tester: Add LL Privacy test cases Tedd Ho-Jeong An
2021-10-22 21:34 ` bluez.test.bot [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=61732e55.1c69fb81.68d6d.b06d@mx.google.com \
--to=bluez.test.bot@gmail.com \
--cc=hj.tedd.an@gmail.com \
--cc=linux-bluetooth@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).