Received-SPF: pass (google.com: domain of linux-kernel+bounces-180124-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Date: Wed, 15 May 2024 17:16:41 +0100
From: Conor Dooley <conor@kernel.org>
To: Mark Brown <broonie@kernel.org>
Cc: Alina Yu <alina_yu@richtek.com>,
	Conor Dooley <conor.dooley@microchip.com>, lgirdwood@gmail.com,
	robh+dt@kernel.org, krzysztof.kozlowski+dt@linaro.org,
	conor+dt@kernel.org, linux-kernel@vger.kernel.org,
	devicetree@vger.kernel.org, johnny_lai@richtek.com,
	cy_huang@richtek.com
Subject: Re: [PATCH v3 6/6] regulator: dt-bindings: rtq2208: Add property to
 get ldo of RTQ2208 is adjustable or not
Message-ID: <20240515-karate-hexagram-0e24d4f595b7@spud>
References: <cover.1715340537.git.alina_yu@richtek.com>
 <6a3a90d9aa2022dfb92e124e417f3e72c2f28b0b.1715340537.git.alina_yu@richtek.com>
 <20240513-tissue-repave-13d2e3bf88fd@spud>
 <d97752ed-4032-4681-b28f-17f149fdc3d4@sirena.org.uk>
 <20240514-plunging-chair-803d9e342e6f@spud>
 <20240515073830.GA12525@linuxcarl2.richtek.com>
 <20240515-wrinkle-engross-ab6b089baae3@wendy>
 <20240515090229.GA15889@linuxcarl2.richtek.com>
 <20240515-gigantic-justifier-1fa140b59de6@spud>
 <35bfdfa3-0b4d-4457-988f-1f861dc5c125@sirena.org.uk>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
	protocol="application/pgp-signature"; boundary="pr6nnWP3Ccms1w04"
Content-Disposition: inline
In-Reply-To: <35bfdfa3-0b4d-4457-988f-1f861dc5c125@sirena.org.uk>


--pr6nnWP3Ccms1w04
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

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:
>=20
> > 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 :)
>=20
> 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.


--pr6nnWP3Ccms1w04
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iHUEABYIAB0WIQRh246EGq/8RLhDjO14tDGHoIJi0gUCZkTf6QAKCRB4tDGHoIJi
0l2JAQCJeMWvRCKSDmWruIf+Td9I9Nh1UlSAhdjk9wLI5NRGvQEA8k8UXspuNV6Q
NWS4+M4ZKDZ7I+bIMbXnMA3QpcAt6Ac=
=crMt
-----END PGP SIGNATURE-----

--pr6nnWP3Ccms1w04--