Re: [PATCH v1 07/11] dt-bindings: clk: microchip: mpfs: remove first reg region

Rob Herring (Arm)
2024-10-02T23:36:31Z

On Wed, 02 Oct 2024 11:48:05 +0100, Conor Dooley wrote:
> From: Conor Dooley <conor.dooley@microchip.com>
>
> 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@microchip.com>
> ---
> .../bindings/clock/microchip,mpfs-clkcfg.yaml | 36 +++++++++++--------
> 1 file changed, 22 insertions(+), 14 deletions(-)
>

Acked-by: Rob Herring (Arm) <robh@kernel.org>