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

Reply via email to