* [PATCH 0/5] Wifi & Bluetooth on LG G Watch R
@ 2022-02-16 21:24 Luca Weiss
2022-02-16 21:24 ` [PATCH 1/5] dt-bindings: bluetooth: broadcom: add BCM43430A0 Luca Weiss
` (6 more replies)
0 siblings, 7 replies; 12+ messages in thread
From: Luca Weiss @ 2022-02-16 21:24 UTC (permalink / raw)
To: linux-arm-msm
Cc: ~postmarketos/upstreaming, phone-devel, Luca Weiss,
Linus Walleij, devicetree, linux-bluetooth, linux-kernel, netdev
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
^ permalink raw reply [flat|nested] 12+ messages in thread
* [PATCH 1/5] dt-bindings: bluetooth: broadcom: add BCM43430A0
2022-02-16 21:24 [PATCH 0/5] Wifi & Bluetooth on LG G Watch R Luca Weiss
@ 2022-02-16 21:24 ` Luca Weiss
2022-02-22 15:21 ` Linus Walleij
2022-02-25 16:29 ` Rob Herring
2022-02-16 21:24 ` [PATCH 2/5] Bluetooth: hci_bcm: " Luca Weiss
` (5 subsequent siblings)
6 siblings, 2 replies; 12+ messages in thread
From: Luca Weiss @ 2022-02-16 21:24 UTC (permalink / raw)
To: linux-arm-msm
Cc: ~postmarketos/upstreaming, phone-devel, Luca Weiss,
David S. Miller, Jakub Kicinski, Rob Herring, Linus Walleij,
netdev, devicetree, linux-kernel
Document the compatible string for BCM43430A0 bluetooth.
Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
---
Documentation/devicetree/bindings/net/broadcom-bluetooth.yaml | 1 +
1 file changed, 1 insertion(+)
diff --git a/Documentation/devicetree/bindings/net/broadcom-bluetooth.yaml b/Documentation/devicetree/bindings/net/broadcom-bluetooth.yaml
index 5aac094fd217..c11cd9bc9f9c 100644
--- a/Documentation/devicetree/bindings/net/broadcom-bluetooth.yaml
+++ b/Documentation/devicetree/bindings/net/broadcom-bluetooth.yaml
@@ -19,6 +19,7 @@ properties:
- brcm,bcm4329-bt
- brcm,bcm4330-bt
- brcm,bcm4334-bt
+ - brcm,bcm43430a0-bt
- brcm,bcm43438-bt
- brcm,bcm4345c5
- brcm,bcm43540-bt
--
2.35.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 2/5] Bluetooth: hci_bcm: add BCM43430A0
2022-02-16 21:24 [PATCH 0/5] Wifi & Bluetooth on LG G Watch R Luca Weiss
2022-02-16 21:24 ` [PATCH 1/5] dt-bindings: bluetooth: broadcom: add BCM43430A0 Luca Weiss
@ 2022-02-16 21:24 ` Luca Weiss
2022-02-16 21:24 ` [PATCH 3/5] ARM: dts: qcom: msm8226: Add pinctrl for sdhci nodes Luca Weiss
` (4 subsequent siblings)
6 siblings, 0 replies; 12+ messages in thread
From: Luca Weiss @ 2022-02-16 21:24 UTC (permalink / raw)
To: linux-arm-msm
Cc: ~postmarketos/upstreaming, phone-devel, Luca Weiss,
Marcel Holtmann, Johan Hedberg, Luiz Augusto von Dentz,
linux-bluetooth, linux-kernel
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
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 3/5] ARM: dts: qcom: msm8226: Add pinctrl for sdhci nodes
2022-02-16 21:24 [PATCH 0/5] Wifi & Bluetooth on LG G Watch R Luca Weiss
2022-02-16 21:24 ` [PATCH 1/5] dt-bindings: bluetooth: broadcom: add BCM43430A0 Luca Weiss
2022-02-16 21:24 ` [PATCH 2/5] Bluetooth: hci_bcm: " Luca Weiss
@ 2022-02-16 21:24 ` Luca Weiss
2022-02-16 21:24 ` [PATCH 4/5] ARM: dts: qcom: apq8026-lg-lenok: Add Wifi Luca Weiss
` (3 subsequent siblings)
6 siblings, 0 replies; 12+ messages in thread
From: Luca Weiss @ 2022-02-16 21:24 UTC (permalink / raw)
To: linux-arm-msm
Cc: ~postmarketos/upstreaming, phone-devel, Luca Weiss, Andy Gross,
Bjorn Andersson, Rob Herring, devicetree, linux-kernel
Also remove the pinctrl from qcom-apq8026-lg-lenok as it is the same
value as the generic pinctrl.
Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
---
arch/arm/boot/dts/qcom-apq8026-lg-lenok.dts | 17 ------
arch/arm/boot/dts/qcom-msm8226.dtsi | 57 +++++++++++++++++++++
2 files changed, 57 insertions(+), 17 deletions(-)
diff --git a/arch/arm/boot/dts/qcom-apq8026-lg-lenok.dts b/arch/arm/boot/dts/qcom-apq8026-lg-lenok.dts
index 1519544029e7..5ce42dd962c4 100644
--- a/arch/arm/boot/dts/qcom-apq8026-lg-lenok.dts
+++ b/arch/arm/boot/dts/qcom-apq8026-lg-lenok.dts
@@ -204,9 +204,6 @@ &sdhc_1 {
bus-width = <8>;
non-removable;
-
- pinctrl-names = "default";
- pinctrl-0 = <&sdhc1_pin_a>;
};
&smbb {
@@ -219,20 +216,6 @@ &smbb {
};
&tlmm {
- sdhc1_pin_a: sdhc1-pin-active {
- clk {
- pins = "sdc1_clk";
- drive-strength = <10>;
- bias-disable;
- };
-
- cmd-data {
- pins = "sdc1_cmd", "sdc1_data";
- drive-strength = <10>;
- bias-pull-up;
- };
- };
-
touch_pins: touch {
irq {
pins = "gpio17";
diff --git a/arch/arm/boot/dts/qcom-msm8226.dtsi b/arch/arm/boot/dts/qcom-msm8226.dtsi
index 103c0ab70814..dfeb47eb41a2 100644
--- a/arch/arm/boot/dts/qcom-msm8226.dtsi
+++ b/arch/arm/boot/dts/qcom-msm8226.dtsi
@@ -116,6 +116,8 @@ sdhc_1: sdhci@f9824900 {
<&gcc GCC_SDCC1_AHB_CLK>,
<&xo_board>;
clock-names = "core", "iface", "xo";
+ pinctrl-names = "default";
+ pinctrl-0 = <&sdhc1_default_state>;
status = "disabled";
};
@@ -130,6 +132,8 @@ sdhc_2: sdhci@f98a4900 {
<&gcc GCC_SDCC2_AHB_CLK>,
<&xo_board>;
clock-names = "core", "iface", "xo";
+ pinctrl-names = "default";
+ pinctrl-0 = <&sdhc2_default_state>;
status = "disabled";
};
@@ -144,6 +148,8 @@ sdhc_3: sdhci@f9864900 {
<&gcc GCC_SDCC3_AHB_CLK>,
<&xo_board>;
clock-names = "core", "iface", "xo";
+ pinctrl-names = "default";
+ pinctrl-0 = <&sdhc3_default_state>;
status = "disabled";
};
@@ -320,6 +326,57 @@ blsp1_i2c5_pins: blsp1-i2c5 {
drive-strength = <2>;
bias-disable;
};
+
+ sdhc1_default_state: sdhc1-default-state {
+ clk {
+ pins = "sdc1_clk";
+ drive-strength = <10>;
+ bias-disable;
+ };
+
+ cmd-data {
+ pins = "sdc1_cmd", "sdc1_data";
+ drive-strength = <10>;
+ bias-pull-up;
+ };
+ };
+
+ sdhc2_default_state: sdhc2-default-state {
+ clk {
+ pins = "sdc2_clk";
+ drive-strength = <10>;
+ bias-disable;
+ };
+
+ cmd-data {
+ pins = "sdc2_cmd", "sdc2_data";
+ drive-strength = <10>;
+ bias-pull-up;
+ };
+ };
+
+ sdhc3_default_state: sdhc3-default-state {
+ clk {
+ pins = "gpio44";
+ function = "sdc3";
+ drive-strength = <8>;
+ bias-disable;
+ };
+
+ cmd {
+ pins = "gpio43";
+ function = "sdc3";
+ drive-strength = <8>;
+ bias-pull-up;
+ };
+
+ data {
+ pins = "gpio39", "gpio40", "gpio41", "gpio42";
+ function = "sdc3";
+ drive-strength = <8>;
+ bias-pull-up;
+ };
+ };
};
restart@fc4ab000 {
--
2.35.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 4/5] ARM: dts: qcom: apq8026-lg-lenok: Add Wifi
2022-02-16 21:24 [PATCH 0/5] Wifi & Bluetooth on LG G Watch R Luca Weiss
` (2 preceding siblings ...)
2022-02-16 21:24 ` [PATCH 3/5] ARM: dts: qcom: msm8226: Add pinctrl for sdhci nodes Luca Weiss
@ 2022-02-16 21:24 ` Luca Weiss
2022-02-16 21:24 ` [PATCH 5/5] ARM: dts: qcom: apq8026-lg-lenok: Add Bluetooth Luca Weiss
` (2 subsequent siblings)
6 siblings, 0 replies; 12+ messages in thread
From: Luca Weiss @ 2022-02-16 21:24 UTC (permalink / raw)
To: linux-arm-msm
Cc: ~postmarketos/upstreaming, phone-devel, Luca Weiss, Andy Gross,
Bjorn Andersson, Rob Herring, devicetree, linux-kernel
The device contains BCM43430A0 for wifi. Add a node for it.
Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
---
arch/arm/boot/dts/qcom-apq8026-lg-lenok.dts | 53 +++++++++++++++++++++
1 file changed, 53 insertions(+)
diff --git a/arch/arm/boot/dts/qcom-apq8026-lg-lenok.dts b/arch/arm/boot/dts/qcom-apq8026-lg-lenok.dts
index 5ce42dd962c4..333fae435913 100644
--- a/arch/arm/boot/dts/qcom-apq8026-lg-lenok.dts
+++ b/arch/arm/boot/dts/qcom-apq8026-lg-lenok.dts
@@ -21,6 +21,20 @@ aliases {
chosen {
stdout-path = "serial0:115200n8";
};
+
+ vreg_wlan: wlan-regulator {
+ compatible = "regulator-fixed";
+
+ regulator-name = "wl-reg";
+ regulator-min-microvolt = <2950000>;
+ regulator-max-microvolt = <2950000>;
+
+ gpio = <&tlmm 46 GPIO_ACTIVE_HIGH>;
+ enable-active-high;
+
+ pinctrl-names = "default";
+ pinctrl-0 = <&wlan_regulator_default_state>;
+ };
};
&blsp1_i2c1 {
@@ -206,6 +220,30 @@ &sdhc_1 {
non-removable;
};
+&sdhc_3 {
+ status = "okay";
+
+ max-frequency = <100000000>;
+ non-removable;
+
+ vmmc-supply = <&vreg_wlan>;
+ vqmmc-supply = <&pm8226_l6>;
+
+ #address-cells = <1>;
+ #size-cells = <0>;
+
+ wifi@1 {
+ compatible = "brcm,bcm43430a0-fmac", "brcm,bcm4329-fmac";
+ reg = <1>;
+
+ interrupts-extended = <&tlmm 37 IRQ_TYPE_EDGE_FALLING>;
+ interrupt-names = "host-wake";
+
+ pinctrl-names = "default";
+ pinctrl-0 = <&wlan_hostwake_default_state>;
+ };
+};
+
&smbb {
qcom,fast-charge-safe-current = <450000>;
qcom,fast-charge-current-limit = <400000>;
@@ -235,6 +273,21 @@ reset {
output-high;
};
};
+
+ wlan_hostwake_default_state: wlan-hostwake-default-state {
+ pins = "gpio37";
+ function = "gpio";
+ drive-strength = <2>;
+ bias-disable;
+ input-enable;
+ };
+
+ wlan_regulator_default_state: wlan-regulator-default-state {
+ pins = "gpio46";
+ function = "gpio";
+ drive-strength = <2>;
+ bias-pull-up;
+ };
};
&usb {
--
2.35.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* [PATCH 5/5] ARM: dts: qcom: apq8026-lg-lenok: Add Bluetooth
2022-02-16 21:24 [PATCH 0/5] Wifi & Bluetooth on LG G Watch R Luca Weiss
` (3 preceding siblings ...)
2022-02-16 21:24 ` [PATCH 4/5] ARM: dts: qcom: apq8026-lg-lenok: Add Wifi Luca Weiss
@ 2022-02-16 21:24 ` Luca Weiss
2022-02-24 4:15 ` [PATCH 0/5] Wifi & Bluetooth on LG G Watch R Bjorn Andersson
2022-02-24 20:54 ` (subset) " Bjorn Andersson
6 siblings, 0 replies; 12+ messages in thread
From: Luca Weiss @ 2022-02-16 21:24 UTC (permalink / raw)
To: linux-arm-msm
Cc: ~postmarketos/upstreaming, phone-devel, Luca Weiss, Andy Gross,
Bjorn Andersson, Rob Herring, devicetree, linux-kernel
The device contains BCM43430A0 for bluetooth. Add a node for it.
Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
---
arch/arm/boot/dts/qcom-apq8026-lg-lenok.dts | 36 +++++++++++++++++++++
1 file changed, 36 insertions(+)
diff --git a/arch/arm/boot/dts/qcom-apq8026-lg-lenok.dts b/arch/arm/boot/dts/qcom-apq8026-lg-lenok.dts
index 333fae435913..42467f705b26 100644
--- a/arch/arm/boot/dts/qcom-apq8026-lg-lenok.dts
+++ b/arch/arm/boot/dts/qcom-apq8026-lg-lenok.dts
@@ -16,6 +16,7 @@ / {
aliases {
serial0 = &blsp1_uart3;
+ serial1 = &blsp1_uart4;
};
chosen {
@@ -80,6 +81,26 @@ &blsp1_uart3 {
status = "okay";
};
+&blsp1_uart4 {
+ status = "okay";
+
+ pinctrl-names = "default";
+ pinctrl-0 = <&blsp1_uart4_default_state>;
+
+ bluetooth {
+ compatible = "brcm,bcm43430a0-bt";
+
+ max-speed = <3000000>;
+
+ pinctrl-names = "default";
+ pinctrl-0 = <&bluetooth_default_state>;
+
+ host-wakeup-gpios = <&tlmm 48 GPIO_ACTIVE_HIGH>;
+ device-wakeup-gpios = <&tlmm 47 GPIO_ACTIVE_HIGH>;
+ shutdown-gpios = <&tlmm 45 GPIO_ACTIVE_HIGH>;
+ };
+};
+
&rpm_requests {
pm8226-regulators {
compatible = "qcom,rpm-pm8226-regulators";
@@ -254,6 +275,21 @@ &smbb {
};
&tlmm {
+ blsp1_uart4_default_state: blsp1-uart4-default-state {
+ pins = "gpio12", "gpio13", "gpio14", "gpio15";
+ function = "blsp_uart4";
+ drive-strength = <8>;
+ bias-disable;
+ };
+
+ bluetooth_default_state: bluetooth-default-state {
+ pins = "gpio47", "gpio48";
+ function = "gpio";
+ drive-strength = <2>;
+ bias-disable;
+ input-enable;
+ };
+
touch_pins: touch {
irq {
pins = "gpio17";
--
2.35.1
^ permalink raw reply related [flat|nested] 12+ messages in thread
* Re: [PATCH 1/5] dt-bindings: bluetooth: broadcom: add BCM43430A0
2022-02-16 21:24 ` [PATCH 1/5] dt-bindings: bluetooth: broadcom: add BCM43430A0 Luca Weiss
@ 2022-02-22 15:21 ` Linus Walleij
2022-02-25 16:29 ` Rob Herring
1 sibling, 0 replies; 12+ messages in thread
From: Linus Walleij @ 2022-02-22 15:21 UTC (permalink / raw)
To: Luca Weiss
Cc: linux-arm-msm, ~postmarketos/upstreaming, phone-devel,
David S. Miller, Jakub Kicinski, Rob Herring, netdev, devicetree,
linux-kernel
On Wed, Feb 16, 2022 at 10:25 PM Luca Weiss <luca@z3ntu.xyz> wrote:
> Document the compatible string for BCM43430A0 bluetooth.
>
> Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
Looks good to me:
Reviewed-by: Linus Walleij <linus.walleij@linaro.org>
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 0/5] Wifi & Bluetooth on LG G Watch R
2022-02-16 21:24 [PATCH 0/5] Wifi & Bluetooth on LG G Watch R Luca Weiss
` (4 preceding siblings ...)
2022-02-16 21:24 ` [PATCH 5/5] ARM: dts: qcom: apq8026-lg-lenok: Add Bluetooth Luca Weiss
@ 2022-02-24 4:15 ` Bjorn Andersson
2022-02-25 20:19 ` Luca Weiss
2022-02-24 20:54 ` (subset) " Bjorn Andersson
6 siblings, 1 reply; 12+ messages in thread
From: Bjorn Andersson @ 2022-02-24 4:15 UTC (permalink / raw)
To: Luca Weiss
Cc: linux-arm-msm, ~postmarketos/upstreaming, phone-devel,
Linus Walleij, devicetree, linux-bluetooth, linux-kernel, netdev
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
>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: (subset) [PATCH 0/5] Wifi & Bluetooth on LG G Watch R
2022-02-16 21:24 [PATCH 0/5] Wifi & Bluetooth on LG G Watch R Luca Weiss
` (5 preceding siblings ...)
2022-02-24 4:15 ` [PATCH 0/5] Wifi & Bluetooth on LG G Watch R Bjorn Andersson
@ 2022-02-24 20:54 ` Bjorn Andersson
6 siblings, 0 replies; 12+ messages in thread
From: Bjorn Andersson @ 2022-02-24 20:54 UTC (permalink / raw)
To: Luca Weiss, linux-arm-msm
Cc: ~postmarketos/upstreaming, linux-kernel, devicetree, phone-devel,
netdev, Linus Walleij, linux-bluetooth
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>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 1/5] dt-bindings: bluetooth: broadcom: add BCM43430A0
2022-02-16 21:24 ` [PATCH 1/5] dt-bindings: bluetooth: broadcom: add BCM43430A0 Luca Weiss
2022-02-22 15:21 ` Linus Walleij
@ 2022-02-25 16:29 ` Rob Herring
1 sibling, 0 replies; 12+ messages in thread
From: Rob Herring @ 2022-02-25 16:29 UTC (permalink / raw)
To: Luca Weiss
Cc: David S. Miller, Linus Walleij, netdev, Rob Herring,
~postmarketos/upstreaming, linux-arm-msm, devicetree,
linux-kernel, Jakub Kicinski, phone-devel
On Wed, 16 Feb 2022 22:24:28 +0100, Luca Weiss wrote:
> Document the compatible string for BCM43430A0 bluetooth.
>
> Signed-off-by: Luca Weiss <luca@z3ntu.xyz>
> ---
> Documentation/devicetree/bindings/net/broadcom-bluetooth.yaml | 1 +
> 1 file changed, 1 insertion(+)
>
Acked-by: Rob Herring <robh@kernel.org>
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 0/5] Wifi & Bluetooth on LG G Watch R
2022-02-24 4:15 ` [PATCH 0/5] Wifi & Bluetooth on LG G Watch R Bjorn Andersson
@ 2022-02-25 20:19 ` Luca Weiss
2022-02-25 21:57 ` Bjorn Andersson
0 siblings, 1 reply; 12+ messages in thread
From: Luca Weiss @ 2022-02-25 20:19 UTC (permalink / raw)
To: Bjorn Andersson
Cc: linux-arm-msm, ~postmarketos/upstreaming, phone-devel,
Linus Walleij, devicetree, linux-bluetooth, linux-kernel, netdev,
Marcel Holtmann
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(-)
^ permalink raw reply [flat|nested] 12+ messages in thread
* Re: [PATCH 0/5] Wifi & Bluetooth on LG G Watch R
2022-02-25 20:19 ` Luca Weiss
@ 2022-02-25 21:57 ` Bjorn Andersson
0 siblings, 0 replies; 12+ messages in thread
From: Bjorn Andersson @ 2022-02-25 21:57 UTC (permalink / raw)
To: Luca Weiss
Cc: linux-arm-msm, ~postmarketos/upstreaming, phone-devel,
Linus Walleij, devicetree, linux-bluetooth, linux-kernel, netdev,
Marcel Holtmann
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(-)
>
>
>
>
^ permalink raw reply [flat|nested] 12+ messages in thread
end of thread, other threads:[~2022-02-25 21:55 UTC | newest]
Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-16 21:24 [PATCH 0/5] Wifi & Bluetooth on LG G Watch R Luca Weiss
2022-02-16 21:24 ` [PATCH 1/5] dt-bindings: bluetooth: broadcom: add BCM43430A0 Luca Weiss
2022-02-22 15:21 ` Linus Walleij
2022-02-25 16:29 ` Rob Herring
2022-02-16 21:24 ` [PATCH 2/5] Bluetooth: hci_bcm: " Luca Weiss
2022-02-16 21:24 ` [PATCH 3/5] ARM: dts: qcom: msm8226: Add pinctrl for sdhci nodes Luca Weiss
2022-02-16 21:24 ` [PATCH 4/5] ARM: dts: qcom: apq8026-lg-lenok: Add Wifi Luca Weiss
2022-02-16 21:24 ` [PATCH 5/5] ARM: dts: qcom: apq8026-lg-lenok: Add Bluetooth Luca Weiss
2022-02-24 4:15 ` [PATCH 0/5] Wifi & Bluetooth on LG G Watch R Bjorn Andersson
2022-02-25 20:19 ` Luca Weiss
2022-02-25 21:57 ` Bjorn Andersson
2022-02-24 20:54 ` (subset) " Bjorn Andersson
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).