Date:   Fri, 10 Dec 2021 21:11:41 +0000
From:   Mark Brown <broonie@kernel.org>
To:     David Collins <quic_collinsd@quicinc.com>
Cc:     "Satya Priya Kakitapalli (Temp)" <quic_c_skakit@quicinc.com>,
        Bjorn Andersson <bjorn.andersson@linaro.org>,
        Rob Herring <robh+dt@kernel.org>,
        Liam Girdwood <lgirdwood@gmail.com>, swboyd@chromium.org,
        subbaram@codeaurora.org, Das Srinagesh <gurus@codeaurora.org>,
        linux-arm-msm@vger.kernel.org, Lee Jones <lee.jones@linaro.org>,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH V4 1/6] dt-bindings: regulator: Add
 "regulator-min-dropout-voltage-microvolt"
Message-ID: <YbPCjbnH6cXQqy6S@sirena.org.uk>
References: <1637314953-4215-1-git-send-email-quic_c_skakit@quicinc.com>
 <1637314953-4215-2-git-send-email-quic_c_skakit@quicinc.com>
 <YZ+o9sQpECZSrieN@sirena.org.uk>
 <d828f2a1-03e8-d6ee-4ab7-39bf677093b7@quicinc.com>
 <Ya5VhkggWdjYyTHL@sirena.org.uk>
 <6a44cb99-6894-c9ce-4f1e-5dee0939598c@quicinc.com>
 <Ya97cnuwM+MuNMg3@sirena.org.uk>
 <23a47965-4ea9-5f6c-7e3c-27f5bd35f5b7@quicinc.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="aY21D3MOZ+5AaGhF"
Content-Disposition: inline
In-Reply-To: <23a47965-4ea9-5f6c-7e3c-27f5bd35f5b7@quicinc.com>
Precedence: bulk


--aY21D3MOZ+5AaGhF
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Wed, Dec 08, 2021 at 04:56:48PM -0800, David Collins wrote:
> On 12/7/21 7:19 AM, Mark Brown wrote:
> > On Tue, Dec 07, 2021 at 08:36:11PM +0530, Satya Priya Kakitapalli (Temp) wrote:

> > that regulator.  We absolutely can and do expect this to be board
> > independent, it's a function of the design of the regulator.  Sharing
> > the input supply has no impact on this, the input voltage that the
> > regulator needs just get fed into the requiremnts on the supply voltage.

> The PM8008 LDOs are low noise LDOs intended to supply noise sensitive
> camera sensor hardware.  They can maintain output regulation with a
> fixed headroom voltage.  However, in order to guarantee high PSRR, the
> headroom voltage must be scaled according to the peak load expected from
> the each LDO on a given board.  Thus, we included support for a DT
> property to specify the headroom per LDO to meet noise requirements
> across boards.

Interesting...  how much extra headroom are we talking about here?  I'd
be unsurprised to see this usually just quoted as part of the standard
headroom requirement and this smells like the sort of thing that's going
to be frequently misused.  If the gains are something worth writing home
about I'd think we should consider if it's better to support this
dynamically at runtime based on load information and provide options for
configuring the peak load information through DT instead for static
configurations.  That would fit in with the stuff we have for managing
modes on DCDCs (which isn't really deployed but is there) and the API we
have for allowing client drivers to indicate their load requirements at
runtime that fits in with that.  That'd allow us to only boost the
headroom when it's really needed.

--aY21D3MOZ+5AaGhF
Content-Type: application/pgp-signature; name="signature.asc"

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

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmGzwo0ACgkQJNaLcl1U
h9D5JAf/UH1JxBePimL0+Pp8KDDhkWrq3f8g7S6s1mh2CgcxSnjDVAhmWiFYh22A
CdS7eEDRRbw9pDfERbScBSjw2NCFKRerD46BS+G/voolysHPatVwe6U/XmayvbJ6
A8Qkj07rDum73b1g1TmrYA+59rYMgtEFke5167sCmqhfo2XjldWcj89kAmr6+3ej
YKmkEn5j9ulNqSskq49pY5QdSRAGxnaziUfuLjnbHwuTFQ3Ce59hdYJcmqny8CJp
gP5pFW4q5Kj4vXHsP+d/Y16qnheo/jaxN4t4z+e8dF+Fdbtg9NIAV58BbsUurGab
QZHp2ThzZotfK5DdkG17CEvYx5OnDg==
=3Eej
-----END PGP SIGNATURE-----

--aY21D3MOZ+5AaGhF--