From: Sameer Pujar <spujar@nvidia.com> To: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> Cc: jonathanh@nvidia.com, nicoleotsuka@gmail.com, alsa-devel@alsa-project.org, atalambedu@nvidia.com, swarren@nvidia.com, linux-kernel@vger.kernel.org, spujar@nvidia.com, 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 11/23] ASoC: simple-card: Loop over all children for 'mclk-fs' Date: Mon, 29 Jun 2020 22:02:24 +0530 [thread overview] Message-ID: <58000bd3-861c-bbc2-75e1-128cf0199a76@nvidia.com> (raw) In-Reply-To: <875zba1y28.wl-kuninori.morimoto.gx@renesas.com> On 6/29/2020 6:35 AM, Kuninori Morimoto wrote: > External email: Use caution opening links or attachments > > > Hi Sameer > >> CPU/Codec in DPCM DAI links are connected as CPU<->Dummy and Dummy<->Codec. >> Though mostly CPU won't use/require 'mclk-fs' property, looping over >> 'np' (current child node in a DAI link) can help in cases where multiple >> Codecs are defined. This further helps to get rid of 'codec' argument >> from simple_dai_link_of_dpcm() function, which gets called for DPCM links. >> >> Signed-off-by: Sameer Pujar <spujar@nvidia.com> >> --- > (snip) >> diff --git a/sound/soc/generic/simple-card.c b/sound/soc/generic/simple-card.c >> index 39cdc71..02d6295 100644 >> --- a/sound/soc/generic/simple-card.c >> +++ b/sound/soc/generic/simple-card.c >> @@ -107,7 +107,9 @@ static void simple_parse_mclk_fs(struct device_node *top, >> snprintf(prop, sizeof(prop), "%smclk-fs", prefix); >> of_property_read_u32(node, prop, &props->mclk_fs); >> of_property_read_u32(cpu, prop, &props->mclk_fs); >> - of_property_read_u32(codec, prop, &props->mclk_fs); >> + >> + if (cpu != codec) >> + of_property_read_u32(codec, prop, &props->mclk_fs); > Maybe we want to have "cpu" in simple_dai_link_of_dpcm() side > without using magical code in simple_parse_mclk_fs() side ? Are you suggesting if we should simplify simple_parse_mclk_fs() by either passing 'cpu' or 'codec'? > > Thank you for your help !! > > Best regards > --- > Kuninori Morimoto
WARNING: multiple messages have this Message-ID (diff)
From: Sameer Pujar <spujar@nvidia.com> To: Kuninori Morimoto <kuninori.morimoto.gx@renesas.com> Cc: <spujar@nvidia.com>, <broonie@kernel.org>, <perex@perex.cz>, <tiwai@suse.com>, <robh+dt@kernel.org>, <lgirdwood@gmail.com>, <thierry.reding@gmail.com>, <jonathanh@nvidia.com>, <digetx@gmail.com>, <alsa-devel@alsa-project.org>, <linux-tegra@vger.kernel.org>, <linux-kernel@vger.kernel.org>, <sharadg@nvidia.com>, <mkumard@nvidia.com>, <viswanathl@nvidia.com>, <rlokhande@nvidia.com>, <dramesh@nvidia.com>, <atalambedu@nvidia.com>, <nwartikar@nvidia.com>, <swarren@nvidia.com>, <nicoleotsuka@gmail.com> Subject: Re: [PATCH v4 11/23] ASoC: simple-card: Loop over all children for 'mclk-fs' Date: Mon, 29 Jun 2020 22:02:24 +0530 [thread overview] Message-ID: <58000bd3-861c-bbc2-75e1-128cf0199a76@nvidia.com> (raw) In-Reply-To: <875zba1y28.wl-kuninori.morimoto.gx@renesas.com> On 6/29/2020 6:35 AM, Kuninori Morimoto wrote: > External email: Use caution opening links or attachments > > > Hi Sameer > >> CPU/Codec in DPCM DAI links are connected as CPU<->Dummy and Dummy<->Codec. >> Though mostly CPU won't use/require 'mclk-fs' property, looping over >> 'np' (current child node in a DAI link) can help in cases where multiple >> Codecs are defined. This further helps to get rid of 'codec' argument >> from simple_dai_link_of_dpcm() function, which gets called for DPCM links. >> >> Signed-off-by: Sameer Pujar <spujar@nvidia.com> >> --- > (snip) >> diff --git a/sound/soc/generic/simple-card.c b/sound/soc/generic/simple-card.c >> index 39cdc71..02d6295 100644 >> --- a/sound/soc/generic/simple-card.c >> +++ b/sound/soc/generic/simple-card.c >> @@ -107,7 +107,9 @@ static void simple_parse_mclk_fs(struct device_node *top, >> snprintf(prop, sizeof(prop), "%smclk-fs", prefix); >> of_property_read_u32(node, prop, &props->mclk_fs); >> of_property_read_u32(cpu, prop, &props->mclk_fs); >> - of_property_read_u32(codec, prop, &props->mclk_fs); >> + >> + if (cpu != codec) >> + of_property_read_u32(codec, prop, &props->mclk_fs); > Maybe we want to have "cpu" in simple_dai_link_of_dpcm() side > without using magical code in simple_parse_mclk_fs() side ? Are you suggesting if we should simplify simple_parse_mclk_fs() by either passing 'cpu' or 'codec'? > > Thank you for your help !! > > Best regards > --- > Kuninori Morimoto
next prev parent reply other threads:[~2020-06-29 16:32 UTC|newest] Thread overview: 181+ 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 ` Sameer Pujar 2020-06-27 4:53 ` 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 ` Sameer Pujar 2020-06-27 4:53 ` 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-27 4:53 ` Sameer Pujar 2020-06-27 4:53 ` Sameer Pujar [not found] ` <1593233625-14961-16-git-send-email-spujar-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2020-06-29 1:38 ` Kuninori Morimoto 2020-06-29 1:38 ` Kuninori Morimoto 2020-06-29 1:38 ` Kuninori Morimoto 2020-06-29 17:19 ` Sameer Pujar 2020-06-29 17:19 ` Sameer Pujar 2020-06-30 6:07 ` Kuninori Morimoto 2020-06-30 6:07 ` Kuninori Morimoto 2020-06-30 6:07 ` Kuninori Morimoto [not found] ` <87h7utytlx.wl-kuninori.morimoto.gx-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org> 2020-06-30 8:03 ` Sameer Pujar 2020-06-30 8:03 ` Sameer Pujar 2020-06-30 8:03 ` Sameer Pujar [not found] ` <9c7871ae-6649-7b0d-4780-c8389c299b04-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2020-07-02 0:52 ` Kuninori Morimoto 2020-07-02 0:52 ` Kuninori Morimoto 2020-07-02 0:52 ` Kuninori Morimoto [not found] ` <87d05ezqlc.wl-kuninori.morimoto.gx-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org> 2020-07-02 3:48 ` Sameer Pujar 2020-07-02 3:48 ` Sameer Pujar 2020-07-02 3:48 ` Sameer Pujar [not found] ` <49bac9c1-093c-d353-cef3-c9c3391cc00d-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2020-07-02 8:50 ` Kuninori Morimoto 2020-07-02 8:50 ` Kuninori Morimoto 2020-07-02 8:50 ` Kuninori Morimoto [not found] ` <875zb6z4fq.wl-kuninori.morimoto.gx-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org> 2020-07-02 9:56 ` Sameer Pujar 2020-07-02 9:56 ` Sameer Pujar 2020-07-02 9:56 ` Sameer Pujar [not found] ` <1593233625-14961-1-git-send-email-spujar-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 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 ` Sameer Pujar 2020-06-27 4:53 ` 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 ` Sameer Pujar 2020-06-27 4:53 ` 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 ` Sameer Pujar 2020-06-27 4:53 ` 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 ` Sameer Pujar 2020-06-27 4:53 ` 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 ` Sameer Pujar 2020-06-27 4:53 ` 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 ` Sameer Pujar 2020-06-27 4:53 ` Sameer Pujar 2020-06-27 4:53 ` [PATCH v4 08/23] ASoC: soc-core: Fix component name_prefix parsing Sameer Pujar 2020-06-27 4:53 ` Sameer Pujar 2020-06-27 4:53 ` Sameer Pujar [not found] ` <1593233625-14961-9-git-send-email-spujar-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2020-06-29 0:38 ` Kuninori Morimoto 2020-06-29 0:38 ` Kuninori Morimoto 2020-06-29 0:38 ` Kuninori Morimoto 2020-06-29 15:46 ` Sameer Pujar 2020-06-29 15:46 ` Sameer Pujar 2020-06-30 10:45 ` Mark Brown 2020-06-30 10:45 ` Mark Brown 2020-06-30 10:45 ` Mark Brown [not found] ` <20200630104514.GE5272-GFdadSzt00ze9xe1eoZjHA@public.gmane.org> 2020-06-30 12:56 ` Sameer Pujar 2020-06-30 12:56 ` Sameer Pujar 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 ` Sameer Pujar 2020-06-27 4:53 ` 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-27 4:53 ` Sameer Pujar 2020-06-27 4:53 ` Sameer Pujar [not found] ` <1593233625-14961-11-git-send-email-spujar-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2020-06-29 0:56 ` Kuninori Morimoto 2020-06-29 0:56 ` Kuninori Morimoto 2020-06-29 0:56 ` Kuninori Morimoto 2020-06-29 16:06 ` Sameer Pujar [not found] ` <6e27daa5-331e-968b-4027-2e30aeb7d382-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2020-06-30 0:56 ` Kuninori Morimoto 2020-06-30 0:56 ` Kuninori Morimoto 2020-06-30 0:56 ` Kuninori Morimoto [not found] ` <87o8p1z81b.wl-kuninori.morimoto.gx-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org> 2020-06-30 3:47 ` Sameer Pujar 2020-06-30 3:47 ` Sameer Pujar 2020-06-30 3:47 ` Sameer Pujar [not found] ` <841ac69f-1c3f-2b13-17f9-6f196811ce52-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2020-06-30 5:07 ` Kuninori Morimoto 2020-06-30 5:07 ` Kuninori Morimoto 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-27 4:53 ` Sameer Pujar 2020-06-27 4:53 ` Sameer Pujar [not found] ` <1593233625-14961-12-git-send-email-spujar-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2020-06-29 1:05 ` Kuninori Morimoto 2020-06-29 1:05 ` Kuninori Morimoto 2020-06-29 1:05 ` Kuninori Morimoto 2020-06-29 16:32 ` Sameer Pujar [this message] 2020-06-29 16:32 ` Sameer Pujar [not found] ` <58000bd3-861c-bbc2-75e1-128cf0199a76-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2020-06-30 2:08 ` Kuninori Morimoto 2020-06-30 2:08 ` Kuninori Morimoto 2020-06-30 2:08 ` Kuninori Morimoto [not found] ` <87lfk5z4ov.wl-kuninori.morimoto.gx-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org> 2020-06-30 4:23 ` Sameer Pujar 2020-06-30 4:23 ` Sameer Pujar 2020-06-30 4:23 ` Sameer Pujar [not found] ` <b33d5a1b-ecd5-3618-4894-c3ab0f4b077d-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2020-06-30 10:55 ` Mark Brown 2020-06-30 10:55 ` Mark Brown 2020-06-30 10:55 ` Mark Brown 2020-06-30 11:56 ` Sameer Pujar 2020-06-30 11:56 ` Sameer Pujar 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-27 4:53 ` Sameer Pujar 2020-06-27 4:53 ` Sameer Pujar [not found] ` <1593233625-14961-13-git-send-email-spujar-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2020-06-29 1:24 ` Kuninori Morimoto 2020-06-29 1:24 ` Kuninori Morimoto 2020-06-29 1:24 ` Kuninori Morimoto 2020-06-29 17:16 ` Sameer Pujar 2020-06-29 17:16 ` Sameer Pujar [not found] ` <1e0cf6d1-bf4e-8808-5390-c8a3b7c7fe7e-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2020-06-30 1:24 ` Kuninori Morimoto 2020-06-30 1:24 ` Kuninori Morimoto 2020-06-30 1:24 ` Kuninori Morimoto [not found] ` <87mu4lz6pt.wl-kuninori.morimoto.gx-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org> 2020-06-30 4:08 ` Sameer Pujar 2020-06-30 4:08 ` Sameer Pujar 2020-06-30 4:08 ` Sameer Pujar [not found] ` <1d7888c7-a8cc-e891-01aa-016e31cc9113-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2020-06-30 6:55 ` Kuninori Morimoto 2020-06-30 6:55 ` Kuninori Morimoto 2020-06-30 6:55 ` Kuninori Morimoto [not found] ` <87ftadyrec.wl-kuninori.morimoto.gx-zM6kxYcvzFBBDgjK7y7TUQ@public.gmane.org> 2020-06-30 7:52 ` Sameer Pujar 2020-06-30 7:52 ` Sameer Pujar 2020-06-30 7:52 ` Sameer Pujar [not found] ` <492079e9-4518-78ba-a227-859d31594369-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2020-06-30 11:01 ` Mark Brown 2020-06-30 11:01 ` Mark Brown 2020-06-30 11:01 ` Mark Brown [not found] ` <20200630110100.GH5272-GFdadSzt00ze9xe1eoZjHA@public.gmane.org> 2020-06-30 12:53 ` Sameer Pujar 2020-06-30 12:53 ` Sameer Pujar 2020-06-30 12:53 ` Sameer Pujar 2020-06-30 15:32 ` Mark Brown 2020-06-30 15:32 ` Mark Brown [not found] ` <20200630153220.GL5272-GFdadSzt00ze9xe1eoZjHA@public.gmane.org> 2020-07-02 10:36 ` Sameer Pujar 2020-07-02 10:36 ` Sameer Pujar 2020-07-02 10:36 ` Sameer Pujar [not found] ` <fb286ab7-21f2-43ad-2751-c76b7b6e4cf2-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2020-07-02 12:26 ` Mark Brown 2020-07-02 12:26 ` Mark Brown 2020-07-02 12:26 ` Mark Brown [not found] ` <20200702122605.GE4483-GFdadSzt00ze9xe1eoZjHA@public.gmane.org> 2020-07-17 12:55 ` Sameer Pujar 2020-07-17 12:55 ` Sameer Pujar 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 ` Sameer Pujar 2020-06-27 4:53 ` 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 ` Sameer Pujar 2020-06-27 4:53 ` Sameer Pujar 2020-06-27 4:53 ` [PATCH v4 16/23] ASoC: soc-pcm: Get all BEs along DAPM path Sameer Pujar 2020-06-27 4:53 ` Sameer Pujar 2020-06-27 4:53 ` Sameer Pujar [not found] ` <1593233625-14961-17-git-send-email-spujar-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> 2020-06-29 1:41 ` Kuninori Morimoto 2020-06-29 1:41 ` Kuninori Morimoto 2020-06-29 1:41 ` Kuninori Morimoto 2020-06-29 17:29 ` Sameer Pujar 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 ` Sameer Pujar 2020-06-27 4:53 ` Sameer Pujar 2020-06-27 4:53 ` [PATCH v4 19/23] arm64: defconfig: Build AHUB component drivers Sameer Pujar 2020-06-27 4:53 ` Sameer Pujar 2020-06-27 4:53 ` Sameer Pujar 2020-06-27 4:53 ` [PATCH v4 20/23] arm64: defconfig: Enable CONFIG_TEGRA210_ADMA Sameer Pujar 2020-06-27 4:53 ` Sameer Pujar 2020-06-27 4:53 ` 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 ` Sameer Pujar 2020-06-27 4:53 ` 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 ` Sameer Pujar 2020-06-27 4:53 ` 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-27 4:53 ` Sameer Pujar 2020-06-27 4:53 ` Sameer Pujar 2020-06-30 10:51 ` [PATCH v4 00/23] Add support for Tegra210 Audio Mark Brown 2020-06-30 10:51 ` Mark Brown 2020-06-30 10:51 ` Mark Brown 2020-06-30 11:54 ` Sameer Pujar 2020-06-30 11:54 ` Sameer Pujar 2020-06-30 11:54 ` Sameer Pujar 2020-06-27 4:53 ` [PATCH v4 18/23] ASoC: simple-card: Add support for component chaining Sameer Pujar 2020-06-27 4:53 ` Sameer Pujar 2020-06-27 4:53 ` 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=58000bd3-861c-bbc2-75e1-128cf0199a76@nvidia.com \ --to=spujar@nvidia.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=kuninori.morimoto.gx@renesas.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=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: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.