Message ID | 1471357649-2483-2-git-send-email-vanessa.maegima@nxp.com (mailing list archive) |
---|---|
State | New, archived |
Headers | show |
On Tue, Aug 16, 2016 at 11:27:29AM -0300, Vanessa Maegima wrote: > Add User Button at GPIO7_1. > > Tested by evtest. > > Signed-off-by: Vanessa Maegima <vanessa.maegima@nxp.com> > --- > arch/arm/boot/dts/imx7s-warp.dts | 19 +++++++++++++++++++ > 1 file changed, 19 insertions(+) > > diff --git a/arch/arm/boot/dts/imx7s-warp.dts b/arch/arm/boot/dts/imx7s-warp.dts > index 71dbece..794aa94 100644 > --- a/arch/arm/boot/dts/imx7s-warp.dts > +++ b/arch/arm/boot/dts/imx7s-warp.dts > @@ -54,6 +54,19 @@ > reg = <0x80000000 0x20000000>; > }; > > + gpio-keys { > + compatible = "gpio-keys"; > + pinctrl-0 = <&pinctrl_gpio>; > + autorepeat; Please have a newline between property list and sub-node. > + back { > + label = "Back"; > + gpios = <&gpio7 1 GPIO_ACTIVE_HIGH>; > + linux,code = <KEY_BACK>; > + gpio-key,wakeup; This property is deprecated, and 'wakeup-source' should be used instead. See Documentation/devicetree/bindings/power/wakeup-source.txt for details. > + autorepeat; Is it really necessary to have this property in both parent and child node? Shawn > + }; > + }; > + > reg_brcm: regulator-brcm { > compatible = "regulator-fixed"; > enable-active-high; > @@ -278,6 +291,12 @@ > >; > }; > > + pinctrl_gpio: gpiogrp { > + fsl,pins = < > + MX7D_PAD_ENET1_RGMII_RD1__GPIO7_IO1 0x14 > + >; > + }; > + > pinctrl_i2c1: i2c1grp { > fsl,pins = < > MX7D_PAD_I2C1_SDA__I2C1_SDA 0x4000007f > -- > 2.7.4 > > > _______________________________________________ > linux-arm-kernel mailing list > linux-arm-kernel@lists.infradead.org > http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
diff --git a/arch/arm/boot/dts/imx7s-warp.dts b/arch/arm/boot/dts/imx7s-warp.dts index 71dbece..794aa94 100644 --- a/arch/arm/boot/dts/imx7s-warp.dts +++ b/arch/arm/boot/dts/imx7s-warp.dts @@ -54,6 +54,19 @@ reg = <0x80000000 0x20000000>; }; + gpio-keys { + compatible = "gpio-keys"; + pinctrl-0 = <&pinctrl_gpio>; + autorepeat; + back { + label = "Back"; + gpios = <&gpio7 1 GPIO_ACTIVE_HIGH>; + linux,code = <KEY_BACK>; + gpio-key,wakeup; + autorepeat; + }; + }; + reg_brcm: regulator-brcm { compatible = "regulator-fixed"; enable-active-high; @@ -278,6 +291,12 @@ >; }; + pinctrl_gpio: gpiogrp { + fsl,pins = < + MX7D_PAD_ENET1_RGMII_RD1__GPIO7_IO1 0x14 + >; + }; + pinctrl_i2c1: i2c1grp { fsl,pins = < MX7D_PAD_I2C1_SDA__I2C1_SDA 0x4000007f
Add User Button at GPIO7_1. Tested by evtest. Signed-off-by: Vanessa Maegima <vanessa.maegima@nxp.com> --- arch/arm/boot/dts/imx7s-warp.dts | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+)