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

2012-05-22 Thread 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.


signature.asc
Description: Digital signature


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

2012-05-21 Thread 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.bra...@gmail.com  wrote:
> From: Yadwinder Singh Brar 
>
> 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 majord...@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


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

2012-05-21 Thread yadi . brar01
From: Yadwinder Singh Brar 

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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html