On 09/04/2024 22:19, mr.nuke.me@xxxxxxxxx wrote: >> Which binding inspired you to such change? No, you need maxItems. See >> your previous patches here how it is done. >> >> >>> >>> clock-names: >>> items: >>> - const: aux >>> - const: cfg_ahb >>> - const: pipe >>> + - const: anoc >>> + - const: snoc >> >> OK, you did not test it. Neither this, nor DTS. I stop review, please >> test first. > > I ran both `checkpatch.pl` and `make dt_binding_check`. What in this > patch makes you say I "did not test it", and what test or tests did I miss? You affect existing bindings, so you must test your and entire existing DTS. You affect, by introducing new errors, in existing DTS. Best regards, Krzysztof
