Received-SPF: pass (google.com: domain of linux-kernel+bounces-107888-linux.lists.archive=gmail.com@vger.kernel.org designates 139.178.88.99 as permitted sender) client-ip=139.178.88.99;
Date: Tue, 19 Mar 2024 17:06:47 +0000
From: Conor Dooley <conor@kernel.org>
To: Javier Carrasco <javier.carrasco@wolfvision.net>
Cc: Dmitry Torokhov <dmitry.torokhov@gmail.com>,
	Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Conor Dooley <conor+dt@kernel.org>,
	Henrik Rydberg <rydberg@bitmath.org>,
	Bastian Hecht <hechtb@gmail.com>,
	Michael Riesch <michael.riesch@wolfvision.net>,
	Jeff LaBundy <jeff@labundy.com>, linux-kernel@vger.kernel.org,
	linux-input@vger.kernel.org, devicetree@vger.kernel.org,
	Rob Herring <robh@kernel.org>
Subject: Re: [PATCH v7 1/4] dt-bindings: touchscreen: add touch-overlay
 property
Message-ID: <20240319-sympathy-vertigo-9bc884e5fef0@spud>
References: <20240119-feature-ts_virtobj_patch-v7-0-eda70985808f@wolfvision.net>
 <20240119-feature-ts_virtobj_patch-v7-1-eda70985808f@wolfvision.net>
 <e373c2c7-0cd9-40d6-a19b-3b9a1a81feb2@wolfvision.net>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
	protocol="application/pgp-signature"; boundary="U9CU1yZFVMXNIoZu"
Content-Disposition: inline
In-Reply-To: <e373c2c7-0cd9-40d6-a19b-3b9a1a81feb2@wolfvision.net>


--U9CU1yZFVMXNIoZu
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

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.
> >=20
> > A clipped touchscreen area avoids getting events from regions that are
> > physically hidden by overlay frames.
> >=20
> > 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.
> >=20
> > Reviewed-by: Jeff LaBundy <jeff@labundy.com>
> > Reviewed-by: Rob Herring <robh@kernel.org>
> > Signed-off-by: Javier Carrasco <javier.carrasco@wolfvision.net>
> > ---
> >  .../bindings/input/touchscreen/touchscreen.yaml    | 119 +++++++++++++=
++++++++
> >  1 file changed, 119 insertions(+)
> >=20
> > diff --git a/Documentation/devicetree/bindings/input/touchscreen/touchs=
creen.yaml b/Documentation/devicetree/bindings/input/touchscreen/touchscree=
n.yaml
> > index 431c13335c40..eb1e86fa86c6 100644
> > --- a/Documentation/devicetree/bindings/input/touchscreen/touchscreen.y=
aml
> > +++ b/Documentation/devicetree/bindings/input/touchscreen/touchscreen.y=
aml
> > @@ -87,6 +87,125 @@ properties:
> >    touchscreen-y-plate-ohms:
> >      description: Resistance of the Y-plate in Ohms
> > =20
> > +  touch-overlay:
>=20
> Even though it has already been reviewed, just in case before it gets
> applied. Should it not be:
>=20
> 	description: |
> 	  List of nodes...
>=20
> instead, to keep formatting?

Yes.

--U9CU1yZFVMXNIoZu
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZfnGJwAKCRB4tDGHoIJi
0r+pAQCQN27PZoCuDBHZdZc6ygr5G/8Gjo17ikuaVKSOz2crTAD/e2nwlGPphrqJ
h1HPjRq7LLIXp5qwQEWUA2oFZPw0TQk=
=K3lV
-----END PGP SIGNATURE-----

--U9CU1yZFVMXNIoZu--