Message ID | 20250301-pocof1-touchscreen-support-v3-1-af01c3b30b55@joelselvaraj.com (mailing list archive) |
---|---|
State | New |
Headers | show |
Series | Add Xiaomi Poco F1 touchscreen support | expand |
On Saturday, March 1st, 2025 at 5:43 PM, Joel Selvaraj via B4 Relay <devnull+foss.joelselvaraj.com@kernel.org> wrote: > ...clipped... > Since the touchscreen controller uses almost all the properties present in touchscreen.yaml... Sorry, should have been "Since the edt-ft5x06 touchscreen binding document uses almost all the properties...". I will wait for further review by others and will fix it v4. Or if it is acceptable with current commit message, let me know. Thanks, Joel Selvaraj
On Sat, Mar 01, 2025 at 05:43:07PM -0600, Joel Selvaraj wrote: > 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 touchscreen controller 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. > > Signed-off-by: Joel Selvaraj <foss@joelselvaraj.com> > --- > .../devicetree/bindings/input/touchscreen/edt-ft5x06.yaml | 9 +-------- > 1 file changed, 1 insertion(+), 8 deletions(-) Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml b/Documentation/devicetree/bindings/input/touchscreen/edt-ft5x06.yaml index 70a922e213f2..ab821490284a 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