Re: [PATCH 3/3] This patch adds wifi to asus tinker board S

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 08/10/2018 10:30, Heiko Stuebner wrote:
Hi David,

Am Samstag, 22. September 2018, 19:02:36 CEST schrieb David Summers:
Now as fair as I understand. the asus tinker board contains a wifi
So should this patch be added to the tinker board dtsi ?

It also far move complex - it adds far more things, so consider

Signed-off-by: David Summers <beagleboard@xxxxxxxxxxxxxxxxxxx>
---
  arch/arm/boot/dts/rk3288-tinker-s.dts | 93 +++++++++++++++++++++++++++
  1 file changed, 93 insertions(+)

diff --git a/arch/arm/boot/dts/rk3288-tinker-s.dts b/arch/arm/boot/dts/rk3288-tinker-s.dts
index bc691e3f9c07..3d91924fd8db 100644
--- a/arch/arm/boot/dts/rk3288-tinker-s.dts
+++ b/arch/arm/boot/dts/rk3288-tinker-s.dts
@@ -10,6 +10,95 @@
  / {
  	model = "Rockchip RK3288 Asus Tinker Board S";
  	compatible = "asus,rk3288-tinker-s", "rockchip,rk3288";
+
+	wireless-bluetooth {
+		compatible = "bluetooth-platdata";
+		uart_rts_gpios = <&gpio4 19 GPIO_ACTIVE_LOW>;
+		pinctrl-names = "default","rts_gpio";
+		pinctrl-0 = <&uart0_rts>;
+		pinctrl-1 = <&uart0_gpios>;
+		BT,reset_gpio    = <&gpio4 29 GPIO_ACTIVE_HIGH>;
+		BT,wake_gpio     = <&gpio4 26 GPIO_ACTIVE_HIGH>;
+		BT,wake_host_irq = <&gpio4 31 GPIO_ACTIVE_HIGH>;
+		status = "okay";
+	};
the basic problem here is, that all these nodes stem from the so called
soc-vendor-tree. Devicetree bindings normally go through a review process
while soc vendors often invent their own shortcut that is not and will
not be supported by the mainline kernel.

Bluetooth-platdata and wlan-platdata are examples of that.

Additionally, the kernel now has the somewhat new "serdev" to manage the
needed connection between bluetooth and uart. See
http://events17.linuxfoundation.org/sites/events/files/slides/serdev-elce-2017-2.pdf
for an introduction.

Sigh - this just got difficult:

https://github.com/torvalds/linux/blob/master/drivers/bluetooth/btrtl.c

Which is the bluetooth driver for Realtek RTL8723BS neither contains the device tree tabs ("static const struct of_device_id") so it can't be referred to in the device tree, nor is it linked in with serdev, so it probably can't go as sub child of uart0.

I'll look at those changes first ...

David




[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux