The alt_ref is present only in Versal-net devices. Other versal devices do not have it so lets make alt_ref optional. Changing the order of the clock items to have the optional ones at the end. Fixes: 352546805a44 ("dt-bindings: clock: Add bindings for versal clock driver") Signed-off-by: Shubhrajyoti Datta <shubhrajyoti.datta@xxxxxxx> --- .../devicetree/bindings/clock/xlnx,versal-clk.yaml | 14 ++++++++------ 1 file changed, 8 insertions(+), 6 deletions(-) diff --git a/Documentation/devicetree/bindings/clock/xlnx,versal-clk.yaml b/Documentation/devicetree/bindings/clock/xlnx,versal-clk.yaml index 1ba687d433b1..746e49383074 100644 --- a/Documentation/devicetree/bindings/clock/xlnx,versal-clk.yaml +++ b/Documentation/devicetree/bindings/clock/xlnx,versal-clk.yaml @@ -31,11 +31,11 @@ properties: clocks: description: List of clock specifiers which are external input clocks to the given clock controller. - minItems: 3 + minItems: 2 maxItems: 8 clock-names: - minItems: 3 + minItems: 2 maxItems: 8 required: @@ -57,16 +57,18 @@ allOf: then: properties: clocks: + minItems: 2 items: - description: reference clock - - description: alternate reference clock - description: alternate reference clock for programmable logic + - description: alternate reference clock clock-names: + minItems: 2 items: - const: ref - - const: alt_ref - const: pl_alt_ref + - const: alt_ref - if: properties: @@ -110,8 +112,8 @@ examples: versal_clk: clock-controller { #clock-cells = <1>; compatible = "xlnx,versal-clk"; - clocks = <&ref>, <&alt_ref>, <&pl_alt_ref>; - clock-names = "ref", "alt_ref", "pl_alt_ref"; + clocks = <&ref>, <&pl_alt_ref>; + clock-names = "ref", "pl_alt_ref"; }; }; }; -- 2.17.1