On Wed, 02 Oct 2024 11:48:05 +0100, Conor Dooley wrote: > From: Conor Dooley <conor.dooley@xxxxxxxxxxxxx> > > The first reg region in this binding is not exclusively for clocks, as > evidenced by the dual role of this device as a reset controller at > present. The first region is however better described by a simple-mfd > syscon, but this would have require a significant re-write of the > devicetree for the platform, so the easy way out was chosen when reset > support was first introduced. The region doesn't just contain clock and > reset registers, it also contains pinctrl and interrupt controller > functionality, so drop the region from the clock binding so that it can > be described instead by a simple-mfd syscon rather than propagate this > incorrect description of the hardware to the new pic64gx SoC. > > Signed-off-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx> > --- > .../bindings/clock/microchip,mpfs-clkcfg.yaml | 36 +++++++++++-------- > 1 file changed, 22 insertions(+), 14 deletions(-) > Acked-by: Rob Herring (Arm) <robh@xxxxxxxxxx>