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@xxxxxxxxxxx > > 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@xxxxxxxxxxxxxxx>
