On 29/08/2024 16:42, Nicolas Ferre wrote:

Alexandre,

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

quoted

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").

I don't understand this patch. The binding without any users makes no
sense. For bindings without drivers, I understand you send it in
parallel from the DTS. But this one here will have a driver, right? In
such case IT MUST GO with the driver.

Best regards,
Krzysztof