2021-05-19 18:32:09

by bluez.test.bot

[permalink] [raw]
Subject: RE: Bluetooth: Add a new QCA_ROME device (0cf3:e500)

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

---Test result---

Test Summary:
CheckPatch FAIL 0.84 seconds
GitLint PASS 0.13 seconds
BuildKernel PASS 589.44 seconds
TestRunner: Setup PASS 390.88 seconds
TestRunner: l2cap-tester PASS 2.83 seconds
TestRunner: bnep-tester PASS 2.04 seconds
TestRunner: mgmt-tester PASS 28.94 seconds
TestRunner: rfcomm-tester PASS 2.26 seconds
TestRunner: sco-tester PASS 2.14 seconds
TestRunner: smp-tester PASS 2.27 seconds
TestRunner: userchan-tester PASS 2.00 seconds

Details
##############################
Test: CheckPatch - FAIL - 0.84 seconds
Run checkpatch.pl script with rule in .checkpatch.conf
Bluetooth: Add a new QCA_ROME device (0cf3:e500)
WARNING: Unknown commit id '0a03f98b98c2', maybe rebased or not pulled?
#9:
commit 0a03f98b98c2 ("Bluetooth: Add a new 04ca:3015 QCA_ROME device").

total: 0 errors, 1 warnings, 8 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.

"[PATCH] Bluetooth: Add a new QCA_ROME device (0cf3:e500)" has style problems, please review.

NOTE: If any of the errors are false positives, please report
them to the maintainer, see CHECKPATCH in MAINTAINERS.


##############################
Test: GitLint - PASS - 0.13 seconds
Run gitlint with rule in .gitlint


##############################
Test: BuildKernel - PASS - 589.44 seconds
Build Kernel with minimal configuration supports Bluetooth


##############################
Test: TestRunner: Setup - PASS - 390.88 seconds
Setup environment for running Test Runner


##############################
Test: TestRunner: l2cap-tester - PASS - 2.83 seconds
Run test-runner with l2cap-tester
Total: 40, Passed: 40 (100.0%), Failed: 0, Not Run: 0

##############################
Test: TestRunner: bnep-tester - PASS - 2.04 seconds
Run test-runner with bnep-tester
Total: 1, Passed: 1 (100.0%), Failed: 0, Not Run: 0

##############################
Test: TestRunner: mgmt-tester - PASS - 28.94 seconds
Run test-runner with mgmt-tester
Total: 416, Passed: 403 (96.9%), Failed: 0, Not Run: 13

##############################
Test: TestRunner: rfcomm-tester - PASS - 2.26 seconds
Run test-runner with rfcomm-tester
Total: 9, Passed: 9 (100.0%), Failed: 0, Not Run: 0

##############################
Test: TestRunner: sco-tester - PASS - 2.14 seconds
Run test-runner with sco-tester
Total: 8, Passed: 8 (100.0%), Failed: 0, Not Run: 0

##############################
Test: TestRunner: smp-tester - PASS - 2.27 seconds
Run test-runner with smp-tester
Total: 8, Passed: 8 (100.0%), Failed: 0, Not Run: 0

##############################
Test: TestRunner: userchan-tester - PASS - 2.00 seconds
Run test-runner with userchan-tester
Total: 3, Passed: 3 (100.0%), Failed: 0, Not Run: 0



---
Regards,
Linux Bluetooth


Attachments:
l2cap-tester.log (43.19 kB)
bnep-tester.log (3.47 kB)
mgmt-tester.log (533.93 kB)
rfcomm-tester.log (11.40 kB)
sco-tester.log (9.68 kB)
smp-tester.log (11.55 kB)
userchan-tester.log (5.33 kB)
Download all attachments

2021-05-19 18:38:35

by Daniel Lenski

[permalink] [raw]
Subject: Re: Bluetooth: Add a new QCA_ROME device (0cf3:e500)

Hi all,
I'm somewhat confused about why CheckPatch is failing to approve this
patch. It says that a commit referenced in the commit message does not
exist:

On Tue, May 18, 2021 at 12:31 PM <[email protected]> wrote:
> Test: CheckPatch - FAIL - 0.84 seconds
> Run checkpatch.pl script with rule in .checkpatch.conf
> Bluetooth: Add a new QCA_ROME device (0cf3:e500)
> WARNING: Unknown commit id '0a03f98b98c2', maybe rebased or not pulled?
> #9:
> commit 0a03f98b98c2 ("Bluetooth: Add a new 04ca:3015 QCA_ROME device").

However, this commit *does* exist, and it is — as far as I can tell —
the *only* commit which exists with this text in its title.

https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=0a03f98b98c201191e3ba15a0e33f46d8660e1fd
https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/log/?qt=grep&q=Add+a+new+04ca%3A3015+QCA_ROME+device

Any suggestions about how to resolve this?

-Dan Lenski

2021-05-19 18:40:12

by Tedd Ho-Jeong An

[permalink] [raw]
Subject: Re: Bluetooth: Add a new QCA_ROME device (0cf3:e500)

Hi Dan

On Tue, 2021-05-18 at 14:41 -0700, Daniel Lenski wrote:
> Hi all,
> I'm somewhat confused about why CheckPatch is failing to approve this
> patch. It says that a commit referenced in the commit message does not
> exist:
>
> On Tue, May 18, 2021 at 12:31 PM <[email protected]> wrote:
> > Test: CheckPatch - FAIL - 0.84 seconds
> > Run checkpatch.pl script with rule in .checkpatch.conf
> > Bluetooth: Add a new QCA_ROME device (0cf3:e500)
> > WARNING: Unknown commit id '0a03f98b98c2', maybe rebased or not pulled?
> > #9:
> > commit 0a03f98b98c2 ("Bluetooth: Add a new 04ca:3015 QCA_ROME device").
>
> However, this commit *does* exist, and it is — as far as I can tell —
> the *only* commit which exists with this text in its title.
>
> https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=0a03f98b98c201191e3ba15a0e33f46d8660e1fd
> https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/log/?qt=grep&q=Add+a+new+04ca%3A3015+QCA_ROME+device
>
> Any suggestions about how to resolve this?

You can ignore this error.
This is an issue in the CI which is not pulling the entire kernel tree to
reduce the running time.

>
> -Dan Lenski

Regards,
Tedd