2017-03-06 12:30:21

by Ramiro Oliveira

[permalink] [raw]
Subject: [PATCH] dma: xilinx: Make Xilinx VDMA support ARC

Add option in Kconfig to use Xilinx VDMA in ARC processors.

Signed-off-by: Ramiro Oliveira <[email protected]>
---
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)
select DMA_ENGINE
help
Enable support for Xilinx AXI VDMA Soft IP.
--
2.11.0



2017-03-09 09:58:09

by Ramiro Oliveira

[permalink] [raw]
Subject: Re: [PATCH] dma: xilinx: Make Xilinx VDMA support ARC

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 <[email protected]>
>> ---
>> 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: [email protected]
> Website: http://www.topicproducts.com
>
> Please consider the environment before printing this e-mail
>
>
>
>
>

--
Best Regards

Ramiro Oliveira
[email protected]

2017-03-09 16:59:12

by Mike Looijmans

[permalink] [raw]
Subject: Re: [PATCH] dma: xilinx: Make Xilinx VDMA support ARC

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 <[email protected]>
> ---
> 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?


> 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: [email protected]
Website: http://www.topicproducts.com

Please consider the environment before printing this e-mail





2017-03-10 04:32:13

by Vinod Koul

[permalink] [raw]
Subject: Re: [PATCH] dma: xilinx: Make Xilinx VDMA support ARC

On Thu, Mar 09, 2017 at 09:55:59AM +0000, Ramiro Oliveira wrote:
> 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 <[email protected]>
> >> ---
> >> 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.

if it works on others, compiles fine then there should not be an issue to
remove.

Feel free to send the patch.

--
~Vinod