[0/8] ASoC: soc-pcm cleanup step5
mbox series

Message ID 87eeuh97k4.wl-kuninori.morimoto.gx@renesas.com
Headers show
Series
  • ASoC: soc-pcm cleanup step5
Related show

Message

Kuninori Morimoto Feb. 26, 2020, 6:39 a.m. UTC
Hi Mark

These are soc-pcm cleanup step5.
These are based on Bard Liao's "ASoC: Add Multi CPU DAI support"
patch-set.

[7/8] patch might exchange the behavior

Kuninori Morimoto (8):
  1) ASoC: soc-pcm: use defined stream
  2) ASoC: soc-pcm: remove duplicate be check from dpcm_add_paths()
  3) ASoC: soc-pcm: move dpcm_fe_dai_close()
  4) ASoC: soc-pcm: add dpcm_fe_dai_clean()
  5) ASoC: soc-pcm: use snd_soc_dai_get_pcm_stream() at dpcm_set_fe_runtime()
  6) ASoC: soc-pcm: check DAI's activity more simply
  7) ASoC: soc-pcm: Do Digital Mute for both CPU/Codec in same timing.
  8) ASoC: soc-pcm: tidyup dulicate handing at dpcm_fe_dai_startup()

 sound/soc/soc-pcm.c | 107 ++++++++++++++++++++------------------------
 1 file changed, 48 insertions(+), 59 deletions(-)

Comments

Pierre-Louis Bossart Feb. 26, 2020, 3:24 p.m. UTC | #1
On 2/26/20 12:39 AM, Kuninori Morimoto wrote:
> 
> Hi Mark
> 
> These are soc-pcm cleanup step5.
> These are based on Bard Liao's "ASoC: Add Multi CPU DAI support"
> patch-set.
> 
> [7/8] patch might exchange the behavior
> 
> Kuninori Morimoto (8):
>    1) ASoC: soc-pcm: use defined stream
>    2) ASoC: soc-pcm: remove duplicate be check from dpcm_add_paths()
>    3) ASoC: soc-pcm: move dpcm_fe_dai_close()
>    4) ASoC: soc-pcm: add dpcm_fe_dai_clean()
>    5) ASoC: soc-pcm: use snd_soc_dai_get_pcm_stream() at dpcm_set_fe_runtime()
>    6) ASoC: soc-pcm: check DAI's activity more simply
>    7) ASoC: soc-pcm: Do Digital Mute for both CPU/Codec in same timing.
>    8) ASoC: soc-pcm: tidyup dulicate handing at dpcm_fe_dai_startup()

Looks good to me (only minor nit-picks)

Reviewed-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>

if you give 2-3 days we'll provide a Tested-by tag.

> 
>   sound/soc/soc-pcm.c | 107 ++++++++++++++++++++------------------------
>   1 file changed, 48 insertions(+), 59 deletions(-)
>