On 10/08/23 11:32 am, Krzysztof Kozlowski wrote: > EXTERNAL EMAIL: Do not click links or open attachments unless you know the content is safe > > 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. Okay. Got your point. I will fix that in the next version. Thanks! > > > Best regards, > Krzysztof > -- Thanks and Regards, Varshini Rajendran.
