Re: [PATCH v2 4/4] lp8788-ldo: use ena_pin of regulator-core for external control

2013-01-26 Thread Mark Brown 
On Tue, Jan 15, 2013 at 04:35:53AM +, Kim, Milo wrote:
>  Regulator core driver provides enable GPIO control for enabling/disabling a
>  regulator. Now, enable GPIO is shared among regulators.
>  Use this internal working, so unnecessary code are removed.
>  GPIO enable pin configurations are added in digital LDO and analog LDO 
> drivers.

Looks good.


signature.asc
Description: Digital signature

Re: [PATCH v2 4/4] lp8788-ldo: use ena_pin of regulator-core for external control

2013-01-26 Thread Mark Brown 
On Tue, Jan 15, 2013 at 04:35:53AM +, Kim, Milo wrote:
  Regulator core driver provides enable GPIO control for enabling/disabling a
  regulator. Now, enable GPIO is shared among regulators.
  Use this internal working, so unnecessary code are removed.
  GPIO enable pin configurations are added in digital LDO and analog LDO 
 drivers.

Looks good.


signature.asc
Description: Digital signature