Re: [PATCH v3 1/4] dmaengine: fsl-edma: add config and makefile changes for mcf-edma

2018-06-14 Thread Vinod Koul
On 12-06-18, 18:34, Angelo Dureghello wrote:
> This patch adds Kconfig and makefile changes to add ColdFire
> mcf5441x family edma support.
> A new fsl-edma-common module has been added, to collect common
> code to fsl-edma.

I think the way you send patch series has some issue and threading is
broken, so the patches are all over my inbox.

Can you please repost with threading fixed

> 
> Signed-off-by: Angelo Dureghello 
> ---
> Changes for v2:
> - patch splitted into 4
> - add fsl-edma-common
> 
> Changes for v3:
> - remove Kconfig FSL_EDMA_COMMON and set fsl-edma-common.o as
>   additional object entry where needed.
> ---
>  drivers/dma/Kconfig  | 13 +
>  drivers/dma/Makefile |  3 ++-
>  2 files changed, 15 insertions(+), 1 deletion(-)
> 
> diff --git a/drivers/dma/Kconfig b/drivers/dma/Kconfig
> index 6d61cd023633..c7d45e69f021 100644
> --- a/drivers/dma/Kconfig
> +++ b/drivers/dma/Kconfig
> @@ -220,6 +220,7 @@ config FSL_EDMA
>   depends on OF
>   select DMA_ENGINE
>   select DMA_VIRTUAL_CHANNELS
> + select FSL_EDMA_COMMON
>   help
> Support the Freescale eDMA engine with programmable channel
> multiplexing capability for DMA request sources(slot).
> @@ -327,6 +328,18 @@ config LPC18XX_DMAMUX
> Enable support for DMA on NXP LPC18xx/43xx platforms
> with PL080 and multiplexed DMA request lines.
>  
> +config MCF_EDMA
> + tristate "Freescale eDMA engine support, ColdFire mcf5441x SoCs"
> + depends on M5441x
> + select DMA_ENGINE
> + select DMA_VIRTUAL_CHANNELS
> + select FSL_EDMA_COMMON
> + help
> +   Support the Freescale ColdFire eDMA engine, 64-channel
> +   implementation that performs complex data transfers with
> +   minimal intervention from a host processor.
> +   This module can be found on Freescale ColdFire mcf5441x SoCs.
> +
>  config MMP_PDMA
>   bool "MMP PDMA support"
>   depends on ARCH_MMP || ARCH_PXA || COMPILE_TEST
> diff --git a/drivers/dma/Makefile b/drivers/dma/Makefile
> index 0f62a4d49aab..cce5e3400580 100644
> --- a/drivers/dma/Makefile
> +++ b/drivers/dma/Makefile
> @@ -32,7 +32,8 @@ obj-$(CONFIG_DW_AXI_DMAC) += dw-axi-dmac/
>  obj-$(CONFIG_DW_DMAC_CORE) += dw/
>  obj-$(CONFIG_EP93XX_DMA) += ep93xx_dma.o
>  obj-$(CONFIG_FSL_DMA) += fsldma.o
> -obj-$(CONFIG_FSL_EDMA) += fsl-edma.o
> +obj-$(CONFIG_FSL_EDMA) += fsl-edma.o fsl-edma-common.o
> +obj-$(CONFIG_MCF_EDMA) += mcf-edma.o fsl-edma-common.o
>  obj-$(CONFIG_FSL_RAID) += fsl_raid.o
>  obj-$(CONFIG_HSU_DMA) += hsu/
>  obj-$(CONFIG_IMG_MDC_DMA) += img-mdc-dma.o
> -- 
> 2.17.0

-- 
~Vinod
--
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v3 1/4] dmaengine: fsl-edma: add config and makefile changes for mcf-edma

2018-06-12 Thread Geert Uytterhoeven
Hi Angelo,

On Tue, Jun 12, 2018 at 6:35 PM Angelo Dureghello  wrote:
> This patch adds Kconfig and makefile changes to add ColdFire
> mcf5441x family edma support.
> A new fsl-edma-common module has been added, to collect common
> code to fsl-edma.
>
> Signed-off-by: Angelo Dureghello 
> ---
> Changes for v2:
> - patch splitted into 4
> - add fsl-edma-common
>
> Changes for v3:
> - remove Kconfig FSL_EDMA_COMMON and set fsl-edma-common.o as
>   additional object entry where needed.

> --- a/drivers/dma/Kconfig
> +++ b/drivers/dma/Kconfig

> @@ -327,6 +328,18 @@ config LPC18XX_DMAMUX
>   Enable support for DMA on NXP LPC18xx/43xx platforms
>   with PL080 and multiplexed DMA request lines.
>
> +config MCF_EDMA
> +   tristate "Freescale eDMA engine support, ColdFire mcf5441x SoCs"
> +   depends on M5441x
> +   select DMA_ENGINE
> +   select DMA_VIRTUAL_CHANNELS
> +   select FSL_EDMA_COMMON
> +   help
> + Support the Freescale ColdFire eDMA engine, 64-channel
> + implementation that performs complex data transfers with
> + minimal intervention from a host processor.
> + This module can be found on Freescale ColdFire mcf5441x SoCs.

I think MCF_EDMA should be added in patch 4/4 only.

> +
>  config MMP_PDMA
> bool "MMP PDMA support"
> depends on ARCH_MMP || ARCH_PXA || COMPILE_TEST
> diff --git a/drivers/dma/Makefile b/drivers/dma/Makefile
> index 0f62a4d49aab..cce5e3400580 100644
> --- a/drivers/dma/Makefile
> +++ b/drivers/dma/Makefile
> @@ -32,7 +32,8 @@ obj-$(CONFIG_DW_AXI_DMAC) += dw-axi-dmac/
>  obj-$(CONFIG_DW_DMAC_CORE) += dw/
>  obj-$(CONFIG_EP93XX_DMA) += ep93xx_dma.o
>  obj-$(CONFIG_FSL_DMA) += fsldma.o
> -obj-$(CONFIG_FSL_EDMA) += fsl-edma.o
> +obj-$(CONFIG_FSL_EDMA) += fsl-edma.o fsl-edma-common.o
> +obj-$(CONFIG_MCF_EDMA) += mcf-edma.o fsl-edma-common.o

Same for the line above: without patch 4/4, it won't build.

Gr{oetje,eeting}s,

Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH v3 1/4] dmaengine: fsl-edma: add config and makefile changes for mcf-edma

2018-06-12 Thread Geert Uytterhoeven
Hi Angelo,

On Tue, Jun 12, 2018 at 6:35 PM Angelo Dureghello  wrote:
> This patch adds Kconfig and makefile changes to add ColdFire
> mcf5441x family edma support.
> A new fsl-edma-common module has been added, to collect common
> code to fsl-edma.
>
> Signed-off-by: Angelo Dureghello 
> ---
> Changes for v2:
> - patch splitted into 4
> - add fsl-edma-common
>
> Changes for v3:
> - remove Kconfig FSL_EDMA_COMMON and set fsl-edma-common.o as
>   additional object entry where needed.

Thanks for the update!

> --- a/drivers/dma/Kconfig
> +++ b/drivers/dma/Kconfig
> @@ -220,6 +220,7 @@ config FSL_EDMA
> depends on OF
> select DMA_ENGINE
> select DMA_VIRTUAL_CHANNELS
> +   select FSL_EDMA_COMMON

Not needed (FSL_EDMA_COMMON is gone).

> help
>   Support the Freescale eDMA engine with programmable channel
>   multiplexing capability for DMA request sources(slot).
> @@ -327,6 +328,18 @@ config LPC18XX_DMAMUX
>   Enable support for DMA on NXP LPC18xx/43xx platforms
>   with PL080 and multiplexed DMA request lines.
>
> +config MCF_EDMA
> +   tristate "Freescale eDMA engine support, ColdFire mcf5441x SoCs"
> +   depends on M5441x
> +   select DMA_ENGINE
> +   select DMA_VIRTUAL_CHANNELS
> +   select FSL_EDMA_COMMON

Not needed (FSL_EDMA_COMMON is gone).

> +   help
> + Support the Freescale ColdFire eDMA engine, 64-channel
> + implementation that performs complex data transfers with
> + minimal intervention from a host processor.
> + This module can be found on Freescale ColdFire mcf5441x SoCs.
> +
>  config MMP_PDMA
> bool "MMP PDMA support"
> depends on ARCH_MMP || ARCH_PXA || COMPILE_TEST

Gr{oetje,eeting}s,

Geert

-- 
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- ge...@linux-m68k.org

In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
--
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH v3 1/4] dmaengine: fsl-edma: add config and makefile changes for mcf-edma

2018-06-12 Thread Angelo Dureghello
This patch adds Kconfig and makefile changes to add ColdFire
mcf5441x family edma support.
A new fsl-edma-common module has been added, to collect common
code to fsl-edma.

Signed-off-by: Angelo Dureghello 
---
Changes for v2:
- patch splitted into 4
- add fsl-edma-common

Changes for v3:
- remove Kconfig FSL_EDMA_COMMON and set fsl-edma-common.o as
  additional object entry where needed.
---
 drivers/dma/Kconfig  | 13 +
 drivers/dma/Makefile |  3 ++-
 2 files changed, 15 insertions(+), 1 deletion(-)

diff --git a/drivers/dma/Kconfig b/drivers/dma/Kconfig
index 6d61cd023633..c7d45e69f021 100644
--- a/drivers/dma/Kconfig
+++ b/drivers/dma/Kconfig
@@ -220,6 +220,7 @@ config FSL_EDMA
depends on OF
select DMA_ENGINE
select DMA_VIRTUAL_CHANNELS
+   select FSL_EDMA_COMMON
help
  Support the Freescale eDMA engine with programmable channel
  multiplexing capability for DMA request sources(slot).
@@ -327,6 +328,18 @@ config LPC18XX_DMAMUX
  Enable support for DMA on NXP LPC18xx/43xx platforms
  with PL080 and multiplexed DMA request lines.
 
+config MCF_EDMA
+   tristate "Freescale eDMA engine support, ColdFire mcf5441x SoCs"
+   depends on M5441x
+   select DMA_ENGINE
+   select DMA_VIRTUAL_CHANNELS
+   select FSL_EDMA_COMMON
+   help
+ Support the Freescale ColdFire eDMA engine, 64-channel
+ implementation that performs complex data transfers with
+ minimal intervention from a host processor.
+ This module can be found on Freescale ColdFire mcf5441x SoCs.
+
 config MMP_PDMA
bool "MMP PDMA support"
depends on ARCH_MMP || ARCH_PXA || COMPILE_TEST
diff --git a/drivers/dma/Makefile b/drivers/dma/Makefile
index 0f62a4d49aab..cce5e3400580 100644
--- a/drivers/dma/Makefile
+++ b/drivers/dma/Makefile
@@ -32,7 +32,8 @@ obj-$(CONFIG_DW_AXI_DMAC) += dw-axi-dmac/
 obj-$(CONFIG_DW_DMAC_CORE) += dw/
 obj-$(CONFIG_EP93XX_DMA) += ep93xx_dma.o
 obj-$(CONFIG_FSL_DMA) += fsldma.o
-obj-$(CONFIG_FSL_EDMA) += fsl-edma.o
+obj-$(CONFIG_FSL_EDMA) += fsl-edma.o fsl-edma-common.o
+obj-$(CONFIG_MCF_EDMA) += mcf-edma.o fsl-edma-common.o
 obj-$(CONFIG_FSL_RAID) += fsl_raid.o
 obj-$(CONFIG_HSU_DMA) += hsu/
 obj-$(CONFIG_IMG_MDC_DMA) += img-mdc-dma.o
-- 
2.17.0

--
To unsubscribe from this list: send the line "unsubscribe linux-m68k" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html