[PATCH 1/3] dt-bindings: clock: mediatek: Add #reset-cells property for MT8188

[Conor Dooley]

On Thu, May 15, 2025 at 05:11:13PM +0200, AngeloGioacchino Del Regno wrote:
> Il 15/05/25 17:03, Conor Dooley ha scritto:
> > On Thu, May 15, 2025 at 03:31:43PM +0200, Julien Massot wrote:
> > > The '#reset-cells' property is required for some of the MT8188
> > > clock controllers, but not listed as a valid property.
> > 
> > "required for some" but not marked required on those platforms.
> > Why not?
> > 
> 
> Yeah now that I read that for the third time, the wording is a bit incorrect.
> 
> It's not "required", some clock controllers do have reset controllers, but it
> is facultative to actually use the latter.

I don't think I've ever seen this word before.

> I'm not sure if the ones that do have reset controllers inside should have the
> #reset-cells property as required...
> 
> Conor, what do you think?

If "required for some" in the OP was meant as "permitted for some",
change the wording to that and it is fine.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-mediatek/attachments/20250515/d0bc1b8b/attachment.sig>