Re: [PATCH QUESTION] ASoC: qcom: sdm845: use DSP_A format for TDM codec DAIs

Mark Brown <[email protected]>
Newsgroups org.kernel.vger.phone-devel,org.kernel.vger.linux-arm-msm,org.kernel.vger.linux-kernel,org.kernel.vger.linux-sound
Message-ID <[email protected]>
On Sat, Jun 13, 2026 at 09:55:59PM +0200, David Heidelberg via B4 Relay wrote:

> Currently this worked only because the cs35l36
> codec mapped both DSP_A and DSP_B to the same hardware register value
> (asp_fmt = 0), which is inherently DSP_A timing.

> The CPU-side AFE is configured with qcom,tdm-data-delay = <1> which
> produces DSP_A framing.
> The codec format should match what is actually on the wire.

> So I'm pretty lost if I should go fixing cs35l36 or sdm845.c.

That sounds like both.  The Cirrus driver is definitely buggy if it's
mapping DSP A and B to the same register value, at least one of those is
wrong.
signature.asc (application/pgp-signature, 488 B)
-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmot7WYACgkQJNaLcl1U
h9AUkAf/ZFJUJ799o4iveN67FSg87nTg6lBOaKtzqMrm4WL5V4qm9ZoTzmtA/3Mw
XYBDOokv+YWQVayhnNNkdjMtTIgtEGU6C2DFVMsPT+vvts+GXwsLhCOpudliQY78
OP8CuJ9/4P4kJslM9ER+g0e3eowpzchXXGBOByzWZ3mgy8mCHPZ6tJsRPYOsZjbp
F7+Bz/XMsDKiju6hnmOKndL8COq849QWC3wmuiT5Eh5h9+U6BzvWa0OM1tmzbION
yQw5CnUGF6IzooHUgdclhFuyMtQaMXby6/vm0f7bIg9VPDNlXFrj3o7moHrNIGux
p6li8hv1jkAojmOHdpzI0t3RiLQxfQ==
=CcoX
-----END PGP SIGNATURE-----
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.