Re: [PATCH 1/3] dt-bindings: clock: fix C3 PLL input parameter

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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@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?

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@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?
This is "fixed" clock.
I will modify "fix" to "fixed",in next version.

Best regards,
Krzysztof





[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux