On Tue, Dec 16, 2025 at 07:11:19PM -0600, Rob Herring wrote: > On Tue, Dec 16, 2025 at 1:14 AM yuanjiey <[email protected]> > wrote: > > > > On Mon, Dec 15, 2025 at 03:36:13AM -0600, Rob Herring (Arm) wrote: > > > > > > On Mon, 15 Dec 2025 16:38:47 +0800, yuanjie yang wrote: > > > > From: Yuanjie Yang <[email protected]> > > > > > > > > Kaanapali introduces DPU 13.0.0 and DSI 2.10. Compared to SM8750, > > > > Kaanapali has significant register changes, making it incompatible > > > > with SM8750. So add MDSS/MDP display subsystem for Qualcomm Kaanapali. > > > > > > > > Co-developed-by: Yongxing Mou <[email protected]> > > > > Signed-off-by: Yongxing Mou <[email protected]> > > > > Signed-off-by: Yuanjie Yang <[email protected]> > > > > --- > > > > .../display/msm/qcom,kaanapali-mdss.yaml | 297 ++++++++++++++++++ > > > > 1 file changed, 297 insertions(+) > > > > create mode 100644 > > > > Documentation/devicetree/bindings/display/msm/qcom,kaanapali-mdss.yaml > > > > > > > > > > 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,kaanapali-mdss.example.dtb: > > > phy@9ac1000 (qcom,kaanapali-dsi-phy-3nm): reg: [[0, 162271232], [0, > > > 460], [0, 162271744], [0, 128], [0, 162272512], [0, 1024]] is too long > > > from schema $id: > > > http://devicetree.org/schemas/display/msm/dsi-phy-7nm.yaml > > > > > > doc reference errors (make refcheckdocs): > > > > > > See > > > https://patchwork.kernel.org/project/devicetree/patch/[email protected] > > > > > > The base for the series is generally the latest rc1. A different > > > dependency > > > should be noted in *this* patch. > > > > > > If you already ran 'make dt_binding_check' and didn't see the above > > > error(s), then make sure 'yamllint' is installed and dt-schema is up to > > > date: > > > > > > pip3 install dtschema --upgrade > > > > pip3 install dtschema --upgrade > > > > After update package, > > and use： make dt_binding_check I see the same issue. > > > > but use single check: > > make dt_binding_check > > DT_SCHEMA_FILES=Documentation/devicetree/bindings/display/msm/qcom,kaanapali-mdss.yaml > > I don't see any error/warning. > > Because DT_SCHEMA_FILES excludes display/msm/dsi-phy-7nm.yaml. Soon > (in the next linux-next tree), you can do: > > make display/msm/qcom,kaanapali-mdss.yaml > > Which will test the example in the specified schema against all schemas. > Thanks for your tips, will fix in next patch.
Thanks, Yuanjie > Rob
