[Sound-open-firmware] [PATCH] FIX: Add dma_stop when receive trigger stop cmd in host

[zhigangw]

Hello Liam:

     could you apply this patch?

Thanks

Br

~zhigangw


On 2018年05月02日 17:33, Wu Zhigang wrote:
> when ALSA lib detect xrun issue, it will send stop and start
> command to DSP without reset. it will hit error in hds_dma_start,
> if we did not add the dma_stop in the host_trigger function.
>
> Signed-off-by: Wu Zhigang <zhigang.wu at linux.intel.com>
>
> ---
> Test with:
> Apl-gpmrb with codec
> Linux sof-v4.14: 735b995d7b66
> SOF master: 85ae8e74181f
> SOF-tools master: b327539e98f1
> ---
> ---
>   src/audio/host.c | 1 +
>   1 file changed, 1 insertion(+)
>
> diff --git a/src/audio/host.c b/src/audio/host.c
> index bee50ce..998c5ca 100644
> --- a/src/audio/host.c
> +++ b/src/audio/host.c
> @@ -466,6 +466,7 @@ static int host_trigger(struct comp_dev *dev, int cmd)
>   
>   	switch (cmd) {
>   	case COMP_TRIGGER_STOP:
> +		ret = dma_stop(hd->dma, hd->chan);
>   		ret = host_stop(dev);
>   		break;
>   	case COMP_TRIGGER_START: