From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S942391AbdDTGne (ORCPT ); Thu, 20 Apr 2017 02:43:34 -0400 Received: from mail-oi0-f68.google.com ([209.85.218.68]:36027 "EHLO mail-oi0-f68.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S968106AbdDTGn1 (ORCPT ); Thu, 20 Apr 2017 02:43:27 -0400 Date: Wed, 19 Apr 2017 23:43:21 -0700 From: Nicolin Chen To: Stefan Agner Cc: timur@tabi.org, Xiubo.Lee@gmail.com, fabio.estevam@nxp.com, lgirdwood@gmail.com, broonie@kernel.org, perex@perex.cz, tiwai@suse.com, alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH] ASoC: fsl_asrc_dma: use correct direction enum type Message-ID: <20170420064320.GA7912@Asurada> References: <20170420063607.23990-1-stefan@agner.ch> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20170420063607.23990-1-stefan@agner.ch> User-Agent: Mutt/1.5.22 (2013-10-16) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Apr 19, 2017 at 11:36:07PM -0700, Stefan Agner wrote: > The direction argument is of type enum dma_transfer_direction, and > not enum dma_data_direction. The enumeration values are the same > so this did not had an effect in practise. > > Signed-off-by: Stefan Agner Acked-by: Nicolin Chen Thanks > --- > sound/soc/fsl/fsl_asrc_dma.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/sound/soc/fsl/fsl_asrc_dma.c b/sound/soc/fsl/fsl_asrc_dma.c > index dc30d780f874..282d841840b1 100644 > --- a/sound/soc/fsl/fsl_asrc_dma.c > +++ b/sound/soc/fsl/fsl_asrc_dma.c > @@ -76,7 +76,7 @@ static int fsl_asrc_dma_prepare_and_submit(struct snd_pcm_substream *substream) > pair->dma_chan[!dir], runtime->dma_addr, > snd_pcm_lib_buffer_bytes(substream), > snd_pcm_lib_period_bytes(substream), > - dir == OUT ? DMA_TO_DEVICE : DMA_FROM_DEVICE, flags); > + dir == OUT ? DMA_MEM_TO_DEV : DMA_DEV_TO_MEM, flags); > if (!pair->desc[!dir]) { > dev_err(dev, "failed to prepare slave DMA for Front-End\n"); > return -ENOMEM; > -- > 2.12.2 >