Re: [PATCH v5 2/5] dt-bindings: clock: spacemit: Add spacemit,k1-pll

From: Krzysztof Kozlowski
Date: Fri Mar 07 2025 - 03:20:42 EST


On Fri, Mar 07, 2025 at 06:34:47AM +0000, Haylen Chu wrote:
> > > +properties:
> > > + compatible:
> > > + const: spacemit,k1-pll
> > > +
> > > + reg:
> > > + maxItems: 1
> > > +
> > > + clocks:
> > > + description: External 24MHz oscillator
> > > +
> > > + spacemit,mpmu:
> > how about naming it as "spacemit,mpmu-syscon" explicitly?
> > to indicate this is a syscon phandle, it's more readable
>
> It's okay but I'm not sure whether this helps a lot... will wait for the
> maintainer to decide the name.

Can mpmu be not a syscon phandle? And what exactly means "syscon" here
that it brings additional information?

Best regards,
Krzysztof