Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933237AbcLBL2d (ORCPT ); Fri, 2 Dec 2016 06:28:33 -0500 Received: from mail.free-electrons.com ([62.4.15.54]:57798 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932766AbcLBL2c (ORCPT ); Fri, 2 Dec 2016 06:28:32 -0500 Date: Fri, 2 Dec 2016 12:28:29 +0100 From: Alexandre Belloni To: Nicolas Ferre Cc: linux-arm-kernel@lists.infradead.org, Boris BREZILLON , Ludovic Desroches , linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/3] ARM: dts: at91: add dma1 definition to sama5d2 Message-ID: <20161202112829.yr3rnylw2ktfc5th@piout.net> References: <20161201104949.23985-1-nicolas.ferre@atmel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20161201104949.23985-1-nicolas.ferre@atmel.com> User-Agent: NeoMutt/20161104 (1.7.1) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1622 Lines: 51 On 01/12/2016 at 11:49:47 +0100, Nicolas Ferre wrote : > The sama5d2 SoC has a second DMA controller and can be used just like DMA0. > By default both DMA controllers are configured as "Secure" in > MATRIX_SPSELR so we can use whichever we want in a "single Secure World" > configuration. > Surprisingly the DMA1 has a lower address than DMA0. To avoid confusion > place it after DMA0 node anyway. > sama5d2.dtsi is probably the only one that is properly ordered and I feel like we should keep it this way. If one of the nodes is not ordered properly, other ones will follow... We don't care about the name, it is just an alias. We only care about the address. > Signed-off-by: Nicolas Ferre > --- > arch/arm/boot/dts/sama5d2.dtsi | 10 ++++++++++ > 1 file changed, 10 insertions(+) > > diff --git a/arch/arm/boot/dts/sama5d2.dtsi b/arch/arm/boot/dts/sama5d2.dtsi > index ceb9783ff7e1..c791ce9c750c 100644 > --- a/arch/arm/boot/dts/sama5d2.dtsi > +++ b/arch/arm/boot/dts/sama5d2.dtsi > @@ -395,6 +395,16 @@ > clock-names = "dma_clk"; > }; > > + /* Place dma1 here despite its address */ > + dma1: dma-controller@f0004000 { > + compatible = "atmel,sama5d4-dma"; > + reg = <0xf0004000 0x1000>; > + interrupts = <7 IRQ_TYPE_LEVEL_HIGH 0>; > + #dma-cells = <1>; > + clocks = <&dma1_clk>; > + clock-names = "dma_clk"; > + }; > + > pmc: pmc@f0014000 { > compatible = "atmel,sama5d2-pmc", "syscon"; > reg = <0xf0014000 0x160>; > -- > 2.9.0 > -- Alexandre Belloni, Free Electrons Embedded Linux and Kernel engineering http://free-electrons.com