mbox series

[V2,0/3] slg51000: regulator driver submission

Message ID cover.1555567784.git.eric.jeong.opensource@diasemi.com
Headers show
Series slg51000: regulator driver submission | expand

Message

Eric Jeong April 18, 2019, 6:09 a.m. UTC
This patch adds support for the Dialog SLG51000 regulator device.

In this patch set the following is provided:

[PATCH V2 1/3] MAINTAINERS file update for SLG51000
[PATCH V2 2/3] SLG51000 DT binding
[PATCH V2 3/3] SLG51000 regulator driver

This patch applies against linux-next and next-20190417 

Thank you,
Eric Jeong, Dialog Semiconductor Ltd.

Eric Jeong (3):
  MAINTAINERS: slg51000 updates to the Dialog Semiconductor search
    terms
  dt-bindings: regulator: add document bindings for slg51000
  regulator: slg51000: add slg51000 regulator driver

 .../devicetree/bindings/regulator/slg51000.txt     |   88 ++++
 MAINTAINERS                                        |    2 +
 drivers/regulator/Kconfig                          |    9 +
 drivers/regulator/Makefile                         |    1 +
 drivers/regulator/slg51000-regulator.c             |  528 ++++++++++++++++++++
 drivers/regulator/slg51000-regulator.h             |  505 +++++++++++++++++++
 6 files changed, 1133 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/regulator/slg51000.txt
 create mode 100644 drivers/regulator/slg51000-regulator.c
 create mode 100644 drivers/regulator/slg51000-regulator.h

Comments

Mark Brown April 18, 2019, 10:16 a.m. UTC | #1
On Thu, Apr 18, 2019 at 03:09:44PM +0900, Eric Jeong wrote:

> I updated regulator_ops structure to use use the helper function 
> which is regulator_is_enabled_regmap() instead reading status bits
> for is_enabled() operation.

> v2: use regulator helper function for is_enabled()

There's also the get_status() operation which you should be able to
implement using the status bits you were reading before - that lets
the framework know if the regualtor is actually working which can help
with error handling so it's good to do.