[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <dbaa1ebc-a0a2-4ca3-bb6d-0c8a07ca1b31@kernel.org>
Date: Mon, 9 Feb 2026 14:15:53 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: chuan.liu@...ogic.com, Neil Armstrong <neil.armstrong@...aro.org>,
Michael Turquette <mturquette@...libre.com>, Stephen Boyd
<sboyd@...nel.org>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>
Cc: linux-amlogic@...ts.infradead.org, linux-clk@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH 03/13] dt-bindings: clock: Add Amlogic A9 misc clock
control units
On 09/02/2026 06:48, Chuan Liu via B4 Relay wrote:
> +
> + '#clock-cells':
> + description:
> + The clock controller of a module may contain one or more child clock(s).
> + When a clock controller has multiple child clocks, an index is required
> + to specify a particular clock within the clock controller.
> + oneOf:
> + - const: 0
> + description: Single clock output, no specifier needed
> + - const: 1
> + description: Multiple clocks, index selects specific output
> +
> + amlogic,clock-max-frequency:
No
> + description: |
> + Each clock's maximum output frequency is constrained during hardware
> + design to ensure proper timing requirements for the clock network. If the
> + clock frequency configured exceeds this design limit, it can lead to
> + abnormal behavior in modules relying on that clock and may even cause
> + cross-talk that affects other modules.
> +
> + In the driver, this property is parsed, and interface functions from the
> + CCF are called to enforce the clock's maximum frequency, preventing
> + potential issues caused by excessive clock frequency configurations.
> + $ref: /schemas/types.yaml#/definitions/uint32-array
> +
> + amlogic,clock-init-regs:
NAK
...
> diff --git a/include/dt-bindings/clock/amlogic,a9-misc-ccu.h b/include/dt-bindings/clock/amlogic,a9-misc-ccu.h
> new file mode 100644
> index 000000000000..102aff8d68e4
> --- /dev/null
> +++ b/include/dt-bindings/clock/amlogic,a9-misc-ccu.h
> @@ -0,0 +1,53 @@
> +/* SPDX-License-Identifier: (GPL-2.0-only OR MIT) */
Use the same license as in binding doc.
Best regards,
Krzysztof
Powered by blists - more mailing lists