Date: Tue, 29 Nov 2016 11:53:06 +0000
From: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>
To: Mark Brown <broonie@kernel.org>
CC: <lgirdwood@gmail.com>, <linux-kernel@vger.kernel.org>,
        <patches@opensource.wolfsonmicro.com>
Subject: Re: [PATCH 2/2] regulator: core: Allow enable GPIO to be specified
 using GPIOD
Message-ID: <20161129115306.GI1867@localhost.localdomain>
References: <1480419045-2275-1-git-send-email-ckeepax@opensource.wolfsonmicro.com>
 <1480419045-2275-2-git-send-email-ckeepax@opensource.wolfsonmicro.com>
 <20161129114547.erdaezyfvzzdtjuk@sirena.org.uk>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Disposition: inline
In-Reply-To: <20161129114547.erdaezyfvzzdtjuk@sirena.org.uk>
User-Agent: Mutt/1.5.23 (2014-03-12)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 846
Lines: 18

On Tue, Nov 29, 2016 at 11:45:47AM +0000, Mark Brown wrote:
> On Tue, Nov 29, 2016 at 11:30:45AM +0000, Charles Keepax wrote:
> > The regulator subsystem has used GPIOs internally for a while, however,
> > end drivers must still specify their enable GPIO using a GPIO number. This
> > patch allows the end drivers to specify the enable GPIO using GPIOD
> > directly.
> 
> The reason we don't support descriptors is that the GPIO API currently
> doesn't support having more than one user of a descriptor but GPIO
> enables for regulators get shared often enough to be interesting.  If
> drivers start using this then that'd mean that the shared enable support
> would become erratic depending on which driver was in use.

Ah ok thanks, guess we can drop this one then. The first patch in
the chain is hopefully still good though.

Thanks,
Charles