Re: [PATCH v5 2/2] ARM: dts: bcm2837-rpi-3-b: Add bcm43438 serial slave

Marcel Holtmann [off-list ref] writes:

Hi,

quoted

Add BCM43438 (bluetooth) as a slave device of uart0 (pl011/ttyAMA0).
This allows to automatically insert the bcm43438 to the bluetooth
subsystem instead of relying on userspace helpers (hciattach).

Overwrite chosen/stdout-path to use 8250 aux uart as console.

Acked-by: Rob Herring <redacted>
Signed-off-by: Loic Poulain <redacted>
---
arch/arm/boot/dts/bcm2837-rpi-3-b.dts | 10 ++++++++++
1 file changed, 10 insertions(+)

v2: dt-bindings as separate patch
    rebase on upcoming pi3 dts changes
v3: changes in bcm serdev drivers:
    name refactoring and additional comments
    Add generic host_set_baudrate method
    Use agnostic device_property_read
v4: changes in doc and dts:
    Add additional params to broadcom bt dt-bindings
    Set the max baudrate to 2000000 instead of 921600
    use chosen/stdout-path instead of console=
    remove useless earlyprintk
    same changes in bcm283x.dtsi
v5: remove 1/4 dt-bindings and 4/4 hci_bcm serdev support
    1/4 and 4/4 merged in bluetooth-next tree

diff --git a/arch/arm/boot/dts/bcm2837-rpi-3-b.dts b/arch/arm/boot/dts/bcm2837-rpi-3-b.dts
index 20725ca..e4488cb 100644
--- a/arch/arm/boot/dts/bcm2837-rpi-3-b.dts
+++ b/arch/arm/boot/dts/bcm2837-rpi-3-b.dts

@@ -8,6 +8,11 @@

	compatible = "raspberrypi,3-model-b", "brcm,bcm2837";
	model = "Raspberry Pi 3 Model B";

+	chosen {
+		/* 8250 auxiliar UART instead of pl011 */
+		stdout-path = "serial1:115200n8";
+	};
+
	memory {
		reg = <0 0x40000000>;
	};

@@ -24,6 +29,11 @@

	pinctrl-names = "default";
	pinctrl-0 = <&uart0_gpio32 &gpclk2_gpio43>;
	status = "okay";
+
+	bluetooth {
+		compatible = "brcm,bcm43438-bt";
+		max-speed = <2000000>;
+	};
};

the driver for this is already in Linus’ tree. And I would really prefer that we get the DT changes merged in as well.

Now that the separated stdout patch is ready to go, I've applied this
for -next (dropping the chosen node) and I'm sending the PR for both.

Thanks so much for working on this!