Received-SPF: pass (google.com: domain of linux-kernel+bounces-183733-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
Date: Mon, 20 May 2024 15:31:02 +0300
From: Matti Vaittinen <mazziesaccount@gmail.com>
To: Matti Vaittinen <mazziesaccount@gmail.com>,
	Matti Vaittinen <matti.vaittinen@fi.rohmeurope.com>
Cc: Liam Girdwood <lgirdwood@gmail.com>, Mark Brown <broonie@kernel.org>,
	=?iso-8859-1?Q?M=E5rten?= Lindahl <marten.lindahl@axis.com>,
	linux-kernel@vger.kernel.org
Subject: [PATCH v2 0/2] regulator: fix pickable ranges vsel caching
Message-ID: <ZktChruNV_Jc4j4u@fedora>
Precedence: bulk
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
	protocol="application/pgp-signature"; boundary="MQgaS3VPhLaQvzhS"
Content-Disposition: inline


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

This series addresses a potential problem where the voltage selector for
pickable-ranges is used in same fashion as a 'apply-bit'. Eg, when a
regulator voltage is changed so that also the range changes, the effect
of the range and voltage selector change is not taken into account until
the voltage selector is written to hardware. In such case caching the
voltage-selector value will cause the voltage range change to not change
the actual output voltage.

Patch 1 addresses this issue by adding a flag to regulator desc to
advertise this behaviour and by always writing the VSEL register value to
hardware if range was changed.

Patch 2 sets that flag to Texsas instrument's tps6287x driver because,
according to the data-sheet, the TI's PMICs VSEL register must be always
written after the range has been changed. Please note that this has not
been tested in TI's hardware but the concept is only quickly verified on
ROHM's BD71851. All testing is appreciated.

Revision history:
- v2:
	- Change series name to reflect content that is still not
	  merged.
	- Drop already applied patches.
	- Add patch fixing TI's tps2687x.
	- Don't bypass cache for pickable-ranges users who don't have.
	  the 'VSEL as apply' behaviour.
	- Bypass cache only if range was changed.
- v1:
	https://lore.kernel.org/all/cover.1715848512.git.mazziesaccount@gmail.com/

series is base on regulator/for-next

---

Matti Vaittinen (2):
  regulator: pickable ranges: don't always cache vsel
  regulator: tps6287x: Force writing VSEL bit

 drivers/regulator/helpers.c            | 43 ++++++++++++++++++--------
 drivers/regulator/tps6287x-regulator.c |  1 +
 include/linux/regulator/driver.h       |  3 ++
 3 files changed, 34 insertions(+), 13 deletions(-)

--=20
2.45.1


--=20
Matti Vaittinen, Linux device drivers
ROHM Semiconductors, Finland SWDC
Kiviharjunlenkki 1E
90220 OULU
FINLAND

~~~ "I don't think so," said Rene Descartes. Just then he vanished ~~~
Simon says - in Latin please.
~~~ "non cogito me" dixit Rene Descarte, deinde evanescavit ~~~
Thanks to Simon Glass for the translation =3D]=20

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

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

iQEzBAEBCAAdFiEEIx+f8wZb28fLKEhTeFA3/03aocUFAmZLQn0ACgkQeFA3/03a
ocVXxAf/eNpKlT3d/ANnNoL1BEmddIi/7e8g3WJrS5H1/1Z5BC+cqZVXPfugKgXy
YZeJo7Pb8+hBafaQSVsZdzHWBgEu//SGabvPUbfhYP6JzlcX9UNSkrNG/EKYX3CZ
GGoUnsxwh0n3lGOf6lc5cwk4ZybqwYHji4BBNVmmGmsgzcjVrX58712XbXdVthNy
w1K9MTJ4elPrgaBrf1cJAbJg+7u7FA9zBxWm8f0SH90Jx65fix63zAvi3chfcqvE
F6QVAGJ2LDWljB4ggXkHsUgt/c2sV7yh5zFdGKlW+6WJu5UC6rf/kV46wL6ORFuS
LWbVDp2LlyVEVeBAG6QswHuaTsPxEA==
=3tAL
-----END PGP SIGNATURE-----

--MQgaS3VPhLaQvzhS--