On Thu, Jun 17, 2021 at 09:09:06PM +0200, Daniel Mack wrote: > CLK_IN is only used in dynamic mode and is hence optional. > > Re-order the clocks so REF_CLK is specified first. It's not a compatible change. You can't do that unless you can explain why it doesn't matter. > > Signed-off-by: Daniel Mack <daniel@xxxxxxxxxx> > --- > .../devicetree/bindings/clock/cirrus,cs2000-cp.yaml | 11 ++++++----- > 1 file changed, 6 insertions(+), 5 deletions(-) > > diff --git a/Documentation/devicetree/bindings/clock/cirrus,cs2000-cp.yaml b/Documentation/devicetree/bindings/clock/cirrus,cs2000-cp.yaml > index 79b90500f6ac..68efed8e5033 100644 > --- a/Documentation/devicetree/bindings/clock/cirrus,cs2000-cp.yaml > +++ b/Documentation/devicetree/bindings/clock/cirrus,cs2000-cp.yaml > @@ -22,14 +22,15 @@ properties: > > clocks: > description: > - Common clock binding for CLK_IN, XTI/REF_CLK > - minItems: 2 > + Common clock binding for XTI/REF_CLK, CLK_IN. > + CLK_IN is optional and only used in dynamic mode. > + minItems: 1 > maxItems: 2 > > clock-names: > items: > - - const: clk_in > - const: ref_clk > + - const: clk_in > > '#clock-cells': > const: 0 > @@ -70,8 +71,8 @@ examples: > #clock-cells = <0>; > compatible = "cirrus,cs2000-cp"; > reg = <0x4f>; > - clocks = <&rcar_sound 0>, <&x12_clk>; > - clock-names = "clk_in", "ref_clk"; > + clocks = <&x12_clk>, <&rcar_sound 0>; > + clock-names = "ref_clk", "clk_in"; > cirrus,aux-output-source = <CS2000CP_AUX_OUTPUT_CLK_OUT>; > }; > }; > -- > 2.31.1 > >
