From mboxrd@z Thu Jan 1 00:00:00 1970 From: Rob Herring Subject: Re: [PATCH 7/9] dt-bindings: input: touchscreen: goodix: Add GT5663 compatible Date: Wed, 19 Dec 2018 11:15:53 -0600 Message-ID: <20181219171553.GA4469@bogus> References: <20181203101547.16835-1-jagan@amarulasolutions.com> <20181203101547.16835-7-jagan@amarulasolutions.com> Reply-To: robh-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Return-path: Sender: linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org Content-Disposition: inline In-Reply-To: <20181203101547.16835-7-jagan-dyjBcgdgk7Pe9wHmmfpqLFaTQe2KTcn/@public.gmane.org> List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , To: Jagan Teki Cc: Maxime Ripard , Chen-Yu Tsai , Dmitry Torokhov , linux-input-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Lee Jones , linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org List-Id: devicetree@vger.kernel.org On Mon, Dec 03, 2018 at 03:45:45PM +0530, Jagan Teki wrote: > GT5663 is capacitive touch controller with customized smart wakeup gestures, > the existing goodix driver will work by phandle vcc-supply regulator. > > So, document compatible and example node for the same. > > Signed-off-by: Jagan Teki > --- > .../bindings/input/touchscreen/goodix.txt | 13 +++++++++++++ > 1 file changed, 13 insertions(+) > > diff --git a/Documentation/devicetree/bindings/input/touchscreen/goodix.txt b/Documentation/devicetree/bindings/input/touchscreen/goodix.txt > index 604766e347ce..1898d3dde8e1 100644 > --- a/Documentation/devicetree/bindings/input/touchscreen/goodix.txt > +++ b/Documentation/devicetree/bindings/input/touchscreen/goodix.txt > @@ -3,6 +3,7 @@ Device tree bindings for Goodix GT9xx series touchscreen controller > Required properties: > > - compatible : Should be "goodix,gt1151" > + or "goodix,gt5663" > or "goodix,gt911" > or "goodix,gt9110" > or "goodix,gt912" > @@ -42,3 +43,15 @@ Example: > > /* ... */ > }; > + > + touchscreen@5d { > + compatible = "goodix,gt5663"; Just a new compatible doesn't warrant a new example. > + reg = <0x5d>; > + vcc-supply = <®_ldo_io0>; > + interrupt-parent = <&pio>; > + interrupts = <7 4 IRQ_TYPE_EDGE_FALLING>; > + irq-gpios = <&pio 7 4 GPIO_ACTIVE_HIGH>; /* CTP-INT: PH4 */ > + reset-gpios = <&pio 7 8 GPIO_ACTIVE_HIGH>; /* CTP-RST: PH8 */ > + touchscreen-inverted-x; > + touchscreen-inverted-y; > + }; > -- > 2.18.0.321.gffc6fa0e3 >