On 10/08/2023 07:31, Varshini.Rajendran@xxxxxxxxxxxxx wrote: on/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml b/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml >>> index c1bdcd9058ed..ce0d99503645 100644 >>> --- a/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml >>> +++ b/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml >>> @@ -43,6 +43,7 @@ properties: >>> - atmel,sama5d4-pmc >>> - microchip,sam9x60-pmc >>> - microchip,sama7g5-pmc >>> + - microchip,sam9x7-pmc >>> - const: syscon >> >> Where is the change in allOf:if:then:? No need for it? Why? Where is the >> driver change? > > Hi Krzysztof, > > This SoC has a new pmc driver and is not dependent on any other > compatible as a fallback. So I think adding it in the enum makes sense. This I did not question. I was speaking about allOf:if:then: section. Best regards, Krzysztof