From mboxrd@z Thu Jan 1 00:00:00 1970 From: sergei.shtylyov@cogentembedded.com (Sergei Shtylyov) Date: Tue, 29 Jul 2014 23:55:08 +0400 Subject: [PATCH 4/4] ARM: dts: rockchip: add rk3188 dma controllers In-Reply-To: <1406661128-7614-5-git-send-email-heiko@sntech.de> References: <1406661128-7614-1-git-send-email-heiko@sntech.de> <1406661128-7614-5-git-send-email-heiko@sntech.de> Message-ID: <53D7FC1C.10607@cogentembedded.com> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Hello. On 07/29/2014 11:12 PM, Heiko Stuebner wrote: > Add both the cpu and peripheral pl330 dma controllers present in rk3188 socs. > The first dma controller can change between secure and non-secure mode and is > left by the bootloader in secure mode, which gets added here. > Signed-off-by: Heiko Stuebner > --- > arch/arm/boot/dts/rk3xxx.dtsi | 27 +++++++++++++++++++++++++++ > 1 file changed, 27 insertions(+) > diff --git a/arch/arm/boot/dts/rk3xxx.dtsi b/arch/arm/boot/dts/rk3xxx.dtsi > index c6f0561..a6c2860 100644 > --- a/arch/arm/boot/dts/rk3xxx.dtsi > +++ b/arch/arm/boot/dts/rk3xxx.dtsi > @@ -28,6 +28,33 @@ > i2c4 = &i2c4; > }; > > + amba { > + compatible = "arm,amba-bus"; > + #address-cells = <1>; > + #size-cells = <1>; > + ranges; > + > + dmac1: dma at 20018000 { The ePAPR standard [1] says: The name of a node should be somewhat generic, reflecting the function of the device and not its precise programming model. If appropriate, the name should be one of the following choices: [...] ? dma-controller > + compatible = "arm,pl330", "arm,primecell"; > + reg = <0x20018000 0x4000>; > + interrupts = , > + ; > + #dma-cells = <1>; > + clocks = <&cru ACLK_DMA1>; > + clock-names = "apb_pclk"; > + }; [...] [1] http://www.power.org/resources/downloads/Power_ePAPR_APPROVED_v1.0.pdf WBR, Sergei