diff mbox

[v7,3/7] ARM: dts: imx27 pinctrl

Message ID 1384246210-9257-4-git-send-email-mpa@pengutronix.de (mailing list archive)
State New, archived
Headers show

Commit Message

Markus Pargmann Nov. 12, 2013, 8:50 a.m. UTC
Pinctrl driver node and pin group definitions for other driver nodes.

Signed-off-by: Markus Pargmann <mpa@pengutronix.de>
---
 arch/arm/boot/dts/imx27.dtsi | 125 +++++++++++++++++++++++--------------------
 1 file changed, 67 insertions(+), 58 deletions(-)

Comments

Shawn Guo Nov. 13, 2013, 2:41 p.m. UTC | #1
On Tue, Nov 12, 2013 at 09:50:06AM +0100, Markus Pargmann wrote:
> Pinctrl driver node and pin group definitions for other driver nodes.

Neither patch subject nor commit log is telling anything useful, like
what is the change and why the changes.

Shawn

> 
> Signed-off-by: Markus Pargmann <mpa@pengutronix.de>
> ---
>  arch/arm/boot/dts/imx27.dtsi | 125 +++++++++++++++++++++++--------------------
>  1 file changed, 67 insertions(+), 58 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/imx27.dtsi b/arch/arm/boot/dts/imx27.dtsi
> index b7a1c6d..f119296 100644
> --- a/arch/arm/boot/dts/imx27.dtsi
> +++ b/arch/arm/boot/dts/imx27.dtsi
> @@ -10,6 +10,7 @@
>   */
>  
>  #include "skeleton.dtsi"
> +#include "imx27-pingrp.h"
>  
>  / {
>  	aliases {
> @@ -235,64 +236,72 @@
>  				status = "disabled";
>  			};
>  
> -			gpio1: gpio@10015000 {
> -				compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
> -				reg = <0x10015000 0x100>;
> -				interrupts = <8>;
> -				gpio-controller;
> -				#gpio-cells = <2>;
> -				interrupt-controller;
> -				#interrupt-cells = <2>;
> -			};
> -
> -			gpio2: gpio@10015100 {
> -				compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
> -				reg = <0x10015100 0x100>;
> -				interrupts = <8>;
> -				gpio-controller;
> -				#gpio-cells = <2>;
> -				interrupt-controller;
> -				#interrupt-cells = <2>;
> -			};
> -
> -			gpio3: gpio@10015200 {
> -				compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
> -				reg = <0x10015200 0x100>;
> -				interrupts = <8>;
> -				gpio-controller;
> -				#gpio-cells = <2>;
> -				interrupt-controller;
> -				#interrupt-cells = <2>;
> -			};
> -
> -			gpio4: gpio@10015300 {
> -				compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
> -				reg = <0x10015300 0x100>;
> -				interrupts = <8>;
> -				gpio-controller;
> -				#gpio-cells = <2>;
> -				interrupt-controller;
> -				#interrupt-cells = <2>;
> -			};
> -
> -			gpio5: gpio@10015400 {
> -				compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
> -				reg = <0x10015400 0x100>;
> -				interrupts = <8>;
> -				gpio-controller;
> -				#gpio-cells = <2>;
> -				interrupt-controller;
> -				#interrupt-cells = <2>;
> -			};
> -
> -			gpio6: gpio@10015500 {
> -				compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
> -				reg = <0x10015500 0x100>;
> -				interrupts = <8>;
> -				gpio-controller;
> -				#gpio-cells = <2>;
> -				interrupt-controller;
> -				#interrupt-cells = <2>;
> +			iomuxc: iomuxc@10015000 {
> +				compatible = "fsl,imx27-iomuxc";
> +				reg = <0x10015000 0x600>;
> +				#address-cells = <1>;
> +				#size-cells = <1>;
> +				ranges;
> +
> +				gpio1: gpio@10015000 {
> +					compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
> +					reg = <0x10015000 0x100>;
> +					interrupts = <8>;
> +					gpio-controller;
> +					#gpio-cells = <2>;
> +					interrupt-controller;
> +					#interrupt-cells = <2>;
> +				};
> +
> +				gpio2: gpio@10015100 {
> +					compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
> +					reg = <0x10015100 0x100>;
> +					interrupts = <8>;
> +					gpio-controller;
> +					#gpio-cells = <2>;
> +					interrupt-controller;
> +					#interrupt-cells = <2>;
> +				};
> +
> +				gpio3: gpio@10015200 {
> +					compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
> +					reg = <0x10015200 0x100>;
> +					interrupts = <8>;
> +					gpio-controller;
> +					#gpio-cells = <2>;
> +					interrupt-controller;
> +					#interrupt-cells = <2>;
> +				};
> +
> +				gpio4: gpio@10015300 {
> +					compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
> +					reg = <0x10015300 0x100>;
> +					interrupts = <8>;
> +					gpio-controller;
> +					#gpio-cells = <2>;
> +					interrupt-controller;
> +					#interrupt-cells = <2>;
> +				};
> +
> +				gpio5: gpio@10015400 {
> +					compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
> +					reg = <0x10015400 0x100>;
> +					interrupts = <8>;
> +					gpio-controller;
> +					#gpio-cells = <2>;
> +					interrupt-controller;
> +					#interrupt-cells = <2>;
> +				};
> +
> +				gpio6: gpio@10015500 {
> +					compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
> +					reg = <0x10015500 0x100>;
> +					interrupts = <8>;
> +					gpio-controller;
> +					#gpio-cells = <2>;
> +					interrupt-controller;
> +					#interrupt-cells = <2>;
> +				};
>  			};
>  
>  			audmux: audmux@10016000 {
> -- 
> 1.8.4.2
>
Markus Pargmann Nov. 13, 2013, 3:02 p.m. UTC | #2
On Wed, Nov 13, 2013 at 10:41:41PM +0800, Shawn Guo wrote:
> On Tue, Nov 12, 2013 at 09:50:06AM +0100, Markus Pargmann wrote:
> > Pinctrl driver node and pin group definitions for other driver nodes.
> 
> Neither patch subject nor commit log is telling anything useful, like
> what is the change and why the changes.

Sorry, I forgot to update the commit message. I will do this for the
next version.

Thanks,

Markus
diff mbox

Patch

diff --git a/arch/arm/boot/dts/imx27.dtsi b/arch/arm/boot/dts/imx27.dtsi
index b7a1c6d..f119296 100644
--- a/arch/arm/boot/dts/imx27.dtsi
+++ b/arch/arm/boot/dts/imx27.dtsi
@@ -10,6 +10,7 @@ 
  */
 
 #include "skeleton.dtsi"
+#include "imx27-pingrp.h"
 
 / {
 	aliases {
@@ -235,64 +236,72 @@ 
 				status = "disabled";
 			};
 
-			gpio1: gpio@10015000 {
-				compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
-				reg = <0x10015000 0x100>;
-				interrupts = <8>;
-				gpio-controller;
-				#gpio-cells = <2>;
-				interrupt-controller;
-				#interrupt-cells = <2>;
-			};
-
-			gpio2: gpio@10015100 {
-				compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
-				reg = <0x10015100 0x100>;
-				interrupts = <8>;
-				gpio-controller;
-				#gpio-cells = <2>;
-				interrupt-controller;
-				#interrupt-cells = <2>;
-			};
-
-			gpio3: gpio@10015200 {
-				compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
-				reg = <0x10015200 0x100>;
-				interrupts = <8>;
-				gpio-controller;
-				#gpio-cells = <2>;
-				interrupt-controller;
-				#interrupt-cells = <2>;
-			};
-
-			gpio4: gpio@10015300 {
-				compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
-				reg = <0x10015300 0x100>;
-				interrupts = <8>;
-				gpio-controller;
-				#gpio-cells = <2>;
-				interrupt-controller;
-				#interrupt-cells = <2>;
-			};
-
-			gpio5: gpio@10015400 {
-				compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
-				reg = <0x10015400 0x100>;
-				interrupts = <8>;
-				gpio-controller;
-				#gpio-cells = <2>;
-				interrupt-controller;
-				#interrupt-cells = <2>;
-			};
-
-			gpio6: gpio@10015500 {
-				compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
-				reg = <0x10015500 0x100>;
-				interrupts = <8>;
-				gpio-controller;
-				#gpio-cells = <2>;
-				interrupt-controller;
-				#interrupt-cells = <2>;
+			iomuxc: iomuxc@10015000 {
+				compatible = "fsl,imx27-iomuxc";
+				reg = <0x10015000 0x600>;
+				#address-cells = <1>;
+				#size-cells = <1>;
+				ranges;
+
+				gpio1: gpio@10015000 {
+					compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
+					reg = <0x10015000 0x100>;
+					interrupts = <8>;
+					gpio-controller;
+					#gpio-cells = <2>;
+					interrupt-controller;
+					#interrupt-cells = <2>;
+				};
+
+				gpio2: gpio@10015100 {
+					compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
+					reg = <0x10015100 0x100>;
+					interrupts = <8>;
+					gpio-controller;
+					#gpio-cells = <2>;
+					interrupt-controller;
+					#interrupt-cells = <2>;
+				};
+
+				gpio3: gpio@10015200 {
+					compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
+					reg = <0x10015200 0x100>;
+					interrupts = <8>;
+					gpio-controller;
+					#gpio-cells = <2>;
+					interrupt-controller;
+					#interrupt-cells = <2>;
+				};
+
+				gpio4: gpio@10015300 {
+					compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
+					reg = <0x10015300 0x100>;
+					interrupts = <8>;
+					gpio-controller;
+					#gpio-cells = <2>;
+					interrupt-controller;
+					#interrupt-cells = <2>;
+				};
+
+				gpio5: gpio@10015400 {
+					compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
+					reg = <0x10015400 0x100>;
+					interrupts = <8>;
+					gpio-controller;
+					#gpio-cells = <2>;
+					interrupt-controller;
+					#interrupt-cells = <2>;
+				};
+
+				gpio6: gpio@10015500 {
+					compatible = "fsl,imx27-gpio", "fsl,imx21-gpio";
+					reg = <0x10015500 0x100>;
+					interrupts = <8>;
+					gpio-controller;
+					#gpio-cells = <2>;
+					interrupt-controller;
+					#interrupt-cells = <2>;
+				};
 			};
 
 			audmux: audmux@10016000 {