Re: [PATCH v5 01/11] ASoC: dt-bindings: renesas,fsi: add support multiple clocks

[Bui Duc Phuc]

Hi Krzysztof,

Thank you for the clarification.

>
> Odd. clock-names are string-array which has uniqueItems by default.
>
> That's the first need of usage it, so I think you should skip it even if
> it does not work correctly - the true fix should be in dtschema.

Understood. I will remove "uniqueItems: true" in the next version.

While testing, I also noticed that the schema accepts cases where the
number of entries in "clock-names" does not match the number of entries
in "clocks". For example:

clocks = <&clk1>, <&clk2>;
clock-names = "fck", "spu", "icka";

This passes dtbs_check on my setup.

Is this also expected to be handled by dtschema, or would an explicit
constraint be needed in the binding?

Best regards,
Phuc