From: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>
To: Sameer Pujar <spujar@nvidia.com>
Cc: jonathanh@nvidia.com, nicoleotsuka@gmail.com,
alsa-devel@alsa-project.org, atalambedu@nvidia.com,
swarren@nvidia.com, linux-kernel@vger.kernel.org,
nwartikar@nvidia.com, lgirdwood@gmail.com, robh+dt@kernel.org,
tiwai@suse.com, viswanathl@nvidia.com, sharadg@nvidia.com,
broonie@kernel.org, thierry.reding@gmail.com,
linux-tegra@vger.kernel.org, digetx@gmail.com,
rlokhande@nvidia.com, mkumard@nvidia.com, dramesh@nvidia.com
Subject: Re: [PATCH v4 12/23] ASoC: simple-card: Support DPCM DAI link with multiple Codecs
Date: 30 Jun 2020 15:55:40 +0900 [thread overview]
Message-ID: <87ftadyrec.wl-kuninori.morimoto.gx@renesas.com> (raw)
In-Reply-To: <1d7888c7-a8cc-e891-01aa-016e31cc9113@nvidia.com>
Hi Sameer
Thank you for explaining detail at off-list mail.
Your issue happen on (C) case, and you are tring to solve it.
It is easy to understand if it was indicated at log area.
I have imagined other system from "multiple CPU/Codec support".
(A) (B)
FE <-> BE
(C) (D)
BE <-> BE
> > I'm not sure, this is just my guess.
> > I'm happy if we can support it more easily :)
> Right now I am trying re-use simple-card driver as much as possible
> and still make it work for flexible sound cards. I will be happy to
> discuss and improve the patch wherever necessary. Please help me
> understand which part you think looks to be hacky.
> > But, if it was difficult to keep compatibility on simple-card,
> > we/you need to have new one.
> Patch 17/23 and 18/23 introduce new compatible
> 'simple-cc-audio-card'. Idea was to use component chaining which
> allows connection of FE<->BE and multiple BE<->BE components along the
> DAPM path (patch 16/23). Do you think it would be fine?
This seems very complex system for current simple-card.
"concept" of simple-card is simple (but "code" is not so simple...)
Because of it, it doesn't assume flexible connection.
Maybe your patch works for you, but might breaks other systems.
Or, makes code or DT settings more complex/ununderstandable.
Not sure, but my guess.
Using cpu@0 node for BE is the 1st confusable point for me.
Using fe/be instead of cpu/codec is easy to understand.
Thank you for your help !!
Best regards
---
Kuninori Morimoto
next prev parent reply other threads:[~2020-06-30 6:56 UTC|newest]
Thread overview: 63+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-06-27 4:53 [PATCH v4 00/23] Add support for Tegra210 Audio Sameer Pujar
2020-06-27 4:53 ` [PATCH v4 01/23] ASoC: dt-bindings: tegra: Add DT bindings for Tegra210 Sameer Pujar
2020-06-27 4:53 ` [PATCH v4 02/23] ASoC: tegra: Add support for CIF programming Sameer Pujar
2020-06-27 4:53 ` [PATCH v4 03/23] ASoC: tegra: Add Tegra210 based DMIC driver Sameer Pujar
2020-06-27 4:53 ` [PATCH v4 04/23] ASoC: tegra: Add Tegra210 based I2S driver Sameer Pujar
2020-06-27 4:53 ` [PATCH v4 05/23] ASoC: tegra: Add Tegra210 based AHUB driver Sameer Pujar
2020-06-27 4:53 ` [PATCH v4 06/23] ASoC: tegra: Add Tegra186 based DSPK driver Sameer Pujar
2020-06-27 4:53 ` [PATCH v4 07/23] ASoC: tegra: Add Tegra210 based ADMAIF driver Sameer Pujar
2020-06-27 4:53 ` [PATCH v4 08/23] ASoC: soc-core: Fix component name_prefix parsing Sameer Pujar
2020-06-29 0:38 ` Kuninori Morimoto
2020-06-29 15:46 ` Sameer Pujar
2020-06-30 10:45 ` Mark Brown
2020-06-30 12:56 ` Sameer Pujar
2020-06-27 4:53 ` [PATCH v4 09/23] ASoC: simple-card: Use of_node and DAI names for DAI link names Sameer Pujar
2020-06-27 4:53 ` [PATCH v4 10/23] ASoC: simple-card: Wrong daifmt for CPU end of DPCM DAI link Sameer Pujar
2020-06-29 0:56 ` Kuninori Morimoto
2020-06-29 16:06 ` Sameer Pujar
2020-06-30 0:56 ` Kuninori Morimoto
2020-06-30 3:47 ` Sameer Pujar
2020-06-30 5:07 ` Kuninori Morimoto
2020-06-27 4:53 ` [PATCH v4 11/23] ASoC: simple-card: Loop over all children for 'mclk-fs' Sameer Pujar
2020-06-29 1:05 ` Kuninori Morimoto
2020-06-29 16:32 ` Sameer Pujar
2020-06-30 2:08 ` Kuninori Morimoto
2020-06-30 4:23 ` Sameer Pujar
2020-06-30 10:55 ` Mark Brown
2020-06-30 11:56 ` Sameer Pujar
2020-06-27 4:53 ` [PATCH v4 12/23] ASoC: simple-card: Support DPCM DAI link with multiple Codecs Sameer Pujar
2020-06-29 1:24 ` Kuninori Morimoto
2020-06-29 17:16 ` Sameer Pujar
2020-06-30 1:24 ` Kuninori Morimoto
2020-06-30 4:08 ` Sameer Pujar
2020-06-30 6:55 ` Kuninori Morimoto [this message]
2020-06-30 7:52 ` Sameer Pujar
2020-06-30 11:01 ` Mark Brown
2020-06-30 12:53 ` Sameer Pujar
2020-06-30 15:32 ` Mark Brown
2020-07-02 10:36 ` Sameer Pujar
2020-07-02 12:26 ` Mark Brown
2020-07-17 12:55 ` Sameer Pujar
2020-06-27 4:53 ` [PATCH v4 13/23] ASoC: simple-card: DPCM DAI link direction as per DAI capability Sameer Pujar
2020-06-27 4:53 ` [PATCH v4 14/23] ASoC: soc-core: Probe auxiliary component before others Sameer Pujar
2020-06-27 4:53 ` [PATCH v4 15/23] ASoC: soc-core: Identify 'no_pcm' DAI links for DPCM Sameer Pujar
2020-06-29 1:38 ` Kuninori Morimoto
2020-06-29 17:19 ` Sameer Pujar
2020-06-30 6:07 ` Kuninori Morimoto
2020-06-30 8:03 ` Sameer Pujar
2020-07-02 0:52 ` Kuninori Morimoto
2020-07-02 3:48 ` Sameer Pujar
2020-07-02 8:50 ` Kuninori Morimoto
2020-07-02 9:56 ` Sameer Pujar
2020-06-27 4:53 ` [PATCH v4 16/23] ASoC: soc-pcm: Get all BEs along DAPM path Sameer Pujar
2020-06-29 1:41 ` Kuninori Morimoto
2020-06-29 17:29 ` Sameer Pujar
2020-06-27 4:53 ` [PATCH v4 17/23] ASoC: dt-bindings: simple-card: Add compatible for component chaining Sameer Pujar
2020-06-27 4:53 ` [PATCH v4 18/23] ASoC: simple-card: Add support " Sameer Pujar
2020-06-27 4:53 ` [PATCH v4 19/23] arm64: defconfig: Build AHUB component drivers Sameer Pujar
2020-06-27 4:53 ` [PATCH v4 20/23] arm64: defconfig: Enable CONFIG_TEGRA210_ADMA Sameer Pujar
2020-06-27 4:53 ` [PATCH v4 21/23] arm64: tegra: Add DT binding for AHUB components Sameer Pujar
2020-06-27 4:53 ` [PATCH v4 22/23] arm64: tegra: Enable AHUB components on few Tegra platforms Sameer Pujar
2020-06-27 4:53 ` [PATCH v4 23/23] arm64: tegra: Add support for APE sound card on Jetson Nano and TX1 Sameer Pujar
2020-06-30 10:51 ` [PATCH v4 00/23] Add support for Tegra210 Audio Mark Brown
2020-06-30 11:54 ` Sameer Pujar
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=87ftadyrec.wl-kuninori.morimoto.gx@renesas.com \
--to=kuninori.morimoto.gx@renesas.com \
--cc=alsa-devel@alsa-project.org \
--cc=atalambedu@nvidia.com \
--cc=broonie@kernel.org \
--cc=digetx@gmail.com \
--cc=dramesh@nvidia.com \
--cc=jonathanh@nvidia.com \
--cc=lgirdwood@gmail.com \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-tegra@vger.kernel.org \
--cc=mkumard@nvidia.com \
--cc=nicoleotsuka@gmail.com \
--cc=nwartikar@nvidia.com \
--cc=rlokhande@nvidia.com \
--cc=robh+dt@kernel.org \
--cc=sharadg@nvidia.com \
--cc=spujar@nvidia.com \
--cc=swarren@nvidia.com \
--cc=thierry.reding@gmail.com \
--cc=tiwai@suse.com \
--cc=viswanathl@nvidia.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