diff mbox

[1/2] ARM: dts: Fix gpio pin names of keypad columns in exynos4x12-pinctrl

Message ID 1354788815-21579-1-git-send-email-sachin.kamat@linaro.org
State Superseded
Headers show

Commit Message

Sachin Kamat Dec. 6, 2012, 10:13 a.m. UTC
Keypad columns are connected to gpx1 pins on 4x12.

Cc: Tomasz Figa <t.figa@samsung.com>
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
---
 arch/arm/boot/dts/exynos4x12-pinctrl.dtsi |   16 ++++++++--------
 1 files changed, 8 insertions(+), 8 deletions(-)

Comments

Kukjin Kim Dec. 21, 2012, 6:34 p.m. UTC | #1
Sachin Kamat wrote:
> 
> Keypad columns are connected to gpx1 pins on 4x12.
> 
> Cc: Tomasz Figa <t.figa@samsung.com>
> Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
> ---
>  arch/arm/boot/dts/exynos4x12-pinctrl.dtsi |   16 ++++++++--------
>  1 files changed, 8 insertions(+), 8 deletions(-)
> 
> diff --git a/arch/arm/boot/dts/exynos4x12-pinctrl.dtsi
> b/arch/arm/boot/dts/exynos4x12-pinctrl.dtsi
> index 8e6115a..0b293fb 100644
> --- a/arch/arm/boot/dts/exynos4x12-pinctrl.dtsi
> +++ b/arch/arm/boot/dts/exynos4x12-pinctrl.dtsi
> @@ -779,56 +779,56 @@
>  		};
> 
>  		keypad_col0: keypad-col0 {
> -			samsung,pins = "gpl2-0";
> +			samsung,pins = "gpx1-0";
>  			samsung,pin-function = <3>;
>  			samsung,pin-pud = <0>;
>  			samsung,pin-drv = <0>;
>  		};
> 
>  		keypad_col1: keypad-col1 {
> -			samsung,pins = "gpl2-1";
> +			samsung,pins = "gpx1-1";
>  			samsung,pin-function = <3>;
>  			samsung,pin-pud = <0>;
>  			samsung,pin-drv = <0>;
>  		};
> 
>  		keypad_col2: keypad-col2 {
> -			samsung,pins = "gpl2-2";
> +			samsung,pins = "gpx1-2";
>  			samsung,pin-function = <3>;
>  			samsung,pin-pud = <0>;
>  			samsung,pin-drv = <0>;
>  		};
> 
>  		keypad_col3: keypad-col3 {
> -			samsung,pins = "gpl2-3";
> +			samsung,pins = "gpx1-3";
>  			samsung,pin-function = <3>;
>  			samsung,pin-pud = <0>;
>  			samsung,pin-drv = <0>;
>  		};
> 
>  		keypad_col4: keypad-col4 {
> -			samsung,pins = "gpl2-4";
> +			samsung,pins = "gpx1-4";
>  			samsung,pin-function = <3>;
>  			samsung,pin-pud = <0>;
>  			samsung,pin-drv = <0>;
>  		};
> 
>  		keypad_col5: keypad-col5 {
> -			samsung,pins = "gpl2-5";
> +			samsung,pins = "gpx1-5";
>  			samsung,pin-function = <3>;
>  			samsung,pin-pud = <0>;
>  			samsung,pin-drv = <0>;
>  		};
> 
>  		keypad_col6: keypad-col6 {
> -			samsung,pins = "gpl2-6";
> +			samsung,pins = "gpx1-6";
>  			samsung,pin-function = <3>;
>  			samsung,pin-pud = <0>;
>  			samsung,pin-drv = <0>;
>  		};
> 
>  		keypad_col7: keypad-col7 {
> -			samsung,pins = "gpl2-7";
> +			samsung,pins = "gpx1-7";
>  			samsung,pin-function = <3>;
>  			samsung,pin-pud = <0>;
>  			samsung,pin-drv = <0>;
> --
> 1.7.4.1

As I commented in your other patch, the value should be defined at the board
dt file not here. And I think, need to fix this file.

- Kukjin
diff mbox

Patch

diff --git a/arch/arm/boot/dts/exynos4x12-pinctrl.dtsi b/arch/arm/boot/dts/exynos4x12-pinctrl.dtsi
index 8e6115a..0b293fb 100644
--- a/arch/arm/boot/dts/exynos4x12-pinctrl.dtsi
+++ b/arch/arm/boot/dts/exynos4x12-pinctrl.dtsi
@@ -779,56 +779,56 @@ 
 		};
 
 		keypad_col0: keypad-col0 {
-			samsung,pins = "gpl2-0";
+			samsung,pins = "gpx1-0";
 			samsung,pin-function = <3>;
 			samsung,pin-pud = <0>;
 			samsung,pin-drv = <0>;
 		};
 
 		keypad_col1: keypad-col1 {
-			samsung,pins = "gpl2-1";
+			samsung,pins = "gpx1-1";
 			samsung,pin-function = <3>;
 			samsung,pin-pud = <0>;
 			samsung,pin-drv = <0>;
 		};
 
 		keypad_col2: keypad-col2 {
-			samsung,pins = "gpl2-2";
+			samsung,pins = "gpx1-2";
 			samsung,pin-function = <3>;
 			samsung,pin-pud = <0>;
 			samsung,pin-drv = <0>;
 		};
 
 		keypad_col3: keypad-col3 {
-			samsung,pins = "gpl2-3";
+			samsung,pins = "gpx1-3";
 			samsung,pin-function = <3>;
 			samsung,pin-pud = <0>;
 			samsung,pin-drv = <0>;
 		};
 
 		keypad_col4: keypad-col4 {
-			samsung,pins = "gpl2-4";
+			samsung,pins = "gpx1-4";
 			samsung,pin-function = <3>;
 			samsung,pin-pud = <0>;
 			samsung,pin-drv = <0>;
 		};
 
 		keypad_col5: keypad-col5 {
-			samsung,pins = "gpl2-5";
+			samsung,pins = "gpx1-5";
 			samsung,pin-function = <3>;
 			samsung,pin-pud = <0>;
 			samsung,pin-drv = <0>;
 		};
 
 		keypad_col6: keypad-col6 {
-			samsung,pins = "gpl2-6";
+			samsung,pins = "gpx1-6";
 			samsung,pin-function = <3>;
 			samsung,pin-pud = <0>;
 			samsung,pin-drv = <0>;
 		};
 
 		keypad_col7: keypad-col7 {
-			samsung,pins = "gpl2-7";
+			samsung,pins = "gpx1-7";
 			samsung,pin-function = <3>;
 			samsung,pin-pud = <0>;
 			samsung,pin-drv = <0>;