| Message ID | 1416211377-9559-1-git-send-email-zhouqiao@marvell.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Mon, Nov 17, 2014 at 04:02:57PM +0800, Qiao Zhou wrote: > Skip dpcm path checking for playback or capture, if corresponding FE > doesn't support playback or capture, or currently is not ready. It > can reduce the unnecessary cost to search connected widgets. Applied, but I did change... > + /* skip if FE doesn't do playback */ > + if (!fe->cpu_dai->playback_active > + || !fe->codec_dai->playback_active) ...this and the corresponding capture comment to say "isn't currently playing" - it's not if the front end can play, it's if it is playing right now.
On 11/22/2014 03:32 AM, Mark Brown wrote: > On Mon, Nov 17, 2014 at 04:02:57PM +0800, Qiao Zhou wrote: >> Skip dpcm path checking for playback or capture, if corresponding FE >> doesn't support playback or capture, or currently is not ready. It >> can reduce the unnecessary cost to search connected widgets. > > Applied, but I did change... > >> + /* skip if FE doesn't do playback */ >> + if (!fe->cpu_dai->playback_active >> + || !fe->codec_dai->playback_active) > > ...this and the corresponding capture comment to say "isn't currently > playing" - it's not if the front end can play, it's if it is playing > right now. > Mark, Thanks for the correction.
diff --git a/sound/soc/soc-pcm.c b/sound/soc/soc-pcm.c index 002311a..3b068d4 100644 --- a/sound/soc/soc-pcm.c +++ b/sound/soc/soc-pcm.c @@ -2248,7 +2248,13 @@ int soc_dpcm_runtime_update(struct snd_soc_card *card) fe->dai_link->name); /* skip if FE doesn't have playback capability */ - if (!fe->cpu_dai->driver->playback.channels_min) + if (!fe->cpu_dai->driver->playback.channels_min + || !fe->codec_dai->driver->playback.channels_min) + goto capture; + + /* skip if FE doesn't do playback */ + if (!fe->cpu_dai->playback_active + || !fe->codec_dai->playback_active) goto capture; paths = dpcm_path_get(fe, SNDRV_PCM_STREAM_PLAYBACK, &list); @@ -2278,7 +2284,13 @@ int soc_dpcm_runtime_update(struct snd_soc_card *card) dpcm_path_put(&list); capture: /* skip if FE doesn't have capture capability */ - if (!fe->cpu_dai->driver->capture.channels_min) + if (!fe->cpu_dai->driver->capture.channels_min + || !fe->codec_dai->driver->capture.channels_min) + continue; + + /* skip if FE doesn't do capture */ + if (!fe->cpu_dai->capture_active + || !fe->codec_dai->capture_active) continue; paths = dpcm_path_get(fe, SNDRV_PCM_STREAM_CAPTURE, &list);
Skip dpcm path checking for playback or capture, if corresponding FE doesn't support playback or capture, or currently is not ready. It can reduce the unnecessary cost to search connected widgets. Signed-off-by: Qiao Zhou <zhouqiao@marvell.com> --- sound/soc/soc-pcm.c | 16 ++++++++++++++-- 1 file changed, 14 insertions(+), 2 deletions(-)