[PATCH 1/3] dt-bindings: clock: fix C3 PLL input parameter
Xianwei Zhao
xianwei.zhao at amlogic.com
Tue Aug 6 18:55:06 PDT 2024
Hi Krzysztof,
Thanks for your review.
On 2024/8/6 21:10, Krzysztof Kozlowski wrote:
> [ EXTERNAL EMAIL ]
>
> On 06/08/2024 12:27, Xianwei Zhao via B4 Relay wrote:
>> From: Xianwei Zhao <xianwei.zhao at amlogic.com>
>>
>> Add C3 PLL controller input clock parameters "fix".
>
> What is "parameters" here? Why you are adding it? Is it missing?
> Something is not working?
>
Yes. The previous submission was lost.
>>
>> Fixes: 0e6be855a96d ("dt-bindings: clock: add Amlogic C3 PLL clock controller")
>
> Why? What bug are you fixing?
The input clock of PLL clock controller need the clock whose fw_name is
called "fix".
>
>> Signed-off-by: Xianwei Zhao <xianwei.zhao at amlogic.com>
>> ---
>> 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?
This is "fixed" clock.
I will modify "fix" to "fixed",in next version.
>
> Best regards,
> Krzysztof
>
More information about the linux-amlogic
mailing list