Messages in this thread Patch in this message | | | Date | Wed, 1 Jul 2015 08:54:56 +0200 (CEST) | From | Paul Osmialowski <> | Subject | Re: [PATCH v2 7/9] arm: twr-k70f120m: use Freescale eDMA driver with Kinetis SoC |
| |
Hi Arnd,
Thanks for pointing this - dma-ranges seem to work properly. This makes the patch much simpler. To be included to the next iteration.
I also removed DMA_OF from Kconfig and CONFIG_DMA_OF from my defconfig and it was automatically added during preparation of .cofing for build.
On Tue, 30 Jun 2015, Arnd Bergmann wrote:
> On Tuesday 30 June 2015 14:27:28 Paul Osmialowski wrote: >> Note that <mach/memory.h> is needed (which is denoted by >> CONFIG_NEED_MACH_MEMORY_H) as it provides macros required for proper >> operation of DMA allocation functions. > > You can't do this, it breaks compilation when multiple platforms > are enabled. > >> Signed-off-by: Paul Osmialowski <pawelo@king.net.pl> >> --- >> arch/arm/Kconfig | 4 ++ >> arch/arm/boot/dts/kinetis.dtsi | 34 ++++++++++++++++ >> arch/arm/mach-kinetis/include/mach/memory.h | 61 +++++++++++++++++++++++++++++ >> 3 files changed, 99 insertions(+) >> create mode 100644 arch/arm/mach-kinetis/include/mach/memory.h >> >> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig >> index b21592b..8ccffee 100644 >> --- a/arch/arm/Kconfig >> +++ b/arch/arm/Kconfig >> @@ -971,6 +971,10 @@ config ARCH_KINETIS >> select CLKSRC_KINETIS >> select PINCTRL >> select PINCTRL_KINETIS >> + select DMADEVICES >> + select FSL_EDMA >> + select DMA_OF >> + select NEED_MACH_MEMORY_H > > I think DMA_OF is implied by dmaengine support in combination with CONFIG_OF > >> + >> +#ifndef _MACH_KINETIS_MEMORY_H >> +#define _MACH_KINETIS_MEMORY_H >> + >> +#ifndef __ASSEMBLY__ >> + >> +/* >> + * On Kinetis K70, consistent DMA memory resides in a special >> + * DDRAM alias region (non-cacheable DDRAM at 0x80000000). >> + * >> + */ >> +#define KINETIS_PHYS_DMA_OFFSET UL(0x80000000) >> + >> +/* >> + * Mask of the field used to distinguish DDRAM aliases >> + */ >> +#define KINETIS_DRAM_ALIAS_MASK UL(0xf8000000) > > This should be expressed using the 'dma-ranges' properties in the > bus nodes above any DMA master, the normal DMA mapping code will > then do the right thing. > > Arnd >From e0b549151b3ee8d4419344a2b5ef461bad19b2af Mon Sep 17 00:00:00 2001 From: Paul Osmialowski <pawelo@king.net.pl> Date: Mon, 29 Jun 2015 23:37:20 +0200 Subject: [PATCH 7/9] arm: twr-k70f120m: use Freescale eDMA driver with Kinetis SoC
Signed-off-by: Paul Osmialowski <pawelo@king.net.pl> --- arch/arm/Kconfig | 2 ++ arch/arm/boot/dts/kinetis.dtsi | 36 ++++++++++++++++++++++++++++++++++++ 2 files changed, 38 insertions(+)
diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index b21592b..81a6328 100644 --- a/arch/arm/Kconfig +++ b/arch/arm/Kconfig @@ -971,6 +971,8 @@ config ARCH_KINETIS select CLKSRC_KINETIS select PINCTRL select PINCTRL_KINETIS + select DMADEVICES + select FSL_EDMA help This enables support for the Freescale Kinetis MCUs diff --git a/arch/arm/boot/dts/kinetis.dtsi b/arch/arm/boot/dts/kinetis.dtsi index 5ff1d3b..75fc6d4 100644 --- a/arch/arm/boot/dts/kinetis.dtsi +++ b/arch/arm/boot/dts/kinetis.dtsi @@ -19,6 +19,42 @@ }; soc { + dma-ranges = <0x80000000 0x00000000 0x08000000>; + + edma: dma-controller@40008000 { + compatible = "fsl,kinetis-edma"; + reg = <0x40008000 0x2000>, /* DMAC */ + <0x40021000 0x1000>, /* DMAMUX0 */ + <0x40022000 0x1000>; /* DMAMUX1 */ + #dma-cells = <2>; + dma-channels = <32>; + interrupts = <0>, <1>, <2>, <3>, + <4>, <5>, <6>, <7>, + <8>, <9>, <10>, <11>, + <12>, <13>, <14>, <15>, + <16>; + interrupt-names = "edma-tx-0,16", + "edma-tx-1,17", + "edma-tx-2,18", + "edma-tx-3,19", + "edma-tx-4,20", + "edma-tx-5,21", + "edma-tx-6,22", + "edma-tx-7,23", + "edma-tx-8,24", + "edma-tx-9,25", + "edma-tx-10,26", + "edma-tx-11,27", + "edma-tx-12,28", + "edma-tx-13,29", + "edma-tx-14,30", + "edma-tx-15,31", + "edma-err"; + clocks = <&mcg_cclk_gate 6 1>, + <&mcg_pclk_gate 5 1>, <&mcg_pclk_gate 5 2>; + clock-names = "edma", "dmamux0", "dmamux1"; + }; + portA: pinmux@40049000 { compatible = "fsl,kinetis-padconf"; reg = <0x40049000 0x1000>; -- 2.3.6
| |