diff mbox series

[v2,03/19] ARM: dts: aspeed: Add LPC and child devices

Message ID 20171215062443.23059-4-joel@jms.id.au
State New
Headers show
Series None | expand

Commit Message

Joel Stanley Dec. 15, 2017, 6:24 a.m. UTC
From: Andrew Jeffery <andrew@aj.id.au>


Ensure the ordering is correct and add all of the children in the SoC
device trees for the ast2400 and ast2500.

Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

Signed-off-by: Joel Stanley <joel@jms.id.au>

---
 arch/arm/boot/dts/aspeed-g4.dtsi | 35 +++++++++++++++++++++++++++++++++++
 arch/arm/boot/dts/aspeed-g5.dtsi | 27 +++++++++++++++++----------
 2 files changed, 52 insertions(+), 10 deletions(-)

-- 
2.14.1

Comments

Cédric Le Goater Dec. 18, 2017, 9:25 a.m. UTC | #1
On 12/15/2017 07:24 AM, Joel Stanley wrote:
> From: Andrew Jeffery <andrew@aj.id.au>

> 

> Ensure the ordering is correct and add all of the children in the SoC

> device trees for the ast2400 and ast2500.

> 

> Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

> Signed-off-by: Joel Stanley <joel@jms.id.au>

> ---

>  arch/arm/boot/dts/aspeed-g4.dtsi | 35 +++++++++++++++++++++++++++++++++++

>  arch/arm/boot/dts/aspeed-g5.dtsi | 27 +++++++++++++++++----------

>  2 files changed, 52 insertions(+), 10 deletions(-)

> 

> diff --git a/arch/arm/boot/dts/aspeed-g4.dtsi b/arch/arm/boot/dts/aspeed-g4.dtsi

> index 100d092e6c07..a3bc5da7d42c 100644

> --- a/arch/arm/boot/dts/aspeed-g4.dtsi

> +++ b/arch/arm/boot/dts/aspeed-g4.dtsi

> @@ -226,6 +226,41 @@

>  				status = "disabled";

>  			};

>  

> +			lpc: lpc@1e789000 {

> +				compatible = "aspeed,ast2400-lpc", "simple-mfd";

> +				reg = <0x1e789000 0x1000>;

> +

> +				#address-cells = <1>;

> +				#size-cells = <1>;

> +				ranges = <0x0 0x1e789000 0x1000>;

> +

> +				lpc_bmc: lpc-bmc@0 {

> +					compatible = "aspeed,ast2400-lpc-bmc";

> +					reg = <0x0 0x80>;

> +				};

> +

> +				lpc_host: lpc-host@80 {

> +					compatible = "aspeed,ast2400-lpc-host", "simple-mfd", "syscon";

> +					reg = <0x80 0x1e0>;

> +					reg-io-width = <4>;

> +

> +					#address-cells = <1>;

> +					#size-cells = <1>;

> +					ranges = <0x0 0x80 0x1e0>;

> +

> +					lpc_ctrl: lpc-ctrl@0 {

> +						compatible = "aspeed,ast2400-lpc-ctrl";

> +						reg = <0x0 0x80>;

> +						status = "disabled";

> +					};

> +

> +					lhc: lhc@20 {

> +						compatible = "aspeed,ast2500-lhc";


aspeed,ast2400-lhc

The layout of the registers are the same but there a couple of differences
in the bit definitions between the two SoCs.

a part from that :

Reviewed-by: Cédric Le Goater <clg@kaod.org>


C. 

> +						reg = <0x20 0x24 0x48 0x8>;

> +					};

> +				};

> +			};

> +

>  			uart2: serial@1e78d000 {

>  				compatible = "ns16550a";

>  				reg = <0x1e78d000 0x20>;

> diff --git a/arch/arm/boot/dts/aspeed-g5.dtsi b/arch/arm/boot/dts/aspeed-g5.dtsi

> index 1f9d28313f82..7861631940fe 100644

> --- a/arch/arm/boot/dts/aspeed-g5.dtsi

> +++ b/arch/arm/boot/dts/aspeed-g5.dtsi

> @@ -266,6 +266,16 @@

>  				status = "disabled";

>  			};

>  

> +			vuart: serial@1e787000 {

> +				compatible = "aspeed,ast2500-vuart";

> +				reg = <0x1e787000 0x40>;

> +				reg-shift = <2>;

> +				interrupts = <10>;

> +				clocks = <&clk_uart>;

> +				no-loopback-test;

> +				status = "disabled";

> +			};

> +

>  			lpc: lpc@1e789000 {

>  				compatible = "aspeed,ast2500-lpc", "simple-mfd";

>  				reg = <0x1e789000 0x1000>;

> @@ -289,6 +299,13 @@

>  

>  					reg-io-width = <4>;

>  

> +					lpc_ctrl: lpc-ctrl@0 {

> +						compatible = "aspeed,ast2500-lpc-ctrl";

> +						reg = <0x0 0x80>;

> +						status = "disabled";

> +					};

> +

> +

>  					lhc: lhc@20 {

>  						compatible = "aspeed,ast2500-lhc";

>  						reg = <0x20 0x24 0x48 0x8>;

> @@ -296,16 +313,6 @@

>  				};

>  			};

>  

> -			vuart: serial@1e787000 {

> -				compatible = "aspeed,ast2500-vuart";

> -				reg = <0x1e787000 0x40>;

> -				reg-shift = <2>;

> -				interrupts = <10>;

> -				clocks = <&clk_uart>;

> -				no-loopback-test;

> -				status = "disabled";

> -			};

> -

>  			uart2: serial@1e78d000 {

>  				compatible = "ns16550a";

>  				reg = <0x1e78d000 0x20>;

>
Joel Stanley Dec. 20, 2017, 3:21 a.m. UTC | #2
j

On Mon, Dec 18, 2017 at 7:55 PM, Cédric Le Goater <clg@kaod.org> wrote:
> On 12/15/2017 07:24 AM, Joel Stanley wrote:

>> From: Andrew Jeffery <andrew@aj.id.au>

>>

>> Ensure the ordering is correct and add all of the children in the SoC

>> device trees for the ast2400 and ast2500.

>>

>> Signed-off-by: Andrew Jeffery <andrew@aj.id.au>

>> Signed-off-by: Joel Stanley <joel@jms.id.au>

>> ---

>>  arch/arm/boot/dts/aspeed-g4.dtsi | 35 +++++++++++++++++++++++++++++++++++

>>  arch/arm/boot/dts/aspeed-g5.dtsi | 27 +++++++++++++++++----------

>>  2 files changed, 52 insertions(+), 10 deletions(-)

>>

>> diff --git a/arch/arm/boot/dts/aspeed-g4.dtsi b/arch/arm/boot/dts/aspeed-g4.dtsi

>> index 100d092e6c07..a3bc5da7d42c 100644

>> --- a/arch/arm/boot/dts/aspeed-g4.dtsi

>> +++ b/arch/arm/boot/dts/aspeed-g4.dtsi

>> @@ -226,6 +226,41 @@

>>                               status = "disabled";

>>                       };

>>

>> +                     lpc: lpc@1e789000 {

>> +                             compatible = "aspeed,ast2400-lpc", "simple-mfd";

>> +                             reg = <0x1e789000 0x1000>;

>> +

>> +                             #address-cells = <1>;

>> +                             #size-cells = <1>;

>> +                             ranges = <0x0 0x1e789000 0x1000>;

>> +

>> +                             lpc_bmc: lpc-bmc@0 {

>> +                                     compatible = "aspeed,ast2400-lpc-bmc";

>> +                                     reg = <0x0 0x80>;

>> +                             };

>> +

>> +                             lpc_host: lpc-host@80 {

>> +                                     compatible = "aspeed,ast2400-lpc-host", "simple-mfd", "syscon";

>> +                                     reg = <0x80 0x1e0>;

>> +                                     reg-io-width = <4>;

>> +

>> +                                     #address-cells = <1>;

>> +                                     #size-cells = <1>;

>> +                                     ranges = <0x0 0x80 0x1e0>;

>> +

>> +                                     lpc_ctrl: lpc-ctrl@0 {

>> +                                             compatible = "aspeed,ast2400-lpc-ctrl";

>> +                                             reg = <0x0 0x80>;

>> +                                             status = "disabled";

>> +                                     };

>> +

>> +                                     lhc: lhc@20 {

>> +                                             compatible = "aspeed,ast2500-lhc";

>

> aspeed,ast2400-lhc

>

> The layout of the registers are the same but there a couple of differences

> in the bit definitions between the two SoCs.

>

> a part from that :

>

> Reviewed-by: Cédric Le Goater <clg@kaod.org>


Good catch. Fixed in v3.

Cheers,

Joel
diff mbox series

Patch

diff --git a/arch/arm/boot/dts/aspeed-g4.dtsi b/arch/arm/boot/dts/aspeed-g4.dtsi
index 100d092e6c07..a3bc5da7d42c 100644
--- a/arch/arm/boot/dts/aspeed-g4.dtsi
+++ b/arch/arm/boot/dts/aspeed-g4.dtsi
@@ -226,6 +226,41 @@ 
 				status = "disabled";
 			};
 
+			lpc: lpc@1e789000 {
+				compatible = "aspeed,ast2400-lpc", "simple-mfd";
+				reg = <0x1e789000 0x1000>;
+
+				#address-cells = <1>;
+				#size-cells = <1>;
+				ranges = <0x0 0x1e789000 0x1000>;
+
+				lpc_bmc: lpc-bmc@0 {
+					compatible = "aspeed,ast2400-lpc-bmc";
+					reg = <0x0 0x80>;
+				};
+
+				lpc_host: lpc-host@80 {
+					compatible = "aspeed,ast2400-lpc-host", "simple-mfd", "syscon";
+					reg = <0x80 0x1e0>;
+					reg-io-width = <4>;
+
+					#address-cells = <1>;
+					#size-cells = <1>;
+					ranges = <0x0 0x80 0x1e0>;
+
+					lpc_ctrl: lpc-ctrl@0 {
+						compatible = "aspeed,ast2400-lpc-ctrl";
+						reg = <0x0 0x80>;
+						status = "disabled";
+					};
+
+					lhc: lhc@20 {
+						compatible = "aspeed,ast2500-lhc";
+						reg = <0x20 0x24 0x48 0x8>;
+					};
+				};
+			};
+
 			uart2: serial@1e78d000 {
 				compatible = "ns16550a";
 				reg = <0x1e78d000 0x20>;
diff --git a/arch/arm/boot/dts/aspeed-g5.dtsi b/arch/arm/boot/dts/aspeed-g5.dtsi
index 1f9d28313f82..7861631940fe 100644
--- a/arch/arm/boot/dts/aspeed-g5.dtsi
+++ b/arch/arm/boot/dts/aspeed-g5.dtsi
@@ -266,6 +266,16 @@ 
 				status = "disabled";
 			};
 
+			vuart: serial@1e787000 {
+				compatible = "aspeed,ast2500-vuart";
+				reg = <0x1e787000 0x40>;
+				reg-shift = <2>;
+				interrupts = <10>;
+				clocks = <&clk_uart>;
+				no-loopback-test;
+				status = "disabled";
+			};
+
 			lpc: lpc@1e789000 {
 				compatible = "aspeed,ast2500-lpc", "simple-mfd";
 				reg = <0x1e789000 0x1000>;
@@ -289,6 +299,13 @@ 
 
 					reg-io-width = <4>;
 
+					lpc_ctrl: lpc-ctrl@0 {
+						compatible = "aspeed,ast2500-lpc-ctrl";
+						reg = <0x0 0x80>;
+						status = "disabled";
+					};
+
+
 					lhc: lhc@20 {
 						compatible = "aspeed,ast2500-lhc";
 						reg = <0x20 0x24 0x48 0x8>;
@@ -296,16 +313,6 @@ 
 				};
 			};
 
-			vuart: serial@1e787000 {
-				compatible = "aspeed,ast2500-vuart";
-				reg = <0x1e787000 0x40>;
-				reg-shift = <2>;
-				interrupts = <10>;
-				clocks = <&clk_uart>;
-				no-loopback-test;
-				status = "disabled";
-			};
-
 			uart2: serial@1e78d000 {
 				compatible = "ns16550a";
 				reg = <0x1e78d000 0x20>;