On Thu, 2020-05-21 at 10:55 +0900, Kuninori Morimoto wrote:
Hi Mark
Current ALSA SoC is handling dai_link related operation, but it is implmemented directly without using function/macro, and at random place.
This v2 patch-set creates new snd_soc_link_xxx() functions which handles dai_link related operation, and implmement these at new soc-link.c.
v1 -> v2
- #include <sound/soc-link.h> is added on each c source file instead of soc.h
- not have extra error message after snd_soc_link_xxx(), because it already indicate it via snc_link_ret()
- snd_soc_link_compr_xxx() doesn't have rtd parameter, because it can be created from cstream
Link: https://lore.kernel.org/r/877dx868op.wl-kuninori.morimoto.gx@renesas.com
Kuninori Morimoto (7): ASoC: add soc-link.c ASoC: soc-link: move soc_rtd_xxx() ASoC: soc-link: remove unneeded parameter from snd_soc_link_xxx() ASoC: soc-link: add snd_soc_link_be_hw_params_fixup() ASoC: soc-link: add snd_soc_link_compr_startup() ASoC: soc-link: add snd_soc_link_compr_shutdown() ASoC: soc-link: add snd_soc_link_compr_set_params()
The series LGTM except for the missing include for soc-link.h in soc- link.c. Thanks, Morimoto-san.
Reviewed-by: Ranjani Sridharan ranjani.sridharan@linux.intel.com