[PATCH v2 0/7] ASoC: add soc-link
Ranjani Sridharan
ranjani.sridharan at linux.intel.com
Thu May 21 20:25:17 CEST 2020
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 at linux.intel.com>
More information about the Alsa-devel
mailing list