On 18/07/2022 17:24, Phil Edworthy wrote: >>> phys: >>> maxItems: 1 >>> @@ -43,7 +65,15 @@ properties: >>> maxItems: 1 >>> >>> resets: >>> - maxItems: 1 >>> + minItems: 1 >>> + items: >>> + - description: Peripheral reset >>> + - description: DRD reset >>> + >>> + reset-names: >>> + items: >>> + - const: aresetn_p >>> + - const: drd_reset >>> >>> usb-role-switch: >>> $ref: /schemas/types.yaml#/definitions/flag >>> @@ -78,6 +108,27 @@ required: >>> - interrupts >>> - clocks >>> >>> +allOf: >>> + - if: >>> + properties: >>> + compatible: >>> + contains: >>> + enum: >>> + - renesas,rzv2m-usb3-peri >>> + then: >>> + properties: >>> + clocks: >>> + minItems: 2 >>> + interrupts: >>> + minItems: 2 >>> + resets: >>> + minItems: 2 >>> + required: >>> + - clock-names >>> + - interrupt-names >>> + - resets >>> + - reset-names >> >> else: >> narrow the number of items > Sorry, I don't understand why we need minItems: 1 for > interrupt-names/clock-names, but then I'm easily confused! > > None of the existing users have any interrupt-names/clock-names > hence they are not in required. The rzv2m is the only device > that needs them so the driver can get them by name, and hence > it sets minItems: 2 They are not required but they can appear. Nothing prevents it, based on your patch. Best regards, Krzysztof
