Unverified Commit 5c0769af authored by Kuninori Morimoto's avatar Kuninori Morimoto Committed by Mark Brown
Browse files

ASoC: soc-dai: add snd_soc_dai_bespoke_trigger()



Current ALSA SoC is directly using dai->driver->ops->xxx,
thus, it has deep nested bracket, and it makes code unreadable.
This patch adds new snd_soc_dai_bespoke_trigger() and use it.

Signed-off-by: default avatarKuninori Morimoto <kuninori.morimoto.gx@renesas.com>
Link: https://lore.kernel.org/r/87r26ihn3u.wl-kuninori.morimoto.gx@renesas.com


Signed-off-by: default avatarMark Brown <broonie@kernel.org>
parent 95aef355
Loading
Loading
Loading
Loading
+2 −0
Original line number Diff line number Diff line
@@ -158,6 +158,8 @@ int snd_soc_dai_prepare(struct snd_soc_dai *dai,
			struct snd_pcm_substream *substream);
int snd_soc_dai_trigger(struct snd_soc_dai *dai,
			struct snd_pcm_substream *substream, int cmd);
int snd_soc_dai_bespoke_trigger(struct snd_soc_dai *dai,
			struct snd_pcm_substream *substream, int cmd);

struct snd_soc_dai_ops {
	/*
+12 −0
Original line number Diff line number Diff line
@@ -330,3 +330,15 @@ int snd_soc_dai_trigger(struct snd_soc_dai *dai,

	return ret;
}

int snd_soc_dai_bespoke_trigger(struct snd_soc_dai *dai,
				struct snd_pcm_substream *substream,
				int cmd)
{
	int ret = 0;

	if (dai->driver->ops->bespoke_trigger)
		ret = dai->driver->ops->bespoke_trigger(substream, cmd, dai);

	return ret;
}
+6 −10
Original line number Diff line number Diff line
@@ -1123,19 +1123,15 @@ static int soc_pcm_bespoke_trigger(struct snd_pcm_substream *substream,
	int i, ret;

	for_each_rtd_codec_dai(rtd, i, codec_dai) {
		if (codec_dai->driver->ops->bespoke_trigger) {
			ret = codec_dai->driver->ops->bespoke_trigger(substream,
								cmd, codec_dai);
		ret = snd_soc_dai_bespoke_trigger(codec_dai, substream, cmd);
		if (ret < 0)
			return ret;
	}
	}

	if (cpu_dai->driver->ops->bespoke_trigger) {
		ret = cpu_dai->driver->ops->bespoke_trigger(substream, cmd, cpu_dai);
	snd_soc_dai_bespoke_trigger(cpu_dai, substream, cmd);
	if (ret < 0)
		return ret;
	}

	return 0;
}
/*