linux-kernel - Re: [PATCH v7 1/4] dt-bindings: touchscreen: add touch-overlay property

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <20240319-sympathy-vertigo-9bc884e5fef0@spud>
Date: Tue, 19 Mar 2024 17:06:47 +0000
From: Conor Dooley <conor@...nel.org>
To: Javier Carrasco <javier.carrasco@...fvision.net>
Cc: Dmitry Torokhov <dmitry.torokhov@...il.com>,
	Rob Herring <robh+dt@...nel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
	Conor Dooley <conor+dt@...nel.org>,
	Henrik Rydberg <rydberg@...math.org>,
	Bastian Hecht <hechtb@...il.com>,
	Michael Riesch <michael.riesch@...fvision.net>,
	Jeff LaBundy <jeff@...undy.com>, linux-kernel@...r.kernel.org,
	linux-input@...r.kernel.org, devicetree@...r.kernel.org,
	Rob Herring <robh@...nel.org>
Subject: Re: [PATCH v7 1/4] dt-bindings: touchscreen: add touch-overlay
 property

On Tue, Mar 19, 2024 at 12:13:08PM +0100, Javier Carrasco wrote:
> On 1/19/24 08:43, Javier Carrasco wrote:
> > The touch-overlay encompasses a number of touch areas that define a
> > clipped touchscreen area and/or buttons with a specific functionality.
> > 
> > A clipped touchscreen area avoids getting events from regions that are
> > physically hidden by overlay frames.
> > 
> > For touchscreens with printed overlay buttons, sub-nodes with a suitable
> > key code can be defined to report key events instead of the original
> > touch events.
> > 
> > Reviewed-by: Jeff LaBundy <jeff@...undy.com>
> > Reviewed-by: Rob Herring <robh@...nel.org>
> > Signed-off-by: Javier Carrasco <javier.carrasco@...fvision.net>
> > ---
> >  .../bindings/input/touchscreen/touchscreen.yaml    | 119 +++++++++++++++++++++
> >  1 file changed, 119 insertions(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/input/touchscreen/touchscreen.yaml b/Documentation/devicetree/bindings/input/touchscreen/touchscreen.yaml
> > index 431c13335c40..eb1e86fa86c6 100644
> > --- a/Documentation/devicetree/bindings/input/touchscreen/touchscreen.yaml
> > +++ b/Documentation/devicetree/bindings/input/touchscreen/touchscreen.yaml
> > @@ -87,6 +87,125 @@ properties:
> >    touchscreen-y-plate-ohms:
> >      description: Resistance of the Y-plate in Ohms
> >  
> > +  touch-overlay:
> 
> Even though it has already been reviewed, just in case before it gets
> applied. Should it not be:
> 
> 	description: |
> 	  List of nodes...
> 
> instead, to keep formatting?

Yes.

Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)