[PATCH v3 0/2] regulator: Add initial suport for max77686

[PATCH v3 0/2] regulator: Add initial suport for max77686

[yadi.brar01 at gmail.com]

From: Yadwinder Singh Brar <yadi.brar@samsung.com>

This patch series adds support for max77686 which is a multifunction device
which includes regulator (pmic), rtc and charger sub-blocks within it. The
support for mfd driver and regulator driver are added by this patch series. This
patch series also includes device tree and irqdomain support for mfd and
regulator portions.

Implemented the required modification, stated in the recieved review comments.
changes since v1:
-added regmap support.
-implemented .get_voltage_sel, .set_voltage_sel and .set_voltage_time_sel after
 removing .get_voltage and .set_voltage in regulator driver.
-used of_regulator_match() for parsing DT.
-added Documentation for Devive Tree binding.

changes since v2:
-converted to use regulator_get_voltage_sel_regmap,
 regulator_set_voltage_sel_regmap, regulator_enable_regmap,
 regulator_disable_regmap, regulator_is_enabled_regmap.

This patch series is based on mark_regulator/for-next and has been tested on
GAIA board.

Yadwinder Singh Brar (2):
  mfd: Add support for MAX77686.
  regulator: Add support for MAX77686.

 Documentation/devicetree/bindings/mfd/max77686.txt |   61 +++
 drivers/mfd/Kconfig                                |   21 +
 drivers/mfd/Makefile                               |    1 +
 drivers/mfd/max77686-irq.c                         |  255 +++++++++++++
 drivers/mfd/max77686.c                             |  322 ++++++++++++++++
 drivers/regulator/Kconfig                          |    9 +
 drivers/regulator/Makefile                         |    1 +
 drivers/regulator/max77686.c                       |  389 ++++++++++++++++++++
 include/linux/mfd/max77686-private.h               |  282 ++++++++++++++
 include/linux/mfd/max77686.h                       |  100 +++++
 10 files changed, 1441 insertions(+), 0 deletions(-)
 create mode 100644 Documentation/devicetree/bindings/mfd/max77686.txt
 create mode 100644 drivers/mfd/max77686-irq.c
 create mode 100644 drivers/mfd/max77686.c
 create mode 100644 drivers/regulator/max77686.c
 create mode 100644 include/linux/mfd/max77686-private.h
 create mode 100644 include/linux/mfd/max77686.h

[PATCH v3 0/2] regulator: Add initial suport for max77686

[Kyungmin Park]

Hi Mark,

BTW, do you know that you're reviewing the same device driver patch
from different person?
One from Mr. Lee and another from Yadwinder.

I wonder how to handle it finally. which one is choose?

Thank you,
Kyungmin Park

On 5/22/12, yadi.brar01 at gmail.com <yadi.brar01@gmail.com> wrote:
> From: Yadwinder Singh Brar <yadi.brar@samsung.com>
>
> This patch series adds support for max77686 which is a multifunction device
> which includes regulator (pmic), rtc and charger sub-blocks within it. The
> support for mfd driver and regulator driver are added by this patch series.
> This
> patch series also includes device tree and irqdomain support for mfd and
> regulator portions.
>
> Implemented the required modification, stated in the recieved review
> comments.
> changes since v1:
> -added regmap support.
> -implemented .get_voltage_sel, .set_voltage_sel and .set_voltage_time_sel
> after
>  removing .get_voltage and .set_voltage in regulator driver.
> -used of_regulator_match() for parsing DT.
> -added Documentation for Devive Tree binding.
>
> changes since v2:
> -converted to use regulator_get_voltage_sel_regmap,
>  regulator_set_voltage_sel_regmap, regulator_enable_regmap,
>  regulator_disable_regmap, regulator_is_enabled_regmap.
>
> This patch series is based on mark_regulator/for-next and has been tested
> on
> GAIA board.
>
> Yadwinder Singh Brar (2):
>   mfd: Add support for MAX77686.
>   regulator: Add support for MAX77686.
>
>  Documentation/devicetree/bindings/mfd/max77686.txt |   61 +++
>  drivers/mfd/Kconfig                                |   21 +
>  drivers/mfd/Makefile                               |    1 +
>  drivers/mfd/max77686-irq.c                         |  255 +++++++++++++
>  drivers/mfd/max77686.c                             |  322 ++++++++++++++++
>  drivers/regulator/Kconfig                          |    9 +
>  drivers/regulator/Makefile                         |    1 +
>  drivers/regulator/max77686.c                       |  389
> ++++++++++++++++++++
>  include/linux/mfd/max77686-private.h               |  282 ++++++++++++++
>  include/linux/mfd/max77686.h                       |  100 +++++
>  10 files changed, 1441 insertions(+), 0 deletions(-)
>  create mode 100644 Documentation/devicetree/bindings/mfd/max77686.txt
>  create mode 100644 drivers/mfd/max77686-irq.c
>  create mode 100644 drivers/mfd/max77686.c
>  create mode 100644 drivers/regulator/max77686.c
>  create mode 100644 include/linux/mfd/max77686-private.h
>  create mode 100644 include/linux/mfd/max77686.h
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc"
> in
> the body of a message to majordomo at vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>

[PATCH v3 0/2] regulator: Add initial suport for max77686

[Mark Brown]

On Tue, May 22, 2012 at 03:53:11PM +0900, Kyungmin Park wrote:

> BTW, do you know that you're reviewing the same device driver patch
> from different person?
> One from Mr. Lee and another from Yadwinder.

> I wonder how to handle it finally. which one is choose?

Yes, I realise there's two different versions of it floating around -
personally I'd hope that you guys could come to some agreement about a
common set of code that works for everyone.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20120522/51e128cc/attachment.sig>