On 11/20/2023 2:56 AM, Krzysztof Kozlowski wrote: > On 17/11/2023 22:18, Elliot Berman wrote: >> - >> -additionalProperties: false >> + - if: >> + properties: >> + compatible: >> + contains: >> + const: arm,psci-1.0 >> + then: >> + $ref: /schemas/power/reset/reboot-mode.yaml# >> + properties: >> + # "mode-normal" is just SYSTEM_RESET >> + mode-normal: false >> + patternProperties: >> + "^mode-.*$": >> + maxItems: 2 > > And if you tested the patch, it would tell you it can be max 1 item. make dt_binding_check DT_SCHEMA_FILES=arm/psci.yaml passes for me. Rob explained why it's working (and why it shouldn't), so I'll fix it according to his recommendation in v2. Thanks, Elliot