[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20160209090131.GB5388@kwain>
Date: Tue, 9 Feb 2016 10:01:31 +0100
From: Antoine Tenart <antoine.tenart@...e-electrons.com>
To: Marc Zyngier <marc.zyngier@....com>
Cc: Antoine Tenart <antoine.tenart@...e-electrons.com>,
catalin.marinas@....com, will.deacon@....com,
tsahee@...apurnalabs.com, linux-arm-kernel@...ts.infradead.org,
rshitrit@...apurnalabs.com, thomas.petazzoni@...e-electrons.com,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
Barak Wasserstrom <barak@...apurnalabs.com>
Subject: Re: [PATCH 2/3] arm64: dts: add the Alpine v2 EVP
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@...00000 {
> > > + 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
Download attachment "signature.asc" of type "application/pgp-signature" (820 bytes)
Powered by blists - more mailing lists