From mboxrd@z Thu Jan 1 00:00:00 1970 From: Dmitry Torokhov Subject: Re: [PATCH v6] input: touchscreen: Add generic driver for Silead touchscreens Date: Tue, 19 Jul 2016 11:44:25 -0700 Message-ID: <20160719184425.GC19250@dtor-ws> References: <1468252304-22850-1-git-send-email-hdegoede@redhat.com> <20160716175605.GA4873@rob-hp-laptop> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <20160716175605.GA4873@rob-hp-laptop> Sender: linux-input-owner@vger.kernel.org To: Rob Herring Cc: Hans de Goede , Maxime Ripard , Chen-Yu Tsai , linux-input@vger.kernel.org, linux-arm-kernel@lists.infradead.org, devicetree , Wessel Blokzijl , Siebren Vroegindeweij , Robert Dolca , Daniel Jansen List-Id: devicetree@vger.kernel.org On Sat, Jul 16, 2016 at 12:56:05PM -0500, Rob Herring wrote: > On Mon, Jul 11, 2016 at 05:51:44PM +0200, Hans de Goede wrote: > > From: Robert Dolca > > > > This driver adds support for Silead touchscreens. It has been tested > > with GSL1680 and GSL3680 touch panels. > > > > It supports ACPI and device tree enumeration. Screen resolution, > > the maximum number of fingers supported and firmware name are > > configurable. > > > > Signed-off-by: Robert Dolca > > Signed-off-by: Daniel Jansen > > Signed-off-by: Hans de Goede > > --- > > Changes in v6: > > -Default to 5 fingers if max-fingers is not specified > > -Improve devicetree binding doc: improve wake-gpios description, use > > "See touchscreen.txt" where applicable > > --- > > .../bindings/input/touchscreen/silead_gsl1680.txt | 36 ++ > > drivers/input/touchscreen/Kconfig | 13 + > > drivers/input/touchscreen/Makefile | 1 + > > drivers/input/touchscreen/silead.c | 652 +++++++++++++++++++++ > > 4 files changed, 702 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt > > create mode 100644 drivers/input/touchscreen/silead.c > > > > diff --git a/Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt b/Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt > > new file mode 100644 > > index 0000000..1e58d37 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/input/touchscreen/silead_gsl1680.txt > > @@ -0,0 +1,36 @@ > > +* GSL 1680 touchscreen controller > > + > > +Required properties: > > +- compatible : "silead,gsl1680" > > +- reg : I2C slave address of the chip (0x40) > > +- interrupt-parent : a phandle pointing to the interrupt controller > > + serving the interrupt for this chip > > +- interrupts : interrupt specification for the gsl1680 interrupt > > +- power-gpios : Specification for the pin connected to the gsl1680's > > + shutdown input. This needs to be driven high to take the > > + gsl1680 out of its low power state > > +- touchscreen-size-x : See touchscreen.txt > > +- touchscreen-size-y : See touchscreen.txt > > +- touchscreen-max-fingers : maximum number of fingers the touchscreen can detect > > If you name it like a standard property, then it should be documented in > touchscreen.txt. Actually this is very much Silead-specific wart, I'd rather we called it "max-fingers" or "silead,max-fingers" to illustrate that it is not at all a generic property. Rob, will you be OK with it? Thanks. -- Dmitry