On Fri, Apr 26, 2013 at 02:47:06PM +0100, Richard Fitzgerald wrote:

> +	if ((data->stream.runtime->state == SNDRV_PCM_STATE_RUNNING) ||
> +	    (data->stream.runtime->state == SNDRV_PCM_STATE_PAUSED))
> +		data->stream.ops->trigger(&data->stream, SNDRV_PCM_TRIGGER_STOP);
> +

A switch statement would be more idiomatic for this sort of thing.