From: Vinod Koul Subject: Re: [PATCH 02/13] dmaengine: Introduce dma_request_slave_channel_compat_reason() Date: Wed, 18 Nov 2015 21:36:18 +0530 Message-ID: <20151118160618.GV25173@localhost> References: <1432646768-12532-1-git-send-email-peter.ujfalusi@ti.com> <6358656.jIv3GGCCXu@wuerfel> <8215351.e99Q2vhQ5T@wuerfel> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: "devicetree@vger.kernel.org" , ALSA Development Mailing List , Tony Lindgren , Linux MMC List , "linux-kernel@vger.kernel.org" , linux-spi , Peter Ujfalusi , Andy Shevchenko , Geert Uytterhoeven , linux-crypto , "linux-serial@vger.kernel.org" , dmaengine , Dan Williams , "linux-omap@vger.kernel.org" , Linux Media Mailing List To: Arnd Bergmann Return-path: Content-Disposition: inline In-Reply-To: <8215351.e99Q2vhQ5T@wuerfel> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: alsa-devel-bounces@alsa-project.org Sender: alsa-devel-bounces@alsa-project.org List-Id: linux-crypto.vger.kernel.org On Wed, Nov 18, 2015 at 04:51:54PM +0100, Arnd Bergmann wrote: > On Wednesday 18 November 2015 17:43:04 Andy Shevchenko wrote: > > > > > > I assume that the sst-firmware.c case is a mistake, it should just use a > > > plain DMA_SLAVE and not DMA_MEMCPY. > > > > Other way around. > > > > Ok, I see. In that case I guess it also shouldn't call > dmaengine_slave_config(), right? I don't think that's valid > on a MEMCPY channel. Yes it is not valid. In this case the dma driver should invoke a generic memcpy and not need slave parameters, knowing the hw and reason for this (firmware download to DSP memory), this doesn't qualify for slave case. In fact filter function doesn't need a channel, any channel in this controller will be good -- ~Vinod