[PATCH 0/3] ARM64: dts: meson-gx: Cleanup for common nodes

public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed

* [PATCH 0/3] ARM64: dts: meson-gx: Cleanup for common nodes
@ 2017-03-09 10:39 Neil Armstrong
  2017-03-09 10:39 ` [PATCH 1/3] ARM64: dts: meson-gx: Finally move common nodes to GX dtsi Neil Armstrong
                   ` (2 more replies)
  0 siblings, 3 replies; 7+ messages in thread
From: Neil Armstrong @ 2017-03-09 10:39 UTC (permalink / raw)
  To: carlo, khilman
  Cc: Neil Armstrong, linux-amlogic, linux-arm-kernel, linux-kernel,
	devicetree

Since we know the GXBBB and GXL/GXM share more hardware, we can safely move
the remaining peripheral nodes present in the GXBB dtsi to the common GX dtsi.
The following patches are small cleanups and helps to uniform the GX common
dtsi compatible scheme.

Neil Armstrong (3):
  ARM64: dts: meson-gx: Finally move common nodes to GX dtsi
  ARM64: dts: meson-gx: empty line cleanup
  ARM64: dts: meson-gx: Prepend GX generic compatible like other nodes

 arch/arm64/boot/dts/amlogic/meson-gx.dtsi   | 39 ++++++++++++++++++++------
 arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 43 ++++++-----------------------
 arch/arm64/boot/dts/amlogic/meson-gxl.dtsi  |  8 ++++++
 3 files changed, 47 insertions(+), 43 deletions(-)

-- 
1.9.1

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH 1/3] ARM64: dts: meson-gx: Finally move common nodes to GX dtsi
  2017-03-09 10:39 [PATCH 0/3] ARM64: dts: meson-gx: Cleanup for common nodes Neil Armstrong
@ 2017-03-09 10:39 ` Neil Armstrong
  2017-03-11  0:23   ` Kevin Hilman
  2017-03-09 10:39 ` [PATCH 2/3] ARM64: dts: meson-gx: empty line cleanup Neil Armstrong
  2017-03-09 10:39 ` [PATCH 3/3] ARM64: dts: meson-gx: Prepend GX generic compatible like other nodes Neil Armstrong
  2 siblings, 1 reply; 7+ messages in thread
From: Neil Armstrong @ 2017-03-09 10:39 UTC (permalink / raw)
  To: carlo, khilman
  Cc: Neil Armstrong, linux-amlogic, linux-arm-kernel, linux-kernel,
	devicetree

Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
---
 arch/arm64/boot/dts/amlogic/meson-gx.dtsi   | 24 ++++++++++++++++
 arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 43 ++++++-----------------------
 arch/arm64/boot/dts/amlogic/meson-gxl.dtsi  |  8 ++++++
 3 files changed, 40 insertions(+), 35 deletions(-)

diff --git a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi
index 5d995f7..2d8dc6f 100644
--- a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi
+++ b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi
@@ -296,6 +296,14 @@
 				status = "disabled";
 			};
 
+			spifc: spi@8c80 {
+				compatible = "amlogic,meson-gxbb-spifc";
+				reg = <0x0 0x08c80 0x0 0x80>;
+				#address-cells = <1>;
+				#size-cells = <0>;
+				status = "disabled";
+			};
+
 			watchdog@98d0 {
 				compatible = "amlogic,meson-gx-wdt", "amlogic,meson-gxbb-wdt";
 				reg = <0x0 0x098d0 0x0 0x10>;
@@ -342,6 +350,13 @@
 			#size-cells = <2>;
 			ranges = <0x0 0x0 0x0 0xc8100000 0x0 0x100000>;
 
+			clkc_AO: clock-controller@040 {
+				compatible = "amlogic,gx-aoclkc", "amlogic,gxbb-aoclkc";
+				reg = <0x0 0x00040 0x0 0x4>;
+				#clock-cells = <1>;
+				#reset-cells = <1>;
+			};
+
 			uart_AO: serial@4c0 {
 				compatible = "amlogic,meson-uart";
 				reg = <0x0 0x004c0 0x0 0x14>;
@@ -358,6 +373,15 @@
 				status = "disabled";
 			};
 
+			i2c_AO: i2c@500 {
+				compatible = "amlogic,meson-gx-i2c", "amlogic,meson-gxbb-i2c";
+				reg = <0x0 0x500 0x0 0x20>;
+				interrupts = <GIC_SPI 195 IRQ_TYPE_EDGE_RISING>;
+				#address-cells = <1>;
+				#size-cells = <0>;
+				status = "disabled";
+			};
+
 			pwm_AO_ab: pwm@550 {
 				compatible = "amlogic,meson-gx-pwm", "amlogic,meson-gxbb-pwm";
 				reg = <0x0 0x00550 0x0 0x10>;
diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi
index 04b3324..c2c41aa 100644
--- a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi
+++ b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi
@@ -97,17 +97,6 @@
 	};
 };
 
-&cbus {
-	spifc: spi@8c80 {
-		compatible = "amlogic,meson-gxbb-spifc";
-		reg = <0x0 0x08c80 0x0 0x80>;
-		#address-cells = <1>;
-		#size-cells = <0>;
-		clocks = <&clkc CLKID_SPI>;
-		status = "disabled";
-	};
-};
-
 &ethmac {
 	clocks = <&clkc CLKID_ETH>,
 		 <&clkc CLKID_FCLK_DIV2>,
@@ -204,30 +193,6 @@
 			};
 		};
 	};
-
-	clkc_AO: clock-controller@040 {
-		compatible = "amlogic,gxbb-aoclkc";
-		reg = <0x0 0x00040 0x0 0x4>;
-		#clock-cells = <1>;
-		#reset-cells = <1>;
-	};
-
-	pwm_ab_AO: pwm@550 {
-		compatible = "amlogic,meson-gxbb-pwm";
-		reg = <0x0 0x0550 0x0 0x10>;
-		#pwm-cells = <3>;
-		status = "disabled";
-	};
-
-	i2c_AO: i2c@500 {
-		compatible = "amlogic,meson-gxbb-i2c";
-		reg = <0x0 0x500 0x0 0x20>;
-		interrupts = <GIC_SPI 195 IRQ_TYPE_EDGE_RISING>;
-		clocks = <&clkc CLKID_AO_I2C>;
-		#address-cells = <1>;
-		#size-cells = <0>;
-		status = "disabled";
-	};
 };
 
 &periphs {
@@ -482,6 +447,10 @@
 	clocks = <&clkc CLKID_I2C>;
 };
 
+&i2c_AO {
+	clocks = <&clkc CLKID_AO_I2C>;
+};
+
 &i2c_B {
 	clocks = <&clkc CLKID_I2C>;
 };
@@ -521,6 +490,10 @@
 	clock-names = "core", "clkin0", "clkin1";
 };
 
+&spifc {
+	clocks = <&clkc CLKID_SPI>;
+};
+
 &vpu {
 	compatible = "amlogic,meson-gxbb-vpu", "amlogic,meson-gx-vpu";
 };
diff --git a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
index 17cd546..37ed7a0 100644
--- a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
+++ b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
@@ -427,6 +427,10 @@
 	clocks = <&clkc CLKID_I2C>;
 };
 
+&i2c_AO {
+	clocks = <&clkc CLKID_AO_I2C>;
+};
+
 &i2c_B {
 	clocks = <&clkc CLKID_I2C>;
 };
@@ -466,6 +470,10 @@
 	clock-names = "core", "clkin0", "clkin1";
 };
 
+&spifc {
+	clocks = <&clkc CLKID_SPI>;
+};
+
 &vpu {
 	compatible = "amlogic,meson-gxl-vpu", "amlogic,meson-gx-vpu";
 };
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH 1/3] ARM64: dts: meson-gx: Finally move common nodes to GX dtsi
  2017-03-09 10:39 ` [PATCH 1/3] ARM64: dts: meson-gx: Finally move common nodes to GX dtsi Neil Armstrong
@ 2017-03-11  0:23   ` Kevin Hilman
  2017-03-13  9:03     ` Neil Armstrong
  0 siblings, 1 reply; 7+ messages in thread
From: Kevin Hilman @ 2017-03-11  0:23 UTC (permalink / raw)
  To: Neil Armstrong
  Cc: carlo, linux-amlogic, linux-arm-kernel, linux-kernel, devicetree

Neil Armstrong <narmstrong@baylibre.com> writes:

> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>

Could use a bit more changelog here (probably what you have in the cover
letter.)

Otherwise, looks good.  Thanks for the cleanup.

Kevin

> ---
>  arch/arm64/boot/dts/amlogic/meson-gx.dtsi   | 24 ++++++++++++++++
>  arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 43 ++++++-----------------------
>  arch/arm64/boot/dts/amlogic/meson-gxl.dtsi  |  8 ++++++
>  3 files changed, 40 insertions(+), 35 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi
> index 5d995f7..2d8dc6f 100644
> --- a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi
> +++ b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi
> @@ -296,6 +296,14 @@
>  				status = "disabled";
>  			};
>  
> +			spifc: spi@8c80 {
> +				compatible = "amlogic,meson-gxbb-spifc";
> +				reg = <0x0 0x08c80 0x0 0x80>;
> +				#address-cells = <1>;
> +				#size-cells = <0>;
> +				status = "disabled";
> +			};
> +
>  			watchdog@98d0 {
>  				compatible = "amlogic,meson-gx-wdt", "amlogic,meson-gxbb-wdt";
>  				reg = <0x0 0x098d0 0x0 0x10>;
> @@ -342,6 +350,13 @@
>  			#size-cells = <2>;
>  			ranges = <0x0 0x0 0x0 0xc8100000 0x0 0x100000>;
>  
> +			clkc_AO: clock-controller@040 {
> +				compatible = "amlogic,gx-aoclkc", "amlogic,gxbb-aoclkc";
> +				reg = <0x0 0x00040 0x0 0x4>;
> +				#clock-cells = <1>;
> +				#reset-cells = <1>;
> +			};
> +
>  			uart_AO: serial@4c0 {
>  				compatible = "amlogic,meson-uart";
>  				reg = <0x0 0x004c0 0x0 0x14>;
> @@ -358,6 +373,15 @@
>  				status = "disabled";
>  			};
>  
> +			i2c_AO: i2c@500 {
> +				compatible = "amlogic,meson-gx-i2c", "amlogic,meson-gxbb-i2c";
> +				reg = <0x0 0x500 0x0 0x20>;
> +				interrupts = <GIC_SPI 195 IRQ_TYPE_EDGE_RISING>;
> +				#address-cells = <1>;
> +				#size-cells = <0>;
> +				status = "disabled";
> +			};
> +
>  			pwm_AO_ab: pwm@550 {
>  				compatible = "amlogic,meson-gx-pwm", "amlogic,meson-gxbb-pwm";
>  				reg = <0x0 0x00550 0x0 0x10>;
> diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi
> index 04b3324..c2c41aa 100644
> --- a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi
> +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi
> @@ -97,17 +97,6 @@
>  	};
>  };
>  
> -&cbus {
> -	spifc: spi@8c80 {
> -		compatible = "amlogic,meson-gxbb-spifc";
> -		reg = <0x0 0x08c80 0x0 0x80>;
> -		#address-cells = <1>;
> -		#size-cells = <0>;
> -		clocks = <&clkc CLKID_SPI>;
> -		status = "disabled";
> -	};
> -};
> -
>  &ethmac {
>  	clocks = <&clkc CLKID_ETH>,
>  		 <&clkc CLKID_FCLK_DIV2>,
> @@ -204,30 +193,6 @@
>  			};
>  		};
>  	};
> -
> -	clkc_AO: clock-controller@040 {
> -		compatible = "amlogic,gxbb-aoclkc";
> -		reg = <0x0 0x00040 0x0 0x4>;
> -		#clock-cells = <1>;
> -		#reset-cells = <1>;
> -	};
> -
> -	pwm_ab_AO: pwm@550 {
> -		compatible = "amlogic,meson-gxbb-pwm";
> -		reg = <0x0 0x0550 0x0 0x10>;
> -		#pwm-cells = <3>;
> -		status = "disabled";
> -	};
> -
> -	i2c_AO: i2c@500 {
> -		compatible = "amlogic,meson-gxbb-i2c";
> -		reg = <0x0 0x500 0x0 0x20>;
> -		interrupts = <GIC_SPI 195 IRQ_TYPE_EDGE_RISING>;
> -		clocks = <&clkc CLKID_AO_I2C>;
> -		#address-cells = <1>;
> -		#size-cells = <0>;
> -		status = "disabled";
> -	};
>  };
>  
>  &periphs {
> @@ -482,6 +447,10 @@
>  	clocks = <&clkc CLKID_I2C>;
>  };
>  
> +&i2c_AO {
> +	clocks = <&clkc CLKID_AO_I2C>;
> +};
> +
>  &i2c_B {
>  	clocks = <&clkc CLKID_I2C>;
>  };
> @@ -521,6 +490,10 @@
>  	clock-names = "core", "clkin0", "clkin1";
>  };
>  
> +&spifc {
> +	clocks = <&clkc CLKID_SPI>;
> +};
> +
>  &vpu {
>  	compatible = "amlogic,meson-gxbb-vpu", "amlogic,meson-gx-vpu";
>  };
> diff --git a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
> index 17cd546..37ed7a0 100644
> --- a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
> +++ b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
> @@ -427,6 +427,10 @@
>  	clocks = <&clkc CLKID_I2C>;
>  };
>  
> +&i2c_AO {
> +	clocks = <&clkc CLKID_AO_I2C>;
> +};
> +
>  &i2c_B {
>  	clocks = <&clkc CLKID_I2C>;
>  };
> @@ -466,6 +470,10 @@
>  	clock-names = "core", "clkin0", "clkin1";
>  };
>  
> +&spifc {
> +	clocks = <&clkc CLKID_SPI>;
> +};
> +
>  &vpu {
>  	compatible = "amlogic,meson-gxl-vpu", "amlogic,meson-gx-vpu";
>  };

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH 1/3] ARM64: dts: meson-gx: Finally move common nodes to GX dtsi
  2017-03-11  0:23   ` Kevin Hilman
@ 2017-03-13  9:03     ` Neil Armstrong
  0 siblings, 0 replies; 7+ messages in thread
From: Neil Armstrong @ 2017-03-13  9:03 UTC (permalink / raw)
  To: Kevin Hilman
  Cc: carlo, linux-amlogic, linux-arm-kernel, linux-kernel, devicetree

On 03/11/2017 01:23 AM, Kevin Hilman wrote:
> Neil Armstrong <narmstrong@baylibre.com> writes:
> 
>> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
> 
> Could use a bit more changelog here (probably what you have in the cover
> letter.)

Oops, sent the wrong files...

Will resend the patchset with changelog for patches 1 and 3.

Neil

> 
> Otherwise, looks good.  Thanks for the cleanup.
> 
> Kevin
> 
>> ---
>>  arch/arm64/boot/dts/amlogic/meson-gx.dtsi   | 24 ++++++++++++++++
>>  arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi | 43 ++++++-----------------------
>>  arch/arm64/boot/dts/amlogic/meson-gxl.dtsi  |  8 ++++++
>>  3 files changed, 40 insertions(+), 35 deletions(-)
>>
>> diff --git a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi
>> index 5d995f7..2d8dc6f 100644
>> --- a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi
>> +++ b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi
>> @@ -296,6 +296,14 @@
>>  				status = "disabled";
>>  			};
>>  
>> +			spifc: spi@8c80 {
>> +				compatible = "amlogic,meson-gxbb-spifc";
>> +				reg = <0x0 0x08c80 0x0 0x80>;
>> +				#address-cells = <1>;
>> +				#size-cells = <0>;
>> +				status = "disabled";
>> +			};
>> +
>>  			watchdog@98d0 {
>>  				compatible = "amlogic,meson-gx-wdt", "amlogic,meson-gxbb-wdt";
>>  				reg = <0x0 0x098d0 0x0 0x10>;
>> @@ -342,6 +350,13 @@
>>  			#size-cells = <2>;
>>  			ranges = <0x0 0x0 0x0 0xc8100000 0x0 0x100000>;
>>  
>> +			clkc_AO: clock-controller@040 {
>> +				compatible = "amlogic,gx-aoclkc", "amlogic,gxbb-aoclkc";
>> +				reg = <0x0 0x00040 0x0 0x4>;
>> +				#clock-cells = <1>;
>> +				#reset-cells = <1>;
>> +			};
>> +
>>  			uart_AO: serial@4c0 {
>>  				compatible = "amlogic,meson-uart";
>>  				reg = <0x0 0x004c0 0x0 0x14>;
>> @@ -358,6 +373,15 @@
>>  				status = "disabled";
>>  			};
>>  
>> +			i2c_AO: i2c@500 {
>> +				compatible = "amlogic,meson-gx-i2c", "amlogic,meson-gxbb-i2c";
>> +				reg = <0x0 0x500 0x0 0x20>;
>> +				interrupts = <GIC_SPI 195 IRQ_TYPE_EDGE_RISING>;
>> +				#address-cells = <1>;
>> +				#size-cells = <0>;
>> +				status = "disabled";
>> +			};
>> +
>>  			pwm_AO_ab: pwm@550 {
>>  				compatible = "amlogic,meson-gx-pwm", "amlogic,meson-gxbb-pwm";
>>  				reg = <0x0 0x00550 0x0 0x10>;
>> diff --git a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi
>> index 04b3324..c2c41aa 100644
>> --- a/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi
>> +++ b/arch/arm64/boot/dts/amlogic/meson-gxbb.dtsi
>> @@ -97,17 +97,6 @@
>>  	};
>>  };
>>  
>> -&cbus {
>> -	spifc: spi@8c80 {
>> -		compatible = "amlogic,meson-gxbb-spifc";
>> -		reg = <0x0 0x08c80 0x0 0x80>;
>> -		#address-cells = <1>;
>> -		#size-cells = <0>;
>> -		clocks = <&clkc CLKID_SPI>;
>> -		status = "disabled";
>> -	};
>> -};
>> -
>>  &ethmac {
>>  	clocks = <&clkc CLKID_ETH>,
>>  		 <&clkc CLKID_FCLK_DIV2>,
>> @@ -204,30 +193,6 @@
>>  			};
>>  		};
>>  	};
>> -
>> -	clkc_AO: clock-controller@040 {
>> -		compatible = "amlogic,gxbb-aoclkc";
>> -		reg = <0x0 0x00040 0x0 0x4>;
>> -		#clock-cells = <1>;
>> -		#reset-cells = <1>;
>> -	};
>> -
>> -	pwm_ab_AO: pwm@550 {
>> -		compatible = "amlogic,meson-gxbb-pwm";
>> -		reg = <0x0 0x0550 0x0 0x10>;
>> -		#pwm-cells = <3>;
>> -		status = "disabled";
>> -	};
>> -
>> -	i2c_AO: i2c@500 {
>> -		compatible = "amlogic,meson-gxbb-i2c";
>> -		reg = <0x0 0x500 0x0 0x20>;
>> -		interrupts = <GIC_SPI 195 IRQ_TYPE_EDGE_RISING>;
>> -		clocks = <&clkc CLKID_AO_I2C>;
>> -		#address-cells = <1>;
>> -		#size-cells = <0>;
>> -		status = "disabled";
>> -	};
>>  };
>>  
>>  &periphs {
>> @@ -482,6 +447,10 @@
>>  	clocks = <&clkc CLKID_I2C>;
>>  };
>>  
>> +&i2c_AO {
>> +	clocks = <&clkc CLKID_AO_I2C>;
>> +};
>> +
>>  &i2c_B {
>>  	clocks = <&clkc CLKID_I2C>;
>>  };
>> @@ -521,6 +490,10 @@
>>  	clock-names = "core", "clkin0", "clkin1";
>>  };
>>  
>> +&spifc {
>> +	clocks = <&clkc CLKID_SPI>;
>> +};
>> +
>>  &vpu {
>>  	compatible = "amlogic,meson-gxbb-vpu", "amlogic,meson-gx-vpu";
>>  };
>> diff --git a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
>> index 17cd546..37ed7a0 100644
>> --- a/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
>> +++ b/arch/arm64/boot/dts/amlogic/meson-gxl.dtsi
>> @@ -427,6 +427,10 @@
>>  	clocks = <&clkc CLKID_I2C>;
>>  };
>>  
>> +&i2c_AO {
>> +	clocks = <&clkc CLKID_AO_I2C>;
>> +};
>> +
>>  &i2c_B {
>>  	clocks = <&clkc CLKID_I2C>;
>>  };
>> @@ -466,6 +470,10 @@
>>  	clock-names = "core", "clkin0", "clkin1";
>>  };
>>  
>> +&spifc {
>> +	clocks = <&clkc CLKID_SPI>;
>> +};
>> +
>>  &vpu {
>>  	compatible = "amlogic,meson-gxl-vpu", "amlogic,meson-gx-vpu";
>>  };

^ permalink raw reply	[flat|nested] 7+ messages in thread

* [PATCH 2/3] ARM64: dts: meson-gx: empty line cleanup
  2017-03-09 10:39 [PATCH 0/3] ARM64: dts: meson-gx: Cleanup for common nodes Neil Armstrong
  2017-03-09 10:39 ` [PATCH 1/3] ARM64: dts: meson-gx: Finally move common nodes to GX dtsi Neil Armstrong
@ 2017-03-09 10:39 ` Neil Armstrong
  2017-03-09 10:39 ` [PATCH 3/3] ARM64: dts: meson-gx: Prepend GX generic compatible like other nodes Neil Armstrong
  2 siblings, 0 replies; 7+ messages in thread
From: Neil Armstrong @ 2017-03-09 10:39 UTC (permalink / raw)
  To: carlo, khilman
  Cc: Neil Armstrong, linux-amlogic, linux-arm-kernel, linux-kernel,
	devicetree

Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
---
 arch/arm64/boot/dts/amlogic/meson-gx.dtsi | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi
index 2d8dc6f..06d70bf 100644
--- a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi
+++ b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi
@@ -410,7 +410,6 @@
 			};
 		};
 
-
 		hiubus: hiubus@c883c000 {
 			compatible = "simple-bus";
 			reg = <0x0 0xc883c000 0x0 0x2000>;
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH 3/3] ARM64: dts: meson-gx: Prepend GX generic compatible like other nodes
  2017-03-09 10:39 [PATCH 0/3] ARM64: dts: meson-gx: Cleanup for common nodes Neil Armstrong
  2017-03-09 10:39 ` [PATCH 1/3] ARM64: dts: meson-gx: Finally move common nodes to GX dtsi Neil Armstrong
  2017-03-09 10:39 ` [PATCH 2/3] ARM64: dts: meson-gx: empty line cleanup Neil Armstrong
@ 2017-03-09 10:39 ` Neil Armstrong
  2017-03-11  0:24   ` Kevin Hilman
  2 siblings, 1 reply; 7+ messages in thread
From: Neil Armstrong @ 2017-03-09 10:39 UTC (permalink / raw)
  To: carlo, khilman
  Cc: Neil Armstrong, linux-amlogic, linux-arm-kernel, linux-kernel,
	devicetree

Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>
---
 arch/arm64/boot/dts/amlogic/meson-gx.dtsi | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi
index 06d70bf..a010ab9 100644
--- a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi
+++ b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi
@@ -233,7 +233,7 @@
 			};
 
 			i2c_A: i2c@8500 {
-				compatible = "amlogic,meson-gxbb-i2c";
+				compatible = "amlogic,meson-gx-i2c", "amlogic,meson-gxbb-i2c";
 				reg = <0x0 0x08500 0x0 0x20>;
 				interrupts = <GIC_SPI 21 IRQ_TYPE_EDGE_RISING>;
 				#address-cells = <1>;
@@ -279,7 +279,7 @@
 			};
 
 			i2c_B: i2c@87c0 {
-				compatible = "amlogic,meson-gxbb-i2c";
+				compatible = "amlogic,meson-gx-i2c", "amlogic,meson-gxbb-i2c";
 				reg = <0x0 0x087c0 0x0 0x20>;
 				interrupts = <GIC_SPI 214 IRQ_TYPE_EDGE_RISING>;
 				#address-cells = <1>;
@@ -288,7 +288,7 @@
 			};
 
 			i2c_C: i2c@87e0 {
-				compatible = "amlogic,meson-gxbb-i2c";
+				compatible = "amlogic,meson-gx-i2c", "amlogic,meson-gxbb-i2c";
 				reg = <0x0 0x087e0 0x0 0x20>;
 				interrupts = <GIC_SPI 215 IRQ_TYPE_EDGE_RISING>;
 				#address-cells = <1>;
@@ -297,7 +297,7 @@
 			};
 
 			spifc: spi@8c80 {
-				compatible = "amlogic,meson-gxbb-spifc";
+				compatible = "amlogic,meson-gx-spifc", "amlogic,meson-gxbb-spifc";
 				reg = <0x0 0x08c80 0x0 0x80>;
 				#address-cells = <1>;
 				#size-cells = <0>;
@@ -325,7 +325,7 @@
 		};
 
 		sram: sram@c8000000 {
-			compatible = "amlogic,meson-gxbb-sram", "mmio-sram";
+			compatible = "amlogic,meson-gx-sram", "amlogic,meson-gxbb-sram", "mmio-sram";
 			reg = <0x0 0xc8000000 0x0 0x14000>;
 
 			#address-cells = <1>;
@@ -333,12 +333,12 @@
 			ranges = <0 0x0 0xc8000000 0x14000>;
 
 			cpu_scp_lpri: scp-shmem@0 {
-				compatible = "amlogic,meson-gxbb-scp-shmem";
+				compatible = "amlogic,meson-gx-scp-shmem", "amlogic,meson-gxbb-scp-shmem";
 				reg = <0x13000 0x400>;
 			};
 
 			cpu_scp_hpri: scp-shmem@200 {
-				compatible = "amlogic,meson-gxbb-scp-shmem";
+				compatible = "amlogic,meson-gx-scp-shmem", "amlogic,meson-gxbb-scp-shmem";
 				reg = <0x13400 0x400>;
 			};
 		};
@@ -390,7 +390,7 @@
 			};
 
 			ir: ir@580 {
-				compatible = "amlogic,meson-gxbb-ir";
+				compatible = "amlogic,meson-gx-ir", "amlogic,meson-gxbb-ir";
 				reg = <0x0 0x00580 0x0 0x40>;
 				interrupts = <GIC_SPI 196 IRQ_TYPE_EDGE_RISING>;
 				status = "disabled";
-- 
1.9.1

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH 3/3] ARM64: dts: meson-gx: Prepend GX generic compatible like other nodes
  2017-03-09 10:39 ` [PATCH 3/3] ARM64: dts: meson-gx: Prepend GX generic compatible like other nodes Neil Armstrong
@ 2017-03-11  0:24   ` Kevin Hilman
  0 siblings, 0 replies; 7+ messages in thread
From: Kevin Hilman @ 2017-03-11  0:24 UTC (permalink / raw)
  To: Neil Armstrong
  Cc: carlo, linux-amlogic, linux-arm-kernel, linux-kernel, devicetree

Neil Armstrong <narmstrong@baylibre.com> writes:

> Signed-off-by: Neil Armstrong <narmstrong@baylibre.com>

Could use a bit more changelog here  (in order to support generic
drivers that work for all GX family, blah, blah...)

Kevin

> ---
>  arch/arm64/boot/dts/amlogic/meson-gx.dtsi | 16 ++++++++--------
>  1 file changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi
> index 06d70bf..a010ab9 100644
> --- a/arch/arm64/boot/dts/amlogic/meson-gx.dtsi
> +++ b/arch/arm64/boot/dts/amlogic/meson-gx.dtsi
> @@ -233,7 +233,7 @@
>  			};
>  
>  			i2c_A: i2c@8500 {
> -				compatible = "amlogic,meson-gxbb-i2c";
> +				compatible = "amlogic,meson-gx-i2c", "amlogic,meson-gxbb-i2c";
>  				reg = <0x0 0x08500 0x0 0x20>;
>  				interrupts = <GIC_SPI 21 IRQ_TYPE_EDGE_RISING>;
>  				#address-cells = <1>;
> @@ -279,7 +279,7 @@
>  			};
>  
>  			i2c_B: i2c@87c0 {
> -				compatible = "amlogic,meson-gxbb-i2c";
> +				compatible = "amlogic,meson-gx-i2c", "amlogic,meson-gxbb-i2c";
>  				reg = <0x0 0x087c0 0x0 0x20>;
>  				interrupts = <GIC_SPI 214 IRQ_TYPE_EDGE_RISING>;
>  				#address-cells = <1>;
> @@ -288,7 +288,7 @@
>  			};
>  
>  			i2c_C: i2c@87e0 {
> -				compatible = "amlogic,meson-gxbb-i2c";
> +				compatible = "amlogic,meson-gx-i2c", "amlogic,meson-gxbb-i2c";
>  				reg = <0x0 0x087e0 0x0 0x20>;
>  				interrupts = <GIC_SPI 215 IRQ_TYPE_EDGE_RISING>;
>  				#address-cells = <1>;
> @@ -297,7 +297,7 @@
>  			};
>  
>  			spifc: spi@8c80 {
> -				compatible = "amlogic,meson-gxbb-spifc";
> +				compatible = "amlogic,meson-gx-spifc", "amlogic,meson-gxbb-spifc";
>  				reg = <0x0 0x08c80 0x0 0x80>;
>  				#address-cells = <1>;
>  				#size-cells = <0>;
> @@ -325,7 +325,7 @@
>  		};
>  
>  		sram: sram@c8000000 {
> -			compatible = "amlogic,meson-gxbb-sram", "mmio-sram";
> +			compatible = "amlogic,meson-gx-sram", "amlogic,meson-gxbb-sram", "mmio-sram";
>  			reg = <0x0 0xc8000000 0x0 0x14000>;
>  
>  			#address-cells = <1>;
> @@ -333,12 +333,12 @@
>  			ranges = <0 0x0 0xc8000000 0x14000>;
>  
>  			cpu_scp_lpri: scp-shmem@0 {
> -				compatible = "amlogic,meson-gxbb-scp-shmem";
> +				compatible = "amlogic,meson-gx-scp-shmem", "amlogic,meson-gxbb-scp-shmem";
>  				reg = <0x13000 0x400>;
>  			};
>  
>  			cpu_scp_hpri: scp-shmem@200 {
> -				compatible = "amlogic,meson-gxbb-scp-shmem";
> +				compatible = "amlogic,meson-gx-scp-shmem", "amlogic,meson-gxbb-scp-shmem";
>  				reg = <0x13400 0x400>;
>  			};
>  		};
> @@ -390,7 +390,7 @@
>  			};
>  
>  			ir: ir@580 {
> -				compatible = "amlogic,meson-gxbb-ir";
> +				compatible = "amlogic,meson-gx-ir", "amlogic,meson-gxbb-ir";
>  				reg = <0x0 0x00580 0x0 0x40>;
>  				interrupts = <GIC_SPI 196 IRQ_TYPE_EDGE_RISING>;
>  				status = "disabled";

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2017-03-13  9:03 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-03-09 10:39 [PATCH 0/3] ARM64: dts: meson-gx: Cleanup for common nodes Neil Armstrong
2017-03-09 10:39 ` [PATCH 1/3] ARM64: dts: meson-gx: Finally move common nodes to GX dtsi Neil Armstrong
2017-03-11  0:23   ` Kevin Hilman
2017-03-13  9:03     ` Neil Armstrong
2017-03-09 10:39 ` [PATCH 2/3] ARM64: dts: meson-gx: empty line cleanup Neil Armstrong
2017-03-09 10:39 ` [PATCH 3/3] ARM64: dts: meson-gx: Prepend GX generic compatible like other nodes Neil Armstrong
2017-03-11  0:24   ` Kevin Hilman

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox