Re: [PATCH 03/13] dt-bindings: clock: Add Amlogic A9 misc clock control units

Krzysztof Kozlowski
2026-02-09T13:16:08Z

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