[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <ebf41227-a0c5-44eb-b6a7-53e30b24e298@linaro.org>
Date: Fri, 21 Feb 2025 14:16:23 +0100
From: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
To: "Rob Herring (Arm)" <robh@...nel.org>
Cc: David Airlie <airlied@...il.com>, Simona Vetter <simona@...ll.ch>,
Maarten Lankhorst <maarten.lankhorst@...ux.intel.com>,
Maxime Ripard <mripard@...nel.org>, Thomas Zimmermann <tzimmermann@...e.de>,
Rob Clark <robdclark@...il.com>, devicetree@...r.kernel.org,
Conor Dooley <conor+dt@...nel.org>, Sean Paul <sean@...rly.run>,
Marijn Suijten <marijn.suijten@...ainline.org>,
dri-devel@...ts.freedesktop.org, Abhinav Kumar <quic_abhinavk@...cinc.com>,
Dmitry Baryshkov <dmitry.baryshkov@...aro.org>,
linux-arm-msm@...r.kernel.org, Krzysztof Kozlowski <krzk+dt@...nel.org>,
freedreno@...ts.freedesktop.org, linux-kernel@...r.kernel.org,
Mahadevan <quic_mahap@...cinc.com>
Subject: Re: [PATCH] dt-bindings: display/msm: qcom,sa8775p-mdss: Add missing
eDP phy
On 21/02/2025 14:14, Rob Herring (Arm) wrote:
>
> On Fri, 21 Feb 2025 13:17:03 +0100, Krzysztof Kozlowski wrote:
>> The Qualcomm SA8775p MDSS display block comes with eDP phy, already used
>> in DTS and already documented in phy/qcom,edp-phy.yaml binding. Add the
>> missing device node in the binding and extend example to silence
>> dtbs_check warnings like:
>>
>> sa8775p-ride.dtb: display-subsystem@...0000: Unevaluated properties are not allowed ('phy@...2a00', 'phy@...5a00' were unexpected)
>>
>> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
>> ---
>> .../display/msm/qcom,sa8775p-mdss.yaml | 32 +++++++++++++++++--
>> 1 file changed, 30 insertions(+), 2 deletions(-)
>>
>
> My bot found errors running 'make dt_binding_check' on your patch:
>
> yamllint warnings/errors:
>
> dtschema/dtc warnings/errors:
> /builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/display/msm/qcom,sa8775p-mdss.example.dtb: phy@...2a00: reg: [[0, 183249408], [0, 512], [0, 183247360], [0, 208], [0, 183248384], [0, 208], [0, 183246848], [0, 456]] is too long
> from schema $id: http://devicetree.org/schemas/phy/qcom,edp-phy.yaml#
>
Insufficient testing. I'll send v2.
Best regards,
Krzysztof
Powered by blists - more mailing lists