On 13/01/2023 17:26, Rob Herring wrote:
On Fri, 13 Jan 2023 10:37:13 +0200, Dmitry Baryshkov wrote:
Follow the 'generic names' rule and rename mdss nodes to
display-subsystem.
Signed-off-by: Dmitry Baryshkov <dmitry.barysh...@linaro.org>
---
.../devicetree/bindings/display/msm/mdss-common.yaml | 8 ++++++++
.../devicetree/bindings/display/msm/qcom,mdss.yaml | 5 ++++-
2 files changed, 12 insertions(+), 1 deletion(-)
My bot found errors running 'make DT_CHECKER_FLAGS=-m dt_binding_check'
on your patch (DT_CHECKER_FLAGS is new in v5.13):
yamllint warnings/errors:
dtschema/dtc warnings/errors:
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.example.dtb:
mdss@5e00000: $nodename:0: 'mdss@5e00000' does not match
'^display-subsystem@[0-9a-f]+$'
From schema:
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/display/msm/qcom,sm6115-mdss.yaml
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.example.dtb:
mdss@5e00000: $nodename:0: 'mdss@5e00000' does not match
'^display-subsystem@[0-9a-f]+$'
From schema:
/builds/robherring/dt-review-ci/linux/Documentation/devicetree/bindings/display/msm/qcom,qcm2290-mdss.yaml
This should be fixed already by the commit e5266ca38294 ("dt-bindings:
display: msm: Rename mdss node name in example")
See https://gitlab.freedesktop.org/drm/msm/-/commit/e5266ca38294
--
With best wishes
Dmitry
