linux-sh.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH/RFC] ARM: shmobile: only enable used I2C interfaces in DT on all Renesas boards
@ 2013-09-26 11:06 Guennadi Liakhovetski
  2013-09-26 11:12 ` Mark Rutland
       [not found] ` <Pine.LNX.4.64.1309261259490.11968-0199iw4Nj15frtckUFj5Ag@public.gmane.org>
  0 siblings, 2 replies; 5+ messages in thread
From: Guennadi Liakhovetski @ 2013-09-26 11:06 UTC (permalink / raw)
  To: linux-sh-u79uwXL29TY76Z2rM5mHXA
  Cc: Laurent Pinchart, Linus Walleij, Magnus Damm,
	devicetree-u79uwXL29TY76Z2rM5mHXA, Simon Horman

Currently all I2C interfaces in all *.dtsi files for various Renesas SoCs
are enabled by default. Switch them all off and only enable populated I2C
interfaces in board-specific *.dts files.

Signed-off-by: Guennadi Liakhovetski <g.liakhovetski+renesas@gmail.com>
---

RFC because this can break things. E.g. if someone relied on an I2C bus 
being available and used it from the user-space. Otherwise I don't think 
it should break anything. Compile-tested all affected .dtb, run-time 
tested on kzm9g.

 arch/arm/boot/dts/r8a73a4-ape6evm-reference.dts    |    1 +
 arch/arm/boot/dts/r8a73a4-ape6evm.dts              |    1 +
 arch/arm/boot/dts/r8a73a4.dtsi                     |    9 +++++++++
 .../boot/dts/r8a7740-armadillo800eva-reference.dts |    1 +
 arch/arm/boot/dts/r8a7740.dtsi                     |    2 ++
 arch/arm/boot/dts/r8a7779.dtsi                     |    4 ++++
 arch/arm/boot/dts/sh73a0-kzm9g-reference.dts       |    2 ++
 arch/arm/boot/dts/sh73a0.dtsi                      |    5 +++++
 8 files changed, 25 insertions(+), 0 deletions(-)

diff --git a/arch/arm/boot/dts/r8a73a4-ape6evm-reference.dts b/arch/arm/boot/dts/r8a73a4-ape6evm-reference.dts
index 2b49b05..9443e93 100644
--- a/arch/arm/boot/dts/r8a73a4-ape6evm-reference.dts
+++ b/arch/arm/boot/dts/r8a73a4-ape6evm-reference.dts
@@ -62,6 +62,7 @@
 };
 
 &i2c5 {
+	status = "okay";
 	vdd_dvfs: max8973@1b {
 		compatible = "maxim,max8973";
 		reg = <0x1b>;
diff --git a/arch/arm/boot/dts/r8a73a4-ape6evm.dts b/arch/arm/boot/dts/r8a73a4-ape6evm.dts
index 72f867e..91436b5 100644
--- a/arch/arm/boot/dts/r8a73a4-ape6evm.dts
+++ b/arch/arm/boot/dts/r8a73a4-ape6evm.dts
@@ -52,6 +52,7 @@
 };
 
 &i2c5 {
+	status = "okay";
 	vdd_dvfs: max8973@1b {
 		compatible = "maxim,max8973";
 		reg = <0x1b>;
diff --git a/arch/arm/boot/dts/r8a73a4.dtsi b/arch/arm/boot/dts/r8a73a4.dtsi
index 658fcc5..6048825 100644
--- a/arch/arm/boot/dts/r8a73a4.dtsi
+++ b/arch/arm/boot/dts/r8a73a4.dtsi
@@ -93,6 +93,7 @@
 		reg = <0 0xe6500000 0 0x428>;
 		interrupt-parent = <&gic>;
 		interrupts = <0 174 0x4>;
+		status = "disabled";
 	};
 
 	i2c1: i2c@e6510000 {
@@ -102,6 +103,7 @@
 		reg = <0 0xe6510000 0 0x428>;
 		interrupt-parent = <&gic>;
 		interrupts = <0 175 0x4>;
+		status = "disabled";
 	};
 
 	i2c2: i2c@e6520000 {
@@ -111,6 +113,7 @@
 		reg = <0 0xe6520000 0 0x428>;
 		interrupt-parent = <&gic>;
 		interrupts = <0 176 0x4>;
+		status = "disabled";
 	};
 
 	i2c3: i2c@e6530000 {
@@ -120,6 +123,7 @@
 		reg = <0 0xe6530000 0 0x428>;
 		interrupt-parent = <&gic>;
 		interrupts = <0 177 0x4>;
+		status = "disabled";
 	};
 
 	i2c4: i2c@e6540000 {
@@ -129,6 +133,7 @@
 		reg = <0 0xe6540000 0 0x428>;
 		interrupt-parent = <&gic>;
 		interrupts = <0 178 0x4>;
+		status = "disabled";
 	};
 
 	i2c5: i2c@e60b0000 {
@@ -138,6 +143,7 @@
 		reg = <0 0xe60b0000 0 0x428>;
 		interrupt-parent = <&gic>;
 		interrupts = <0 179 0x4>;
+		status = "disabled";
 	};
 
 	i2c6: i2c@e6550000 {
@@ -147,6 +153,7 @@
 		reg = <0 0xe6550000 0 0x428>;
 		interrupt-parent = <&gic>;
 		interrupts = <0 184 0x4>;
+		status = "disabled";
 	};
 
 	i2c7: i2c@e6560000 {
@@ -156,6 +163,7 @@
 		reg = <0 0xe6560000 0 0x428>;
 		interrupt-parent = <&gic>;
 		interrupts = <0 185 0x4>;
+		status = "disabled";
 	};
 
 	i2c8: i2c@e6570000 {
@@ -165,6 +173,7 @@
 		reg = <0 0xe6570000 0 0x428>;
 		interrupt-parent = <&gic>;
 		interrupts = <0 173 0x4>;
+		status = "disabled";
 	};
 
 	mmcif0: mmcif@ee200000 {
diff --git a/arch/arm/boot/dts/r8a7740-armadillo800eva-reference.dts b/arch/arm/boot/dts/r8a7740-armadillo800eva-reference.dts
index 801442b..8b2aab5 100644
--- a/arch/arm/boot/dts/r8a7740-armadillo800eva-reference.dts
+++ b/arch/arm/boot/dts/r8a7740-armadillo800eva-reference.dts
@@ -62,6 +62,7 @@
 };
 
 &i2c0 {
+	status = "okay";
 	touchscreen: st1232@55 {
 		compatible = "sitronix,st1232";
 		reg = <0x55>;
diff --git a/arch/arm/boot/dts/r8a7740.dtsi b/arch/arm/boot/dts/r8a7740.dtsi
index 44d3d52..868bdde 100644
--- a/arch/arm/boot/dts/r8a7740.dtsi
+++ b/arch/arm/boot/dts/r8a7740.dtsi
@@ -131,6 +131,7 @@
 			      0 202 0x4
 			      0 203 0x4
 			      0 204 0x4>;
+		status = "disabled";
 	};
 
 	i2c1: i2c@e6c20000 {
@@ -143,6 +144,7 @@
 			      0 71 0x4
 			      0 72 0x4
 			      0 73 0x4>;
+		status = "disabled";
 	};
 
 	pfc: pfc@e6050000 {
diff --git a/arch/arm/boot/dts/r8a7779.dtsi b/arch/arm/boot/dts/r8a7779.dtsi
index ebbe507..912b3a0 100644
--- a/arch/arm/boot/dts/r8a7779.dtsi
+++ b/arch/arm/boot/dts/r8a7779.dtsi
@@ -156,6 +156,7 @@
 		reg = <0xffc70000 0x1000>;
 		interrupt-parent = <&gic>;
 		interrupts = <0 79 0x4>;
+		status = "disabled";
 	};
 
 	i2c1: i2c@ffc71000 {
@@ -165,6 +166,7 @@
 		reg = <0xffc71000 0x1000>;
 		interrupt-parent = <&gic>;
 		interrupts = <0 82 0x4>;
+		status = "disabled";
 	};
 
 	i2c2: i2c@ffc72000 {
@@ -174,6 +176,7 @@
 		reg = <0xffc72000 0x1000>;
 		interrupt-parent = <&gic>;
 		interrupts = <0 80 0x4>;
+		status = "disabled";
 	};
 
 	i2c3: i2c@ffc73000 {
@@ -183,6 +186,7 @@
 		reg = <0xffc73000 0x1000>;
 		interrupt-parent = <&gic>;
 		interrupts = <0 81 0x4>;
+		status = "disabled";
 	};
 
 	pfc: pfc@fffc0000 {
diff --git a/arch/arm/boot/dts/sh73a0-kzm9g-reference.dts b/arch/arm/boot/dts/sh73a0-kzm9g-reference.dts
index 2122306..8ee06dd 100644
--- a/arch/arm/boot/dts/sh73a0-kzm9g-reference.dts
+++ b/arch/arm/boot/dts/sh73a0-kzm9g-reference.dts
@@ -108,6 +108,7 @@
 };
 
 &i2c0 {
+	status = "okay";
 	as3711@40 {
 		compatible = "ams,as3711";
 		reg = <0x40>;
@@ -183,6 +184,7 @@
 &i2c3 {
 	pinctrl-0 = <&i2c3_pins>;
 	pinctrl-names = "default";
+	status = "okay";
 };
 
 &mmcif {
diff --git a/arch/arm/boot/dts/sh73a0.dtsi b/arch/arm/boot/dts/sh73a0.dtsi
index 3955c76..fcf2688 100644
--- a/arch/arm/boot/dts/sh73a0.dtsi
+++ b/arch/arm/boot/dts/sh73a0.dtsi
@@ -135,6 +135,7 @@
 			      0 168 0x4
 			      0 169 0x4
 			      0 170 0x4>;
+		status = "disabled";
 	};
 
 	i2c1: i2c@e6822000 {
@@ -147,6 +148,7 @@
 			      0 52 0x4
 			      0 53 0x4
 			      0 54 0x4>;
+		status = "disabled";
 	};
 
 	i2c2: i2c@e6824000 {
@@ -159,6 +161,7 @@
 			      0 172 0x4
 			      0 173 0x4
 			      0 174 0x4>;
+		status = "disabled";
 	};
 
 	i2c3: i2c@e6826000 {
@@ -171,6 +174,7 @@
 			      0 184 0x4
 			      0 185 0x4
 			      0 186 0x4>;
+		status = "disabled";
 	};
 
 	i2c4: i2c@e6828000 {
@@ -183,6 +187,7 @@
 			      0 188 0x4
 			      0 189 0x4
 			      0 190 0x4>;
+		status = "disabled";
 	};
 
 	mmcif: mmcif@e6bd0000 {
-- 
1.7.2.5


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

* Re: [PATCH/RFC] ARM: shmobile: only enable used I2C interfaces in DT on all Renesas boards
  2013-09-26 11:06 [PATCH/RFC] ARM: shmobile: only enable used I2C interfaces in DT on all Renesas boards Guennadi Liakhovetski
@ 2013-09-26 11:12 ` Mark Rutland
       [not found] ` <Pine.LNX.4.64.1309261259490.11968-0199iw4Nj15frtckUFj5Ag@public.gmane.org>
  1 sibling, 0 replies; 5+ messages in thread
From: Mark Rutland @ 2013-09-26 11:12 UTC (permalink / raw)
  To: Guennadi Liakhovetski
  Cc: linux-sh@vger.kernel.org, Laurent Pinchart, Linus Walleij,
	Magnus Damm, devicetree@vger.kernel.org, Simon Horman

On Thu, Sep 26, 2013 at 12:06:01PM +0100, Guennadi Liakhovetski wrote:
> Currently all I2C interfaces in all *.dtsi files for various Renesas SoCs
> are enabled by default. Switch them all off and only enable populated I2C
> interfaces in board-specific *.dts files.

I agree with this approach. Having unusable/non-present devices enabled
by default in DTs is not a great situation.

Acked-by: Mark Rutland <mark.rutland@arm.com>

> 
> Signed-off-by: Guennadi Liakhovetski <g.liakhovetski+renesas@gmail.com>
> ---
> 
> RFC because this can break things. E.g. if someone relied on an I2C bus 
> being available and used it from the user-space. Otherwise I don't think 
> it should break anything. Compile-tested all affected .dtb, run-time 
> tested on kzm9g.
> 
>  arch/arm/boot/dts/r8a73a4-ape6evm-reference.dts    |    1 +
>  arch/arm/boot/dts/r8a73a4-ape6evm.dts              |    1 +
>  arch/arm/boot/dts/r8a73a4.dtsi                     |    9 +++++++++
>  .../boot/dts/r8a7740-armadillo800eva-reference.dts |    1 +
>  arch/arm/boot/dts/r8a7740.dtsi                     |    2 ++
>  arch/arm/boot/dts/r8a7779.dtsi                     |    4 ++++
>  arch/arm/boot/dts/sh73a0-kzm9g-reference.dts       |    2 ++
>  arch/arm/boot/dts/sh73a0.dtsi                      |    5 +++++
>  8 files changed, 25 insertions(+), 0 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/r8a73a4-ape6evm-reference.dts b/arch/arm/boot/dts/r8a73a4-ape6evm-reference.dts
> index 2b49b05..9443e93 100644
> --- a/arch/arm/boot/dts/r8a73a4-ape6evm-reference.dts
> +++ b/arch/arm/boot/dts/r8a73a4-ape6evm-reference.dts
> @@ -62,6 +62,7 @@
>  };
>  
>  &i2c5 {
> +	status = "okay";
>  	vdd_dvfs: max8973@1b {
>  		compatible = "maxim,max8973";
>  		reg = <0x1b>;
> diff --git a/arch/arm/boot/dts/r8a73a4-ape6evm.dts b/arch/arm/boot/dts/r8a73a4-ape6evm.dts
> index 72f867e..91436b5 100644
> --- a/arch/arm/boot/dts/r8a73a4-ape6evm.dts
> +++ b/arch/arm/boot/dts/r8a73a4-ape6evm.dts
> @@ -52,6 +52,7 @@
>  };
>  
>  &i2c5 {
> +	status = "okay";
>  	vdd_dvfs: max8973@1b {
>  		compatible = "maxim,max8973";
>  		reg = <0x1b>;
> diff --git a/arch/arm/boot/dts/r8a73a4.dtsi b/arch/arm/boot/dts/r8a73a4.dtsi
> index 658fcc5..6048825 100644
> --- a/arch/arm/boot/dts/r8a73a4.dtsi
> +++ b/arch/arm/boot/dts/r8a73a4.dtsi
> @@ -93,6 +93,7 @@
>  		reg = <0 0xe6500000 0 0x428>;
>  		interrupt-parent = <&gic>;
>  		interrupts = <0 174 0x4>;
> +		status = "disabled";
>  	};
>  
>  	i2c1: i2c@e6510000 {
> @@ -102,6 +103,7 @@
>  		reg = <0 0xe6510000 0 0x428>;
>  		interrupt-parent = <&gic>;
>  		interrupts = <0 175 0x4>;
> +		status = "disabled";
>  	};
>  
>  	i2c2: i2c@e6520000 {
> @@ -111,6 +113,7 @@
>  		reg = <0 0xe6520000 0 0x428>;
>  		interrupt-parent = <&gic>;
>  		interrupts = <0 176 0x4>;
> +		status = "disabled";
>  	};
>  
>  	i2c3: i2c@e6530000 {
> @@ -120,6 +123,7 @@
>  		reg = <0 0xe6530000 0 0x428>;
>  		interrupt-parent = <&gic>;
>  		interrupts = <0 177 0x4>;
> +		status = "disabled";
>  	};
>  
>  	i2c4: i2c@e6540000 {
> @@ -129,6 +133,7 @@
>  		reg = <0 0xe6540000 0 0x428>;
>  		interrupt-parent = <&gic>;
>  		interrupts = <0 178 0x4>;
> +		status = "disabled";
>  	};
>  
>  	i2c5: i2c@e60b0000 {
> @@ -138,6 +143,7 @@
>  		reg = <0 0xe60b0000 0 0x428>;
>  		interrupt-parent = <&gic>;
>  		interrupts = <0 179 0x4>;
> +		status = "disabled";
>  	};
>  
>  	i2c6: i2c@e6550000 {
> @@ -147,6 +153,7 @@
>  		reg = <0 0xe6550000 0 0x428>;
>  		interrupt-parent = <&gic>;
>  		interrupts = <0 184 0x4>;
> +		status = "disabled";
>  	};
>  
>  	i2c7: i2c@e6560000 {
> @@ -156,6 +163,7 @@
>  		reg = <0 0xe6560000 0 0x428>;
>  		interrupt-parent = <&gic>;
>  		interrupts = <0 185 0x4>;
> +		status = "disabled";
>  	};
>  
>  	i2c8: i2c@e6570000 {
> @@ -165,6 +173,7 @@
>  		reg = <0 0xe6570000 0 0x428>;
>  		interrupt-parent = <&gic>;
>  		interrupts = <0 173 0x4>;
> +		status = "disabled";
>  	};
>  
>  	mmcif0: mmcif@ee200000 {
> diff --git a/arch/arm/boot/dts/r8a7740-armadillo800eva-reference.dts b/arch/arm/boot/dts/r8a7740-armadillo800eva-reference.dts
> index 801442b..8b2aab5 100644
> --- a/arch/arm/boot/dts/r8a7740-armadillo800eva-reference.dts
> +++ b/arch/arm/boot/dts/r8a7740-armadillo800eva-reference.dts
> @@ -62,6 +62,7 @@
>  };
>  
>  &i2c0 {
> +	status = "okay";
>  	touchscreen: st1232@55 {
>  		compatible = "sitronix,st1232";
>  		reg = <0x55>;
> diff --git a/arch/arm/boot/dts/r8a7740.dtsi b/arch/arm/boot/dts/r8a7740.dtsi
> index 44d3d52..868bdde 100644
> --- a/arch/arm/boot/dts/r8a7740.dtsi
> +++ b/arch/arm/boot/dts/r8a7740.dtsi
> @@ -131,6 +131,7 @@
>  			      0 202 0x4
>  			      0 203 0x4
>  			      0 204 0x4>;
> +		status = "disabled";
>  	};
>  
>  	i2c1: i2c@e6c20000 {
> @@ -143,6 +144,7 @@
>  			      0 71 0x4
>  			      0 72 0x4
>  			      0 73 0x4>;
> +		status = "disabled";
>  	};
>  
>  	pfc: pfc@e6050000 {
> diff --git a/arch/arm/boot/dts/r8a7779.dtsi b/arch/arm/boot/dts/r8a7779.dtsi
> index ebbe507..912b3a0 100644
> --- a/arch/arm/boot/dts/r8a7779.dtsi
> +++ b/arch/arm/boot/dts/r8a7779.dtsi
> @@ -156,6 +156,7 @@
>  		reg = <0xffc70000 0x1000>;
>  		interrupt-parent = <&gic>;
>  		interrupts = <0 79 0x4>;
> +		status = "disabled";
>  	};
>  
>  	i2c1: i2c@ffc71000 {
> @@ -165,6 +166,7 @@
>  		reg = <0xffc71000 0x1000>;
>  		interrupt-parent = <&gic>;
>  		interrupts = <0 82 0x4>;
> +		status = "disabled";
>  	};
>  
>  	i2c2: i2c@ffc72000 {
> @@ -174,6 +176,7 @@
>  		reg = <0xffc72000 0x1000>;
>  		interrupt-parent = <&gic>;
>  		interrupts = <0 80 0x4>;
> +		status = "disabled";
>  	};
>  
>  	i2c3: i2c@ffc73000 {
> @@ -183,6 +186,7 @@
>  		reg = <0xffc73000 0x1000>;
>  		interrupt-parent = <&gic>;
>  		interrupts = <0 81 0x4>;
> +		status = "disabled";
>  	};
>  
>  	pfc: pfc@fffc0000 {
> diff --git a/arch/arm/boot/dts/sh73a0-kzm9g-reference.dts b/arch/arm/boot/dts/sh73a0-kzm9g-reference.dts
> index 2122306..8ee06dd 100644
> --- a/arch/arm/boot/dts/sh73a0-kzm9g-reference.dts
> +++ b/arch/arm/boot/dts/sh73a0-kzm9g-reference.dts
> @@ -108,6 +108,7 @@
>  };
>  
>  &i2c0 {
> +	status = "okay";
>  	as3711@40 {
>  		compatible = "ams,as3711";
>  		reg = <0x40>;
> @@ -183,6 +184,7 @@
>  &i2c3 {
>  	pinctrl-0 = <&i2c3_pins>;
>  	pinctrl-names = "default";
> +	status = "okay";
>  };
>  
>  &mmcif {
> diff --git a/arch/arm/boot/dts/sh73a0.dtsi b/arch/arm/boot/dts/sh73a0.dtsi
> index 3955c76..fcf2688 100644
> --- a/arch/arm/boot/dts/sh73a0.dtsi
> +++ b/arch/arm/boot/dts/sh73a0.dtsi
> @@ -135,6 +135,7 @@
>  			      0 168 0x4
>  			      0 169 0x4
>  			      0 170 0x4>;
> +		status = "disabled";
>  	};
>  
>  	i2c1: i2c@e6822000 {
> @@ -147,6 +148,7 @@
>  			      0 52 0x4
>  			      0 53 0x4
>  			      0 54 0x4>;
> +		status = "disabled";
>  	};
>  
>  	i2c2: i2c@e6824000 {
> @@ -159,6 +161,7 @@
>  			      0 172 0x4
>  			      0 173 0x4
>  			      0 174 0x4>;
> +		status = "disabled";
>  	};
>  
>  	i2c3: i2c@e6826000 {
> @@ -171,6 +174,7 @@
>  			      0 184 0x4
>  			      0 185 0x4
>  			      0 186 0x4>;
> +		status = "disabled";
>  	};
>  
>  	i2c4: i2c@e6828000 {
> @@ -183,6 +187,7 @@
>  			      0 188 0x4
>  			      0 189 0x4
>  			      0 190 0x4>;
> +		status = "disabled";
>  	};
>  
>  	mmcif: mmcif@e6bd0000 {
> -- 
> 1.7.2.5
> 
> --
> To unsubscribe from this list: send the line "unsubscribe devicetree" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
> 

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

* Re: [PATCH/RFC] ARM: shmobile: only enable used I2C interfaces in DT on all Renesas boards
       [not found] ` <Pine.LNX.4.64.1309261259490.11968-0199iw4Nj15frtckUFj5Ag@public.gmane.org>
@ 2013-09-26 23:34   ` Laurent Pinchart
  2013-09-27  0:10   ` Magnus Damm
  1 sibling, 0 replies; 5+ messages in thread
From: Laurent Pinchart @ 2013-09-26 23:34 UTC (permalink / raw)
  To: Guennadi Liakhovetski
  Cc: linux-sh-u79uwXL29TY76Z2rM5mHXA, Linus Walleij, Magnus Damm,
	devicetree-u79uwXL29TY76Z2rM5mHXA, Simon Horman

Hi Guennadi,

On Thursday 26 September 2013 13:06:01 Guennadi Liakhovetski wrote:
> Currently all I2C interfaces in all *.dtsi files for various Renesas SoCs
> are enabled by default. Switch them all off and only enable populated I2C
> interfaces in board-specific *.dts files.
> 
> Signed-off-by: Guennadi Liakhovetski <g.liakhovetski+renesas@gmail.com>

I had the exact same patch in my tree, you were faster than me to send it out.

Acked-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

-- 
Regards,

Laurent Pinchart


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

* Re: [PATCH/RFC] ARM: shmobile: only enable used I2C interfaces in DT on all Renesas boards
       [not found] ` <Pine.LNX.4.64.1309261259490.11968-0199iw4Nj15frtckUFj5Ag@public.gmane.org>
  2013-09-26 23:34   ` Laurent Pinchart
@ 2013-09-27  0:10   ` Magnus Damm
  2013-09-27  4:46     ` Simon Horman
  1 sibling, 1 reply; 5+ messages in thread
From: Magnus Damm @ 2013-09-27  0:10 UTC (permalink / raw)
  To: Guennadi Liakhovetski
  Cc: SH-Linux, Laurent Pinchart, Linus Walleij,
	devicetree-u79uwXL29TY76Z2rM5mHXA, Simon Horman

On Thu, Sep 26, 2013 at 4:06 AM, Guennadi Liakhovetski
<g.liakhovetski@gmx.de> wrote:
> Currently all I2C interfaces in all *.dtsi files for various Renesas SoCs
> are enabled by default. Switch them all off and only enable populated I2C
> interfaces in board-specific *.dts files.
>
> Signed-off-by: Guennadi Liakhovetski <g.liakhovetski+renesas@gmail.com>

Excellent, thank you!

/ magnus

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

* Re: [PATCH/RFC] ARM: shmobile: only enable used I2C interfaces in DT on all Renesas boards
  2013-09-27  0:10   ` Magnus Damm
@ 2013-09-27  4:46     ` Simon Horman
  0 siblings, 0 replies; 5+ messages in thread
From: Simon Horman @ 2013-09-27  4:46 UTC (permalink / raw)
  To: Magnus Damm
  Cc: Guennadi Liakhovetski, SH-Linux, Laurent Pinchart, Linus Walleij,
	devicetree

On Thu, Sep 26, 2013 at 05:10:58PM -0700, Magnus Damm wrote:
> On Thu, Sep 26, 2013 at 4:06 AM, Guennadi Liakhovetski
> <g.liakhovetski@gmx.de> wrote:
> > Currently all I2C interfaces in all *.dtsi files for various Renesas SoCs
> > are enabled by default. Switch them all off and only enable populated I2C
> > interfaces in board-specific *.dts files.
> >
> > Signed-off-by: Guennadi Liakhovetski <g.liakhovetski+renesas@gmail.com>
> 
> Excellent, thank you!

Thanks, I will queue this up with supplied Acks for v3.13.

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

end of thread, other threads:[~2013-09-27  4:46 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2013-09-26 11:06 [PATCH/RFC] ARM: shmobile: only enable used I2C interfaces in DT on all Renesas boards Guennadi Liakhovetski
2013-09-26 11:12 ` Mark Rutland
     [not found] ` <Pine.LNX.4.64.1309261259490.11968-0199iw4Nj15frtckUFj5Ag@public.gmane.org>
2013-09-26 23:34   ` Laurent Pinchart
2013-09-27  0:10   ` Magnus Damm
2013-09-27  4:46     ` Simon Horman

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).