Re: [RFC PATCH v2 2/9] dt-bindings: soc: microchip: document the simple-mfd syscon on PolarFire SoC

Conor Dooley
2025-03-26T11:01:11Z

[-- Attachment #1.1: Type: text/plain, Size: 1241 bytes --]

On Wed, Mar 26, 2025 at 07:53:55AM +0100, Krzysztof Kozlowski wrote:
> On 25/03/2025 17:03, Conor Dooley wrote:
> > On Tue, Mar 25, 2025 at 09:13:22AM +0100, Krzysztof Kozlowski wrote:
> >> On Fri, Mar 21, 2025 at 05:22:35PM +0000, Conor Dooley wrote:
> >>> +title: Microchip PolarFire SoC Microprocessor Subsystem (MSS) sysreg register region
> >>> +
> >>> +maintainers:
> >>> + - Conor Dooley <conor.dooley@microchip.com>
> >>> +
> >>> +description:
> >>> + An wide assortment of registers that control elements of the MSS on PolarFire
> >>> + SoC, including pinmuxing, resets and clocks among others.
> >>> +
> >>> +properties:
> >>> + compatible:
> >>> + items:
> >>> + - const: microchip,mpfs-mss-top-sysreg
> >>> + - const: syscon
> >>> + - const: simple-mfd
> >>
> >> You need to list the children if you use simple-mfd. Commit msg
> >> mentioned clock controller, so where is it?
> >
> > I don't think a child node is required here, there's not enough
>
> Then this is not a simple-mfd.

The pinctrl will have one, whenever I get around to actually working on
that. I can leave the simple-mfd out until I establish exactly what
that's going to look like if that's what you want?

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

[-- Attachment #2: Type: text/plain, Size: 161 bytes --]