Alexandre,

On 29/08/2024 at 16:10, Alexandre Belloni wrote:

On 29/08/2024 15:08:45+0530, Dharma Balasubiramani wrote:

quoted

Add the `microchip,sama7d65-pmc` compatible string to the existing binding,
since the SAMA7D65 PMC shares the same properties and clock requirements
as the SAMA7G5.

Shouldn't you rather use a fallback if you currently have no driver
change?

The clock/pmc driver is (will be) different. Only the binding of the PMC 
uses the same properties and clocks specification as our recent SoCs (so 
can be added to the "enum").

Regards,
   Nicolas

quoted

Signed-off-by: Dharma Balasubiramani <dharma.b@microchip.com>
---
  Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml | 2 ++
  1 file changed, 2 insertions(+)

diff --git a/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml b/Documentation/devicetree/bindings/clock/atmel,at91rm9200-pmc.yaml
index c9eb60776b4d..885d47dd5724 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,sam9x7-pmc
+              - microchip,sama7d65-pmc
                - microchip,sama7g5-pmc
            - const: syscon

@@ -90,6 +91,7 @@ allOf:
              enum:
                - microchip,sam9x60-pmc
                - microchip,sam9x7-pmc
+              - microchip,sama7d65-pmc
                - microchip,sama7g5-pmc
      then:
        properties:

---
base-commit: b18bbfc14a38b5234e09c2adcf713e38063a7e6e
change-id: 20240829-sama7d65-next-a91d089b56a3

Best regards,
--
Dharma Balasubiramani [off-list ref]

--
Alexandre Belloni, co-owner and COO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com