Re: [PATCH v2 1/3] dt-bindings: clock: Add bindings for Canaan K230 clock controller

[Xukai Wang]

On 2025/1/9 17:02, Krzysztof Kozlowski wrote:
> On Wed, Jan 08, 2025 at 07:53:07PM +0800, Xukai Wang wrote:
> > +properties:
> > +  compatible:
> > +    const: canaan,k230-clk
> > +
> > +  clocks:
> > +    maxItems: 1
> > +
> > +  reg:
> > +    items:
> > +      - description: PLL control registers.
> > +      - description: Sysclk control registers.
> > +
> > +  '#clock-cells':
> > +    const: 1
> > +
> > +required:
> > +  - compatible
> > +  - reg
> > +  - clocks
>
> If there is going to be any new version, then keep the same order in
> required: as in properties:
>
> In any case:
>
> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@xxxxxxxxxx>
Thank you for the review and your suggestions.

I will ensure that the order of `required:` matches the order in 
`properties:` in any future versions of the patch.

> Best regards,
> Krzysztof