[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20250515-deploy-wiring-5829acc3d82e@spud>
Date: Thu, 15 May 2025 17:17:11 +0100
From: Conor Dooley <conor@...nel.org>
To: AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>
Cc: Julien Massot <julien.massot@...labora.com>, kernel@...labora.com,
Michael Turquette <mturquette@...libre.com>,
Stephen Boyd <sboyd@...nel.org>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>,
Conor Dooley <conor+dt@...nel.org>,
Matthias Brugger <matthias.bgg@...il.com>,
Garmin Chang <garmin.chang@...iatek.com>,
Friday Yang <friday.yang@...iatek.com>,
Conor Dooley <conor.dooley@...rochip.com>,
linux-clk@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-mediatek@...ts.infradead.org
Subject: Re: [PATCH 1/3] dt-bindings: clock: mediatek: Add #reset-cells
property for MT8188
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.
Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)
Powered by blists - more mailing lists