Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Wed, 24 May 2023 10:48:40 +0100
From:   Mark Brown <broonie@kernel.org>
To:     Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
Cc:     perex@perex.cz, tiwai@suse.com, lgirdwood@gmail.com,
        ckeepax@opensource.cirrus.com, kuninori.morimoto.gx@renesas.com,
        linux-kernel@vger.kernel.org, pierre-louis.bossart@linux.intel.com,
        alsa-devel@alsa-project.org
Subject: Re: [PATCH 1/2] ASoC: codecs: wsa883x: do not set can_multi_write
 flag
Message-ID: <c24b2f1c-3b08-492f-954c-c4f10ae5ad81@sirena.org.uk>
References: <20230523154605.4284-1-srinivas.kandagatla@linaro.org>
 <00283665-e44f-457b-b2c9-1acf59d1cbd8@sirena.org.uk>
 <c37b88ae-7f54-3c07-666f-010a5fd84bd1@linaro.org>
 <d94e734c-e816-4b3f-9fb3-a6589063c05a@sirena.org.uk>
 <af7b7d4f-d7ab-b5ef-e639-9a8b13de54ee@linaro.org>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="Z25BQUiQltUZUpg7"
Content-Disposition: inline
In-Reply-To: <af7b7d4f-d7ab-b5ef-e639-9a8b13de54ee@linaro.org>
Precedence: bulk


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

On Wed, May 24, 2023 at 10:42:21AM +0100, Srinivas Kandagatla wrote:
> On 24/05/2023 09:57, Mark Brown wrote:

> > I'm unclear, is this a limitation of the hardware or of the current
> > Soundwire code?

> Its both.

> Codec itself does not have any private write callback to support this and
> AFAIU Qualcomm Soundwire controller does not have any such hw facility to
> program multi-registers for device at one shot.

How about the *CODEC* hardware?

> Is write callback used for both Bulk writes and multi-writes?

Only multi-write at this point but we probably should consider redoing
bulk writes to use it as well.

> Is multi-write feature of regmap bus or device?

Well, I don't think any buses that understand registers have implemented
it yet but there's nothing fundamental that means that a bus couldn't
usefully do something with multi-write.  The current users all use raw
buses that don't know anything about registers or values.

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

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

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmRt3XcACgkQJNaLcl1U
h9CdRgf+Nhy3duqWiV/9INno44wrOWxhqUR8tBBmnKtwdQwe1czT+kEXwFf8iJ4k
Yi6z8w3tWfDOXyW7cxhrgWqraV9oE7XMyhNK0NNx9qWR4gDWItAmxzriB67iIJnd
3wyVoz61coztitpVVkdEgI8mFqqeKluRkDxtYrWDPRSHguQtt3e+Pw+bTY8QvtvW
GDbPdrLrUSIl7ZUn/J6Rfbs6C3iuQ+6Mh8udyNIq8angXzBVT0Dyn9pKb5hjPusf
RN7PlaPjYDG00J8cg9QPhkfY9RWJs1S3JlaxBJ9lkTQx5gffEe4rPhBNkqXpAOVH
/40RVfp952NEWhOjy/LlgCwEZW68hA==
=9Wc1
-----END PGP SIGNATURE-----

--Z25BQUiQltUZUpg7--