2015-12-19 0:25 GMT+03:00 Maxime Ripard : > Hi, > > On Fri, Dec 18, 2015 at 11:51:50AM +0300, Aleksei Mamlin wrote: > > Add a node for the Goodix GT911 touchscreen found on the Wexler TAB7200 > tablet > > > > Signed-off-by: Aleksei Mamlin > > --- > > arch/arm/boot/dts/sun7i-a20-wexler-tab7200.dts | 19 +++++++++++++++++++ > > 1 file changed, 19 insertions(+) > > > > diff --git a/arch/arm/boot/dts/sun7i-a20-wexler-tab7200.dts > b/arch/arm/boot/dts/sun7i-a20-wexler-tab7200.dts > > index 239b5d2..ec3b837 100644 > > --- a/arch/arm/boot/dts/sun7i-a20-wexler-tab7200.dts > > +++ b/arch/arm/boot/dts/sun7i-a20-wexler-tab7200.dts > > @@ -102,6 +102,18 @@ > > pinctrl-names = "default"; > > pinctrl-0 = <&i2c2_pins_a>; > > status = "okay"; > > + > > + gt911: touchscreen@5d { > > + compatible = "goodix,gt911"; > > + reg = <0x5d>; > > + interrupt-parent = <&pio>; > > + interrupts = <7 21 IRQ_TYPE_EDGE_FALLING>; /* EINT21 > (PH21) */ > > + pinctrl-names = "default"; > > + pinctrl-0 = <&ts_reset_pin>; > > + irq-gpios = <&pio 7 21 GPIO_ACTIVE_HIGH>; /* INT (PH21) */ > > It seems odd that you need both irq-gpios and interrupts. These two > are completely redundant, and you should even actually use only one in > your driver, since the second request_irq will fail. > > We need both interrupts and irq-gpios because the driver uses the interrupt gpio pin as output to reset the device. See bindings documentation [1] [1] https://git.kernel.org/cgit/linux/kernel/git/dtor/input.git/tree/Documentation/devicetree/bindings/input/touchscreen/goodix.txt?h=goodix Thanks! > Maxime > > -- > Maxime Ripard, Free Electrons > Embedded Linux, Kernel and Android engineering > http://free-electrons.com > -- Thanks and regards, Aleksei Mamlin -- You received this message because you are subscribed to the Google Groups "linux-sunxi" group. To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org For more options, visit https://groups.google.com/d/optout.