* [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-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
---
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-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
^ 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[parent not found: <Pine.LNX.4.64.1309261259490.11968-0199iw4Nj15frtckUFj5Ag@public.gmane.org>]
* 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-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
I had the exact same patch in my tree, you were faster than me to send it out.
Acked-by: Laurent Pinchart <laurent.pinchart-ryLnwIuWjnjg/C1BVhZhaw@public.gmane.org>
--
Regards,
Laurent Pinchart
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.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
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-Mmb7MZpHnFY@public.gmane.org> 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-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Excellent, thank you!
/ magnus
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.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
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).