From: Charles Keepax <ckeepax@opensource.cirrus.com>
To: anish kumar <yesanishhere@gmail.com>
Cc: <lgirdwood@gmail.com>, <broonie@kernel.org>, <perex@perex.cz>,
<tiwai@suse.com>, <corbet@lwn.net>,
<linux-kernel@vger.kernel.org>, <linux-doc@vger.kernel.org>,
<linux-sound@vger.kernel.org>
Subject: Re: [PATCH V5] Docs/sound: Update codec-to-codec documentation
Date: Tue, 10 Dec 2024 11:06:41 +0000 [thread overview]
Message-ID: <Z1ggwXLAu+/Y817c@opensource.cirrus.com> (raw)
In-Reply-To: <20241207193441.15767-1-yesanishhere@gmail.com>
On Sat, Dec 07, 2024 at 11:34:41AM -0800, anish kumar wrote:
> +Boot-up logs will display message similar to:
> +
> +.. code-block:: text
> +
> + ASoC: registered pcm #0 codec2codec(Playback Codec)
> +
> +To trigger this DAI link, a control interface is established by the
> +DAPM core during internal DAI creation. This interface links to
> +the ``snd_soc_dai_link_event`` function, which is invoked when a
> +path connects in the DAPM core. A mixer must be created to trigger
> +the connection, prompting the DAPM core to evaluate path
> +connections and call the ``snd_soc_dai_link_event`` callback with
> +SND_SOC_DAPM_*_PMU and SND_SOC_DAPM_*_PMD events.
> +
I am still not really following this, what are we trying to
convey in this paragraph?
What do we mean by a "control interface" here, is that referring
to the kcontrol that is created to select the rate in the case
of multiple rates? Or is it simply refering to the fact we call
snd_soc_dapm_new_control_unlocked? But really that function is
adding widgets, as I noted in my last email.
Also what do we mean by a "A mixer must be created", I am not
aware of any requirement for a mixer? The DAI link needs to be on
a valid DAPM path, but I don't see any requirements other than
that.
Thanks,
Charles
next prev parent reply other threads:[~2024-12-10 11:07 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-12-07 19:34 [PATCH V5] Docs/sound: Update codec-to-codec documentation anish kumar
2024-12-10 11:06 ` Charles Keepax [this message]
2024-12-12 21:35 ` anish kumar
2025-01-03 18:10 ` anish kumar
2025-01-06 12:13 ` Charles Keepax
2024-12-10 12:54 ` Mark Brown
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=Z1ggwXLAu+/Y817c@opensource.cirrus.com \
--to=ckeepax@opensource.cirrus.com \
--cc=broonie@kernel.org \
--cc=corbet@lwn.net \
--cc=lgirdwood@gmail.com \
--cc=linux-doc@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-sound@vger.kernel.org \
--cc=perex@perex.cz \
--cc=tiwai@suse.com \
--cc=yesanishhere@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).