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

From: Conor Dooley
Date: Tue Mar 19 2024 - 13:07:02 EST


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@xxxxxxxxxxx>
> > Reviewed-by: Rob Herring <robh@xxxxxxxxxx>
> > Signed-off-by: Javier Carrasco <javier.carrasco@xxxxxxxxxxxxxx>
> > ---
> > .../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.

Attachment: signature.asc
Description: PGP signature