Thread (1 message) 1 message, 1 author, 2017-01-16

[PATCH 3/4] ARM64: dts: meson-gx-p23x-q20x: enable the Bluetooth module

From: martin.blumenstingl@googlemail.com (Martin Blumenstingl)
Date: 2017-01-16 09:44:36
Also in: linux-amlogic, linux-devicetree, linux-serial

On Mon, Jan 16, 2017 at 1:47 AM, Andreas F?rber [off-list ref] wrote:
Am 15.01.2017 um 23:32 schrieb Martin Blumenstingl:
quoted
This takes the Bluetooth module out of reset (the reset line is
connected to GPIOX_17) and enables uart_A which is used to configure the
module.
This is identical for all boards which inherit meson-gx-p23x-q20x:
- GXL S905D P230
- GXL S905D P231
- GXM S912 Q200
- GXM S912 Q201

To get the HCI interface up one has to install bluez-utils and run:
hciattach -s115200 /dev/ttyAML1 bcm43xx 2000000 flow -

Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com>
---
 arch/arm64/boot/dts/amlogic/meson-gx-p23x-q20x.dtsi | 12 +++++++++++-
 1 file changed, 11 insertions(+), 1 deletion(-)
diff --git a/arch/arm64/boot/dts/amlogic/meson-gx-p23x-q20x.dtsi b/arch/arm64/boot/dts/amlogic/meson-gx-p23x-q20x.dtsi
index 7a078bef04cd..7db779048091 100644
--- a/arch/arm64/boot/dts/amlogic/meson-gx-p23x-q20x.dtsi
+++ b/arch/arm64/boot/dts/amlogic/meson-gx-p23x-q20x.dtsi
@@ -48,6 +48,7 @@
 / {
      aliases {
              serial0 = &uart_AO;
+             serial1 = &uart_A;
      };

      chosen {
@@ -94,12 +95,21 @@

      sdio_pwrseq: sdio-pwrseq {
              compatible = "mmc-pwrseq-simple";
-             reset-gpios = <&gpio GPIOX_6 GPIO_ACTIVE_LOW>;
+             reset-gpios = <&gpio GPIOX_6 GPIO_ACTIVE_LOW>,
+                           <&gpio GPIOX_17 GPIO_ACTIVE_LOW>;
              clocks = <&wifi32k>;
              clock-names = "ext_clock";
      };
 };

+/* This is connected to the Bluetooth module of the wifi/BT combo chip: */
+&uart_A {
+     status = "okay";
+     pinctrl-0 = <&uart_a_pins &uart_a_cts_rts_pins>;
Nit: <&uart_a_pins>, <&uart_a_cts_rts_pins> please, like you've done for
reset-gpios above.
indeed, not sure why I mixed it up. should I also send an update for
the pinctrl-documentation (as it seems to use the same pattern): [0]?
Regards,
Andreas
quoted
+     pinctrl-names = "default";
+     uart-has-rtscts;
+};
+
 /* This UART is brought out to the DB9 connector */
 &uart_AO {
      status = "okay";

--
SUSE Linux GmbH, Maxfeldstr. 5, 90409 N?rnberg, Germany
GF: Felix Imend?rffer, Jane Smithard, Graham Norton
HRB 21284 (AG N?rnberg)

[0] http://lxr.free-electrons.com/source/Documentation/devicetree/bindings/pinctrl/pinctrl-bindings.txt?v=4.9#L74
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help