[PATCH RESEND v4 1/4] dt-bindings: input: touchscreen: edt-ft5x06: use unevaluatedProperties

Joel Selvaraj via B4 Relay posted 4 patches 7 months, 2 weeks ago
[PATCH RESEND v4 1/4] dt-bindings: input: touchscreen: edt-ft5x06: use unevaluatedProperties
Posted by Joel Selvaraj via B4 Relay 7 months, 2 weeks ago
From: Joel Selvaraj <foss@joelselvaraj.com>

In Xiaomi Poco F1 (qcom/sdm845-xiaomi-beryllium-ebbg.dts), the FocalTech
FT8719 touchscreen is integrally connected to the display panel
(EBBG FT8719) and thus should be power sequenced together with display
panel using the panel property. Since the edt-ft5x06 touchscreen binding
uses almost all the properties present in touchscreen.yaml, let's remove
additionalProperties: false and use unevaluatedProperties to include all
the properties, including the needed panel property.

Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Acked-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
Signed-off-by: Joel Selvaraj <foss@joelselvaraj.com>
---
 .../devicetree/bindings/input/touchscreen/edt-ft5x06.yaml        | 9 +--------
 1 file changed, 1 insertion(+), 8 deletions(-)

diff --git a/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml b/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml
index 70a922e213f2a62e3d0bd403e24ed9c3ccffd2d5..ab821490284ac0ea13568cb9ee357aea3faaa1b7 100644
--- a/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml
+++ b/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml
@@ -103,16 +103,9 @@ properties:
     minimum: 0
     maximum: 255
 
-  touchscreen-size-x: true
-  touchscreen-size-y: true
-  touchscreen-fuzz-x: true
-  touchscreen-fuzz-y: true
-  touchscreen-inverted-x: true
-  touchscreen-inverted-y: true
-  touchscreen-swapped-x-y: true
   interrupt-controller: true
 
-additionalProperties: false
+unevaluatedProperties: false
 
 required:
   - compatible

-- 
2.49.0
Re: [PATCH RESEND v4 1/4] dt-bindings: input: touchscreen: edt-ft5x06: use unevaluatedProperties
Posted by Dmitry Torokhov 7 months, 2 weeks ago
On Tue, May 06, 2025 at 01:18:38PM -0500, Joel Selvaraj via B4 Relay wrote:
> From: Joel Selvaraj <foss@joelselvaraj.com>
> 
> In Xiaomi Poco F1 (qcom/sdm845-xiaomi-beryllium-ebbg.dts), the FocalTech
> FT8719 touchscreen is integrally connected to the display panel
> (EBBG FT8719) and thus should be power sequenced together with display
> panel using the panel property. Since the edt-ft5x06 touchscreen binding
> uses almost all the properties present in touchscreen.yaml, let's remove
> additionalProperties: false and use unevaluatedProperties to include all
> the properties, including the needed panel property.
> 
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Acked-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> Signed-off-by: Joel Selvaraj <foss@joelselvaraj.com>
> ---

Applied to the input tree now, sorry about the delay.

Thanks.

-- 
Dmitry
Re: [PATCH RESEND v4 1/4] dt-bindings: input: touchscreen: edt-ft5x06: use unevaluatedProperties
Posted by Joel Selvaraj 7 months, 2 weeks ago
On 5/6/25 14:24, Dmitry Torokhov wrote:
> On Tue, May 06, 2025 at 01:18:38PM -0500, Joel Selvaraj via B4 Relay wrote:
>> From: Joel Selvaraj <foss@joelselvaraj.com>
>>
>> In Xiaomi Poco F1 (qcom/sdm845-xiaomi-beryllium-ebbg.dts), the FocalTech
>> FT8719 touchscreen is integrally connected to the display panel
>> (EBBG FT8719) and thus should be power sequenced together with display
>> panel using the panel property. Since the edt-ft5x06 touchscreen binding
>> uses almost all the properties present in touchscreen.yaml, let's remove
>> additionalProperties: false and use unevaluatedProperties to include all
>> the properties, including the needed panel property.
>>
>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>> Acked-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
>> Signed-off-by: Joel Selvaraj <foss@joelselvaraj.com>
>> ---
> 
> Applied to the input tree now, sorry about the delay.

Awesome. Thanks!

Joel