Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1164414AbdD1NiA (ORCPT ); Fri, 28 Apr 2017 09:38:00 -0400 Received: from smtprelay4.synopsys.com ([198.182.47.9]:48153 "EHLO smtprelay.synopsys.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1031028AbdD1Nhw (ORCPT ); Fri, 28 Apr 2017 09:37:52 -0400 From: Eugeniy Paltsev To: dmaengine@vger.kernel.org Cc: linux-kernel@vger.kernel.org, linux-snps-arc@lists.infradead.org, Dan Williams , Vinod Koul , Andy Shevchenko , Alexey Brodkin , Eugeniy Paltsev Subject: [PATCH] Allow to use DMA_CTRL_REUSE flag for all channel types Date: Fri, 28 Apr 2017 16:37:46 +0300 Message-Id: <20170428133746.1855-1-Eugeniy.Paltsev@synopsys.com> X-Mailer: git-send-email 2.9.3 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1122 Lines: 34 In the current implementation dma_get_slave_caps is used to check state of descriptor_reuse option. But dma_get_slave_caps includes check if the channel supports slave transactions. So DMA_CTRL_REUSE flag can be set (even for MEM-TO-MEM tranfers) only if channel supports slave transactions. Now we can use DMA_CTRL_REUSE flag for all channel types. Also it allows to test reusing mechanism with simply mem-to-mem dma test. Signed-off-by: Eugeniy Paltsev --- include/linux/dmaengine.h | 6 +----- 1 file changed, 1 insertion(+), 5 deletions(-) diff --git a/include/linux/dmaengine.h b/include/linux/dmaengine.h index 5336808..92cf8b0 100644 --- a/include/linux/dmaengine.h +++ b/include/linux/dmaengine.h @@ -1376,11 +1376,7 @@ static inline int dma_get_slave_caps(struct dma_chan *chan, static inline int dmaengine_desc_set_reuse(struct dma_async_tx_descriptor *tx) { - struct dma_slave_caps caps; - - dma_get_slave_caps(tx->chan, &caps); - - if (caps.descriptor_reuse) { + if (tx->chan->device->descriptor_reuse) { tx->flags |= DMA_CTRL_REUSE; return 0; } else { -- 2.9.3