[RESEND PATCH] dt-bindings: display/msm: add missing brace in dpu-qcm2290.yaml

Rob Herring robh+dt at kernel.org
Tue Mar 8 19:36:10 UTC 2022


On Tue, Mar 1, 2022 at 6:14 PM Dmitry Baryshkov
<dmitry.baryshkov at linaro.org> wrote:
>
> Add missing brace in dpu-qcm2290.yaml. While we are at it, also fix
> indentation for another brace, so it matches the corresponding line.
>
> Reported-by: Rob Herring <robh at kernel.org>
> Cc: Loic Poulain <loic.poulain at linaro.org>
> Reviewed-by: Bjorn Andersson <bjorn.andersson at linaro.org>
> Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov at linaro.org>
> ---
> Didn't include freedreno@ in the first email, so resending.
> ---
>  Documentation/devicetree/bindings/display/msm/dpu-qcm2290.yaml | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

Now that the example actually builds, we get just schema warnings:

/builds/robherring/linux-dt/Documentation/devicetree/bindings/display/msm/dpu-qcm2290.example.dt.yaml:
mdss at 5e00000: compatible: ['qcom,qcm2290-mdss', 'qcom,mdss'] is too
long
>From schema: /builds/robherring/linux-dt/Documentation/devicetree/bindings/display/msm/dpu-qcm2290.yaml
/builds/robherring/linux-dt/Documentation/devicetree/bindings/display/msm/dpu-qcm2290.example.dt.yaml:
mdss at 5e00000: 'mdp at 5e01000' does not match any of the regexes:
'^display-controller@[0-9a-f]+$', 'pinctrl-[0-9]+'
>From schema: /builds/robherring/linux-dt/Documentation/devicetree/bindings/display/msm/dpu-qcm2290.yaml


I would have assumed upon reporting errors with 'make
dt_binding_check' that the fixes would be tested with 'make
dt_binding_check'...

Rob


More information about the dri-devel mailing list