[U-Boot] [PATCH v1 1/2] x86: edison: Use proper number of serial interface

[U-Boot] [PATCH v1 1/2] x86: edison: Use proper number of serial interface

[Andy Shevchenko]

The console is actually serial #2. When we would like to enable other ports,
this would be not okay to mess up with the ordering.

Thus, fix the number of default console interface to be 2.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 arch/x86/dts/edison.dts | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/arch/x86/dts/edison.dts b/arch/x86/dts/edison.dts
index 0dd72407ea..b59171cb50 100644
--- a/arch/x86/dts/edison.dts
+++ b/arch/x86/dts/edison.dts
@@ -17,11 +17,11 @@
 	compatible = "intel,edison";
 
 	aliases {
-		serial0 = &serial0;
+		serial2 = &serial2;
 	};
 
 	chosen {
-		stdout-path = &serial0;
+		stdout-path = &serial2;
 	};
 
 	cpus {
@@ -53,7 +53,7 @@
 			  0x01000000 0x0 0x2000 0x2000 0 0xe000>;
 	};
 
-	serial0: serial at ff010180 {
+	serial2: serial at ff010180 {
 		compatible = "intel,mid-uart";
 		reg = <0xff010180 0x100>;
 		reg-shift = <0>;
-- 
2.20.1

[U-Boot] [PATCH v1 2/2] x86: edison: Add the rest of UARTs present on board

[Andy Shevchenko]

Intel Edison has three UART ports, i.e.
 port 0 - Bluetooth
 port 1 - auxiliary, available for general purpose use
 port 2 - debugging, usually console output is here

Enable all of them for future use.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 arch/x86/dts/edison.dts | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)

diff --git a/arch/x86/dts/edison.dts b/arch/x86/dts/edison.dts
index b59171cb50..8629f7bcf2 100644
--- a/arch/x86/dts/edison.dts
+++ b/arch/x86/dts/edison.dts
@@ -17,6 +17,8 @@
 	compatible = "intel,edison";
 
 	aliases {
+		serial0 = &serial0;
+		serial1 = &serial1;
 		serial2 = &serial2;
 	};
 
@@ -53,6 +55,22 @@
 			  0x01000000 0x0 0x2000 0x2000 0 0xe000>;
 	};
 
+	serial0: serial at ff010180 {
+		compatible = "intel,mid-uart";
+		reg = <0xff010080 0x100>;
+		reg-shift = <0>;
+		clock-frequency = <29491200>;
+		current-speed = <115200>;
+	};
+
+	serial1: serial at ff010180 {
+		compatible = "intel,mid-uart";
+		reg = <0xff010100 0x100>;
+		reg-shift = <0>;
+		clock-frequency = <29491200>;
+		current-speed = <115200>;
+	};
+
 	serial2: serial at ff010180 {
 		compatible = "intel,mid-uart";
 		reg = <0xff010180 0x100>;
-- 
2.20.1

[U-Boot] [PATCH v1 1/2] x86: edison: Use proper number of serial interface

[Bin Meng]

On Tue, Feb 26, 2019 at 7:21 PM Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
>
> The console is actually serial #2. When we would like to enable other ports,
> this would be not okay to mess up with the ordering.
>
> Thus, fix the number of default console interface to be 2.
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> ---
>  arch/x86/dts/edison.dts | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>

Reviewed-by: Bin Meng <bmeng.cn@gmail.com>

[U-Boot] [PATCH v1 2/2] x86: edison: Add the rest of UARTs present on board

[Bin Meng]

Hi Andy,

On Tue, Feb 26, 2019 at 7:21 PM Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
>
> Intel Edison has three UART ports, i.e.
>  port 0 - Bluetooth
>  port 1 - auxiliary, available for general purpose use
>  port 2 - debugging, usually console output is here
>
> Enable all of them for future use.
>
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> ---
>  arch/x86/dts/edison.dts | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
>
> diff --git a/arch/x86/dts/edison.dts b/arch/x86/dts/edison.dts
> index b59171cb50..8629f7bcf2 100644
> --- a/arch/x86/dts/edison.dts
> +++ b/arch/x86/dts/edison.dts
> @@ -17,6 +17,8 @@
>         compatible = "intel,edison";
>
>         aliases {
> +               serial0 = &serial0;
> +               serial1 = &serial1;
>                 serial2 = &serial2;
>         };
>
> @@ -53,6 +55,22 @@
>                           0x01000000 0x0 0x2000 0x2000 0 0xe000>;
>         };
>
> +       serial0: serial at ff010180 {

The reg base is wrong here, should be @ff010080

> +               compatible = "intel,mid-uart";
> +               reg = <0xff010080 0x100>;
> +               reg-shift = <0>;
> +               clock-frequency = <29491200>;
> +               current-speed = <115200>;
> +       };
> +
> +       serial1: serial at ff010180 {

ditto

> +               compatible = "intel,mid-uart";
> +               reg = <0xff010100 0x100>;
> +               reg-shift = <0>;
> +               clock-frequency = <29491200>;
> +               current-speed = <115200>;
> +       };
> +
>         serial2: serial at ff010180 {
>                 compatible = "intel,mid-uart";
>                 reg = <0xff010180 0x100>;
> --

Regards,
Bin