[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <220cbe91-07d1-4512-bc7e-a5af8c03eb5b@joelselvaraj.com>
Date: Tue, 22 Apr 2025 19:02:26 +0000
From: Joel Selvaraj <foss@...lselvaraj.com>
To: Bjorn Andersson <andersson@...nel.org>, Dmitry Torokhov <dmitry.torokhov@...il.com>
Cc: Konrad Dybcio <konradybcio@...nel.org>, Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>, linux-arm-msm@...r.kernel.org, devicetree@...r.kernel.org, linux-kernel@...r.kernel.org, linux-input@...r.kernel.org, Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
Subject: Re: [PATCH v4 1/4] dt-bindings: input: touchscreen: edt-ft5x06: use unevaluatedProperties
Hi Bjorn Andersson and Dmitry Torokhov,
On 3/13/25 17:18, Bjorn Andersson wrote:
> On Tue, Mar 04, 2025 at 10:45:35PM -0800, Dmitry Torokhov wrote:
>> On Mon, Mar 03, 2025 at 04:36:55PM -0600, Joel Selvaraj via B4 Relay wrote:
>>> From: Joel Selvaraj <foss@...lselvaraj.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@...aro.org>
>>> Signed-off-by: Joel Selvaraj <foss@...lselvaraj.com>
>>
>> I believe this is better be merged through the arch tree together with
>> the dts changes.
>>
>
> Then I will have to come up with a motivation to why it needed to go
> together with the dts change ;)
>
> Please pick the binding change through the input tree and I'll pick the
> dts changes once the binding has landed in linux-next.
>
>> Acked-by: Dmitry Torokhov <dmitry.torokhov@...il.com>
>>
A gently ping on this patch. It seems there was a confusion on which
tree the patch should go, but ended up not landing yet. Kindly let me
know if any further fixes are needed from my end. Would be nice to get
this landed. Thank you both.
Regards,
Joel Selvaraj
Powered by blists - more mailing lists