[PATCH] ASoC: rt1316: Add RT1316 SDCA vendor-specific driver

Mark Brown broonie at kernel.org
Mon Feb 8 18:52:28 CET 2021


On Mon, Feb 08, 2021 at 09:26:10AM -0600, Pierre-Louis Bossart wrote:

> > > +	rt1316->rx_mask = rx_mask;
> > > +	rt1316->slots = slots;
> > > +	/* slot_width is not used since it's irrelevant for SoundWire */

> > I wouldn't expect to see any TDM stuff at all for SoundWire?  I do see
> > some crept through though :/

> We only use the mask information to let codecs/amplifiers know which
> channels they need to pick in a stream. This is useful e.g. when we have two
> (or more) amplifiers on the same link, and you want to tell amplifierA to
> use ch0 and amplifierB to use ch1.

...

> In most cases though all amplifiers on Intel-based solutions extract the
> same information, and the output will be dynamically selected based on
> kcontrol information. This is easier to use for e.g. orientation and posture
> changes.

Right, this was my expectation for how this would work - dynamic control
at runtime so you can do stuff like that rather than a more static setup
like slot assignment is normally..
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://mailman.alsa-project.org/pipermail/alsa-devel/attachments/20210208/8ddb8309/attachment.sig>


More information about the Alsa-devel mailing list