On Thu, Aug 25, 2016 at 7:58 AM, Icenowy Zheng <icenowy@xxxxxxxx> wrote: > > > 25.08.2016, 03:36, "Maxime Ripard" <maxime.ripard@xxxxxxxxxxxxxxxxxx>: >> Hi, >> >> On Tue, Aug 23, 2016 at 02:44:51PM +0800, Icenowy Zheng wrote: >>> UART1 is connected to the bluetooth part of RTL8723BS WiFi/BT combo card >>> on iNet D978 Rev2 board. >>> >>> Enable the UART1 to make it possible to use the modified hciattach by >>> Realtek to drive the BT part of RTL8723BS. >>> >>> Signed-off-by: Icenowy Zheng <icenowy@xxxxxxxx> >> >> I guess you could fold that patch into the previous one. >> >>> --- >>> arch/arm/boot/dts/sun8i-a33-inet-d978-rev2.dts | 11 +++++++++++ >>> 1 file changed, 11 insertions(+) >>> >>> diff --git a/arch/arm/boot/dts/sun8i-a33-inet-d978-rev2.dts b/arch/arm/boot/dts/sun8i-a33-inet-d978-rev2.dts >>> index 78823d8..3ac22d4 100644 >>> --- a/arch/arm/boot/dts/sun8i-a33-inet-d978-rev2.dts >>> +++ b/arch/arm/boot/dts/sun8i-a33-inet-d978-rev2.dts >>> @@ -48,6 +48,10 @@ >>> model = "INet-D978 Rev 02"; >>> compatible = "primux,inet-d978-rev2", "allwinner,sun8i-a33"; >>> >>> + aliases { >>> + serial1 = &uart1; >>> + }; >>> + >> >> Is there any other UART in the system? > > serial0 is defined in sun8i-reference-design-tablet.dtsi, as r_uart. If your board does not have r_uart pads, then the right thing to do would be to disable it. You can then have uart1 as serial0. AFAIK Linux requires a console, but that console can be simplefb. ChenYu >> >> The alias is not supposed to be about the controller index, but the >> index in the board, meaning that if you only have a single UART, that >> would be serial0. >> >> Thanks! >> Maxime >> >> -- >> Maxime Ripard, Free Electrons >> Embedded Linux and Kernel engineering >> http://free-electrons.com -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html