diff mbox

[3/6] arm: dts: vt8500: Update Wondermedia SoC dtsi files for pinctrl driver

Message ID 1362807578-23089-4-git-send-email-linux@prisktech.co.nz (mailing list archive)
State New, archived
Headers show

Commit Message

Tony Prisk March 9, 2013, 5:39 a.m. UTC
This patch adds pinctrl nodes to the VIA VT8500 and Wondermedia SoC dtsi
files to support the pinctrl driver.

Signed-off-by: Tony Prisk <linux@prisktech.co.nz>
---
 arch/arm/boot/dts/vt8500.dtsi |    8 ++++++++
 arch/arm/boot/dts/wm8505.dtsi |    8 ++++++++
 arch/arm/boot/dts/wm8650.dtsi |    8 ++++++++
 arch/arm/boot/dts/wm8850.dtsi |    8 ++++++++
 4 files changed, 32 insertions(+)

Comments

Stephen Warren March 11, 2013, 4:46 p.m. UTC | #1
On 03/08/2013 10:39 PM, Tony Prisk wrote:
> This patch adds pinctrl nodes to the VIA VT8500 and Wondermedia SoC dtsi
> files to support the pinctrl driver.

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

> +		pinctrl: pinctrl@d8110000 {
> +			compatible = "via,vt8500-pinctrl";
> +			gpio-controller;
> +			reg = <0xd8110000 0x10000>;
> +			#gpio-cells = <2>;
> +			wm,pinmux = <0x00000001 0x00000001>;
> +		};

In the binding documentation, "wm,pinmux" is listed under "optional
sub-node properties", but here the property is included in the main
node, not a sub-node.

I'm a little confused what this property means then...
Tony Prisk March 12, 2013, 4:10 a.m. UTC | #2
On Mon, 2013-03-11 at 10:46 -0600, Stephen Warren wrote:
> On 03/08/2013 10:39 PM, Tony Prisk wrote:
> > This patch adds pinctrl nodes to the VIA VT8500 and Wondermedia SoC dtsi
> > files to support the pinctrl driver.
> 
> > diff --git a/arch/arm/boot/dts/vt8500.dtsi b/arch/arm/boot/dts/vt8500.dtsi
> 
> > +		pinctrl: pinctrl@d8110000 {
> > +			compatible = "via,vt8500-pinctrl";
> > +			gpio-controller;
> > +			reg = <0xd8110000 0x10000>;
> > +			#gpio-cells = <2>;
> > +			wm,pinmux = <0x00000001 0x00000001>;
> > +		};
> 
> In the binding documentation, "wm,pinmux" is listed under "optional
> sub-node properties", but here the property is included in the main
> node, not a sub-node.
> 
> I'm a little confused what this property means then...
> 
> _______________________________________________
> linux-arm-kernel mailing list
> linux-arm-kernel@lists.infradead.org
> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

Sorry - this was copied in large part from the brcm,bcm2835-gpio.txt
binding. I added wm,pinmux alongside the existing wm,pins, wm,function
and wm,pull properties.

You are correct that this shouldn't be listed as a subnode property.

Regards
Tony P
diff mbox

Patch

diff --git a/arch/arm/boot/dts/vt8500.dtsi b/arch/arm/boot/dts/vt8500.dtsi
index cf31ced..16388c2 100644
--- a/arch/arm/boot/dts/vt8500.dtsi
+++ b/arch/arm/boot/dts/vt8500.dtsi
@@ -32,6 +32,14 @@ 
 			#gpio-cells = <3>;
 		};
 
+		pinctrl: pinctrl@d8110000 {
+			compatible = "via,vt8500-pinctrl";
+			gpio-controller;
+			reg = <0xd8110000 0x10000>;
+			#gpio-cells = <2>;
+			wm,pinmux = <0x00000001 0x00000001>;
+		};
+
 		pmc@d8130000 {
 			compatible = "via,vt8500-pmc";
 			reg = <0xd8130000 0x1000>;
diff --git a/arch/arm/boot/dts/wm8505.dtsi b/arch/arm/boot/dts/wm8505.dtsi
index e74a1c0..c5aa4c9 100644
--- a/arch/arm/boot/dts/wm8505.dtsi
+++ b/arch/arm/boot/dts/wm8505.dtsi
@@ -47,6 +47,14 @@ 
 			#gpio-cells = <3>;
 		};
 
+		pinctrl: pinctrl@d8110000 {
+			compatible = "wm,wm8505-pinctrl";
+			gpio-controller;
+			reg = <0xd8110000 0x10000>;
+			#gpio-cells = <2>;
+			wm,pinmux = <0x80000000 0x80000000>;
+		};
+
 		pmc@d8130000 {
 			compatible = "via,vt8500-pmc";
 			reg = <0xd8130000 0x1000>;
diff --git a/arch/arm/boot/dts/wm8650.dtsi b/arch/arm/boot/dts/wm8650.dtsi
index db3c0a1..99c45e5 100644
--- a/arch/arm/boot/dts/wm8650.dtsi
+++ b/arch/arm/boot/dts/wm8650.dtsi
@@ -41,6 +41,14 @@ 
 			#gpio-cells = <3>;
 		};
 
+		pinctrl: pinctrl@d8110000 {
+			compatible = "wm,wm8650-pinctrl";
+			gpio-controller;
+			reg = <0xd8110000 0x10000>;
+			#gpio-cells = <2>;
+			wm,pinmux = <0x80000000 0x80000000>;
+		};
+
 		pmc@d8130000 {
 			compatible = "via,vt8500-pmc";
 			reg = <0xd8130000 0x1000>;
diff --git a/arch/arm/boot/dts/wm8850.dtsi b/arch/arm/boot/dts/wm8850.dtsi
index e8cbfdc..60094cd 100644
--- a/arch/arm/boot/dts/wm8850.dtsi
+++ b/arch/arm/boot/dts/wm8850.dtsi
@@ -48,6 +48,14 @@ 
 			#gpio-cells = <3>;
 		};
 
+		pinctrl: pinctrl@d8110000 {
+			compatible = "wm,wm8850-pinctrl";
+			gpio-controller;
+			reg = <0xd8110000 0x10000>;
+			#gpio-cells = <2>;
+			wm,pinmux = <0x80000000 0x80000000>;
+		};
+
 		pmc@d8130000 {
 			compatible = "via,vt8500-pmc";
 			reg = <0xd8130000 0x1000>;