Re: [PATCH RESEND] sst: fix missing breaks that would cause the wrong operation to execute

[Yang Jie <yang.jie@linux.intel.com>]

On 2016年04月06日 21:44, Alan wrote:
> From: Alan <gnomes@lxorguk.ukuu.org.uk>
>
> Now we correctly error an attempt to execute an unsupported operation. The
> current code does something else random.
>
> Signed-off-by: Alan Cox <alan@linux.intel.com>

I think this is nice fix. + Vinod who is the owner of the atom sound driver.

Thanks,
~Keyon

> ---
>   sound/soc/intel/atom/sst-mfld-platform-compress.c |    9 +++++++--
>   1 file changed, 7 insertions(+), 2 deletions(-)
>
> diff --git a/sound/soc/intel/atom/sst-mfld-platform-compress.c b/sound/soc/intel/atom/sst-mfld-platform-compress.c
> index 3951689..1bead81 100644
> --- a/sound/soc/intel/atom/sst-mfld-platform-compress.c
> +++ b/sound/soc/intel/atom/sst-mfld-platform-compress.c
> @@ -182,24 +182,29 @@ static int sst_platform_compr_trigger(struct snd_compr_stream *cstream, int cmd)
>   	case SNDRV_PCM_TRIGGER_START:
>   		if (stream->compr_ops->stream_start)
>   			return stream->compr_ops->stream_start(sst->dev, stream->id);
> +		break;
>   	case SNDRV_PCM_TRIGGER_STOP:
>   		if (stream->compr_ops->stream_drop)
>   			return stream->compr_ops->stream_drop(sst->dev, stream->id);
> +		break;
>   	case SND_COMPR_TRIGGER_DRAIN:
>   		if (stream->compr_ops->stream_drain)
>   			return stream->compr_ops->stream_drain(sst->dev, stream->id);
> +		break;
>   	case SND_COMPR_TRIGGER_PARTIAL_DRAIN:
>   		if (stream->compr_ops->stream_partial_drain)
>   			return stream->compr_ops->stream_partial_drain(sst->dev, stream->id);
> +		break;
>   	case SNDRV_PCM_TRIGGER_PAUSE_PUSH:
>   		if (stream->compr_ops->stream_pause)
>   			return stream->compr_ops->stream_pause(sst->dev, stream->id);
> +		break;
>   	case SNDRV_PCM_TRIGGER_PAUSE_RELEASE:
>   		if (stream->compr_ops->stream_pause_release)
>   			return stream->compr_ops->stream_pause_release(sst->dev, stream->id);
> -	default:
> -		return -EINVAL;
> +		break;
>   	}
> +	return -EINVAL;
>   }
>
>   static int sst_platform_compr_pointer(struct snd_compr_stream *cstream,
>
>