On Fri, Oct 20, 2023 at 06:03:29PM +0200, Sebastian Reichel wrote: > Hi Conor, > > On Fri, Oct 20, 2023 at 04:36:19PM +0100, Conor Dooley wrote: > > On Fri, Oct 20, 2023 at 04:11:40PM +0200, Sebastian Reichel wrote: > > > [...] > > > +allOf: > > > + - $ref: snps,dwc3.yaml# > > > + - if: > > > + properties: > > > + compatible: > > > + contains: > > > + const: rockchip,rk3328-dwc3 > > > + then: > > > + properties: > > > + clocks: > > > + minItems: 3 > > > + maxItems: 4 > > > + clock-names: > > > + minItems: 3 > > > + items: > > > + - const: ref_clk > > > + - const: suspend_clk > > > + - const: bus_clk > > > + - const: grf_clk > > > > minItems for clocks and clock-names is already 3, is it not? > > Yes, but the following 'maxItems: 4' implicitly sets it to 4, > so I had to set it again. The same is true for clock-names - > providings new 'items:' effectively drops the "minItems: 3" > from the generic section. Are you sure? We don't add anything implicit in the if/then schemas. Could be a tool issue though. Rob
