]> git.baikalelectronics.ru Git - kernel.git/commit
regulator: lp87565: enable voltage regardless of ENx pin
authorLuca Ceresoli <luca@lucaceresoli.net>
Mon, 22 Jun 2020 20:43:26 +0000 (22:43 +0200)
committerMark Brown <broonie@kernel.org>
Wed, 1 Jul 2020 20:53:02 +0000 (21:53 +0100)
commit28e31d61722e9fa43980a8d5494d8b3040705bdc
tree150b362bdfca6e54e841fafb0a98354d7ad5a027
parent073c5a2943d6c0625c46cd3522b8c73fe21f925b
regulator: lp87565: enable voltage regardless of ENx pin

This driver enables outputs by setting bit EN_BUCKn in the BUCKn_CTRL1
register. However, if bit EN_PIN_CTRLn in the same register is set, the
output is actually enabled only if EN_BUCKn is set AND an enable pin is
active. Since the driver does not touch EN_PIN_CTRLn, the choice is left to
the hardware, which in turn gets this bit from OTP memory, and in absence
of OTP data it uses a default value that is documented in the datasheet for
LP8752x, but not for LP8756x.

Thus the driver doesn't really "know" whether it is actually enabling the
output or not.

In order to make sure activation is always driver-controlled, just clear
the EN_PIN_CTRLn bit. Now all activation solely depend on the EN_BUCKn bit.

Signed-off-by: Luca Ceresoli <luca@lucaceresoli.net>
Link: https://lore.kernel.org/r/20200622204329.11147-2-luca@lucaceresoli.net
Signed-off-by: Mark Brown <broonie@kernel.org>
drivers/regulator/lp87565-regulator.c