[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAL_JsqLR8e4FND5LfAVB+cJDMiFpgBTKZtpEDDQiuQfGJu2LKg@mail.gmail.com>
Date: Tue, 16 Dec 2025 19:11:19 -0600
From: Rob Herring <robh@...nel.org>
To: yuanjiey <yuanjie.yang@....qualcomm.com>
Cc: yongxing.mou@....qualcomm.com, robin.clark@....qualcomm.com,
tzimmermann@...e.de, dri-devel@...ts.freedesktop.org, conor+dt@...nel.org,
devicetree@...r.kernel.org, konrad.dybcio@....qualcomm.com,
freedreno@...ts.freedesktop.org, linux-kernel@...r.kernel.org,
krzk+dt@...nel.org, neil.armstrong@...aro.org, jesszhan0024@...il.com,
airlied@...il.com, maarten.lankhorst@...ux.intel.com,
linux-arm-msm@...r.kernel.org, lumag@...nel.org, simona@...ll.ch,
tingwei.zhang@....qualcomm.com, aiqun.yu@....qualcomm.com, sean@...rly.run,
marijn.suijten@...ainline.org, abhinav.kumar@...ux.dev, mripard@...nel.org
Subject: Re: [PATCH v3 04/11] dt-bindings: display/msm: qcom,kaanapali-mdss:
Add Kaanapali
On Tue, Dec 16, 2025 at 1:14 AM yuanjiey <yuanjie.yang@....qualcomm.com> 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 <yuanjie.yang@....qualcomm.com>
> > >
> > > 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 <yongxing.mou@....qualcomm.com>
> > > Signed-off-by: Yongxing Mou <yongxing.mou@....qualcomm.com>
> > > Signed-off-by: Yuanjie Yang <yuanjie.yang@....qualcomm.com>
> > > ---
> > > .../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@...1000 (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/20251215083854.577-5-yuanjie.yang@oss.qualcomm.com
> >
> > 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.
Rob
Powered by blists - more mailing lists