From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754479AbdCIJ6J (ORCPT ); Thu, 9 Mar 2017 04:58:09 -0500 Received: from us01smtprelay-2.synopsys.com ([198.182.47.9]:55771 "EHLO smtprelay.synopsys.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753746AbdCIJ5e (ORCPT ); Thu, 9 Mar 2017 04:57:34 -0500 Subject: Re: [PATCH] dma: xilinx: Make Xilinx VDMA support ARC To: Mike Looijmans , Ramiro Oliveira , , , , References: <1ff465a7-69ed-6cd1-0e02-4b9ff752b3cd@topic.nl> CC: Vinod Koul , Dan Williams , From: Ramiro Oliveira Message-ID: Date: Thu, 9 Mar 2017 09:55:59 +0000 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:45.0) Gecko/20100101 Thunderbird/45.7.1 MIME-Version: 1.0 In-Reply-To: <1ff465a7-69ed-6cd1-0e02-4b9ff752b3cd@topic.nl> Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit X-Originating-IP: [10.107.25.136] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Mike On 3/9/2017 7:37 AM, Mike Looijmans wrote: > On 06-03-17 13:27, Ramiro Oliveira wrote: >> Add option in Kconfig to use Xilinx VDMA in ARC processors. >> >> Signed-off-by: Ramiro Oliveira >> --- >> drivers/dma/Kconfig | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/drivers/dma/Kconfig b/drivers/dma/Kconfig >> index 263495d0adbd..b6b6c0581ad7 100644 >> --- a/drivers/dma/Kconfig >> +++ b/drivers/dma/Kconfig >> @@ -546,7 +546,7 @@ config XGENE_DMA >> >> config XILINX_DMA >> tristate "Xilinx AXI DMAS Engine" >> - depends on (ARCH_ZYNQ || MICROBLAZE || ARM64) >> + depends on (ARCH_ZYNQ || MICROBLAZE || ARM64|| ARC) > > One can put a Xilinx FPGA on a PCIe card, instantiate a PCIe-AXI bridge and then > use it on any platform that supports PCIe. > > I've had to remove this line in order to be able to use the engine on an x86 > desktop. > > How about plainly removing this "depends on" line and be done with it, instead > of adding platforms one by one? > > I agree with you. From my point of view it would be better to remove this line entirely. I'm just not sure if everyone agrees. >> select DMA_ENGINE >> help >> Enable support for Xilinx AXI VDMA Soft IP. >> > > > > Kind regards, > > Mike Looijmans > System Expert > > TOPIC Products > Materiaalweg 4, NL-5681 RJ Best > Postbus 440, NL-5680 AK Best > Telefoon: +31 (0) 499 33 69 79 > E-mail: mike.looijmans@topicproducts.com > Website: www.topicproducts.com > > Please consider the environment before printing this e-mail > > > > > -- Best Regards Ramiro Oliveira Ramiro.Oliveira@synopsys.com