From mboxrd@z Thu Jan 1 00:00:00 1970 From: Simon Horman Subject: Re: [PATCH 00/30] ARM: dts: r8a779[0-4]: Reference both DMA controllers Date: Fri, 11 Mar 2016 10:16:28 +0900 Message-ID: <20160311011627.GA20699@verge.net.au> References: <1457609782-24108-1-git-send-email-niklas.soderlund+renesas@ragnatech.se> <20160310134325.GA1111@bigcity.dyn.berto.se> <20160310233855.GD7423@verge.net.au> <20160311002326.GC1111@bigcity.dyn.berto.se> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: Content-Disposition: inline In-Reply-To: <20160311002326.GC1111@bigcity.dyn.berto.se> Sender: linux-renesas-soc-owner@vger.kernel.org To: Geert Uytterhoeven , Magnus Damm , Laurent Pinchart , linux-renesas-soc@vger.kernel.org, "devicetree@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" List-Id: devicetree@vger.kernel.org On Fri, Mar 11, 2016 at 01:23:26AM +0100, Niklas S=C3=B6derlund wrote: > On 2016-03-11 08:38:56 +0900, Simon Horman wrote: > > On Thu, Mar 10, 2016 at 02:43:25PM +0100, Niklas S=C3=B6derlund wro= te: > > > Hi Geert, > > >=20 > > > On 2016-03-10 13:20:56 +0100, Geert Uytterhoeven wrote: > > > > Hi Niklas, > > > >=20 > > > > On Thu, Mar 10, 2016 at 12:35 PM, Niklas S=C3=B6derlund > > > > wrote: > > > > > R-Car Gen2 have two DMA controllers, which are equivalent. Ho= wever DMA > > > > > slaves were tied only to a single DMA controller in .dtsi. Th= is series > > > > > add references to to both DMA controllers for all nodes that = previously > > > > > only referenced one of them. It also adds dma references to t= he SDHI > > > > > nodes for r8a7794 which according to the datasheet should wor= k but I > > > > > have not access to HW to verify. > > > > > > > > > > Tested on Koelsch (r8a7791) by altering the compatible value = of either > > > > > dmac0 or dmac1 and verifying that the DMA slaves pick the DMA= controller > > > > > that is instantiated. > > > >=20 > > > > Thanks for your series! > > > >=20 > > > > If you fix the subject of patch 30: > > > > Acked-by: Geert Uytterhoeven > > >=20 > > > Thanks for your review and that you found my error in the subject= =2E > > >=20 > > > >=20 > > > > I guess this is against v4.5-rc7? It doesn't apply against > > > > renesas-devel-20160308-v4.5-rc7. > > > > Unfortunately you'll have to rebase against Simon's tree, and a= dd scif2 > > > > on r8a7790, and i2c[6-8] on r8a7793. > > >=20 > > > I will rebase and update the relevant patches to include the addi= tions=20 > > > to r8a7790 and r8a7793. > >=20 > > If its not too much bother could you provide a git branch as well a= s > > patches posted to the mailing list. For larger series that is help= ful to me. >=20 > No problem I will include a link when i send v2. Excellent!