All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] dma: xilinx: Make Xilinx VDMA support ARC
@ 2017-03-06 12:27 ` Ramiro Oliveira
  0 siblings, 0 replies; 12+ messages in thread
From: Ramiro Oliveira @ 2017-03-06 12:27 UTC (permalink / raw)
  To: linux-kernel, dmaengine, devicetree, linux-arm-kernel
  Cc: CARLOS.PALMINHA, Ramiro Oliveira, Dan Williams, Vinod Koul

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

Signed-off-by: Ramiro Oliveira <roliveir@synopsys.com>
---
 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

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH] dma: xilinx: Make Xilinx VDMA support ARC
@ 2017-03-06 12:27 ` Ramiro Oliveira
  0 siblings, 0 replies; 12+ messages in thread
From: Ramiro Oliveira @ 2017-03-06 12:27 UTC (permalink / raw)
  To: linux-kernel, dmaengine, devicetree, linux-arm-kernel
  Cc: Vinod Koul, Ramiro Oliveira, Dan Williams, CARLOS.PALMINHA

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

Signed-off-by: Ramiro Oliveira <roliveir@synopsys.com>
---
 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

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* [PATCH] dma: xilinx: Make Xilinx VDMA support ARC
@ 2017-03-06 12:27 ` Ramiro Oliveira
  0 siblings, 0 replies; 12+ messages in thread
From: Ramiro Oliveira @ 2017-03-06 12:27 UTC (permalink / raw)
  To: linux-arm-kernel

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

Signed-off-by: Ramiro Oliveira <roliveir@synopsys.com>
---
 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

^ permalink raw reply related	[flat|nested] 12+ messages in thread

* Re: [PATCH] dma: xilinx: Make Xilinx VDMA support ARC
  2017-03-06 12:27 ` Ramiro Oliveira
  (?)
@ 2017-03-09  7:37   ` Mike Looijmans
  -1 siblings, 0 replies; 12+ messages in thread
From: Mike Looijmans @ 2017-03-09  7:37 UTC (permalink / raw)
  To: Ramiro Oliveira, linux-kernel, dmaengine, devicetree, linux-arm-kernel
  Cc: Vinod Koul, Dan Williams, CARLOS.PALMINHA

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 <roliveir@synopsys.com>
> ---
>  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: mike.looijmans@topicproducts.com
Website: www.topicproducts.com

Please consider the environment before printing this e-mail

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] dma: xilinx: Make Xilinx VDMA support ARC
@ 2017-03-09  7:37   ` Mike Looijmans
  0 siblings, 0 replies; 12+ messages in thread
From: Mike Looijmans @ 2017-03-09  7:37 UTC (permalink / raw)
  To: Ramiro Oliveira, linux-kernel, dmaengine, devicetree, linux-arm-kernel
  Cc: Vinod Koul, Dan Williams, CARLOS.PALMINHA

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 <roliveir@synopsys.com>
> ---
>  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: mike.looijmans@topicproducts.com
Website: www.topicproducts.com

Please consider the environment before printing this e-mail






_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH] dma: xilinx: Make Xilinx VDMA support ARC
@ 2017-03-09  7:37   ` Mike Looijmans
  0 siblings, 0 replies; 12+ messages in thread
From: Mike Looijmans @ 2017-03-09  7:37 UTC (permalink / raw)
  To: linux-arm-kernel

?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 <roliveir@synopsys.com>
> ---
>  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: mike.looijmans at topicproducts.com
Website: www.topicproducts.com

Please consider the environment before printing this e-mail

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] dma: xilinx: Make Xilinx VDMA support ARC
  2017-03-09  7:37   ` Mike Looijmans
  (?)
@ 2017-03-09  9:55     ` Ramiro Oliveira
  -1 siblings, 0 replies; 12+ messages in thread
From: Ramiro Oliveira @ 2017-03-09  9:55 UTC (permalink / raw)
  To: Mike Looijmans, Ramiro Oliveira, linux-kernel, dmaengine,
	devicetree, linux-arm-kernel
  Cc: Vinod Koul, Dan Williams, CARLOS.PALMINHA

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 <roliveir@synopsys.com>
>> ---
>>  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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] dma: xilinx: Make Xilinx VDMA support ARC
@ 2017-03-09  9:55     ` Ramiro Oliveira
  0 siblings, 0 replies; 12+ messages in thread
From: Ramiro Oliveira @ 2017-03-09  9:55 UTC (permalink / raw)
  To: Mike Looijmans, Ramiro Oliveira, linux-kernel, dmaengine,
	devicetree, linux-arm-kernel
  Cc: Vinod Koul, Dan Williams, CARLOS.PALMINHA

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 <roliveir@synopsys.com>
>> ---
>>  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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH] dma: xilinx: Make Xilinx VDMA support ARC
@ 2017-03-09  9:55     ` Ramiro Oliveira
  0 siblings, 0 replies; 12+ messages in thread
From: Ramiro Oliveira @ 2017-03-09  9:55 UTC (permalink / raw)
  To: linux-arm-kernel

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 <roliveir@synopsys.com>
>> ---
>>  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 at topicproducts.com
> Website: www.topicproducts.com
> 
> Please consider the environment before printing this e-mail
> 
> 
> 
> 
> 

-- 
Best Regards

Ramiro Oliveira
Ramiro.Oliveira at synopsys.com

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] dma: xilinx: Make Xilinx VDMA support ARC
@ 2017-03-10  4:33       ` Vinod Koul
  0 siblings, 0 replies; 12+ messages in thread
From: Vinod Koul @ 2017-03-10  4:33 UTC (permalink / raw)
  To: Ramiro Oliveira
  Cc: Mike Looijmans, linux-kernel, dmaengine, devicetree,
	linux-arm-kernel, Dan Williams, CARLOS.PALMINHA

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 <roliveir@synopsys.com>
> >> ---
> >>  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

^ permalink raw reply	[flat|nested] 12+ messages in thread

* Re: [PATCH] dma: xilinx: Make Xilinx VDMA support ARC
@ 2017-03-10  4:33       ` Vinod Koul
  0 siblings, 0 replies; 12+ messages in thread
From: Vinod Koul @ 2017-03-10  4:33 UTC (permalink / raw)
  To: Ramiro Oliveira
  Cc: Mike Looijmans, linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	dmaengine-u79uwXL29TY76Z2rM5mHXA,
	devicetree-u79uwXL29TY76Z2rM5mHXA,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r, Dan Williams,
	CARLOS.PALMINHA-HKixBCOQz3hWk0Htik3J/w

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 <roliveir-HKixBCOQz3hWk0Htik3J/w@public.gmane.org>
> >> ---
> >>  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
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 12+ messages in thread

* [PATCH] dma: xilinx: Make Xilinx VDMA support ARC
@ 2017-03-10  4:33       ` Vinod Koul
  0 siblings, 0 replies; 12+ messages in thread
From: Vinod Koul @ 2017-03-10  4:33 UTC (permalink / raw)
  To: linux-arm-kernel

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 <roliveir@synopsys.com>
> >> ---
> >>  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

^ permalink raw reply	[flat|nested] 12+ messages in thread

end of thread, other threads:[~2017-03-10  4:33 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-03-06 12:27 [PATCH] dma: xilinx: Make Xilinx VDMA support ARC Ramiro Oliveira
2017-03-06 12:27 ` Ramiro Oliveira
2017-03-06 12:27 ` Ramiro Oliveira
2017-03-09  7:37 ` Mike Looijmans
2017-03-09  7:37   ` Mike Looijmans
2017-03-09  7:37   ` Mike Looijmans
2017-03-09  9:55   ` Ramiro Oliveira
2017-03-09  9:55     ` Ramiro Oliveira
2017-03-09  9:55     ` Ramiro Oliveira
2017-03-10  4:33     ` Vinod Koul
2017-03-10  4:33       ` Vinod Koul
2017-03-10  4:33       ` Vinod Koul

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.