Re: [PATCH 2/3] ARM: edma: Rename header file for dmaengine filter function definition

2014-11-28 Thread Arnd Bergmann
On Friday 28 November 2014 09:16:24 Peter Ujfalusi wrote: On 11/27/2014 11:52 PM, Arnd Bergmann wrote: On Thursday 27 November 2014 20:46:12 Peter Ujfalusi wrote: I see. With this series I did not planed to fix all edma related issues, just as a start clean up the related header files.

Re: [PATCH 2/3] ARM: edma: Rename header file for dmaengine filter function definition

2014-11-28 Thread Ulf Hansson
On 27 November 2014 at 11:41, Peter Ujfalusi peter.ujfal...@ti.com wrote: Rename the include/linux/edma.h to include/linux/edma-dmaengine.h Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com For the mmc parts: Acked-by: Ulf Hansson ulf.hans...@linaro.org --- arch/arm/common/edma.c

Re: [PATCH 2/3] ARM: edma: Rename header file for dmaengine filter function definition

2014-11-28 Thread Peter Ujfalusi
On 11/28/2014 12:51 PM, Arnd Bergmann wrote: On Friday 28 November 2014 09:16:24 Peter Ujfalusi wrote: On 11/27/2014 11:52 PM, Arnd Bergmann wrote: On Thursday 27 November 2014 20:46:12 Peter Ujfalusi wrote: I see. With this series I did not planed to fix all edma related issues, just as a

[PATCH 2/3] ARM: edma: Rename header file for dmaengine filter function definition

2014-11-27 Thread Peter Ujfalusi
Rename the include/linux/edma.h to include/linux/edma-dmaengine.h Signed-off-by: Peter Ujfalusi peter.ujfal...@ti.com --- arch/arm/common/edma.c | 2 +- drivers/mmc/host/davinci_mmc.c | 3 +-- drivers/spi/spi-davinci.c | 2 +- include/linux/edma-dmaengine.h | 29

Re: [PATCH 2/3] ARM: edma: Rename header file for dmaengine filter function definition

2014-11-27 Thread Arnd Bergmann
On Thursday 27 November 2014 12:41:30 Peter Ujfalusi wrote: diff --git a/include/linux/edma-dmaengine.h b/include/linux/edma-dmaengine.h new file mode 100644 index ..8a2602809a77 --- /dev/null +++ b/include/linux/edma-dmaengine.h @@ -0,0 +1,29 @@ +struct dma_chan; + +#if

Re: [PATCH 2/3] ARM: edma: Rename header file for dmaengine filter function definition

2014-11-27 Thread Peter Ujfalusi
On 11/27/2014 01:14 PM, Arnd Bergmann wrote: On Thursday 27 November 2014 12:41:30 Peter Ujfalusi wrote: diff --git a/include/linux/edma-dmaengine.h b/include/linux/edma-dmaengine.h new file mode 100644 index ..8a2602809a77 --- /dev/null +++ b/include/linux/edma-dmaengine.h @@

Re: [PATCH 2/3] ARM: edma: Rename header file for dmaengine filter function definition

2014-11-27 Thread Arnd Bergmann
On Thursday 27 November 2014 16:23:31 Peter Ujfalusi wrote: This will only work in case of legacy boot. When booting with DT we do not have pdata and after this patch in dt boot we are not going to be able to get the DMA resources either. No, when booting with DT, the filter_fn and data are

Re: [PATCH 2/3] ARM: edma: Rename header file for dmaengine filter function definition

2014-11-27 Thread Peter Ujfalusi
On 11/27/2014 04:50 PM, Arnd Bergmann wrote: On Thursday 27 November 2014 16:23:31 Peter Ujfalusi wrote: This will only work in case of legacy boot. When booting with DT we do not have pdata and after this patch in dt boot we are not going to be able to get the DMA resources either. No,

Re: [PATCH 2/3] ARM: edma: Rename header file for dmaengine filter function definition

2014-11-27 Thread Arnd Bergmann
On Thursday 27 November 2014 20:46:12 Peter Ujfalusi wrote: I see. With this series I did not planed to fix all edma related issues, just as a start clean up the related header files. I would rather not add fixes to mmc, spi, etc drivers since while you have valid point it is not in the scope

Re: [PATCH 2/3] ARM: edma: Rename header file for dmaengine filter function definition

2014-11-27 Thread Peter Ujfalusi
On 11/27/2014 11:52 PM, Arnd Bergmann wrote: On Thursday 27 November 2014 20:46:12 Peter Ujfalusi wrote: I see. With this series I did not planed to fix all edma related issues, just as a start clean up the related header files. I would rather not add fixes to mmc, spi, etc drivers since