Received-SPF: pass (google.com: domain of linux-kernel+bounces-9698-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Date: Fri, 22 Dec 2023 12:24:01 +0000
From: Mark Brown <broonie@kernel.org>
To: Martin Blumenstingl <martin.blumenstingl@googlemail.com>
Cc: lgirdwood@gmail.com,
	Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= <u.kleine-koenig@pengutronix.de>,
	linux-pwm@vger.kernel.org, linux-amlogic@lists.infradead.org,
	linux-kernel@vger.kernel.org,
	Thierry Reding <thierry.reding@gmail.com>,
	Heiner Kallweit <hkallweit1@gmail.com>,
	Dmitry Rokosov <ddrokosov@sberdevices.ru>
Subject: Re: [RFC PATCH v1] regulator: pwm-regulator: Fix continuous
 get_voltage for disabled PWM
Message-ID: <9bea64d5-8689-48f0-a081-5da60434e6c0@sirena.org.uk>
References: <20231221211222.1380658-1-martin.blumenstingl@googlemail.com>
 <0c99b575-5cf2-4bd6-8cfd-af19f5fd58da@sirena.org.uk>
 <CAFBinCDJnVzE2sMwu52MQGTKW7dtCuUoj63ZZHhJPJO0+dZDkg@mail.gmail.com>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
	protocol="application/pgp-signature"; boundary="juKFXjvLaxOmiLZO"
Content-Disposition: inline
In-Reply-To: <CAFBinCDJnVzE2sMwu52MQGTKW7dtCuUoj63ZZHhJPJO0+dZDkg@mail.gmail.com>


--juKFXjvLaxOmiLZO
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Thu, Dec 21, 2023 at 11:42:29PM +0100, Martin Blumenstingl wrote:

> The vendor BSP includes a custom u-boot with lots of relevant
> information for which there's seemingly no documentation.
> It seems that 1.1V is what should be used during normal operation.
> 0.86V is what can be used during system suspend (when power to the
> Cortex-A5 cores is turned off and an integrated ARC core is taking
> over for wakeup purposes).
> Hence the supported voltage range of 0.86..1.1V

That sounds like the constraints are wrong actually, if 0.86V can only
be used during system suspend then it shouldn't be in the valid voltage
range - the suspend voltage doesn't need to be in the range used during
normal operation.  If we might use it during runtime suspend then it
does need to be there though.

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

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

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmWFf+AACgkQJNaLcl1U
h9DTWgf/fUEmRlyJxxj7iLDuRYL+wIRD8ml/KWBIrzLV0ywoeLasM4HG3AgPcOlo
gk4P9Zo3xCf2smtb/Dqal7hjM6i/DvKFNN/u4aQiVZdBA9v1LL+Qtd+ZSfs4/f49
/l8mGJn5/S9FDZroHkIaVVyJMIucUua3dLfGdFA0Wpmcufd+W6TYYCuy1nx4zF+8
d++E/GpQY6O/sZdb5McjKcN/17ylJo29NgyTRRtT6lwua+jTeDzEv0iOSEENwjUl
Wws4Vg79aFs0amAtbLjuKKPQEEgQWNcpRB2Ge52JsFT/69ol9mBeAozdCTgFbP5j
Dr3W71uo+hsk9kjSnL/ZmcjRnmjCog==
=ijuO
-----END PGP SIGNATURE-----

--juKFXjvLaxOmiLZO--