[PATCH v7 0/3] mfd: lp873x: Add lp873x PMIC support

[PATCH v7 0/3] mfd: lp873x: Add lp873x PMIC support

[Keerthy]

The LP873X chip is a power management IC for Portable Navigation Systems
    and Tablet Computing devices. It contains the following components:

     - Regulators.
     - Configurable General Purpose Output Signals(GPO).

PMIC interacts with the main processor through i2c. PMIC has
couple of LDOs(Linear Regulators), couple of BUCKs (Step-Down DC-DC
Converter Cores) and GPOs(General Purpose Output Signals).

The regulator patch is already queued:
http://marc.info/?l=linux-kernel&m=146298767110771&w=2

DT patch also queued:

Hence posting the remaining patches of the series with
the comments fixed on mfd driver.

Chages in v7:

Reverted the dependency on http://www.gossamer-threads.com/lists/linux/kernel/2457552.

Changes in v6:

Series depends on: http://www.gossamer-threads.com/lists/linux/kernel/2457552
probe_new used.

Chnages in v4:

Added GPO driver support. 

Keerthy (3):
  mfd: lp873x: Add lp873x PMIC support
  gpio: lp873x: Add support for General Purpose Outputs
  regulator: lp873x: Change the MFD config option as per latest naming

 drivers/gpio/Kconfig       |  10 ++
 drivers/gpio/Makefile      |   1 +
 drivers/gpio/gpio-lp873x.c | 185 +++++++++++++++++++++++++++++++
 drivers/mfd/Kconfig        |  14 +++
 drivers/mfd/Makefile       |   2 +
 drivers/mfd/lp873x.c       |  96 +++++++++++++++++
 drivers/regulator/Kconfig  |   2 +-
 include/linux/mfd/lp873x.h | 264 +++++++++++++++++++++++++++++++++++++++++++++
 8 files changed, 573 insertions(+), 1 deletion(-)
 create mode 100644 drivers/gpio/gpio-lp873x.c
 create mode 100644 drivers/mfd/lp873x.c
 create mode 100644 include/linux/mfd/lp873x.h

-- 
1.9.1

[PATCH v7 3/3] regulator: lp873x: Change the MFD config option as per latest naming

[Keerthy]

Change the MFD config option as per latest naming

Signed-off-by: Keerthy <j-keerthy@ti.com>
---
 drivers/regulator/Kconfig | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/regulator/Kconfig b/drivers/regulator/Kconfig
index 6c88e31..97dc4cc 100644
--- a/drivers/regulator/Kconfig
+++ b/drivers/regulator/Kconfig
@@ -323,7 +323,7 @@ config REGULATOR_LP872X
 
 config REGULATOR_LP873X
 	tristate "TI LP873X Power regulators"
-	depends on MFD_LP873X && OF
+	depends on MFD_TI_LP873X && OF
 	help
 	  This driver supports LP873X voltage regulator chips. LP873X
 	  provides two step-down converters and two general-purpose LDO
-- 
1.9.1

Re: [PATCH v7 3/3] regulator: lp873x: Change the MFD config option as per latest naming

[Mark Brown]

[-- Attachment #1: Type: text/plain, Size: 152 bytes --]

On Wed, Aug 10, 2016 at 03:48:11PM +0530, Keerthy wrote:
> Change the MFD config option as per latest naming

Acked-by: Mark Brown <broonie@kernel.org>

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

Re: [PATCH v7 0/3] mfd: lp873x: Add lp873x PMIC support

[Keerthy]

Hi Lee Jones,

On Wednesday 10 August 2016 03:48 PM, Keerthy wrote:
> The LP873X chip is a power management IC for Portable Navigation Systems
>      and Tablet Computing devices. It contains the following components:
>
>       - Regulators.
>       - Configurable General Purpose Output Signals(GPO).
>
> PMIC interacts with the main processor through i2c. PMIC has
> couple of LDOs(Linear Regulators), couple of BUCKs (Step-Down DC-DC
> Converter Cores) and GPOs(General Purpose Output Signals).
>
> The regulator patch is already queued:
> http://marc.info/?l=linux-kernel&m=146298767110771&w=2
>

Seems like patch 1 of the series titled "mfd: lp873x: Add lp873x PMIC 
support" never made it to the list for some reason i got a failure 
notice only on that patch and not the entire set x :-(.

I will resend v7 series with Linus's comments addressed on the GPIO 
driver and add Mark's ack on the regulator patch.

Thanks,
Keerthy

> DT patch also queued:
>
> Hence posting the remaining patches of the series with
> the comments fixed on mfd driver.
>
> Chages in v7:
>
> Reverted the dependency on http://www.gossamer-threads.com/lists/linux/kernel/2457552.
>
> Changes in v6:
>
> Series depends on: http://www.gossamer-threads.com/lists/linux/kernel/2457552
> probe_new used.
>
> Chnages in v4:
>
> Added GPO driver support.
>
> Keerthy (3):
>    mfd: lp873x: Add lp873x PMIC support
>    gpio: lp873x: Add support for General Purpose Outputs
>    regulator: lp873x: Change the MFD config option as per latest naming
>
>   drivers/gpio/Kconfig       |  10 ++
>   drivers/gpio/Makefile      |   1 +
>   drivers/gpio/gpio-lp873x.c | 185 +++++++++++++++++++++++++++++++
>   drivers/mfd/Kconfig        |  14 +++
>   drivers/mfd/Makefile       |   2 +
>   drivers/mfd/lp873x.c       |  96 +++++++++++++++++
>   drivers/regulator/Kconfig  |   2 +-
>   include/linux/mfd/lp873x.h | 264 +++++++++++++++++++++++++++++++++++++++++++++
>   8 files changed, 573 insertions(+), 1 deletion(-)
>   create mode 100644 drivers/gpio/gpio-lp873x.c
>   create mode 100644 drivers/mfd/lp873x.c
>   create mode 100644 include/linux/mfd/lp873x.h
>

Re: [PATCH v7 0/3] mfd: lp873x: Add lp873x PMIC support

[Keerthy]

Lee Jones,

On Wednesday 10 August 2016 03:48 PM, Keerthy wrote:
> The LP873X chip is a power management IC for Portable Navigation Systems
>     and Tablet Computing devices. It contains the following components:
>
>      - Regulators.
>      - Configurable General Purpose Output Signals(GPO).
>
> PMIC interacts with the main processor through i2c. PMIC has
> couple of LDOs(Linear Regulators), couple of BUCKs (Step-Down DC-DC
> Converter Cores) and GPOs(General Purpose Output Signals).
>

Kindly pull this set if there are no further comments.

> The regulator patch is already queued:
> http://marc.info/?l=linux-kernel&m=146298767110771&w=2
>
> DT patch also queued:
>
> Hence posting the remaining patches of the series with
> the comments fixed on mfd driver.
>
> Chages in v7:
>
> Reverted the dependency on http://www.gossamer-threads.com/lists/linux/kernel/2457552.
>
> Changes in v6:
>
> Series depends on: http://www.gossamer-threads.com/lists/linux/kernel/2457552
> probe_new used.
>
> Chnages in v4:
>
> Added GPO driver support.
>
> Keerthy (3):
>   mfd: lp873x: Add lp873x PMIC support
>   gpio: lp873x: Add support for General Purpose Outputs
>   regulator: lp873x: Change the MFD config option as per latest naming
>
>  drivers/gpio/Kconfig       |  10 ++
>  drivers/gpio/Makefile      |   1 +
>  drivers/gpio/gpio-lp873x.c | 185 +++++++++++++++++++++++++++++++
>  drivers/mfd/Kconfig        |  14 +++
>  drivers/mfd/Makefile       |   2 +
>  drivers/mfd/lp873x.c       |  96 +++++++++++++++++
>  drivers/regulator/Kconfig  |   2 +-
>  include/linux/mfd/lp873x.h | 264 +++++++++++++++++++++++++++++++++++++++++++++
>  8 files changed, 573 insertions(+), 1 deletion(-)
>  create mode 100644 drivers/gpio/gpio-lp873x.c
>  create mode 100644 drivers/mfd/lp873x.c
>  create mode 100644 include/linux/mfd/lp873x.h
>