Re: [PATCH v3 1/2] dt-bindings: clock: eswin: Documentation for eic7700 SoC

[Krzysztof Kozlowski]

On 24/06/2025 13:00, Krzysztof Kozlowski wrote:
> On 24/06/2025 12:32, dongxuyang@xxxxxxxxxxxxxxxxxx wrote:
>> +
>> +properties:
>> +  compatible:
>> +    const: eswin,eic7700-clock
>> +
>> +  reg:
>> +    maxItems: 1
>> +
>> +  '#clock-cells':
>> +    const: 1
>> +
>> +  cpu-default-frequency:
> 
> Frequency has a type - hz, use it as unit suffix if this stays.
> 
>> +    $ref: /schemas/types.yaml#/definitions/uint32
> 
> Drop
> 
> Anyway, why do you need it? Why would this be a different per board and
> why would you ever need to encode it in DT? If firmware initializes
> device, just the registers.

I missed here a word: ", just read the registers".


Best regards,
Krzysztof