Re: [PATCH v3 2/5] dt-bindings: clock: document exynos7870 clock driver CMU bindings
From: Krzysztof Kozlowski
Date: Wed Feb 19 2025 - 03:07:32 EST
On Wed, Feb 19, 2025 at 12:20:29AM +0530, Kaustabh Chakraborty wrote:
> + Exynos7870 clock controller is comprised of several CMU units, generating
> + clocks for different domains. Those CMU units are modeled as separate device
> + tree nodes, and might depend on each other. The root clock in that root tree
> + is an external clock: OSCCLK (26 MHz). This external clock must be defined
> + as a fixed-rate clock in dts.
> +
> + Each clock is assigned an identifier and client nodes can use this identifier
> + to specify the clock which they consume. All clocks available for usage
> + in clock consumer nodes are defined as preprocessor macros in
> + 'dt-bindings/clock/exynos7870.h' header.
Full path and drop quotes
> +
> +properties:
> + compatible:
> + enum:
> + - samsung,exynos7870-cmu-mif
> + - samsung,exynos7870-cmu-dispaud
> + - samsung,exynos7870-cmu-fsys
> + - samsung,exynos7870-cmu-g3d
> + - samsung,exynos7870-cmu-isp
> + - samsung,exynos7870-cmu-mfcmscl
> + - samsung,exynos7870-cmu-peri
> +
> + clocks:
> + minItems: 1
> + maxItems: 10
> +
> + clock-names:
> + minItems: 1
> + maxItems: 10
> +
> + "#clock-cells":
> + const: 1
> +
> + reg:
> + maxItems: 1
> +
> +required:
> + - compatible
> + - "#clock-cells"
This goes after clock-names, to keep the same order. No need to repeat
old mistakes.
> + - clocks
> + - clock-names
> + - reg
> +
> +allOf:
> + - if:
> + properties:
> + compatible:
> + contains:
> + const: samsung,exynos7870-cmu-mif
> + then:
> + properties:
> + clocks:
> + items:
> + - description: External reference clock (26 MHz)
> +
> + clock-names:
> + items:
> + - const: oscclk
> +
> + - if:
> + properties:
> + compatible:
> + contains:
> + const: samsung,exynos7870-cmu-dispaud
> + then:
> + properties:
> + clocks:
> + items:
> + - description: External reference clock (26 MHz)
> + - description: CMU_DISPAUD bus clock (from CMU_MIF)
> + - description: DECON external clock (from CMU_MIF)
> + - description: DECON vertical clock (from CMU_MIF)
> +
> + clock-names:
> + items:
> + - const: oscclk
> + - const: bus
> + - const: decon_eclk
> + - const: decon_vclk
> +
> + - if:
> + properties:
> + compatible:
> + contains:
> + const: samsung,exynos7870-cmu-fsys
> + then:
> + properties:
> + clocks:
> + items:
> + - description: External reference clock (26 MHz)
> + - description: CMU_FSYS bus clock (from CMU_MIF)
> + - description: USB20DRD clock (from CMU_MIF)
> +
> + clock-names:
> + items:
> + - const: oscclk
> + - const: bus
> + - const: usb20drd_refclk
Drop _refclk suffix
Best regards,
Krzysztof