Re: [PATCH RFT] regulator: lp8788-ldo: Use ldo->en_pin to check ifregulator is enabled by external pin

From: Mark Brown
Date: Wed Jan 09 2013 - 10:40:01 EST


On Wed, Jan 09, 2013 at 02:34:40AM +0000, Kim, Milo wrote:

> - goto wash;
> +
> + if (ret == -EBUSY && config->is_gpio_shared)
> + shared_ena_pin = true;
> +
> + if (!shared_ena_pin)
> + goto wash;

It'd be much nicer if we were able to just keep a list of GPIOs
controlling regulators as we see them and automatically work out if
they're shared. Probably we should just refactor the code so that the
GPIO object is a separate thing rather than just storing the state
directly in the regulator struct.

Attachment: signature.asc
Description: Digital signature