ASoC: Intel: atom: fix missing breaks that would cause the wrong operation to execute
Now we correctly error an attempt to execute an unsupported operation. Signed-off-by: Alan Cox <alan@linux.intel.com> Signed-off-by: Vinod Koul <vinod.koul@intel.com> Signed-off-by: Mark Brown <broonie@kernel.org>
This commit is contained in:
parent
e68aca08d7
commit
9c4639ec95
@ -182,24 +182,29 @@ static int sst_platform_compr_trigger(struct snd_compr_stream *cstream, int cmd)
|
|||||||
case SNDRV_PCM_TRIGGER_START:
|
case SNDRV_PCM_TRIGGER_START:
|
||||||
if (stream->compr_ops->stream_start)
|
if (stream->compr_ops->stream_start)
|
||||||
return stream->compr_ops->stream_start(sst->dev, stream->id);
|
return stream->compr_ops->stream_start(sst->dev, stream->id);
|
||||||
|
break;
|
||||||
case SNDRV_PCM_TRIGGER_STOP:
|
case SNDRV_PCM_TRIGGER_STOP:
|
||||||
if (stream->compr_ops->stream_drop)
|
if (stream->compr_ops->stream_drop)
|
||||||
return stream->compr_ops->stream_drop(sst->dev, stream->id);
|
return stream->compr_ops->stream_drop(sst->dev, stream->id);
|
||||||
|
break;
|
||||||
case SND_COMPR_TRIGGER_DRAIN:
|
case SND_COMPR_TRIGGER_DRAIN:
|
||||||
if (stream->compr_ops->stream_drain)
|
if (stream->compr_ops->stream_drain)
|
||||||
return stream->compr_ops->stream_drain(sst->dev, stream->id);
|
return stream->compr_ops->stream_drain(sst->dev, stream->id);
|
||||||
|
break;
|
||||||
case SND_COMPR_TRIGGER_PARTIAL_DRAIN:
|
case SND_COMPR_TRIGGER_PARTIAL_DRAIN:
|
||||||
if (stream->compr_ops->stream_partial_drain)
|
if (stream->compr_ops->stream_partial_drain)
|
||||||
return stream->compr_ops->stream_partial_drain(sst->dev, stream->id);
|
return stream->compr_ops->stream_partial_drain(sst->dev, stream->id);
|
||||||
|
break;
|
||||||
case SNDRV_PCM_TRIGGER_PAUSE_PUSH:
|
case SNDRV_PCM_TRIGGER_PAUSE_PUSH:
|
||||||
if (stream->compr_ops->stream_pause)
|
if (stream->compr_ops->stream_pause)
|
||||||
return stream->compr_ops->stream_pause(sst->dev, stream->id);
|
return stream->compr_ops->stream_pause(sst->dev, stream->id);
|
||||||
|
break;
|
||||||
case SNDRV_PCM_TRIGGER_PAUSE_RELEASE:
|
case SNDRV_PCM_TRIGGER_PAUSE_RELEASE:
|
||||||
if (stream->compr_ops->stream_pause_release)
|
if (stream->compr_ops->stream_pause_release)
|
||||||
return stream->compr_ops->stream_pause_release(sst->dev, stream->id);
|
return stream->compr_ops->stream_pause_release(sst->dev, stream->id);
|
||||||
default:
|
break;
|
||||||
return -EINVAL;
|
|
||||||
}
|
}
|
||||||
|
return -EINVAL;
|
||||||
}
|
}
|
||||||
|
|
||||||
static int sst_platform_compr_pointer(struct snd_compr_stream *cstream,
|
static int sst_platform_compr_pointer(struct snd_compr_stream *cstream,
|
||||||
|
Loading…
Reference in New Issue
Block a user