Re: [PATCH 4/5] ARM: shmobile: r8a7791: Add IPMMU DT nodes

2014-04-01 Thread Laurent Pinchart
Hi Ben, On Monday 31 March 2014 15:30:00 Ben Dooks wrote: > On 31/03/14 15:01, Laurent Pinchart wrote: > > On Monday 31 March 2014 10:52:28 Geert Uytterhoeven wrote: > >> On Sat, Mar 29, 2014 at 12:36 AM, Laurent Pinchart wrote: > >>> + ipmmu_sy0: mmu@e6280800 { > >>> + compati

Re: [PATCH 4/5] ARM: shmobile: r8a7791: Add IPMMU DT nodes

2014-03-31 Thread Ben Dooks
On 31/03/14 15:01, Laurent Pinchart wrote: Hi Geert, On Monday 31 March 2014 10:52:28 Geert Uytterhoeven wrote: On Sat, Mar 29, 2014 at 12:36 AM, Laurent Pinchart wrote: + ipmmu_sy0: mmu@e6280800 { + compatible = "renesas,ipmmu-vmsa"; + reg = <0 0xe6280800 0 0

Re: [PATCH 4/5] ARM: shmobile: r8a7791: Add IPMMU DT nodes

2014-03-31 Thread Laurent Pinchart
Hi Geert, On Monday 31 March 2014 10:52:28 Geert Uytterhoeven wrote: > On Sat, Mar 29, 2014 at 12:36 AM, Laurent Pinchart wrote: > > + ipmmu_sy0: mmu@e6280800 { > > + compatible = "renesas,ipmmu-vmsa"; > > + reg = <0 0xe6280800 0 0x800>; > > Shouldn't this be "re

Re: [PATCH 4/5] ARM: shmobile: r8a7791: Add IPMMU DT nodes

2014-03-31 Thread Geert Uytterhoeven
Hi Laurent, On Sat, Mar 29, 2014 at 12:36 AM, Laurent Pinchart wrote: > + ipmmu_sy0: mmu@e6280800 { > + compatible = "renesas,ipmmu-vmsa"; > + reg = <0 0xe6280800 0 0x800>; Shouldn't this be "reg = <0 0xe628 0 0x1000>", i.e. expose both banks? Is there any

[PATCH 4/5] ARM: shmobile: r8a7791: Add IPMMU DT nodes

2014-03-28 Thread Laurent Pinchart
Signed-off-by: Laurent Pinchart --- arch/arm/boot/dts/r8a7791.dtsi | 49 ++ 1 file changed, 49 insertions(+) diff --git a/arch/arm/boot/dts/r8a7791.dtsi b/arch/arm/boot/dts/r8a7791.dtsi index 2e22cb2..bbadf7a 100644 --- a/arch/arm/boot/dts/r8a7791.dtsi +++