Re: [PATCH 2/7] dt-bindings: clock: qcom: Add DISPCC clocks for SM4450

From: Krzysztof Kozlowski
Date: Sun Mar 31 2024 - 04:17:56 EST


On 30/03/2024 19:28, Ajit Pandey wrote:
> Add support for qcom display clock controller bindings
> for SM4450 platform.

You cannot add support for a binding. Either you add a binding or not.
Please look at git history for inspiration.

>
> Signed-off-by: Ajit Pandey <quic_ajipan@xxxxxxxxxxx>


> +
> + clocks:
> + items:
> + - description: Board XO source
> + - description: Board active XO source
> + - description: Display AHB clock source from GCC
> + - description: sleep clock source
> + - description: Byte clock from DSI PHY0
> + - description: Pixel clock from DSI PHY0
> +
> + '#clock-cells':
> + const: 1
> +
> + '#reset-cells':
> + const: 1
> +
> + '#power-domain-cells':
> + const: 1

No power-domain? This looks incomplete.

Best regards,
Krzysztof