MIME-Version: 1.0
References: <20230126074356.431306-2-francesco@dolcini.it> <63d23c50.170a0220.124e7.0ce5@mx.google.com>
 <Y9KYpw+V1e7wLiT/@francesco-nb.int.toradex.com>
In-Reply-To: <Y9KYpw+V1e7wLiT/@francesco-nb.int.toradex.com>
From:   Luiz Augusto von Dentz <luiz.dentz@gmail.com>
Date:   Thu, 26 Jan 2023 12:59:36 -0800
Message-ID: <CABBYNZKsD1FAXz1y+8=ogWLmrN0OhctcoRJJXEz=dorv8H8nfQ@mail.gmail.com>
Subject: Re: Bluetooth: hci_mrvl: Add serdev support for 88W8997
To:     Francesco Dolcini <francesco@dolcini.it>
Cc:     linux-bluetooth@vger.kernel.org
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Precedence: bulk

Hi Francesco,

On Thu, Jan 26, 2023 at 7:15 AM Francesco Dolcini <francesco@dolcini.it> wr=
ote:
>
> On Thu, Jan 26, 2023 at 12:39:44AM -0800, bluez.test.bot@gmail.com wrote:
> > Thank you for submitting the patches to the linux bluetooth mailing lis=
t.
> > This is a CI test results with your patch series:
> > PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=3D7=
15765
> [snip]
> > Test: SubjectPrefix - FAIL
> > Desc: Check subject contains "Bluetooth" prefix
> > Output:
> > "Bluetooth: " prefix is not specified in the subject
> > "Bluetooth: " prefix is not specified in the subject
> > "Bluetooth: " prefix is not specified in the subject
>
> These warning are bogus, the reason is that the series include a couple
> dts/binding changes that have a different rule on the commit message
> prefix.

Or CI only expects patches to Bluetooth subsystem or drivers, so
perhaps the dt-bindings changes shouldn't actually be send here if
there are not meant to be applied to bluetooth-next? Or we need some
sort of smart filtering so PW don't assume everything sent to
linux-bluetooth shall be tested by our CI.

> > ##############################
> > Test: CheckSparse - WARNING
> > Desc: Run sparse tool with linux kernel
> > Output:
> > drivers/bluetooth/hci_mrvl.c:170:23: warning: restricted __le16 degrade=
s to integerdrivers/bluetooth/hci_mrvl.c:203:23: warning: restricted __le16=
 degrades to integerdrivers/bluetooth/hci_mrvl.c:170:23: warning: restricte=
d __le16 degrades to integerdrivers/bluetooth/hci_mrvl.c:203:23: warning: r=
estricted __le16 degrades to integer
> This sparse warning was not introduced by this series.

Yep, that is why they are treated as warnings not errors, still it
would be a good idea to fix those to avoid it popping up again.

> Francesco
>


--=20
Luiz Augusto von Dentz