Alsa-Devel Archive on lore.kernel.org
 help / color / Atom feed
From: "Sridharan, Ranjani" <ranjani.sridharan@intel.com>
To: Daniel Baluta <daniel.baluta@gmail.com>
Cc: Linux-ALSA <alsa-devel@alsa-project.org>,
	Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Subject: Re: [alsa-devel] Question about FE & BE DPCM
Date: Fri, 11 Oct 2019 14:56:54 -0700
Message-ID: <CAFQqKeV+Q0vrHZDMniteEratq09p50CpQHHygJJLvCe+zSHA3Q@mail.gmail.com> (raw)
In-Reply-To: <CAEnQRZAen_Aa7SMdnPY4ezzbYtUypShto7ENGV_FhnsNhnxmBQ@mail.gmail.com>

On Fri, Oct 11, 2019 at 2:05 PM Daniel Baluta <daniel.baluta@gmail.com>
wrote:

> Hi Morimoto-san,
>
> I'm really confused on what exactly is Front End and Back End.
>
> Is FE a DAI or a DAI link or part of a DAI link?
>
> In simple-card.c there is simple_dai_link_of_dpcm function:
>
> struct snd_soc_dai_link_component *codecs = dai_link->codecs;
>
> »       /* BE is dummy */
> »       »       codecs->of_node»»       = NULL;
> »       »       codecs->dai_name»       = "snd-soc-dummy-dai";
> »       »       codecs->name»   »       = "snd-soc-dummy";
>
> »       »       /* FE settings */
> »       »       dai_link->dynamic»      »       = 1;
> »       »       dai_link->dpcm_merged_format»   = 1;
>
> So, from here it looks like BE/FE are components being part of the same
> link.
>
> Another thing is:
>
> asoc_simple_canonicalize_platform function where it does say:
>
> »        * DPCM BE can be no platform.
> »        * Alloced memory will be waste, but not leak.
> »        */
> »       if (!dai_link->platforms->of_node)
> »       »       dai_link->num_platforms = 0;
>
> But I do want to create a BE with a platform. So, should I try to
> revert the following
> patch:
>
> commit 794fcee8da3c0c8a01b08ecad1c181cb0a622868
> Author: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
> Date:   Wed Jul 10 17:01:12 2019 +0900
>
>     ASoC: simple-card-utils: care no Platform for DPCM
>
>     commit 34614739988ad ("ASoC: soc-core: support dai_link with
>     platforms_num != 1") supports multi Platform, and
>     commit 9f3eb91753451 ("ASoC: simple-card-utils: consider CPU-Platform
>     possibility") removed no Platform from simple-card.
>
>     Multi Platform is now checking both Platform name/of_node are NULL
> case.
>     But in normal case, DPCM be doesn't have Platform.
>
>
>
> Basically, what I want to do is: after the DAI link is created to call:
> snd_soc_fixup_dai_links_platform_name function to set a proper name
> for BE platform.
>
> The problem is that above mentioned patch sets dai_link->num_platforms = 0;
> so even the name is set the platform component is not probed.


Hi Daniel,
Have you looked into how SOF does it at the time the dai_link elem in
topology is parsed?
https://github.com/thesofproject/linux/blob/5edcfdf3907354de7a260a451ba611f22c5e6a07/sound/soc/sof/topology.c#L2926

Is this what you are asking about?

Thanks,
Ranjani

>
> thanks,
> Daniel.
>
_______________________________________________
Alsa-devel mailing list
Alsa-devel@alsa-project.org
https://mailman.alsa-project.org/mailman/listinfo/alsa-devel

  reply index

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-11 21:05 Daniel Baluta
2019-10-11 21:56 ` Sridharan, Ranjani [this message]
2019-10-12  9:20   ` Daniel Baluta
2019-10-15  1:15 ` Kuninori Morimoto

Reply instructions:

You may reply publically 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=CAFQqKeV+Q0vrHZDMniteEratq09p50CpQHHygJJLvCe+zSHA3Q@mail.gmail.com \
    --to=ranjani.sridharan@intel.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=daniel.baluta@gmail.com \
    --cc=kuninori.morimoto.gx@renesas.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

Alsa-Devel Archive on lore.kernel.org

Archives are clonable:
	git clone --mirror https://lore.kernel.org/alsa-devel/0 alsa-devel/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 alsa-devel alsa-devel/ https://lore.kernel.org/alsa-devel \
		alsa-devel@alsa-project.org alsa-devel@archiver.kernel.org
	public-inbox-index alsa-devel

Example config snippet for mirrors

Newsgroup available over NNTP:
	nntp://nntp.lore.kernel.org/org.alsa-project.alsa-devel


AGPL code for this site: git clone https://public-inbox.org/ public-inbox