[PATCH 0/5] Wifi & Bluetooth on LG G Watch R

[PATCH 0/5] Wifi & Bluetooth on LG G Watch R

[Luca Weiss]

This series adds the BCM43430A0 chip providing Bluetooth & Wifi on the
LG G Watch R.

Luca Weiss (5):
  dt-bindings: bluetooth: broadcom: add BCM43430A0
  Bluetooth: hci_bcm: add BCM43430A0
  ARM: dts: qcom: msm8226: Add pinctrl for sdhci nodes
  ARM: dts: qcom: apq8026-lg-lenok: Add Wifi
  ARM: dts: qcom: apq8026-lg-lenok: Add Bluetooth

 .../bindings/net/broadcom-bluetooth.yaml      |  1 +
 arch/arm/boot/dts/qcom-apq8026-lg-lenok.dts   | 98 ++++++++++++++++---
 arch/arm/boot/dts/qcom-msm8226.dtsi           | 57 +++++++++++
 drivers/bluetooth/hci_bcm.c                   |  1 +
 4 files changed, 144 insertions(+), 13 deletions(-)

-- 
2.35.1

[PATCH 2/5] Bluetooth: hci_bcm: add BCM43430A0

[Luca Weiss]

Add the compatible for BCM43430A0 bluetooth to the driver.

Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
---
 drivers/bluetooth/hci_bcm.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/drivers/bluetooth/hci_bcm.c b/drivers/bluetooth/hci_bcm.c
index d634a27bc850..e19c7a41a4f6 100644
--- a/drivers/bluetooth/hci_bcm.c
+++ b/drivers/bluetooth/hci_bcm.c
@@ -1513,6 +1513,7 @@ static const struct of_device_id bcm_bluetooth_of_match[] = {
 	{ .compatible = "brcm,bcm4330-bt" },
 	{ .compatible = "brcm,bcm4334-bt" },
 	{ .compatible = "brcm,bcm4345c5" },
+	{ .compatible = "brcm,bcm43430a0-bt" },
 	{ .compatible = "brcm,bcm43438-bt", .data = &bcm43438_device_data },
 	{ .compatible = "brcm,bcm43540-bt", .data = &bcm4354_device_data },
 	{ .compatible = "brcm,bcm4335a0" },
-- 
2.35.1

RE: Wifi & Bluetooth on LG G Watch R

[bluez.test.bot]

[-- Attachment #1: Type: text/plain, Size: 1938 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=615144

---Test result---

Test Summary:
CheckPatch                    FAIL      1.78 seconds
GitLint                       PASS      1.02 seconds
SubjectPrefix                 PASS      0.87 seconds
BuildKernel                   PASS      37.42 seconds
BuildKernel32                 PASS      32.54 seconds
Incremental Build with patchesPASS      44.92 seconds
TestRunner: Setup             PASS      579.45 seconds
TestRunner: l2cap-tester      PASS      16.06 seconds
TestRunner: bnep-tester       PASS      7.36 seconds
TestRunner: mgmt-tester       PASS      121.35 seconds
TestRunner: rfcomm-tester     PASS      9.99 seconds
TestRunner: sco-tester        PASS      9.33 seconds
TestRunner: smp-tester        PASS      9.44 seconds
TestRunner: userchan-tester   PASS      7.63 seconds

Details
##############################
Test: CheckPatch - FAIL - 1.78 seconds
Run checkpatch.pl script with rule in .checkpatch.conf
[2/5] Bluetooth: hci_bcm: add BCM43430A0\WARNING:UNDOCUMENTED_DT_STRING: DT compatible string "brcm,bcm43430a0-bt" appears un-documented -- check ./Documentation/devicetree/bindings/
#71: FILE: drivers/bluetooth/hci_bcm.c:1516:
+	{ .compatible = "brcm,bcm43430a0-bt" },

total: 0 errors, 1 warnings, 7 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/12749111.patch has style problems, please review.

NOTE: Ignored message types: UNKNOWN_COMMIT_ID

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




---
Regards,
Linux Bluetooth

Re: [PATCH 0/5] Wifi & Bluetooth on LG G Watch R

[Bjorn Andersson]

On Wed 16 Feb 15:24 CST 2022, Luca Weiss wrote:

> This series adds the BCM43430A0 chip providing Bluetooth & Wifi on the
> LG G Watch R.
> 

I picked the dts changes, but would prefer that the other two changes
goes through the BT tree. I see that you haven't copied Marcel on the
dt-binding change though, so please resubmit those two patches together.

Thanks,
Bjorn

> Luca Weiss (5):
>   dt-bindings: bluetooth: broadcom: add BCM43430A0
>   Bluetooth: hci_bcm: add BCM43430A0
>   ARM: dts: qcom: msm8226: Add pinctrl for sdhci nodes
>   ARM: dts: qcom: apq8026-lg-lenok: Add Wifi
>   ARM: dts: qcom: apq8026-lg-lenok: Add Bluetooth
> 
>  .../bindings/net/broadcom-bluetooth.yaml      |  1 +
>  arch/arm/boot/dts/qcom-apq8026-lg-lenok.dts   | 98 ++++++++++++++++---
>  arch/arm/boot/dts/qcom-msm8226.dtsi           | 57 +++++++++++
>  drivers/bluetooth/hci_bcm.c                   |  1 +
>  4 files changed, 144 insertions(+), 13 deletions(-)
> 
> -- 
> 2.35.1
> 

Re: (subset) [PATCH 0/5] Wifi & Bluetooth on LG G Watch R

[Bjorn Andersson]

On Wed, 16 Feb 2022 22:24:27 +0100, Luca Weiss wrote:
> This series adds the BCM43430A0 chip providing Bluetooth & Wifi on the
> LG G Watch R.
> 
> Luca Weiss (5):
>   dt-bindings: bluetooth: broadcom: add BCM43430A0
>   Bluetooth: hci_bcm: add BCM43430A0
>   ARM: dts: qcom: msm8226: Add pinctrl for sdhci nodes
>   ARM: dts: qcom: apq8026-lg-lenok: Add Wifi
>   ARM: dts: qcom: apq8026-lg-lenok: Add Bluetooth
> 
> [...]

Applied, thanks!

[2/5] Bluetooth: hci_bcm: add BCM43430A0
      (no commit info)
[3/5] ARM: dts: qcom: msm8226: Add pinctrl for sdhci nodes
      commit: a5683471b68d81898db14e1dee347bfe7469540d
[4/5] ARM: dts: qcom: apq8026-lg-lenok: Add Wifi
      commit: 81ecc39d0dd3fbbcc3c8505d0eefe511eb422d03
[5/5] ARM: dts: qcom: apq8026-lg-lenok: Add Bluetooth
      commit: e8880a10f970d86d1b3fdd8bc36400e0e01ba4a9

Best regards,
-- 
Bjorn Andersson <bjorn.andersson@linaro.org>

Re: [PATCH 0/5] Wifi & Bluetooth on LG G Watch R

[Luca Weiss]

Hi Bjorn

On Donnerstag, 24. Februar 2022 05:15:06 CET Bjorn Andersson wrote:
> On Wed 16 Feb 15:24 CST 2022, Luca Weiss wrote:
> > This series adds the BCM43430A0 chip providing Bluetooth & Wifi on the
> > LG G Watch R.
> 
> I picked the dts changes, but would prefer that the other two changes
> goes through the BT tree. I see that you haven't copied Marcel on the
> dt-binding change though, so please resubmit those two patches together.

Thank you, will resubmit the first two!

Just to be clear, as far as I understand each patch gets sent based on its own 
get_maintainer.pl, and the cover letter gets sent to the superset of all 
individual patch recipients?
I'm using this script that's largely based on something I found online a while 
ago
https://github.com/z3ntu/dotfiles/blob/master/scripts/usr/local/bin/cocci_cc

Also just checked and Marcel isn't listed as maintainer of the relevant dt 
bindings in MAINTAINERS, maybe they should get added there?

(also CCed Marcel on this email)

Regards
Luca

> 
> Thanks,
> Bjorn
> 
> > Luca Weiss (5):
> >   dt-bindings: bluetooth: broadcom: add BCM43430A0
> >   Bluetooth: hci_bcm: add BCM43430A0
> >   ARM: dts: qcom: msm8226: Add pinctrl for sdhci nodes
> >   ARM: dts: qcom: apq8026-lg-lenok: Add Wifi
> >   ARM: dts: qcom: apq8026-lg-lenok: Add Bluetooth
> >  
> >  .../bindings/net/broadcom-bluetooth.yaml      |  1 +
> >  arch/arm/boot/dts/qcom-apq8026-lg-lenok.dts   | 98 ++++++++++++++++---
> >  arch/arm/boot/dts/qcom-msm8226.dtsi           | 57 +++++++++++
> >  drivers/bluetooth/hci_bcm.c                   |  1 +
> >  4 files changed, 144 insertions(+), 13 deletions(-)

Re: [PATCH 0/5] Wifi & Bluetooth on LG G Watch R

[Bjorn Andersson]

On Fri 25 Feb 12:19 PST 2022, Luca Weiss wrote:

> Hi Bjorn
> 
> On Donnerstag, 24. Februar 2022 05:15:06 CET Bjorn Andersson wrote:
> > On Wed 16 Feb 15:24 CST 2022, Luca Weiss wrote:
> > > This series adds the BCM43430A0 chip providing Bluetooth & Wifi on the
> > > LG G Watch R.
> > 
> > I picked the dts changes, but would prefer that the other two changes
> > goes through the BT tree. I see that you haven't copied Marcel on the
> > dt-binding change though, so please resubmit those two patches together.
> 
> Thank you, will resubmit the first two!
> 
> Just to be clear, as far as I understand each patch gets sent based on its own 
> get_maintainer.pl, and the cover letter gets sent to the superset of all 
> individual patch recipients?

It's rather annoying to be maintainer and only get 1-2 patches out of a
larger series and having to browse lore.kernel.org to find the rest. So
based on that I tend to make sure that everyone is Cc'ed on all patches
in my series.

And then there's the general advice that if there isn't a strong
dependency between the patches, it might be better to just submit them
separately in the first place. Simply to make it easier for each
maintainer to merge your patches.

> I'm using this script that's largely based on something I found online a while 
> ago
> https://github.com/z3ntu/dotfiles/blob/master/scripts/usr/local/bin/cocci_cc
> 
> Also just checked and Marcel isn't listed as maintainer of the relevant dt 
> bindings in MAINTAINERS, maybe they should get added there?
> 

In general I think the DT bindings goes through the relevant subsystem
maintainer, so I would suggest that you post a patch. If for some reason
Marcel shouldn't merge BT related DT patches, then the discussion that
follows will make that clear and you could make sure that MAINTAINERS
reflects the outcome.

Regards,
Bjorn

> (also CCed Marcel on this email)
> 
> Regards
> Luca
> 
> > 
> > Thanks,
> > Bjorn
> > 
> > > Luca Weiss (5):
> > >   dt-bindings: bluetooth: broadcom: add BCM43430A0
> > >   Bluetooth: hci_bcm: add BCM43430A0
> > >   ARM: dts: qcom: msm8226: Add pinctrl for sdhci nodes
> > >   ARM: dts: qcom: apq8026-lg-lenok: Add Wifi
> > >   ARM: dts: qcom: apq8026-lg-lenok: Add Bluetooth
> > >  
> > >  .../bindings/net/broadcom-bluetooth.yaml      |  1 +
> > >  arch/arm/boot/dts/qcom-apq8026-lg-lenok.dts   | 98 ++++++++++++++++---
> > >  arch/arm/boot/dts/qcom-msm8226.dtsi           | 57 +++++++++++
> > >  drivers/bluetooth/hci_bcm.c                   |  1 +
> > >  4 files changed, 144 insertions(+), 13 deletions(-)
> 
> 
> 
>