Hi David, > Since v2, the compatible binding names have been changed to end in > -bt, as requested by robh. Also the example has been expanded. > > As before, for these Realtek devices, are typicaly joint wifi and > bluetooth devices; with seperate interfaces which will be > independently wired to the cpu. For these devices wifi is sdio, whilst > bluetooth is a uart. This patchset just sets the device tree links for > the bluetooth interface. > > The example device tree source code, alas hasn't been tested. It is a > copy of that used on the ASUS Tinker Board: > > https://github.com/TinkerBoard/debian_kernel/commit/6a3128ade33f758887048578ada61a4b7ab8e678 > > If these patches are accepted, my next job will be to mainline the > rest of the Tinker Board patches - so if this element of the device > tree has to change, I will also update the example here. > > Signed-off-by: David Summers <beagleboard@xxxxxxxxxxxxxxxxxxx> > --- > .../bindings/net/realtek-bluetooth-serial.txt | 32 +++++++++++++++++++ > 1 file changed, 32 insertions(+) > create mode 100644 Documentation/devicetree/bindings/net/realtek-bluetooth-serial.txt > > diff --git a/Documentation/devicetree/bindings/net/realtek-bluetooth-serial.txt b/Documentation/devicetree/bindings/net/realtek-bluetooth-serial.txt > new file mode 100644 > index 000000000000..119f9db4f87f > --- /dev/null > +++ b/Documentation/devicetree/bindings/net/realtek-bluetooth-serial.txt > @@ -0,0 +1,32 @@ > +Realtek bluetooth devices connected via a UART. > +These devices typically also have a wi-fi connected via sdio - the > +compatible described here is used just for referencing the bluetooth. If it is not in a file name or code, I prefer you write it Bluetooth since that is the approving naming in written text. Same as Wi-Fi and SDIO etc. The tendency to lower-case everything in written text is just bad style. > + > +- compatible: should be "realtek,<name>-bt" > + except for "realtek,trl8761atv" - which only has a serial bluetooth connection > + "realtek,rtl8723as-bt" > + "realtek,rtl8723bs-bt" > + "realtek,rtl8723ds-bt" > + "realtek,rtl8761atv" > + "realtek,rtl8821as-bt" > + "realtek,rtl8821cs-bt" > + "realtek,rtl8822bs-bt" > + > +Example: > + > +&uart0 { > + status = "okay"; > + pinctrl-0 = <&uart0_xfer>, <&uart0_cts>; > + bluetooth { > + compatible = "realtek,rtl8723bs-bluetooth”; I think you need to correct the example to also use -bt. Regards Marcel
