Re: [PATCH 07/15] dt-bindings: clock: Add Ambarella clock bindings

From: Krzysztof Kozlowski
Date: Fri Jan 27 2023 - 10:08:22 EST


On 27/01/2023 15:48, Li Chen wrote:
> >
> > but what you are saying is that there is no separate clock controller
> > device with its own IO address but these clocks are part of rct_syscon.
> > Then model it that way in DTS. The rct_syscon is then your clock
> > controller and all these fake gclk-core and gclk-ddr nodes should be gone.
>
> Ok, I will remove these fake nodes, and model the hardware as:
>
> rct_syscon node
> | clock node(pll, div, mux, composite clocks live in the same driver)
> | other periphal nodes

You need clock node if it takes any resources. If it doesn't, you do not
need it.

Best regards,
Krzysztof