Re: [PATCH 7/8] ARM: dts: aspeed: Add aliases for UARTs

From: Andrew Jeffery
Date: Wed Oct 04 2017 - 01:13:34 EST


On Thu, 2017-09-28 at 17:21 +0930, Joel Stanley wrote:
> Existing userspace expects the console (UART5) to be at /dev/ttyS4.ÂÂTo
> ensure the UARTs show up where users expect them, we give them fixed
> aliases starting at 0.
>Â
> Signed-off-by: Joel Stanley <joel@xxxxxxxxx>

Reviewed-by: Andrew Jeffery <andrew@xxxxxxxx>

> ---
> Âarch/arm/boot/dts/aspeed-bmc-opp-palmetto.dts | 4 ----
> Âarch/arm/boot/dts/aspeed-g4.dtsiÂÂÂÂÂÂÂÂÂÂÂÂÂÂ| 5 +++++
> Âarch/arm/boot/dts/aspeed-g5.dtsiÂÂÂÂÂÂÂÂÂÂÂÂÂÂ| 5 +++++
> Â3 files changed, 10 insertions(+), 4 deletions(-)
>Â
> diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-palmetto.dts b/arch/arm/boot/dts/aspeed-bmc-opp-palmetto.dts
> index e387c80b7f4f..be51be5a5f39 100644
> --- a/arch/arm/boot/dts/aspeed-bmc-opp-palmetto.dts
> +++ b/arch/arm/boot/dts/aspeed-bmc-opp-palmetto.dts
> @@ -6,10 +6,6 @@
> Â model = "Palmetto BMC";
> Â compatible = "tyan,palmetto-bmc", "aspeed,ast2400";
> Â
> - aliases {
> - serial4 = &uart5;
> - };
> -
> Â chosen {
> Â stdout-path = &uart5;
> Â bootargs = "console=ttyS4,115200 earlyprintk";
> diff --git a/arch/arm/boot/dts/aspeed-g4.dtsi b/arch/arm/boot/dts/aspeed-g4.dtsi
> index f8611d5a6465..191c33d18122 100644
> --- a/arch/arm/boot/dts/aspeed-g4.dtsi
> +++ b/arch/arm/boot/dts/aspeed-g4.dtsi
> @@ -22,6 +22,11 @@
> Â i2c11 = &i2c11;
> Â i2c12 = &i2c12;
> Â i2c13 = &i2c13;
> + serial0 = &uart1;
> + serial1 = &uart2;
> + serial2 = &uart3;
> + serial3 = &uart4;
> + serial4 = &uart5;
> Â };
> Â
> Â cpus {
> diff --git a/arch/arm/boot/dts/aspeed-g5.dtsi b/arch/arm/boot/dts/aspeed-g5.dtsi
> index de127370131d..251fc9f4637e 100644
> --- a/arch/arm/boot/dts/aspeed-g5.dtsi
> +++ b/arch/arm/boot/dts/aspeed-g5.dtsi
> @@ -22,6 +22,11 @@
> Â i2c11 = &i2c11;
> Â i2c12 = &i2c12;
> Â i2c13 = &i2c13;
> + serial0 = &uart1;
> + serial1 = &uart2;
> + serial2 = &uart3;
> + serial3 = &uart4;
> + serial4 = &uart5;
> Â };
> Â
> Â cpus {

Attachment: signature.asc
Description: This is a digitally signed message part