On 26-02-27 10:57:25, Rob Herring (Arm) wrote: > > On Fri, 27 Feb 2026 16:23:00 +0200, Abel Vesa wrote: > > The Glymur platform has four DisplayPort controllers. All the > > controllers support four streams (MST). However, the first three only > > have two streams wired up physically to the display subsystem, while the > > fourth controller has only one stream (SST). > > > > So add a dedicated clause for Glymur compatible to enforce reg ranges to > > describing all four streams while allowing either one pixel clock, for the > > third DP controller, or two pixel clocks, for the rest of them. > > > > Cc: [email protected] # v6.19 > > Fixes: 8f63bf908213 ("dt-bindings: display: msm: Document the Glymur > > DiplayPort controller") > > Signed-off-by: Abel Vesa <[email protected]> > > --- > > .../bindings/display/msm/dp-controller.yaml | 21 > > ++++++++++++++++++++- > > 1 file changed, 20 insertions(+), 1 deletion(-) > > > > 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,glymur-mdss.example.dtb: > displayport-controller@ae90000 (qcom,glymur-dp): reg: [[183042048, 512], > [183042560, 512], [183043072, 1536], [183046144, 1024], [183047168, 1024]] is > too short > from schema $id: > http://devicetree.org/schemas/display/msm/dp-controller.yaml
Yep, I need to fix the qcom,glymur-mdss.yaml as well. Will do in v2.
