3 Sep
2018
3 Sep
'18
6:19 p.m.
On Fri, Aug 31, 2018 at 04:19:04PM +0100, Liam Girdwood wrote:
+static int sof_widget_load(struct snd_soc_component *scomp, int index,
struct snd_soc_dapm_widget *w,struct snd_soc_tplg_dapm_widget *tw)+{
- /* nothing todo atm */
- return 0;
+}
+static int sof_link_unload(struct snd_soc_component *scomp,
struct snd_soc_dobj *dobj)+{
- return 0;
+}
Missing implementation here as well? If these functions can sensibly be optional shouldn't the framework support that?
+static int sof_route_unload(struct snd_soc_component *scomp,
struct snd_soc_dobj *dobj)+{
- /* TODO: unload routes when topology is changed */
- return 0;
+}
This seems like it should at least return an error.