From mboxrd@z Thu Jan 1 00:00:00 1970 From: LW@KARO-electronics.de (Lothar =?UTF-8?B?V2HDn21hbm4=?=) Date: Fri, 25 Oct 2013 07:56:39 +0200 Subject: [PATCHv7][ 1/4] Input: tsc2007: Add device tree support. In-Reply-To: <20131024164746.3CB36C403B6@trevor.secretlab.ca> References: <1382618536-25320-1-git-send-email-denis@eukrea.com> <20131024164746.3CB36C403B6@trevor.secretlab.ca> Message-ID: <20131025075639.75f45fb0@ipc1.ka-ro> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hi, Grant Likely wrote: > On Thu, 24 Oct 2013 14:42:13 +0200, Denis Carikli wrote: > > Cc: Rob Herring > > Cc: Pawel Moll > > Cc: Mark Rutland > > Cc: Stephen Warren > > Cc: Ian Campbell > > Cc: devicetree at vger.kernel.org > > Cc: Dmitry Torokhov > > Cc: linux-input at vger.kernel.org > > Cc: Sascha Hauer > > Cc: linux-arm-kernel at lists.infradead.org > > Cc: Lothar Wa??mann > > Cc: Eric B??nard > > Signed-off-by: Denis Carikli > > --- > > ChangeLog v6->v7: > > - One small whitespace cleanup. > > - The properties specific to that driver are now prefixed with "ti,". > > - The ti,fuzzy property has now better documentation. > > --- > > .../bindings/input/touchscreen/tsc2007.txt | 45 +++++ > > drivers/input/touchscreen/tsc2007.c | 194 +++++++++++++++----- > > 2 files changed, 198 insertions(+), 41 deletions(-) > > create mode 100644 Documentation/devicetree/bindings/input/touchscreen/tsc2007.txt > > > > diff --git a/Documentation/devicetree/bindings/input/touchscreen/tsc2007.txt b/Documentation/devicetree/bindings/input/touchscreen/tsc2007.txt > > new file mode 100644 > > index 0000000..516b63b > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/input/touchscreen/tsc2007.txt > > @@ -0,0 +1,45 @@ > > +* Texas Instruments tsc2007 touchscreen controller > > + > > +Required properties: > > +- compatible: must be "ti,tsc2007". > > +- reg: I2C address of the chip. > > +- ti,x-plate-ohms: X-plate resistance in ohms. > > + > > +Optional properties: > > +- gpios: the interrupt gpio the chip is connected to (trough the penirq pin) > > + (see GPIO binding[2] for more details). > > Hmmm, why is this needed? Is the line used for anything other than > finding the irq number? If it is just an interrupt then an interrupts > property is all that should be specified. > It's also the pendown detect GPIO. It asserts an interrupt when first activated and is then polled in the driver until it goes inactive again. Lothar Wa?mann -- ___________________________________________________________ Ka-Ro electronics GmbH | Pascalstra?e 22 | D - 52076 Aachen Phone: +49 2408 1402-0 | Fax: +49 2408 1402-10 Gesch?ftsf?hrer: Matthias Kaussen Handelsregistereintrag: Amtsgericht Aachen, HRB 4996 www.karo-electronics.de | info at karo-electronics.de ___________________________________________________________