| Message ID | 20220811084331.83715-2-krzysztof.kozlowski@linaro.org (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | dt-bindings: display/msm: dpu: opp-table fixes | expand |
On Thu, Aug 11, 2022 at 11:43:27AM +0300, Krzysztof Kozlowski wrote: > The 'display-controller' child (DPU) of Display SubSystem (MDSS) uses > opp-table, so reference it which allows restricting DPU schema to fixed > list of properties. > > Fixes: 6e986a8f1cf1 ("dt-bindings: display: msm: Add binding for msm8998 dpu") > Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> > > --- > > Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> > Cc: Rob Clark <robdclark@gmail.com> > Cc: Sean Paul <sean@poorly.run> > Cc: Abhinav Kumar <quic_abhinavk@quicinc.com> > --- > Documentation/devicetree/bindings/display/msm/dpu-msm8998.yaml | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/Documentation/devicetree/bindings/display/msm/dpu-msm8998.yaml b/Documentation/devicetree/bindings/display/msm/dpu-msm8998.yaml > index 2df64afb76e6..7ed438bc7dce 100644 > --- a/Documentation/devicetree/bindings/display/msm/dpu-msm8998.yaml > +++ b/Documentation/devicetree/bindings/display/msm/dpu-msm8998.yaml > @@ -62,6 +62,7 @@ patternProperties: > "^display-controller@[0-9a-f]+$": > type: object > description: Node containing the properties of DPU. > + additionalProperties: false > > properties: > compatible: > @@ -105,6 +106,8 @@ patternProperties: > maxItems: 1 > > operating-points-v2: true > + opp-table: true type: object Otherwise, 'opp-table;' would be accepted. > + > ports: > $ref: /schemas/graph.yaml#/properties/ports > description: | > -- > 2.34.1 > >
On 15/08/2022 00:25, Rob Herring wrote: >> properties: >> compatible: >> @@ -105,6 +106,8 @@ patternProperties: >> maxItems: 1 >> >> operating-points-v2: true >> + opp-table: true > > type: object > > Otherwise, 'opp-table;' would be accepted. > Thanks, I'll fix it. Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/display/msm/dpu-msm8998.yaml b/Documentation/devicetree/bindings/display/msm/dpu-msm8998.yaml index 2df64afb76e6..7ed438bc7dce 100644 --- a/Documentation/devicetree/bindings/display/msm/dpu-msm8998.yaml +++ b/Documentation/devicetree/bindings/display/msm/dpu-msm8998.yaml @@ -62,6 +62,7 @@ patternProperties: "^display-controller@[0-9a-f]+$": type: object description: Node containing the properties of DPU. + additionalProperties: false properties: compatible: @@ -105,6 +106,8 @@ patternProperties: maxItems: 1 operating-points-v2: true + opp-table: true + ports: $ref: /schemas/graph.yaml#/properties/ports description: |
The 'display-controller' child (DPU) of Display SubSystem (MDSS) uses opp-table, so reference it which allows restricting DPU schema to fixed list of properties. Fixes: 6e986a8f1cf1 ("dt-bindings: display: msm: Add binding for msm8998 dpu") Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> --- Cc: Dmitry Baryshkov <dmitry.baryshkov@linaro.org> Cc: Rob Clark <robdclark@gmail.com> Cc: Sean Paul <sean@poorly.run> Cc: Abhinav Kumar <quic_abhinavk@quicinc.com> --- Documentation/devicetree/bindings/display/msm/dpu-msm8998.yaml | 3 +++ 1 file changed, 3 insertions(+)