[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240515-karate-hexagram-0e24d4f595b7@spud>
Date: Wed, 15 May 2024 17:16:41 +0100
From: Conor Dooley <conor@...nel.org>
To: Mark Brown <broonie@...nel.org>
Cc: Alina Yu <alina_yu@...htek.com>,
Conor Dooley <conor.dooley@...rochip.com>, lgirdwood@...il.com,
robh+dt@...nel.org, krzysztof.kozlowski+dt@...aro.org,
conor+dt@...nel.org, linux-kernel@...r.kernel.org,
devicetree@...r.kernel.org, johnny_lai@...htek.com,
cy_huang@...htek.com
Subject: Re: [PATCH v3 6/6] regulator: dt-bindings: rtq2208: Add property to
get ldo of RTQ2208 is adjustable or not
On Wed, May 15, 2024 at 05:04:32PM +0100, Mark Brown wrote:
> On Wed, May 15, 2024 at 04:51:30PM +0100, Conor Dooley wrote:
>
> > BTW, you should probably change the example so that the voltage you add
> > there is actually outside of the range, rather than identical to one of
> > the range's constraints :)
>
> No, that'd be invalid - the constraints need to include a value offered
> by the regulator, in this case the one fixed voltage. For a fixed
> voltage regulator it's probably better to just not specify a voltage
> range since it can't be changed anyway.
I see. Clearly I got your previous explanation of why the property was
needed arseways.
Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)
Powered by blists - more mailing lists