Re: [PATCH v2 4/6] arm64: dts: axiado: Add initial support for AX3000 SoC and eval board

[Krzysztof Kozlowski <krzk@kernel.org>]

On 20/06/2025 23:18, Harshit Shah wrote:
> On 6/19/2025 11:08 PM, Krzysztof Kozlowski wrote:
>>>>> +
>>>>> +             spi_clk: spi_clk {
>>>>> +                     compatible = "fixed-clock";
>>>>> +                     #clock-cells = <0>;
>>>>> +                     clock-frequency = <25000000>;
>>>>> +             };
>>>>> +
>>>>> +             apb_pclk: apb_pclk {
>>>> No underscores in node names, but all these look incorrect - don't you
>>>> have clock controller?
>>> Noted, we will remove the "_" from the nodes. We do have clock
>>> controller however that is being accessed by other CPU before Linux will
>>> come-up.
>> What does it mean? Is the clock controller not available at all for
>> Linux or any other OS?
> 
> Apologies for the confusion. Yes, clock controller is available however 
> it is only accessible by another CPU which boots up before Linux comes up.
> 
> This another CPU is setting up the various output clocks (& PLLs) before 
> the Linux comes up.
> 
> So, that's the reason haven not added the clock controller in this DTS 
> but only will add fixed clocks.

And what happens if that other part decides to change frequencies?

Best regards,
Krzysztof