[PATCH v4 0/4] LP3943 MFD driver for a GPIO expander and a PWM generator

[PATCH v4 0/4] LP3943 MFD driver for a GPIO expander and a PWM generator

[Milo Kim]

LP3943 is an integrated device capable of driving 16 output channels.
It can be used for GPIO expander and PWM generators.
LP3493 registers are controlled via the I2C interface.

This patch-set consists of four parts - MFD, GPIO, PWM and documents.

Update from v3 to v4:
  Move the driver description from the documentation to the MFD driver file.

Milo Kim (4):
  mfd: add LP3943 MFD driver
  gpio: add LP3943 I2C GPIO expander driver
  pwm: add LP3943 PWM driver
  Documentation: add LP3943 DT bindings and document

 .../devicetree/bindings/gpio/gpio-lp3943.txt       |   37 +++
 Documentation/devicetree/bindings/mfd/lp3943.txt   |   33 ++
 .../devicetree/bindings/pwm/pwm-lp3943.txt         |   58 ++++
 drivers/gpio/Kconfig                               |    8 +
 drivers/gpio/Makefile                              |    1 +
 drivers/gpio/gpio-lp3943.c                         |  242 +++++++++++++++
 drivers/mfd/Kconfig                                |   11 +
 drivers/mfd/Makefile                               |    1 +
 drivers/mfd/lp3943.c                               |  167 +++++++++++
 drivers/pwm/Kconfig                                |   10 +
 drivers/pwm/Makefile                               |    1 +
 drivers/pwm/pwm-lp3943.c                           |  314 ++++++++++++++++++++
 include/linux/mfd/lp3943.h                         |  114 +++++++
 13 files changed, 997 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/gpio/gpio-lp3943.txt
 create mode 100644 Documentation/devicetree/bindings/mfd/lp3943.txt
 create mode 100644 Documentation/devicetree/bindings/pwm/pwm-lp3943.txt
 create mode 100644 drivers/gpio/gpio-lp3943.c
 create mode 100644 drivers/mfd/lp3943.c
 create mode 100644 drivers/pwm/pwm-lp3943.c
 create mode 100644 include/linux/mfd/lp3943.h

-- 
1.7.9.5

--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH v4 0/4] LP3943 MFD driver for a GPIO expander and a PWM generator

[Milo Kim]

Hello Guys,

Could you check this patchset?
Driver documentation was moved from 'Documentation' to 
'drivers/mfd/lp3943.c'.

Thanks,
   Milo

On 09/25/2013 01:22 PM, Milo Kim wrote:
> LP3943 is an integrated device capable of driving 16 output channels.
> It can be used for GPIO expander and PWM generators.
> LP3493 registers are controlled via the I2C interface.
>
> This patch-set consists of four parts - MFD, GPIO, PWM and documents.
>
> Update from v3 to v4:
>    Move the driver description from the documentation to the MFD driver file.
>
> Milo Kim (4):
>    mfd: add LP3943 MFD driver
>    gpio: add LP3943 I2C GPIO expander driver
>    pwm: add LP3943 PWM driver
>    Documentation: add LP3943 DT bindings and document
>
>   .../devicetree/bindings/gpio/gpio-lp3943.txt       |   37 +++
>   Documentation/devicetree/bindings/mfd/lp3943.txt   |   33 ++
>   .../devicetree/bindings/pwm/pwm-lp3943.txt         |   58 ++++
>   drivers/gpio/Kconfig                               |    8 +
>   drivers/gpio/Makefile                              |    1 +
>   drivers/gpio/gpio-lp3943.c                         |  242 +++++++++++++++
>   drivers/mfd/Kconfig                                |   11 +
>   drivers/mfd/Makefile                               |    1 +
>   drivers/mfd/lp3943.c                               |  167 +++++++++++
>   drivers/pwm/Kconfig                                |   10 +
>   drivers/pwm/Makefile                               |    1 +
>   drivers/pwm/pwm-lp3943.c                           |  314 ++++++++++++++++++++
>   include/linux/mfd/lp3943.h                         |  114 +++++++
>   13 files changed, 997 insertions(+)
>   create mode 100644 Documentation/devicetree/bindings/gpio/gpio-lp3943.txt
>   create mode 100644 Documentation/devicetree/bindings/mfd/lp3943.txt
>   create mode 100644 Documentation/devicetree/bindings/pwm/pwm-lp3943.txt
>   create mode 100644 drivers/gpio/gpio-lp3943.c
>   create mode 100644 drivers/mfd/lp3943.c
>   create mode 100644 drivers/pwm/pwm-lp3943.c
>   create mode 100644 include/linux/mfd/lp3943.h
>

Re: [PATCH v4 0/4] LP3943 MFD driver for a GPIO expander and a PWM generator

[Lee Jones]

> Could you check this patchset?
> Driver documentation was moved from 'Documentation' to
> 'drivers/mfd/lp3943.c'.

Well because the reset of the patch-set wasn't sent as replies to
[PATCH 0/X], it's now lost in the ulu. You will have to resubmit.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

Re: [PATCH v4 0/4] LP3943 MFD driver for a GPIO expander and a PWM generator

[Thierry Reding]

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

On Wed, Sep 25, 2013 at 01:22:55PM +0900, Milo Kim wrote:
> LP3943 is an integrated device capable of driving 16 output channels.
> It can be used for GPIO expander and PWM generators.
> LP3493 registers are controlled via the I2C interface.
> 
> This patch-set consists of four parts - MFD, GPIO, PWM and documents.
> 
> Update from v3 to v4:
>   Move the driver description from the documentation to the MFD driver file.
> 
> Milo Kim (4):
>   mfd: add LP3943 MFD driver
>   gpio: add LP3943 I2C GPIO expander driver
>   pwm: add LP3943 PWM driver
>   Documentation: add LP3943 DT bindings and document

I suspect that given the build-time dependencies Lee may want to take
all four patches through the MFD tree, so I've acked the PWM parts.

Thierry

[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]

Re: [PATCH v4 0/4] LP3943 MFD driver for a GPIO expander and a PWM generator

[Lee Jones]

> > LP3943 is an integrated device capable of driving 16 output channels.
> > It can be used for GPIO expander and PWM generators.
> > LP3493 registers are controlled via the I2C interface.
> > 
> > This patch-set consists of four parts - MFD, GPIO, PWM and documents.
> > 
> > Update from v3 to v4:
> >   Move the driver description from the documentation to the MFD driver file.
> > 
> > Milo Kim (4):
> >   mfd: add LP3943 MFD driver
> >   gpio: add LP3943 I2C GPIO expander driver
> >   pwm: add LP3943 PWM driver
> >   Documentation: add LP3943 DT bindings and document
> 
> I suspect that given the build-time dependencies Lee may want to take
> all four patches through the MFD tree, so I've acked the PWM parts.

Thanks Thierry.

Milo, would you mind re-sending the patch-set complete with
Thierry's Acks please? Also, can you send [PATCH 1/X]..[PATCH X/X] as
replies to [PATCH 0/X], as it aids greatly with status tracking.

Please see git send-email options:
  `--[no-]thread` and `--[no-]chain-reply-to`

For shallow threading, which is what I believe you want, you need:
  `--thread --no-chain-reply-to`

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog

Re: [PATCH v4 0/4] LP3943 MFD driver for a GPIO expander and a PWM generator

[Mark Brown]

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

On Wed, Dec 04, 2013 at 11:34:32AM +0100, Thierry Reding wrote:
> On Wed, Sep 25, 2013 at 01:22:55PM +0900, Milo Kim wrote:

> >   mfd: add LP3943 MFD driver
> >   gpio: add LP3943 I2C GPIO expander driver
> >   pwm: add LP3943 PWM driver
> >   Documentation: add LP3943 DT bindings and document

> I suspect that given the build-time dependencies Lee may want to take
> all four patches through the MFD tree, so I've acked the PWM parts.

FWIW this isn't an issue with new MFDs - since the function drivers will
depend on the core driver it's not possible to enable them in Kconfig
until the core driver is also merged and you don't get bisect issues.

[-- Attachment #2: Digital signature --]
[-- Type: application/pgp-signature, Size: 836 bytes --]

Re: [PATCH v4 0/4] LP3943 MFD driver for a GPIO expander and a PWM generator

[Thierry Reding]

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

On Thu, Dec 05, 2013 at 09:43:41PM +0000, Mark Brown wrote:
> On Wed, Dec 04, 2013 at 11:34:32AM +0100, Thierry Reding wrote:
> > On Wed, Sep 25, 2013 at 01:22:55PM +0900, Milo Kim wrote:
> 
> > >   mfd: add LP3943 MFD driver
> > >   gpio: add LP3943 I2C GPIO expander driver
> > >   pwm: add LP3943 PWM driver
> > >   Documentation: add LP3943 DT bindings and document
> 
> > I suspect that given the build-time dependencies Lee may want to take
> > all four patches through the MFD tree, so I've acked the PWM parts.
> 
> FWIW this isn't an issue with new MFDs - since the function drivers will
> depend on the core driver it's not possible to enable them in Kconfig
> until the core driver is also merged and you don't get bisect issues.

Right. I hadn't thought about that. Thanks for mentioning it.

Thierry

[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]

Re: [PATCH v4 0/4] LP3943 MFD driver for a GPIO expander and a PWM generator

[Lee Jones]

On Fri, 06 Dec 2013, Thierry Reding wrote:

> On Thu, Dec 05, 2013 at 09:43:41PM +0000, Mark Brown wrote:
> > On Wed, Dec 04, 2013 at 11:34:32AM +0100, Thierry Reding wrote:
> > > On Wed, Sep 25, 2013 at 01:22:55PM +0900, Milo Kim wrote:
> > 
> > > >   mfd: add LP3943 MFD driver
> > > >   gpio: add LP3943 I2C GPIO expander driver
> > > >   pwm: add LP3943 PWM driver
> > > >   Documentation: add LP3943 DT bindings and document
> > 
> > > I suspect that given the build-time dependencies Lee may want to take
> > > all four patches through the MFD tree, so I've acked the PWM parts.
> > 
> > FWIW this isn't an issue with new MFDs - since the function drivers will
> > depend on the core driver it's not possible to enable them in Kconfig
> > until the core driver is also merged and you don't get bisect issues.
> 
> Right. I hadn't thought about that. Thanks for mentioning it.

I have applied them all with the relevant Acks in any case.

-- 
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog