On 06/08/2024 12:27, Xianwei Zhao via B4 Relay wrote: > From: Xianwei Zhao <xianwei.zhao@xxxxxxxxxxx> > > Add C3 PLL controller input clock parameters "fix". What is "parameters" here? Why you are adding it? Is it missing? Something is not working? > > Fixes: 0e6be855a96d ("dt-bindings: clock: add Amlogic C3 PLL clock controller") Why? What bug are you fixing? > Signed-off-by: Xianwei Zhao <xianwei.zhao@xxxxxxxxxxx> > --- > Documentation/devicetree/bindings/clock/amlogic,c3-pll-clkc.yaml | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/clock/amlogic,c3-pll-clkc.yaml b/Documentation/devicetree/bindings/clock/amlogic,c3-pll-clkc.yaml > index 43de3c6fc1cf..700865cc9792 100644 > --- a/Documentation/devicetree/bindings/clock/amlogic,c3-pll-clkc.yaml > +++ b/Documentation/devicetree/bindings/clock/amlogic,c3-pll-clkc.yaml > @@ -24,11 +24,13 @@ properties: > items: > - description: input top pll > - description: input mclk pll > + - description: input fix pll > > clock-names: > items: > - const: top > - const: mclk > + - const: fix and that's not an ABI break because? Best regards, Krzysztof