[PATCH 2/2] ARM: dts: sun8i: enable UART1 for iNet D978 Rev2 board

25.08.2016, 16:03, "Maxime Ripard" [off-list ref]:

Hi,

On Thu, Aug 25, 2016 at 02:57:24PM +0800, Icenowy Zheng wrote:

quoted

?>>>> ??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.

?I don't know it. The RX/TX is wired on PF2/PF4 on the board (UART0 muxed with
?MMC)

Then you don't support it and you need to disable it.

quoted

?But many codes will require a ttyS0 as console... So I think the
?bluetooth should be located at ttyS1...

Most of them would use /dev/console anyway. What do you have in mind?

As a general configuration, /dev/ttyS0 will be indicated as the default console=
value... (Otherwise more boards will fail)

Or maybe we can specify uart0 as serial0, and leave it as disabled? (Thus if the
debug port is needed, we can easily apply a overlay with &mmc0 status as disabled,
and &uart0 status as okay)

On all my A33 boards, the official boot0, u-boot all uses uart0 as debugging port.

Maxime

--
Maxime Ripard, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com