On Wed, Nov 06, 2024 at 05:31:41PM +0200, Ivaylo Ivanov wrote:
> Convert the Actions Semi Owl CMU bindings to DT schema.
>
> Changes during conversion:
> - Since all Actions Semi Owl SoCs utilize the internal low frequency
> oscillator as a parent for some clocks, require it.
>
> Signed-off-by: Ivaylo Ivanov <ivo.ivanov.ivanov1@xxxxxxxxx>
> ---
> v2: drop address and size cells from example
...
> +description: |
> + The Actions Semi Owl Clock Management Unit generates and supplies clock
> + to various controllers within the SoC.
> +
> + All available clocks are defined as preprocessor macros in
> + include/dt-bindings/clock/ headers.
You could list the headers. Otherwise it is pretty obvious and could be
skipped.
> +
> +properties:
> + compatible:
> + enum:
> + - actions,s500-cmu
> + - actions,s700-cmu
> + - actions,s900-cmu
> +
> + clocks:
> + items:
> + - description: Host oscillator source
> + - description: Internal low frequency oscillator source
> +
> + "#clock-cells":
> + const: 1
> +
> + reg:
> + maxItems: 1
> +
> + "#reset-cells":
> + const: 1
> +
> +required:
> + - compatible
> + - reg
> + - "#clock-cells"
In the future: please keep the same order of items here in required: as
in properties:. No need to resend just for that... but:
The clocks property were required in the old binding.
> + - "#reset-cells"
> +
> +additionalProperties: false
> +
> +examples:
> + - |
> + cmu: clock-controller@e0160000 {
Drop unused label 'cmu'
Best regards,
Krzysztof
