Re: [PATCH 1/2] ASoC: SOF: (cosmetic) use for_each_pcm_streams() in sof_dai_load()

Guennadi, who are the intended reviewers/maintainers here and which tree is this supposed to be merged on?

Patch0 has everyone copied, patch 1/2 only target mailing lists, and I haven't seen any PR/CI for this?

I am flexible with processes, I am just lost here on what you had in mind.

Thanks -Pierre

On 3/24/20 7:29 AM, Guennadi Liakhovetski wrote:

Use for_each_pcm_streams() to enumerate streams in sof_dai_load() instead of doing that manually.

Signed-off-by: Guennadi Liakhovetski guennadi.liakhovetski@linux.intel.com

sound/soc/sof/topology.c | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/sound/soc/sof/topology.c b/sound/soc/sof/topology.c index 058de94..54437ca 100644 --- a/sound/soc/sof/topology.c +++ b/sound/soc/sof/topology.c @@ -2448,7 +2448,7 @@ static int sof_dai_load(struct snd_soc_component *scomp, int index, struct snd_soc_tplg_stream_caps *caps; struct snd_soc_tplg_private *private = &pcm->priv; struct snd_sof_pcm *spcm;

• int stream = SNDRV_PCM_STREAM_PLAYBACK;

• int stream; int ret = 0;

  /* nothing to do for BEs atm */

@@ -2460,8 +2460,9 @@ static int sof_dai_load(struct snd_soc_component *scomp, int index, return -ENOMEM;

spcm->scomp = scomp;

• spcm->stream[SNDRV_PCM_STREAM_PLAYBACK].comp_id = COMP_ID_UNASSIGNED;
• spcm->stream[SNDRV_PCM_STREAM_CAPTURE].comp_id = COMP_ID_UNASSIGNED;

• for_each_pcm_streams(stream)

• spcm->stream[stream].comp_id = COMP_ID_UNASSIGNED;
  
  

  spcm->pcm = *pcm; dev_dbg(scomp->dev, "tplg: load pcm %s\n", pcm->dai_name);

@@ -2482,8 +2483,10 @@ static int sof_dai_load(struct snd_soc_component *scomp, int index, if (!spcm->pcm.playback) goto capture;

• stream = SNDRV_PCM_STREAM_PLAYBACK;
• dev_vdbg(scomp->dev, "tplg: pcm %s stream tokens: playback d0i3:%d\n",

•  spcm->pcm.pcm_name, spcm->stream[0].d0i3_compatible);
  

•  spcm->pcm.pcm_name, spcm->stream[stream].d0i3_compatible);
  
  

  caps = &spcm->pcm.caps[stream];

@@ -2513,7 +2516,7 @@ static int sof_dai_load(struct snd_soc_component *scomp, int index, return ret;

dev_vdbg(scomp->dev, "tplg: pcm %s stream tokens: capture d0i3:%d\n",

•  spcm->pcm.pcm_name, spcm->stream[1].d0i3_compatible);
  

•  spcm->pcm.pcm_name, spcm->stream[stream].d0i3_compatible);
  
  

  caps = &spcm->pcm.caps[stream];