On Monday 08 June 2015 09:48:29 Y Vo wrote: > On Fri, Jun 5, 2015 at 7:50 PM, Arnd Bergmann <arnd@xxxxxxxx> wrote: > > On Thursday 04 June 2015 14:25:12 Y Vo wrote: > >> GIC is designed to support two of trigger mechanisms - active level > >> high or edge rising. But in the gpio_keys driver, it tries to use both > >> edge rising and edge falling trigger. This patch fixes the gpio_keys > >> driver to request only the edge rising event when failed to configure the > >> interrupt. > >> > >> Signed-off-by: Y Vo <yvo@xxxxxxx> > > > > I think you want to use an 'interrupts' property instead of a 'gpios' > > property (or possibly both), so you can pass the right polarity. > > gpio-keys { > compatible = "gpio-keys"; > apm_ctrl_name = "Power Button"; > btn2 { > label = "EXT_INT2"; > gpios = <&sbgpio 13 0x0>; > linux,code = <116>; > linux,input-type = <0x1>; /* EV_KEY */ > }; > }; > > sbgpio: sbgpio@17001000{ > compatible = "apm,xgene-gpio-sb"; > reg = <0x0 0x17001000 0x0 0x400>; > #gpio-cells = <2>; > gpio-controller; > interrupts = <0x0 0x28 0x1>, > <0x0 0x29 0x1>, > <0x0 0x2a 0x1>, > <0x0 0x2b 0x1>, > <0x0 0x2c 0x1>, > <0x0 0x2d 0x1>; > }; > I can change the polarity of interrupt in the sbgpio node, but the > issue here is the gpio_key driver always set interrupt type to > (irqflags = IRQF_TRIGGER_RISING | IRQF_TRIGGER_FALLING). And the GIC > driver only support edge rising or level high. So that's our issue. No, please do as I wrote, and use an interrupts property in the gpio-keys node. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html