Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) client-ip=23.128.96.33;
Date:   Wed, 18 Oct 2023 13:56:11 +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: <b7aeda24-d638-45b7-8e30-80d287f498f8@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>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="SeEMerEnyAsMxP6P"
Content-Disposition: inline
In-Reply-To: <84f9f1c4-0627-4986-8160-b4ab99469b81@linaro.org>
Precedence: bulk


--SeEMerEnyAsMxP6P
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Wed, Oct 18, 2023 at 02:38:00PM +0200, Krzysztof Kozlowski wrote:
> On 18/10/2023 14:35, Mark Brown wrote:

> > How do the speakers coordinate?

> They don't and that's the generic problem of many Linux drivers. Not
> only this one, but others as well.

> Device unbind (remove()) or runtime suspend of one speaker will affect
> other speaker. I don't think any other drivers solved this, because this
> is rather core's GPIO issue, thus I am not solving it here either. :(

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.

--SeEMerEnyAsMxP6P
Content-Type: application/pgp-signature; name="signature.asc"

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

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmUv1eoACgkQJNaLcl1U
h9Arjwf+MY/q21wh5tBoMbw8vn1r4Ag4O1t2M+4gxxtLbqzpaWKPc99mHcw04BMC
1ddMktsfr9bnWH0PBXYhrKahMdfDlXrotllyDtWS0i9hQUOhrgFKj0r4192NDc4q
do9drmxc8ikNhIP4AWqQTj0uLXEwRuHqj4gXnak6M0ScAi9cLYo0BvIBi/g8f+jY
U609DqfVDLVbntcAVB+b6j087AwW1drZ4cj2XV7TppJtUE1IWku3rlb95zm2jC0B
SwQI49ZkObNk2YiErX33Q9qmIitZbtgLvBR3DHsZHlBaswBT/IApJRnu1VOJsHX4
/56/6Ela2XeuYcFE3rPB0ExOstsO+g==
=lF0c
-----END PGP SIGNATURE-----

--SeEMerEnyAsMxP6P--