Re: [PATCH v3 4/9] dt-bindings: clock: brcm,kona-ccu: Add BCM281xx bus clocks

[Krzysztof Kozlowski]

On Sat, Mar 08, 2025 at 08:50:42AM +0100, Artur Weber wrote:
> Add bus clocks corresponding to peripheral clocks currently supported
> by the BCM281xx clock driver and add the relevant clock IDs to the
> clock/bcm281xx.h dt-bindings header.

Please squash the patch so we see complete change.

> 
> Signed-off-by: Artur Weber <aweber.kernel@xxxxxxxxx>
> ---
> Changes in v3:
> - Add more clock output names to example
> - Drop CLOCK_COUNT defines from the DT binding header
> 
> Changes in v2:
> - Add this commit (BCM281xx bus clocks)
> ---
>  .../devicetree/bindings/clock/brcm,kona-ccu.yaml   | 33 ++++++++++++++++++++--
>  include/dt-bindings/clock/bcm281xx.h               | 19 +++++++++++++
>  2 files changed, 50 insertions(+), 2 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/clock/brcm,kona-ccu.yaml b/Documentation/devicetree/bindings/clock/brcm,kona-ccu.yaml
> index dff04e24e92829b890bf7cd336f0e083bdb30fa6..d00dcf916b45904177614c6f19a5df02abdf42f7 100644
> --- a/Documentation/devicetree/bindings/clock/brcm,kona-ccu.yaml
> +++ b/Documentation/devicetree/bindings/clock/brcm,kona-ccu.yaml
> @@ -40,7 +40,7 @@ properties:
>  
>    clock-output-names:
>      minItems: 1
> -    maxItems: 14
> +    maxItems: 20
>  
>  required:
>    - compatible
> @@ -61,6 +61,8 @@ allOf:
>              - const: hub_timer
>              - const: pmu_bsc
>              - const: pmu_bsc_var
> +            - const: hub_timer_apb
> +            - const: pmu_bsc_apb
>    - if:
>        properties:
>          compatible:
> @@ -86,6 +88,13 @@ allOf:
>              - const: usb_ic
>              - const: hsic2_48m
>              - const: hsic2_12m
> +            - const: sdio1_ahb
> +            - const: sdio2_ahb
> +            - const: sdio3_ahb
> +            - const: sdio4_ahb
> +            - const: usb_ic_ahb
> +            - const: hsic2_ahb
> +            - const: usb_otg_ahb
>    - if:
>        properties:
>          compatible:
> @@ -116,6 +125,16 @@ allOf:
>              - const: bsc2
>              - const: bsc3
>              - const: pwm
> +            - const: uartb_apb
> +            - const: uartb2_apb
> +            - const: uartb3_apb
> +            - const: uartb4_apb
> +            - const: ssp0_apb
> +            - const: ssp2_apb
> +            - const: bsc1_apb
> +            - const: bsc2_apb
> +            - const: bsc3_apb
> +            - const: pwm_apb

Why pwm_apb cannot be after pwm? Any idea for sorting here?

Best regards,
Krzysztof