Re: [PATCH 1/6] dt-bindings: clock: imx8m-anatop: support spread spectrum clocking

Krzysztof Kozlowski
2024-09-30T06:45:33Z

On 29/09/2024 22:00, Dario Binacchi wrote:
>>
>>
>>> + properties:
>>> + compatible:
>>> + contains:
>>> + enum:
>>> + - fsl,imx8mm-anatop
>>> +
>>> +then:
>>> + properties:
>>> + fsl,ssc-clocks:
>>
>> Nope. Properties must be defined in top-level.
>>
>>> + $ref: /schemas/types.yaml#/definitions/phandle-array
>>> + description:
>>> + The phandles to the PLLs with spread spectrum clock generation
>>> + hardware capability.
>>
>> These should be clocks.
>
> Sorry, but I can't understand what you're asking me.
> Could you kindly explain it to me in more detail?

You added new property instead of using existing one for this purpose:
'clocks'.

Best regards,
Krzysztof