On Mon, Nov 11, 2024 at 4:54 PM Bagas Sanjaya <bagasdotme@xxxxxxxxx> wrote: > > On Sun, Nov 10, 2024 at 01:30:12PM -0800, anish kumar wrote: > > On Thu, Oct 31, 2024 at 7:13 PM Bagas Sanjaya <bagasdotme@xxxxxxxxx> wrote: > > > > > > On Thu, Oct 31, 2024 at 02:14:11PM -0700, anish kumar wrote: > > > > Updated documentation to provide more details > > > > for codec-to-codec connection especially around > > > > the scenarios and DAPM core details for C2C > > > > creation. > > > "Describe in more detail codec-to-codec connection, especially on > > > use cases and DAPM core details for C2C creation." > > > > Sorry, I like the way it is written. > > What do you mean? > > > > > +An ALSA-based audio system typically involves playback and capture > > > > +functionality, where users may require audio file playback through > > > > +speakers or recording from microphones. However, certain systems > > > > +necessitate audio data routing directly between components, such as FM > > > > +radio to speakers, without CPU involvement. For such scenarios, ASoC( > > > > +ALSA system on chip) provides a mechanism known as codec-to-codec (C2C) > > > "For such scenarios, ASoC (ALSA system on chip) ..." > > > > +connections, leveraging the Dynamic Audio Power Management (DAPM) > > > > +framework to facilitate direct data transfers between codecs. > > > > <snipped>... > > > > +To better understand the configuration inspired by the setup found in > > > > +``sound/soc/samsung/speyside.c``, here are several key points: > > > > + > > > > +1. The presence of ``c2c_params`` informs the DAPM core that the DAI link > > > > + represents a C2C connection. > > > > + > > > > +2. ``c2c_params`` can be an array, and ``num_c2c_params`` defines the size > > > > + of this array. > > > > + > > > > +3. If ``num_c2c_params`` is 1: > > > > + > > > > + - The C2C DAI is configured with the provided ``snd_soc_pcm_stream`` > > > > + parameters. > > > > + > > > > +4. If ``num_c2c_params`` is greater than 1: > > > > + > > > > + - A kcontrol is created, allowing the user to select the index of the > > > > + ``c2c_params`` array to be used. > > > > > > I guess #3 and #4 can be combined, i.e. "If ``num_c2c_params`` is 1, ...; > > > otherwise if ``num_c2c_params`` is greater than 1, ...". > > > > I like it this way. > > Sorry again but what do you mean? Keep it as-is? Yes it is more developer friendly. > > Confused... > > -- > An old man doll... just what I always wanted! - Clara
