Re: [PATCH 2/2] ARM: dts: am335x-icev2: Add nodes for gpio-decoder

2016-08-12 Thread Rob Herring
On Fri, Aug 12, 2016 at 10:22:29AM +0530, Vignesh R wrote:
> AM335x ICE board has a rotary-switch connected to PCA9536 I2C GPIO
> expander. The position of the rotary-switch is reflected by status of
> GPIO lines. Add gpio-decoder node to read these GPIO line status via
> gpio-decoder driver and report it as an input event to the system.
> 
> Signed-off-by: Vignesh R 
> ---
>  arch/arm/boot/dts/am335x-icev2.dts | 9 +
>  1 file changed, 9 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/am335x-icev2.dts 
> b/arch/arm/boot/dts/am335x-icev2.dts
> index 7d8b8fefdf08..f2de594343f2 100644
> --- a/arch/arm/boot/dts/am335x-icev2.dts
> +++ b/arch/arm/boot/dts/am335x-icev2.dts
> @@ -139,6 +139,15 @@
>   default-state = "off";
>   };
>   };
> + gpio-decoder0 {

Drop the 0 here.

> + compatible = "gpio-decoder";
> + gpios = < 3 GPIO_ACTIVE_HIGH>,
> + < 2 GPIO_ACTIVE_HIGH>,
> + < 1 GPIO_ACTIVE_HIGH>,
> + < 0 GPIO_ACTIVE_HIGH>;
> + linux,axis = <0>; /* ABS_X */
> + gpio-decoder,max-value = <9>;
> + };
>  };
>  
>  _pinmux {
> -- 
> 2.9.2
> 


Re: [PATCH 2/2] ARM: dts: am335x-icev2: Add nodes for gpio-decoder

2016-08-12 Thread Rob Herring
On Fri, Aug 12, 2016 at 10:22:29AM +0530, Vignesh R wrote:
> AM335x ICE board has a rotary-switch connected to PCA9536 I2C GPIO
> expander. The position of the rotary-switch is reflected by status of
> GPIO lines. Add gpio-decoder node to read these GPIO line status via
> gpio-decoder driver and report it as an input event to the system.
> 
> Signed-off-by: Vignesh R 
> ---
>  arch/arm/boot/dts/am335x-icev2.dts | 9 +
>  1 file changed, 9 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/am335x-icev2.dts 
> b/arch/arm/boot/dts/am335x-icev2.dts
> index 7d8b8fefdf08..f2de594343f2 100644
> --- a/arch/arm/boot/dts/am335x-icev2.dts
> +++ b/arch/arm/boot/dts/am335x-icev2.dts
> @@ -139,6 +139,15 @@
>   default-state = "off";
>   };
>   };
> + gpio-decoder0 {

Drop the 0 here.

> + compatible = "gpio-decoder";
> + gpios = < 3 GPIO_ACTIVE_HIGH>,
> + < 2 GPIO_ACTIVE_HIGH>,
> + < 1 GPIO_ACTIVE_HIGH>,
> + < 0 GPIO_ACTIVE_HIGH>;
> + linux,axis = <0>; /* ABS_X */
> + gpio-decoder,max-value = <9>;
> + };
>  };
>  
>  _pinmux {
> -- 
> 2.9.2
> 


[PATCH 2/2] ARM: dts: am335x-icev2: Add nodes for gpio-decoder

2016-08-11 Thread Vignesh R
AM335x ICE board has a rotary-switch connected to PCA9536 I2C GPIO
expander. The position of the rotary-switch is reflected by status of
GPIO lines. Add gpio-decoder node to read these GPIO line status via
gpio-decoder driver and report it as an input event to the system.

Signed-off-by: Vignesh R 
---
 arch/arm/boot/dts/am335x-icev2.dts | 9 +
 1 file changed, 9 insertions(+)

diff --git a/arch/arm/boot/dts/am335x-icev2.dts 
b/arch/arm/boot/dts/am335x-icev2.dts
index 7d8b8fefdf08..f2de594343f2 100644
--- a/arch/arm/boot/dts/am335x-icev2.dts
+++ b/arch/arm/boot/dts/am335x-icev2.dts
@@ -139,6 +139,15 @@
default-state = "off";
};
};
+   gpio-decoder0 {
+   compatible = "gpio-decoder";
+   gpios = < 3 GPIO_ACTIVE_HIGH>,
+   < 2 GPIO_ACTIVE_HIGH>,
+   < 1 GPIO_ACTIVE_HIGH>,
+   < 0 GPIO_ACTIVE_HIGH>;
+   linux,axis = <0>; /* ABS_X */
+   gpio-decoder,max-value = <9>;
+   };
 };
 
 _pinmux {
-- 
2.9.2



[PATCH 2/2] ARM: dts: am335x-icev2: Add nodes for gpio-decoder

2016-08-11 Thread Vignesh R
AM335x ICE board has a rotary-switch connected to PCA9536 I2C GPIO
expander. The position of the rotary-switch is reflected by status of
GPIO lines. Add gpio-decoder node to read these GPIO line status via
gpio-decoder driver and report it as an input event to the system.

Signed-off-by: Vignesh R 
---
 arch/arm/boot/dts/am335x-icev2.dts | 9 +
 1 file changed, 9 insertions(+)

diff --git a/arch/arm/boot/dts/am335x-icev2.dts 
b/arch/arm/boot/dts/am335x-icev2.dts
index 7d8b8fefdf08..f2de594343f2 100644
--- a/arch/arm/boot/dts/am335x-icev2.dts
+++ b/arch/arm/boot/dts/am335x-icev2.dts
@@ -139,6 +139,15 @@
default-state = "off";
};
};
+   gpio-decoder0 {
+   compatible = "gpio-decoder";
+   gpios = < 3 GPIO_ACTIVE_HIGH>,
+   < 2 GPIO_ACTIVE_HIGH>,
+   < 1 GPIO_ACTIVE_HIGH>,
+   < 0 GPIO_ACTIVE_HIGH>;
+   linux,axis = <0>; /* ABS_X */
+   gpio-decoder,max-value = <9>;
+   };
 };
 
 _pinmux {
-- 
2.9.2