Quoting Dmitry Baryshkov (2022-07-07 14:32:00) > diff --git a/Documentation/devicetree/bindings/display/msm/dp-controller.yaml > b/Documentation/devicetree/bindings/display/msm/dp-controller.yaml > index f00eae66196f..1ef845005b14 100644 > --- a/Documentation/devicetree/bindings/display/msm/dp-controller.yaml > +++ b/Documentation/devicetree/bindings/display/msm/dp-controller.yaml > @@ -119,6 +111,50 @@ required: > - power-domains > - ports > > +allOf: > + - if: > + properties: > + compatible: > + contains: > + enum: > + - qcom,sc7280-edp > + then: > + properties: > + clocks: > + items: > + - description: XO clock
What is this for? I would guess it's for the eDP phy, but that isn't part of the eDP controller, so probably it can be removed. > + - description: eDP reference clock Same for this one, looking at the binding for qcom,sc7280-edp-phy. Can we simply remove these two clks from sc7280? I think it will be fine.
