Re: [RFC 7/7] MFD: TWL6040: Add regulator support for VIO, V2V1supplies

From: Mark Brown
Date: Thu Feb 02 2012 - 08:32:09 EST


On Thu, Feb 02, 2012 at 03:18:09PM +0200, Peter Ujfalusi wrote:
> On 02/02/2012 02:52 PM, Mark Brown wrote:

> > Looks like you want regulator_bulk_get() here. Or (better yet though
> > it'd be a potential issue for merge via MFD and the benefits aren't that
> > exciting since you still need to disable) devm_regulator_bulk_get().

> I need separate control for the two power source since we can hit
> different power levels depending on which is powered/not powered:
> power down: vio, v2v1 is OFF
> deep sleep: vio is ON, v2v1 is OFF
> sleep/power on: vio, v2v1 is ON

> And we have certain sequence to move between power states.

That's not a problem for using the bulk get - the array is part of the
API so you can use regulator_bulk_get() and still look at individual
supplies within the array later on when enabling and disabling them.

> >> + ret = regulator_enable(twl6040->vio);
> >> + if (ret != 0) {
> >> + dev_err(&client->dev, "Failed to enable VIO: %d\n", ret);
> >> + goto power_err;
> >> + }
> >> + ret = regulator_enable(twl6040->v2v1);
> >> + if (ret != 0) {
> >> + dev_err(&client->dev, "Failed to enable V2V1: %d\n", ret);
> >> + regulator_disable(twl6040->vio);

> I disable the vio here, if the v2v1 enable fails.

Oh, that's quite confusing when mixed in with the goto/unwind - it'd be
clearer to have the extra lable to jump to.

Attachment: signature.asc
Description: Digital signature