Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.34 as permitted sender) client-ip=23.128.96.34;
Date:   Wed, 18 Oct 2023 16:07:57 +0100
From:   Mark Brown <broonie@kernel.org>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Cc:     Srinivas Kandagatla <srinivas.kandagatla@linaro.org>,
        Banajit Goswami <bgoswami@quicinc.com>,
        Liam Girdwood <lgirdwood@gmail.com>,
        Jaroslav Kysela <perex@perex.cz>,
        Takashi Iwai <tiwai@suse.com>, alsa-devel@alsa-project.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] ASoC: codecs: wsa884x: allow sharing reset GPIO
Message-ID: <c17f56c1-a629-4b05-b807-42010f206f6c@sirena.org.uk>
References: <20231018100055.140847-1-krzysztof.kozlowski@linaro.org>
 <3aa9e3a7-9417-44b4-87d8-fcf1a8b46daf@sirena.org.uk>
 <84f9f1c4-0627-4986-8160-b4ab99469b81@linaro.org>
 <b7aeda24-d638-45b7-8e30-80d287f498f8@sirena.org.uk>
 <b35a21a2-6e69-4033-8d51-2b67b08ac7b3@linaro.org>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="0DVAadVcjJVHZ/F3"
Content-Disposition: inline
In-Reply-To: <b35a21a2-6e69-4033-8d51-2b67b08ac7b3@linaro.org>
Precedence: bulk


--0DVAadVcjJVHZ/F3
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Wed, Oct 18, 2023 at 02:57:59PM +0200, Krzysztof Kozlowski wrote:
> On 18/10/2023 14:56, Mark Brown wrote:

> > I'd expect that the GPIO users should coordiante directly rather than
> > rely on the GPIO API to do the coordination for them - there aren't
> > enough semantics in the GPIO itself to do much more except possibly
> > provide discovery services (which would be nice).  Look at how the
> > regulator API manages multiple regulators sharing an enable GPIO for
> > example, it adds an additional layer of reference counting when it
> > identifies a shared GPIO.

> OK, it is still regulator core, though. Not individual drivers problem.

> Several other existing drivers have the same issue, so this should be
> solved in a generic or shared way.

Indeed.

--0DVAadVcjJVHZ/F3
Content-Type: application/pgp-signature; name="signature.asc"

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

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmUv9MwACgkQJNaLcl1U
h9Bv+Qf/Rvb5zyWFaHHrWVDS/7ye9dIL/po1Jvc9RP6d4AYn6QD0oyu/NbUNIHFm
LeB/Xwa73t6cIk7NqGMfpiGr494gsJIZkIYFXA2sRbIViBed9lfXkPIp/lbosz5P
sPjpR/gzT3+PqyyaRdEzXfmc2Ninu10PEvR1US8HUfNntlkGOjgnJLp8mXM0whUm
U+Fbl5W2+2TrIstqes8bJRzBSzpZgM5EqkhXUapqmr0EBVHOn7OjfB1e2/DqA+PF
k3ZedVR9V7xi7S3nhLurr5zS4E8OYUebnt61PfdxqsZjqlQNjEQrQfMtwALjarOg
yOL/dyC9UX7+UUWQlxh5j1GOYjBCdA==
=sw3k
-----END PGP SIGNATURE-----

--0DVAadVcjJVHZ/F3--