On Tue, Feb 09, 2016 at 09:56:33AM +0100, Antoine Tenart wrote: > Hi Marc, > > On Mon, Feb 08, 2016 at 03:29:33PM +0000, Marc Zyngier wrote: > > On 08/02/16 09:11, Antoine Tenart wrote: > > > > > + gic: gic@f0100000 { > > > + compatible = "arm,gic-v3"; > > > + reg = <0x0 0xf0200000 0x0 0x10000>, /* GIC Dist */ > > > + <0x0 0xf0280000 0x0 0x200000>, /* GICR */ > > > + <0x0 0xf0100000 0x0 0x2000>; /* GICC */ > > > + interrupt-controller; > > > + #interrupt-cells = <3>; > > > + }; > > > > Something is wrong here. Either you are missing GICH and GICV (assuming > > you have legacy support), or you have an extra GICC region (which > > doesn't make sense on its own). > > I'll add the missing regions. Hmm, in fact the GICC region shouldn't be there. I'll make some tests and remove it. Antoine -- Antoine Ténart, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com
Attachment:
signature.asc
Description: Digital signature