DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 1AC1E2085B
Subject: Re: [PATCH 3/3] This patch adds wifi to asus tinker board S
To:     Heiko Stuebner <heiko@sntech.de>,
        Marcel Holtmann <marcel@holtmann.org>,
        Johan Hedberg <johan.hedberg@gmail.com>
References: <20180922170236.8690-1-beagleboard@davidjohnsummers.uk>
 <20180922170236.8690-4-beagleboard@davidjohnsummers.uk>
 <5682345.Yzn33l6ONg@phil>
Cc:     linux-bluetooth@vger.kernel.org, linux-rockchip@lists.infradead.org
From:   David Summers <beagleboard@davidjohnsummers.uk>
Message-ID: <efb16675-0382-64b1-ceb5-acee3f95628d@davidjohnsummers.uk>
Date:   Thu, 11 Oct 2018 11:09:51 +0100
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101
 Thunderbird/60.0
MIME-Version: 1.0
In-Reply-To: <5682345.Yzn33l6ONg@phil>
Content-Type: text/plain; charset=utf-8; format=flowed
Content-Transfer-Encoding: 7bit
Content-Language: en-GB
Sender: linux-bluetooth-owner@vger.kernel.org
Precedence: bulk

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@davidjohnsummers.uk>
>> ---
>>   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