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

From: Luca Weiss <luca@z3ntu.xyz>
To: Bjorn Andersson <bjorn.andersson@linaro.org>
Cc: linux-arm-msm@vger.kernel.org,
	~postmarketos/upstreaming@lists.sr.ht,
	phone-devel@vger.kernel.org,
	Linus Walleij <linus.walleij@linaro.org>,
	devicetree@vger.kernel.org, linux-bluetooth@vger.kernel.org,
	linux-kernel@vger.kernel.org, netdev@vger.kernel.org,
	Marcel Holtmann <marcel@holtmann.org>
Subject: Re: [PATCH 0/5] Wifi & Bluetooth on LG G Watch R
Date: Fri, 25 Feb 2022 21:19:23 +0100	[thread overview]
Message-ID: <4379033.LvFx2qVVIh@g550jk> (raw)
In-Reply-To: <YhcGSmd5M3W+fI6c@builder.lan>

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(-)