Commit 973a8429 authored by Rander Wang's avatar Rander Wang Committed by Vinod Koul
Browse files

soundwire: intel: add trigger support in sdw dai driver 



The existing code does not expose a trigger callback, which is very
much required for streaming.

The SoundWire stream is enabled and disabled in trigger function.

Signed-off-by: default avatarPierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Signed-off-by: default avatarRander Wang <rander.wang@linux.intel.com>
Acked-by: default avatarSanyog Kale <sanyog.r.kale@intel.com>
Link: https://lore.kernel.org/r/20200215014740.27580-4-pierre-louis.bossart@linux.intel.com


Signed-off-by: default avatarVinod Koul <vkoul@kernel.org>
parent 27b198f4

drivers/soundwire/intel.c

+39 −0
Original line number Diff line number Diff line
@@ -714,6 +714,43 @@ static int intel_prepare(struct snd_pcm_substream *substream, 
	return sdw_prepare_stream(dma->stream);

}



static int intel_trigger(struct snd_pcm_substream *substream, int cmd,

			 struct snd_soc_dai *dai)

{

	struct sdw_cdns_dma_data *dma;

	int ret;



	dma = snd_soc_dai_get_dma_data(dai, substream);

	if (!dma) {

		dev_err(dai->dev, "failed to get dma data in %s", __func__);

		return -EIO;

	}



	switch (cmd) {

	case SNDRV_PCM_TRIGGER_START:

	case SNDRV_PCM_TRIGGER_PAUSE_RELEASE:

	case SNDRV_PCM_TRIGGER_RESUME:

		ret = sdw_enable_stream(dma->stream);

		break;



	case SNDRV_PCM_TRIGGER_PAUSE_PUSH:

	case SNDRV_PCM_TRIGGER_SUSPEND:

	case SNDRV_PCM_TRIGGER_STOP:

		ret = sdw_disable_stream(dma->stream);

		break;



	default:

		ret = -EINVAL;

		break;

	}



	if (ret)

		dev_err(dai->dev,

			"%s trigger %d failed: %d",

			__func__, cmd, ret);

	return ret;

}



static int

intel_hw_free(struct snd_pcm_substream *substream, struct snd_soc_dai *dai)

{
@@ -761,6 +798,7 @@ static int intel_pdm_set_sdw_stream(struct snd_soc_dai *dai, 
static const struct snd_soc_dai_ops intel_pcm_dai_ops = {

	.hw_params = intel_hw_params,

	.prepare = intel_prepare,

	.trigger = intel_trigger,

	.hw_free = intel_hw_free,

	.shutdown = intel_shutdown,

	.set_sdw_stream = intel_pcm_set_sdw_stream,
@@ -769,6 +807,7 @@ static const struct snd_soc_dai_ops intel_pcm_dai_ops = { 
static const struct snd_soc_dai_ops intel_pdm_dai_ops = {

	.hw_params = intel_hw_params,

	.prepare = intel_prepare,

	.trigger = intel_trigger,

	.hw_free = intel_hw_free,

	.shutdown = intel_shutdown,

	.set_sdw_stream = intel_pdm_set_sdw_stream,