Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Fri, 23 Nov 2018 13:25:22 +0000
From:   Mark Brown <broonie@kernel.org>
To:     Charles Keepax <ckeepax@opensource.cirrus.com>
Cc:     Linus Walleij <linus.walleij@linaro.org>,
        Liam Girdwood <lgirdwood@gmail.com>,
        Marek Szyprowski <m.szyprowski@samsung.com>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        patches@opensource.cirrus.com
Subject: Re: [PATCH 3/3] gpio: Add reference counting for non-exclusive GPIOs
Message-ID: <20181123132522.GE2089@sirena.org.uk>
References: <20181122173015.23905-1-ckeepax@opensource.cirrus.com>
 <20181122173015.23905-3-ckeepax@opensource.cirrus.com>
 <CACRpkdZEu44WHScd5VibYT5C9RikqcvKsgpVOpkkdrYbLaV8Ng@mail.gmail.com>
 <20181123105729.GM16508@imbe.wolfsonmicro.main>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="Z79+P6LHXcBZ81w+"
Content-Disposition: inline
In-Reply-To: <20181123105729.GM16508@imbe.wolfsonmicro.main>
User-Agent: Mutt/1.10.1 (2018-07-13)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


--Z79+P6LHXcBZ81w+
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Fri, Nov 23, 2018 at 10:57:29AM +0000, Charles Keepax wrote:

> It's fixing something in the case of two regulators using the
> same GPIO. The direction this patch chain takes is that the end
> drivers own the GPIOs not the regulator core (except for the
> legacy case). So both of the end drivers will devm_ request their

The situation with descriptor based GPIOs is a bug - the reason why the
core does the request in the legacy case is precisely the problem Linus
identified.  Once a GPIO is shared all the users of the GPIO need to
coordinate with each other in order to set the value so the refcount by
itself in the GPIO core isn't super useful unless it can also serve to
help the multiple users find each other somehow.  I think what we want
to do here is either push the gpiod requests into the regulator core or
change things so that once the regulator is registered with the
regulator core the regulator core owns and is responsible for freeing
the regulator.

--Z79+P6LHXcBZ81w+
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAlv3/8EACgkQJNaLcl1U
h9CngAf/VsSmqhxP8B88PyMCu9jvoNGZV92/zflmxH90MoJCULb/hYaovGGWRr8i
ywXaT2Z2v60U9nb2gIAsXV0OcuzCSPKz+546I47AKdP9L6Ib168Y90qKLEe+GN0r
HR18HPVfzd30RDPykJfFpLh7CZVPuPQcZOxTOQup2TRzYTFU6iEAwjMBHwd0IckW
bk+DFxAmeF9a+uDeb3BfIWlBN/qRDarUW/H8ihBgj33RSxmtg24WuMO6v1TvH3Do
qom5VoHbPuPtk75wdvbetJQ1/RZAmJ/mNf6HVPGkDPbH9Qel50wk1XYuYCtbDLZN
j7/UA2NddoyPWsVgGxnKEXklYVswMA==
=SGoR
-----END PGP SIGNATURE-----

--Z79+P6LHXcBZ81w+--