[PATCH v2 00/21] ASoC: cleanup DAI/Component activity

Ranjani Sridharan ranjani.sridharan at linux.intel.com
Thu May 14 19:26:30 CEST 2020


On Thu, 2020-05-14 at 17:36 +0900, Kuninori Morimoto wrote:
> Hi Mark
> 
> These are v2 of DAI/Component activity cleanup.
> 
> This patch-set exchanges soc-dapm.c :: snd_soc_dai_link_event_xxx()
> behavior which updates dai->active but not cares other actives.
> But I think original code was wrong.
> So, I believe these works correctly, but maybe need some tests or
> deep review.
> 
> v1 patch-set used "activity", but v2 is using "active" for each
> function/macro naming.
> 
> Link: 
> https://lore.kernel.org/r/875zd39frp.wl-kuninori.morimoto.gx@renesas.com
> 
> Kuninori Morimoto (21):
>   ASoC: soc-pcm: replace snd_soc_runtime_activate()/deactivate() to
> macro
>   ASoC: soc-dai: add snd_soc_dai_action()
>   ASoC: soc-dapm: use snd_soc_dai_activate()/deactivate()
>   ASoC: soc-dai: add snd_soc_dai_active()
>   ASoC: soc-component: add snd_soc_component_active()
>   ASoC: soc-dai: add snd_soc_dai_stream_active()
>   ASoC: use snd_soc_xxx_active()
>   ASoC: atomel: use snd_soc_xxx_active()
>   ASoC: bcm: use snd_soc_xxx_active()
>   ASoC: cirrus: use snd_soc_xxx_active()
>   ASoC: codecs: use snd_soc_xxx_active()
>   ASoC: fsl: use snd_soc_xxx_active()
>   ASoC: intel: use snd_soc_xxx_active()
>   ASoC: jz4740: use snd_soc_xxx_active()
>   ASoC: mediatek: use snd_soc_xxx_active()
>   ASoC: meson: use snd_soc_xxx_active()
>   ASoC: pxa: use snd_soc_xxx_active()
>   ASoC: ti: use snd_soc_xxx_active()
>   ASoC: uniphier: use snd_soc_xxx_active()
>   ASoC: dwc: use snd_soc_xxx_active()
>   ASoC: cleanup dai / component active code

V2 looks good, Morimoto-san. Thanks!

Reviewed-by: Ranjani Sridharan <ranjani.sridharan at linux.intel.com>



More information about the Alsa-devel mailing list