On Sat, 17 Feb 2024 at 13:39, Abel Vesa <abel.vesa@xxxxxxxxxx> wrote: > > On 24-02-16 12:32:02, Rob Herring wrote: > > > > On Fri, 16 Feb 2024 19:01:06 +0200, Abel Vesa wrote: > > > Document the MDSS hardware found on the Qualcomm X1E80100 platform. > > > > > > Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx> > > > Signed-off-by: Abel Vesa <abel.vesa@xxxxxxxxxx> > > > --- > > > .../bindings/display/msm/qcom,x1e80100-mdss.yaml | 253 +++++++++++++++++++++ > > > 1 file changed, 253 insertions(+) > > > > > > > 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: > > Documentation/devicetree/bindings/display/msm/qcom,x1e80100-mdss.example.dts:24:18: fatal error: dt-bindings/clock/qcom,x1e80100-dispcc.h: No such file or directory > > 24 | #include <dt-bindings/clock/qcom,x1e80100-dispcc.h> > > | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ > > compilation terminated. > > make[2]: *** [scripts/Makefile.lib:419: Documentation/devicetree/bindings/display/msm/qcom,x1e80100-mdss.example.dtb] Error 1 > > make[2]: *** Waiting for unfinished jobs.... > > make[1]: *** [/builds/robherring/dt-review-ci/linux/Makefile:1428: dt_binding_check] Error 2 > > make: *** [Makefile:240: __sub-make] Error 2 > > > > These bindings headers are already in -next. However they are not a part of the drm/msm branch. We can probably merge the rest of the patches from the series (once patch 1, DPU bindings, gets reviewed) and merge this one in the next cycle. Does that sound suitable? -- With best wishes Dmitry
