| Message ID | 20220630120845.3356144-6-bryan.odonoghue@linaro.org |
|---|---|
| State | New |
| Headers | show |
| Series | msm8916/msm8939 DSI controller and DSI PHY bindings fixes | expand |
On 30/06/2022 14:08, Bryan O'Donoghue wrote: > When converting from .txt to .yaml dt-binding descriptions we appear to > have missed some of the previous detail on the number and names of > permissible clocks. > > Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings") > Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> > --- > .../display/msm/dsi-controller-main.yaml | 23 +++++++++++++------ > 1 file changed, 16 insertions(+), 7 deletions(-) > > diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml > index fd9d472437853..b24ba6c346a77 100644 > --- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml > +++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml > @@ -28,22 +28,31 @@ properties: > maxItems: 1 > > clocks: > - items: > - - description: Display byte clock > - - description: Display byte interface clock > - - description: Display pixel clock > - - description: Display escape clock > - - description: Display AHB clock > - - description: Display AXI clock > + minItems: 6 > + maxItems: 9 > > clock-names: > items: > - const: byte > + description: Display byte clock Descriptions should stay where they were. Just add minItems and new items. > - const: byte_intf > + description: Display byte interface clock > - const: pixel > + description: Display pixel clock > - const: core > + description: Display escape clock > - const: iface > + description: Display AHB clock > - const: bus > + description: Display AXI clock > + - const: core_mmss > + description: Core MultiMedia SubSystem clock > + - const: mdp_core > + description: MDP Core clock > + - const: mnoc > + description: MNOC clock > + minItems: 6 > + maxItems: 9 maxItems are not needed. > > phys: > maxItems: 1 Best regards, Krzysztof
diff --git a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml index fd9d472437853..b24ba6c346a77 100644 --- a/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml +++ b/Documentation/devicetree/bindings/display/msm/dsi-controller-main.yaml @@ -28,22 +28,31 @@ properties: maxItems: 1 clocks: - items: - - description: Display byte clock - - description: Display byte interface clock - - description: Display pixel clock - - description: Display escape clock - - description: Display AHB clock - - description: Display AXI clock + minItems: 6 + maxItems: 9 clock-names: items: - const: byte + description: Display byte clock - const: byte_intf + description: Display byte interface clock - const: pixel + description: Display pixel clock - const: core + description: Display escape clock - const: iface + description: Display AHB clock - const: bus + description: Display AXI clock + - const: core_mmss + description: Core MultiMedia SubSystem clock + - const: mdp_core + description: MDP Core clock + - const: mnoc + description: MNOC clock + minItems: 6 + maxItems: 9 phys: maxItems: 1
When converting from .txt to .yaml dt-binding descriptions we appear to have missed some of the previous detail on the number and names of permissible clocks. Fixes: 4dbe55c97741 ("dt-bindings: msm: dsi: add yaml schemas for DSI bindings") Signed-off-by: Bryan O'Donoghue <bryan.odonoghue@linaro.org> --- .../display/msm/dsi-controller-main.yaml | 23 +++++++++++++------ 1 file changed, 16 insertions(+), 7 deletions(-)