RE: [PATCH Resend 1/2] ARM: dts: Use active low flag for gpio-keys on Origen

[Kukjin Kim <kgene.kim@samsung.com>]

Kukjin Kim wrote:
> 
> Sachin Kamat wrote:
> >
> > Commit: f447ed8b31d (gpio: samsung: add flags specifier to
> > device-tree binding) adds a flag to represent active low state
> > for gpio line. Since gpio-keys on Origen board are active low,
> > using this flag to represent the same.
> >
> > Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
> > Reviewed-by: Thomas Abraham <thomas.abraham@linaro.org>
> > ---
> >  arch/arm/boot/dts/exynos4210-origen.dts |   10 +++++-----
> >  1 files changed, 5 insertions(+), 5 deletions(-)
> >
> > diff --git a/arch/arm/boot/dts/exynos4210-origen.dts
> > b/arch/arm/boot/dts/exynos4210-origen.dts
> > index 0c49caa..c37edf0 100644
> > --- a/arch/arm/boot/dts/exynos4210-origen.dts
> > +++ b/arch/arm/boot/dts/exynos4210-origen.dts
> > @@ -62,31 +62,31 @@
> >
> >  		up {
> >  			label = "Up";
> > -			gpios = <&gpx2 0 0 0 2>;
> > +			gpios = <&gpx2 0 0 0x10000 2>;
> >  			linux,code = <103>;
> >  		};
> >
> >  		down {
> >  			label = "Down";
> > -			gpios = <&gpx2 1 0 0 2>;
> > +			gpios = <&gpx2 1 0 0x10000 2>;
> >  			linux,code = <108>;
> >  		};
> >
> >  		back {
> >  			label = "Back";
> > -			gpios = <&gpx1 7 0 0 2>;
> > +			gpios = <&gpx1 7 0 0x10000 2>;
> >  			linux,code = <158>;
> >  		};
> >
> >  		home {
> >  			label = "Home";
> > -			gpios = <&gpx1 6 0 0 2>;
> > +			gpios = <&gpx1 6 0 0x10000 2>;
> >  			linux,code = <102>;
> >  		};
> >
> >  		menu {
> >  			label = "Menu";
> > -			gpios = <&gpx1 5 0 0 2>;
> > +			gpios = <&gpx1 5 0 0x10000 2>;
> >  			linux,code = <139>;
> >  		};
> >  	};
> > --
> > 1.7.4.1
> 
> Applied, thanks.
> 
Note, since this is not a bug fix, queued this for v3.7.

If any problem, let me know.

Thanks.

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.